[PATCH 0/2] hw/nvme: support smart AEN
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-pizhen...@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(-) -- 2.20.1
Re: [PATCH 0/2] hw/nvme: support smart AEN
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-pizhen...@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
Re: Re: [PATCH 0/2] hw/nvme: support smart AEN
On 5/9/22 19:12, Klaus Jensen wrote: 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-pizhen...@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. Oh, sorry, I misunderstand OAES[7:0]. Thanks for pointing this out! -- zhenwei pi