When we use the 'ovs-appctl rstp/show', the root bridge
of rstp is always 'unknown root port'. We don't expect
that. The reason is that the committer added the check
for var 'p'. In the rstp, if a bridge is root bridge,
there is not root port, and we don't use the root port
'p', 'rstp/show' in the same case. If we check only rstp
root port, the root info will not shown any more.

CC: Ben Pfaff <[email protected]>
Signed-off-by: nickcooper-zhangtonghao <[email protected]>
---
 lib/rstp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/rstp.c b/lib/rstp.c
index 9280b3a..b30f930 100644
--- a/lib/rstp.c
+++ b/lib/rstp.c
@@ -1602,7 +1602,7 @@ rstp_print_details(struct ds *ds, const struct rstp *rstp)
 
     bool is_root = rstp_is_root_bridge__(rstp);
     struct rstp_port *p = rstp_get_root_port__(rstp);
-    if (!p) {
+    if (!is_root && !p) {
         ds_put_cstr(ds, "unknown root port\n");
         return;
     }
-- 
1.8.3.1




_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to