<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 20, 2023 at 7:17 PM Andrea Bolognani <<a href="mailto:abologna@redhat.com">abologna@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Feb 20, 2023 at 05:12:53PM +0100, Kristina Hanicova wrote:<br>
> v1 here:<br>
> <a href="https://listman.redhat.com/archives/libvir-list/2023-February/237622.html" rel="noreferrer" target="_blank">https://listman.redhat.com/archives/libvir-list/2023-February/237622.html</a><br>
><br>
> diff to v1:<br>
> * reduced test files (thanks Andrea)<br>
> * removed redundant check for address type (noticed by Peter)<br>
> * plugging the device directly into pcie.0 if its address was not<br>
>   specified<br>
<br>
Just for completeness' sake, note that the use of<br>
VIR_PCI_CONNECT_INTEGRATED results in libvirt outright rejecting<br>
attempts to put the device anywhere but on pcie.0, even when the<br>
address comes directly from the user. This is technically a<br>
limitation compared to what QEMU allows, but in practice the other<br>
configurations are untested and overall it feels like a fair<br>
trade-off. We can also decide to lift this limitation in the future,<br>
if it ever comes to that.<br>
<br>
> Kristina Hanicova (6):<br>
>   qemu: introduce QEMU_CAPS_DEVICE_PANIC_PCI<br>
>   conf: add panic model 'pvpanic'<br>
>   tests: add test cases for device pvpanic-pci<br>
>   qemu: assign PCI address to device pvpanic-pci<br>
>   tests: add case for pvpanic-pci without address<br>
>   docs: document panic device 'pvpanic-pci'<br>
<br>
Everything looks great, so<br>
<br>
  Reviewed-by: Andrea Bolognani <<a href="mailto:abologna@redhat.com" target="_blank">abologna@redhat.com</a>><br>
<br>
The first patch is missing the Signed-off-by tag. Can you please<br>
confirm that you're okay with me adding it before pushing?<br></blockquote><div><br></div><div>Yes, thank you.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I'm also not seeing any updates to the NEWS file. Please post that as<br>
a follow-up patch before release :)<br>
<br></blockquote><div><br></div><div>Thanks for the reminder, the patch updating NEWS is on the list now.</div><div><br></div><div><br></div><div>Kristina<br></div></div></div>