[PATCH 0/1] Fix the bug about virsh attach-interface from an ovs

2021-11-02 Thread jx8zjs
bridge_driver: use ovs-vsctl to setup and clean Qos when using an OVS bridge Fix bug 1826168: bridge type network with ovs bridge can start with Qos setting which do not take any effect Resolves:https://bugzilla.redhat.com/show_bug.cgi?id=1826168 zhangjl02 (1): bridge_driver: use ovs-vsctl to

[PATCH 1/1] bridge_driver: use ovs-vsctl to setup and clean Qos when

2021-11-02 Thread jx8zjs
From: zhangjl02 Fix bug 1826168: bridge type network with ovs bridge can start with Qos setting which do not take any effect Resolves:https://bugzilla.redhat.com/show_bug.cgi?id=1826168 Signed-off-by: zhangjl02 --- src/network/bridge_driver.c | 65 +++-- 1 file

[PATCH 2/2] virnetdevopenvswitchtest: fix inbound value calculation

2021-11-01 Thread jx8zjs
From: zhangjl02 Signed-off-by: zhangjl02 --- tests/virnetdevopenvswitchtest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/virnetdevopenvswitchtest.c b/tests/virnetdevopenvswitchtest.c index 2a20ba82d0..57860f4bc4 100644 --- a/tests/virnetdevopenvswitchtest.c

[PATCH 1/2] virnetdevopenvswitch: unify calculation of ovs and tc. For other interface type, values in tc rules are calculated by multiply by 8*1000 instead of 8*1024.

2021-11-01 Thread jx8zjs
From: zhangjl02 Signed-off-by: zhangjl02 --- src/util/virnetdevopenvswitch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virnetdevopenvswitch.c b/src/util/virnetdevopenvswitch.c index 8955f0fa66..250726a633 100644 --- a/src/util/virnetdevopenvswitch.c +++

[PATCH 0/2] Unify calculation of ovs and tc.

2021-11-01 Thread jx8zjs
For other interface type, values in tc rules are calculated by multiply by 8*1000 instead of 8*1024. Related link: https://listman.redhat.com/archives/libvir-list/2021-October/msg01121.html zhangjl02 (2): virnetdevopenvswitch: unify calculation of ovs and tc. For other interface type,

[PATCH 0/1] Fix the bug about virsh attach-interface from a

2021-11-01 Thread jx8zjs
virsh: inherit qos from network definition when attaching an interface Resolved: https://bugzilla.redhat.com/show_bug.cgi?id=1826168 jx8zjs (1): virsh: inherit qos from network definition when attaching an interface tools/virsh-domain.c | 34 +++--- 1 file changed

[PATCH 1/1] virsh: inherit qos from network definition when attaching an interface

2021-11-01 Thread jx8zjs
Fix bug 1826168: bridge type network with ovs bridge can start with Qos setting which do not take any effect Resolves:https://bugzilla.redhat.com/show_bug.cgi?id=1826168 Signed-off-by: jx8zjs --- tools/virsh-domain.c | 34 +++--- 1 file changed, 31 insertions(+), 3

[PATCHv2 1/6] virnetdevovs: Add vmuuid notes on virNetDevOpenvswitchInterfaceSetQos

2021-08-16 Thread jx8zjs
From: Jinsheng Zhang Add vmuuid notes on virNetDevOpenvswitchInterfaceSetQos, and change vmid to vmuuid. Signed-off-by: Jinsheng Zhang --- src/util/virnetdevopenvswitch.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/util/virnetdevopenvswitch.c

[PATCHv2 3/6] virnetdevovs: Extract common code block to a single function

2021-08-16 Thread jx8zjs
From: Jinsheng Zhang Signed-off-by: zhangjl02 s Signed-off-by: zhangjl02 s Signed-off-by: zhangjl02 --- src/util/virnetdevopenvswitch.c | 60 ++--- 1 file changed, 26 insertions(+), 34 deletions(-) diff --git a/src/util/virnetdevopenvswitch.c

[PATCHv2 5/6] virnetdevovs: Fix qos cleaning residual on multi interfaces

2021-08-16 Thread jx8zjs
From: Jinsheng Zhang Warn these error instead of return when removing qos or queues. This will avoid residual qos clearance on multiple interfaces. Signed-off-by: zhangjl02 --- src/util/virnetdevopenvswitch.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git

[PATCHv2 4/6] virnetdevovs: Introduce virNetDevOpenvswitchInterfaceClearTxQos and virNetDevOpenvswitchInterfaceClearRxQos

2021-08-16 Thread jx8zjs
From: Jinsheng Zhang Separate virNetDevOpenvswitchInterfaceClearQos into two steps. When setting qos, we can set only rx or tx and the other one should be cleared. Signed-off-by: zhangjl02 --- src/libvirt_private.syms| 2 ++ src/util/virnetdevopenvswitch.c | 50

[PATCHv2 6/6] tests: add test on virNetDevOpenvswitchInterfaceSetQos and virNetDevOpenvswitchInterfaceClearQos

2021-08-16 Thread jx8zjs
From: Jinsheng Zhang Test virNetDevOpenvswitchInterfaceSetQos and virNetDevOpenvswitchInterfaceClearQos with dryrun method. Signed-off-by: zhangjl02 --- tests/virnetdevopenvswitchtest.c | 196 ++- 1 file changed, 195 insertions(+), 1 deletion(-) diff --git

[PATCHv2 0/6] virDomain: fix problems when setting qos on ovs managed

2021-08-16 Thread jx8zjs
--- diff to v1: - Modify commit message to no longer then 80 characters. - Split patch into several commits - Add notes on some function and defination - Extract common code block to a single function Thanks to Pavel Hrdina for helping to review these patches. Two problems are found

[PATCHv2 2/6] virnetdevovs: Extract conversion parameters between virNetDevBandwidth and ovs

2021-08-16 Thread jx8zjs
From: Jinsheng Zhang Signed-off-by: zhangjl02 --- src/util/virnetdevopenvswitch.c | 12 +++- src/util/virnetdevopenvswitch.h | 7 +++ 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/src/util/virnetdevopenvswitch.c b/src/util/virnetdevopenvswitch.c index