mbox series

[0/2] hw/nvme: fix namespace attachment on controller hotplug

Message ID 20210924072742.306473-1-its@irrelevant.dk
Headers show
Series hw/nvme: fix namespace attachment on controller hotplug | expand

Message

Klaus Jensen Sept. 24, 2021, 7:27 a.m. UTC
From: Klaus Jensen <k.jensen@samsung.com>

First patch from Hannes fixes the subsystem registration process such
that shared (but non-detached) namespaces are automatically attached to
hotplugged controllers.

The second patch changes the default for 'shared' such that namespaces
are shared by default and will thus by default be attached to hotplugged
controllers. This adds a compat property for older machine versions and
updates the documentation to reflect this.

Hannes Reinecke (1):
  hw/nvme: reattach subsystem namespaces on hotplug

Klaus Jensen (1):
  hw/nvme: change nvme-ns 'shared' default

 docs/system/devices/nvme.rst | 24 ++++++++++++++----------
 hw/core/machine.c            |  4 +++-
 hw/nvme/ns.c                 |  8 +-------
 hw/nvme/subsys.c             | 10 +++++++++-
 4 files changed, 27 insertions(+), 19 deletions(-)

Comments

Klaus Jensen Nov. 15, 2021, 5:57 p.m. UTC | #1
On Sep 24 09:27, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
> 
> First patch from Hannes fixes the subsystem registration process such
> that shared (but non-detached) namespaces are automatically attached to
> hotplugged controllers.
> 
> The second patch changes the default for 'shared' such that namespaces
> are shared by default and will thus by default be attached to hotplugged
> controllers. This adds a compat property for older machine versions and
> updates the documentation to reflect this.
> 
> Hannes Reinecke (1):
>   hw/nvme: reattach subsystem namespaces on hotplug
> 
> Klaus Jensen (1):
>   hw/nvme: change nvme-ns 'shared' default
> 
>  docs/system/devices/nvme.rst | 24 ++++++++++++++----------
>  hw/core/machine.c            |  4 +++-
>  hw/nvme/ns.c                 |  8 +-------
>  hw/nvme/subsys.c             | 10 +++++++++-
>  4 files changed, 27 insertions(+), 19 deletions(-)
> 

Ping :)
Keith Busch Nov. 16, 2021, 3:55 p.m. UTC | #2
On Fri, Sep 24, 2021 at 09:27:40AM +0200, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
> 
> First patch from Hannes fixes the subsystem registration process such
> that shared (but non-detached) namespaces are automatically attached to
> hotplugged controllers.
> 
> The second patch changes the default for 'shared' such that namespaces
> are shared by default and will thus by default be attached to hotplugged
> controllers. This adds a compat property for older machine versions and
> updates the documentation to reflect this.

Series looks good.

Reviewed-by: Keith Busch <kbusch@kernel.org>