<div dir="ltr">Thank you very much Dann for finishing that. And sorry for lack of response for so long. <div>I somehow missed your last email and feeling bad that you had to ping me so long for this.<div><br></div><div>Thank you Michal for accepting this.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 23 Jan 2019 at 10:29, Michal Privoznik <<a href="mailto:mprivozn@redhat.com">mprivozn@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 1/22/19 8:26 PM, dann frazier wrote:<br>
> Following up on my offer to take the baton[*], here's a v3 of this series<br>
> that should address the provided feedback provided for Radoslaw's v2.<br>
> <br>
> ThunderX is Cavium SoC. This platform contain SRIOV NIC.<br>
> Unlike other commonly known network devices it does not have VF functionality<br>
> duplicated in its PF. PF is purely management device (on HW level).<br>
> <br>
> This creates several problems with existing libvirt code as in many places<br>
> libvirt assumes that each VF netdev has PF netdev assigned. <br>
> <br>
> This patch series trying to address issues which can be easily fixed.<br>
> (mostly bug fixes found while working on full featured solution)<br>
> <br>
> First patch in series is most important as it allows to unblock the netdev<br>
> detection and use <hostdev> on this platform.i <interface type="hostdev"<br>
> still does not work as it requires bigger changes both on netdev driver <br>
> and in libvirt itself.<br>
> More details about those issues can be found at:<br>
> <a href="https://bugs.linaro.org/show_bug.cgi?id=3778" rel="noreferrer" target="_blank">https://bugs.linaro.org/show_bug.cgi?id=3778</a><br>
> <a href="https://bugs.launchpad.net/charm-nova-compute/+bug/1771662" rel="noreferrer" target="_blank">https://bugs.launchpad.net/charm-nova-compute/+bug/1771662</a><br>
> <br>
> v3:<br>
> - Reinstated error path in virNetDevGetPhysicalFunction()<br>
> - Add missing free of *pfname in error path<br>
> - Use proper < 0 comparisons when checking for errors<br>
> <br>
> v2:<br>
> - error reporting taken out of virNetDevGetPhysicalFunction() and moved<br>
>   to calling function virNetDevGetVirtualFunctionInfo()<br>
> - curly braces removed from single line<br>
> - net-> model check removed as STREQ_NULLABLE() follows<br>
> <br>
> [*] <a href="https://www.redhat.com/archives/libvir-list/2019-January/msg00201.html" rel="noreferrer" target="_blank">https://www.redhat.com/archives/libvir-list/2019-January/msg00201.html</a><br>
> <br>
> Radoslaw Biernacki (4):<br>
>   util: fixing wrong assumption that PF has to have netdev assigned<br>
>   util: Code simplification<br>
>   util: Fix for NULL dereference<br>
>   util: Fixing invalid error checking from virPCIGetNetname()<br>
> <br>
>  src/qemu/qemu_domain_address.c | 13 ++++----<br>
>  src/util/virhostdev.c          |  2 +-<br>
>  src/util/virnetdev.c           | 55 ++++++++++++----------------------<br>
>  3 files changed, 25 insertions(+), 45 deletions(-)<br>
> <br>
<br>
I've fixed all the small issues I've raised, ACKed the whole thing and<br>
pushed.<br>
<br>
Congratulations Radoslaw on your first libvirt contribution and thank<br>
you Dann for picking this up.<br>
<br>
Michal<br>
<br>
</blockquote></div>