Thread (8 messages) 8 messages, 6 authors, 2016-08-10

Re: [PATCH net] bridge: Fix problems around fdb entries pointing to the bridge device

From: Toshiaki Makita <hidden>
Date: 2016-08-04 10:11:19
Also in: bridge

On 2016/08/04 17:15, Toshiaki Makita wrote:
On 2016/08/04 16:24, Roopa Prabhu wrote:
quoted
On 8/3/16, 7:11 PM, Toshiaki Makita wrote:
quoted
Adding fdb entries pointing to the bridge device uses fdb_insert(),
which lacks various checks and does not respect added_by_user flag.

As a result, some inconsistent behavior can happen:
* Adding temporary entries succeeds but results in permanent entries.
IIRC this is not specific to fdb entries on the bridge device. all temp
fdb entries via iproute2 result in permanent entries. thats why 'dynamic'
was added.
Sorry for confusing you, I meant "temp" (i.e., "static") of bridge fdb
command.
"temp", "dynamic" and "use" should not result in "permanent".
I probably misread you in my previous reply so the previous answer looks
weird...

What I should have said is:
Other temp fdb entries via iproute2 result in static (== temp), not
permanent.
"dynamic" and "use" should be meant for dynamic entries, not static (temp).

-- 
Toshiaki Makita
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help