Re: [PATCH 0/2] hw/nvme: support smart AEN

[Klaus Jensen]

On May  7 15:26, zhenwei pi wrote:
> Hi,
> In this series, firstly introduce smart related bits of aen cfg, then
> support this in oaes.
> 
> Linux guest does not support this currently, I also send a series to
> enable smart AEN:
> https://lore.kernel.org/lkml/20220507065026.260306-1-pizhenwei@bytedance.com/T/#t
> 
> Test the two series together, works fine.
> 
> Zhenwei Pi (2):
>   hw/nvme: introduce smart bits of aen cfg
>   hw/nvme: support smart AEN
> 
>  hw/nvme/ctrl.c       | 9 ++++++++-
>  include/block/nvme.h | 8 +++++++-
>  2 files changed, 15 insertions(+), 2 deletions(-)
> 

I'm not quite sure what you are trying to achieve here. Bits 0 through 7
are reserved in OAES, they are not optional.

We already simply accept whatever the host configured with the
Asynchronous Event Configuration feature. And you added the support for
triggering custom AENs for those through the monitor.

signature.asc
Description: PGP signature