<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 30, 2021 at 18:52 Laine Stump <<a href="mailto:laine@redhat.com">laine@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On 9/30/21 2:16 AM, Ani Sinha wrote:<br>
> On Fri, Sep 24, 2021 at 2:16 AM Laine Stump <<a href="mailto:laine@redhat.com" target="_blank">laine@redhat.com</a>> wrote:<br>
>><br>
>> On 9/11/21 11:26 PM, Ani Sinha wrote:<br>
>>> The above two options are only available for qemu driver and that too for x86<br>
>>> guests only. Both of them are global options.<br>
>>><br>
>>> ``acpi-hotplug-bridge`` option enables or disables ACPI hotplug support for cold<br>
>>> plugged bridges. Examples of cold plugged bridges include PCI-PCI bridge<br>
>>> (pci-bridge controller) for pc machines and pcie-root-port controller for q35<br>
>>> machines. The corresponding commandline options to qemu for x86 guests are:<br>
>><br>
>> The "cold plugged bridges" term here throws me for a loop - it implies<br>
>> that hotplugging bridges is something that's supported, and I think it<br>
>> still isn't. Of course this is just the cover letter, so it won't go<br>
>> into git anywhere, but I think it should be enough to say "enables ACPI<br>
>> hotplug into non-root bus PCI bridges/ports".<br>
> <br>
> I think emphasizing cold plugged bridges is important as Igor (CC'd)<br>
> has clarified in the other email on patch #3 of this series.<br>
<br>
Okay, so the implication in Igor's email is that a) it is possible to <br>
hotplug a pcie controller, but b) any controller that is hotplugged will <br>
not have ACPI enabled. Note though that libvirt doesn't allow <br>
hotplugging *any* PCI controller, since we were told long ago that no OS <br>
will actually rescan the PCI topology when this is done, and so the new <br>
controller wouldn't be usable anyway. (that information may well be <br>
outdated).</blockquote><div dir="auto"><br></div><div dir="auto">From i440fx side all empty ports in the pci root controller are described as hotplug capable from ACPI. So I do not see why we cannot hotplug a pci bridge in one of the pci root ports and OS should be able to detect it without reboot. I have not tried it though. </div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)" dir="auto"><br>
<br>
I think if you're going to mention that it is specifically for <br>
"cold-plugged bridges" then you should also 1) define what <br>
"cold-plugged" means, i.e. "(PCI controllers that were present in the <br>
domain definition when the guest was first started"), and 2) note that <br>
"ACPI is not enabled for bridges that are hot-plugged (but currently <br>
libvirt doesn't support hotplugging a pci controller anyway)" or <br>
something like that.<br>
<br>
</blockquote></div></div>