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
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
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
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
+++
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,
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
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
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
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
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
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
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
---
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
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
14 matches
Mail list logo