Re: [PATCH v3] pcie_root_port: Add hotplug disabling option

["Ján Tomko" <jtomko@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1572 bytes --]

On a Wednesday in 2020, Julia Suvorova wrote:
>Make hot-plug/hot-unplug on PCIe Root Ports optional to allow libvirt
>manage it and restrict unplug for the whole machine. This is going to
>prevent user-initiated unplug in guests (Windows mostly).
>Hotplug is enabled by default.
>Usage:
>    -device pcie-root-port,hotplug=off,...
>
>If you want to disable hot-unplug on some downstream ports of one
>switch, disable hot-unplug on PCIe Root Port connected to the upstream
>port as well as on the selected downstream ports.
>
>Discussion related:
>    https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg00530.html
>
>Signed-off-by: Julia Suvorova <jusual@redhat.com>
>---
>v1: https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg04868.html
>
>v2: https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg05192.html
>    * change name of the option to 'enable-hotplug' [Laine]
>    * change order of enabling capability bits [Igor]
>    * enable HPS bit [Igor]
>    * add option to xio3130_downstream [Ján]
>
>v3:
>    * change name of the option to 'hotplug'. Naming is hard! [Laine]
>    * move property under TYPE_PCIE_SLOT [Michael]
>
> hw/pci-bridge/pcie_root_port.c     |  2 +-
> hw/pci-bridge/xio3130_downstream.c |  2 +-
> hw/pci/pcie.c                      | 11 +++++++----
> hw/pci/pcie_port.c                 |  1 +
> include/hw/pci/pcie.h              |  2 +-
> include/hw/pci/pcie_port.h         |  3 +++
> 6 files changed, 14 insertions(+), 7 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]