[PATCH] Add explanation about the attribute "delay"

Yalan Zhang posted 1 patch 3 months, 2 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20240110045913.14389-1-yalzhang@redhat.com
docs/formatnetwork.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
[PATCH] Add explanation about the attribute "delay"
Posted by Yalan Zhang 3 months, 2 weeks ago
The libvirt created linux bridge has a configurable value "delay",
the default value is "0", but it will not take effect. That's because
kernel has a minimum value for linux bridge. Add some explanation
about it in the document.

Signed-off-by: Yalan Zhang <yalzhang@redhat.com>
---
 docs/formatnetwork.rst | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/docs/formatnetwork.rst b/docs/formatnetwork.rst
index 5335da5e11..aee70bbd0d 100644
--- a/docs/formatnetwork.rst
+++ b/docs/formatnetwork.rst
@@ -106,7 +106,8 @@ to the physical LAN (if at all).
    prefix "virbr" is recommended (and that is what is auto-generated), but not
    enforced. Attribute ``stp`` specifies if Spanning Tree Protocol is 'on' or
    'off' (default is 'on'). Attribute ``delay`` sets the bridge's forward delay
-   value in seconds (default is 0). :since:`Since 0.3.0`
+   value in seconds (default is 0. As the kernel has a minimum delay, values 
+   below it may not be counted). :since:`Since 0.3.0`
 
    The ``macTableManager`` attribute of the bridge element is used to tell
    libvirt how the bridge's MAC address table (used to determine the correct
-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org
Re: [PATCH] Add explanation about the attribute "delay"
Posted by Ján Tomko 3 months ago
On a Wednesday in 2024, Yalan Zhang wrote:
>The libvirt created linux bridge has a configurable value "delay",
>the default value is "0", but it will not take effect. That's because
>kernel has a minimum value for linux bridge. Add some explanation
>about it in the document.
>
>Signed-off-by: Yalan Zhang <yalzhang@redhat.com>
>---
> docs/formatnetwork.rst | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org