diff options
author | Vlad Yasevich <vyasevich@gmail.com> | 2015-11-10 06:15:32 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-11-10 15:41:36 -0500 |
commit | 8a921265e2cd31e61a0c2eda582af54c5bfef897 (patch) | |
tree | 39fcefd08dd897840cc92ea1969cf354046aa06d /security | |
parent | a31d82d85afdcbdb8c4128dfd6146992dc6b3576 (diff) | |
download | linux-8a921265e2cd31e61a0c2eda582af54c5bfef897.tar.gz linux-8a921265e2cd31e61a0c2eda582af54c5bfef897.tar.bz2 linux-8a921265e2cd31e61a0c2eda582af54c5bfef897.zip |
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>
Diffstat (limited to 'security')
0 files changed, 0 insertions, 0 deletions