Re: [PATCH v5 1/5] vfio: Add the device features for the low power entry and exit

From: Cornelia Huck
Date: Tue Jul 26 2022 - 09:13:16 EST


On Tue, Jul 26 2022, Abhishek Sahu <abhsahu@xxxxxxxxxx> wrote:

> #define VFIO_DEVICE_FEATURE_LOW_POWER_ENTRY_WITH_WAKEUP 4
>
> /*
> * Upon VFIO_DEVICE_FEATURE_SET, disallow use of device low power states as
> * previously enabled via VFIO_DEVICE_FEATURE_LOW_POWER_ENTRY or
> * VFIO_DEVICE_FEATURE_LOW_POWER_ENTRY_WITH_WAKEUP device features.
> * This device feature IOCTL may itself generate a wakeup eventfd notification
> * in the latter case if the device has previously entered a low power state.

Nit: s/has/had/

> */
> #define VFIO_DEVICE_FEATURE_LOW_POWER_EXIT 5

I haven't followed this closely, and I'm not that familiar with power
management, but at least I can't spot anything obviously problematic.