Re: [libvirt] [PATCH v3] lxc: Include support to lxc version 3.0 or higher.

2018-11-10 Thread John Ferlan
On 11/9/18 12:30 PM, Julio Faracco wrote: > This patch introduce the new settings for LXC 3.0 or higher. The older > versions keep the compatibility to deprecated settings for LXC, but > after release 3.0, the compatibility was removed. This commit adds the > support to the refactored settings.

[libvirt] [PATCH 4/4] util: Fixing invalid error checking from virPCIGetNetname()

2018-11-10 Thread Radoslaw Biernacki
linkdev is In/Out function parameter as second order reference pointer so requires first order dereference for checking NULLs which can be a result from virPCIGetNetName() Fixes: d6ee56d7237 (util: change virPCIGetNetName() to not return error if device has no net name) Signed-off-by: Radoslaw

[libvirt] [PATCH 0/4] util: Fixing libvirt errors on cavium/thunder-nicvf

2018-11-10 Thread Radoslaw Biernacki
ThunderX is Cavium SoC. This platform contain SRIOV NIC. Unlike other commonly known network devices it does not have VF functionality duplicated in its PF. PF is purely management device (on HW level). This creates several problems with existing libvirt code as in many places libvirt assumes

[libvirt] [PATCH 2/4] util: Code simplification

2018-11-10 Thread Radoslaw Biernacki
Removing redundant sections of the code Signed-off-by: Radoslaw Biernacki --- src/util/virnetdev.c | 40 +++- 1 file changed, 5 insertions(+), 35 deletions(-) diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c index e55c538a29..117ec41869 100644 --- a/src/util/virnetdev.c

[libvirt] [PATCH 1/4] util: fixing wrong assumption that PF has to have netdev assigned

2018-11-10 Thread Radoslaw Biernacki
libvirt wrongly assumes that VF netdev has to have the netdev assigned to PF. There is no such requirement in SRIOV standard. This patch change the virNetDevSwitchdevFeature() function to deal with SRIOV devices which does not have netdev on PF. Also removes one comment about PF netdev assumption.

[libvirt] [PATCH 3/4] util: Fix for NULL dereference

2018-11-10 Thread Radoslaw Biernacki
The device xml parser code does not set "model" while parsing virDomainDefPtr def->nets[i]->model can be NULL while latter compares strings with STREQ instead of STREQ_NULLABLE. Fixes: ac47e4a6225 (qemu: replace "def->nets[i]" with "net" and "def->sounds[i]" with "sound") Fixes: