]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
Revert "bridge: Allow forward delay to be cfgd when STP enabled"
authorVlad Yasevich <vyasevich@gmail.com>
Tue, 10 Nov 2015 11:15:32 +0000 (06:15 -0500)
committerDavid S. Miller <davem@davemloft.net>
Tue, 10 Nov 2015 20:41:36 +0000 (15:41 -0500)
commit8a921265e2cd31e61a0c2eda582af54c5bfef897
tree39fcefd08dd897840cc92ea1969cf354046aa06d
parenta31d82d85afdcbdb8c4128dfd6146992dc6b3576
Revert "bridge: Allow forward delay to be cfgd when STP enabled"

This reverts commit 34c2d9fb0498c066afbe610b15e18995fd8be792.

There are 2 reasons for this revert:
 1)  The commit in question doesn't do what it says it does.  The
     description reads: "Allow bridge forward delay to be configured
     when Spanning Tree is enabled."  This was already the case before
     the commit was made.  What the commit actually do was disallow
     invalid values or 'forward_delay' when STP was turned off.

 2)  The above change was actually a change in the user observed
     behavior and broke things like libvirt and other network configs
     that set 'forward_delay' to 0 without enabling STP.  The value
     of 0 is actually used when STP is turned off to immediately mark
     the bridge as forwarding.

Signed-off-by: Vlad Yasevich <vyasevic@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_stp.c