Re: [PATCH V4 0/8] hw/block/nvme: support namespace attachment

[Klaus Jensen]

On Mar  2 22:26, Minwoo Im wrote:
> Hello,
> 
> This series supports namespace attachment: attach and detach.  This is
> the fourth version of series with replacing changed namespace list to
> bitmap to indicate changed namespace IDs.
> 
> Please review.
> 
> Thanks,
> 
> Since V3:
>   - Replace changed ns list to bitmap to not only represent relationship
>     between controller and namespace, but also avoid duplications of
>     nsids in the list.  (Klaus)
> 
> Since V2:
>   - Added command effects (namespace inventory changed) for namespace
>     attach command.  (Keith)
>   - Added [7/8] patch to support asynchronus event when namespace
>     inventory is updated.  (Keith)
>   - Added review and tested tag from Klaus to all the patches, but [6/8]
>     and [7/8].
> 
> Since V1:
>   - Fix to take 'ctrl' which is given from the command rather than 'n'.
>     (Klaus)
>   - Add a [7/7] patch to support CNS 12h Identify command (Namespace
>     Attached Controller list).
> 
> Minwoo Im (8):
>   hw/block/nvme: support namespace detach
>   hw/block/nvme: fix namespaces array to 1-based
>   hw/block/nvme: fix allocated namespace list to 256
>   hw/block/nvme: support allocated namespace type
>   hw/block/nvme: refactor nvme_select_ns_iocs
>   hw/block/nvme: support namespace attachment command
>   hw/block/nvme: support changed namespace asyncrohous event
>   hw/block/nvme: support Identify NS Attached Controller List
> 
>  hw/block/nvme-ns.c     |   1 +
>  hw/block/nvme-ns.h     |   2 +
>  hw/block/nvme-subsys.h |  28 +++-
>  hw/block/nvme.c        | 300 ++++++++++++++++++++++++++++++++++++-----
>  hw/block/nvme.h        |  37 +++++
>  hw/block/trace-events  |   3 +
>  include/block/nvme.h   |  14 ++
>  7 files changed, 349 insertions(+), 36 deletions(-)
> 
> -- 
> 2.27.0
> 

Tested and applied to nvme-next! Thanks!

signature.asc
Description: PGP signature