From: Joachim Wiberg <troglobit@gmail.com>
To: Nikolay Aleksandrov <razor@blackwall.org>,
Roopa Prabhu <roopa@nvidia.com>
Cc: netdev@vger.kernel.org, bridge@lists.linux-foundation.org,
Vladimir Oltean <vladimir.oltean@nxp.com>,
Jakub Kicinski <kuba@kernel.org>,
"David S . Miller" <davem@davemloft.net>,
Tobias Waldekranz <tobias@waldekranz.com>
Subject: Re: [Bridge] [PATCH RFC net-next 01/13] net: bridge: add control of bum flooding to bridge itself
Date: Wed, 13 Apr 2022 12:09:28 +0200 [thread overview]
Message-ID: <87czhl9twn.fsf@gmail.com> (raw)
In-Reply-To: <2cf8c40c-c122-24c0-1c01-b61da9830e9d@blackwall.org>
On Wed, Apr 13, 2022 at 12:58, Nikolay Aleksandrov <razor@blackwall.org> wrote:
> On 13/04/2022 12:51, Joachim Wiberg wrote:
>> Interesting, you mean by speculatively setting local_rcv = true and
>> postpone the decsion to br_pass_frame_up(), right? Yeah that would
>> indeed be a bit more work.
> Yes, I was thinking maybe local_rcv can become an enum with an exact reason for the
> local_rcv, so if it's > 0 do the local_rcv and br_pass_frame_up() then
> can make a proper decision without passing all of the vars. I haven't tried it,
> so not sure if it's feasible. :)
Ah, yeah that could definitely work. Thanks, I'll keep that in mind! :)
WARNING: multiple messages have this Message-ID (diff)
From: Joachim Wiberg <troglobit@gmail.com>
To: Nikolay Aleksandrov <razor@blackwall.org>,
Roopa Prabhu <roopa@nvidia.com>
Cc: netdev@vger.kernel.org, bridge@lists.linux-foundation.org,
"David S . Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>,
Tobias Waldekranz <tobias@waldekranz.com>,
Vladimir Oltean <vladimir.oltean@nxp.com>
Subject: Re: [PATCH RFC net-next 01/13] net: bridge: add control of bum flooding to bridge itself
Date: Wed, 13 Apr 2022 12:09:28 +0200 [thread overview]
Message-ID: <87czhl9twn.fsf@gmail.com> (raw)
In-Reply-To: <2cf8c40c-c122-24c0-1c01-b61da9830e9d@blackwall.org>
On Wed, Apr 13, 2022 at 12:58, Nikolay Aleksandrov <razor@blackwall.org> wrote:
> On 13/04/2022 12:51, Joachim Wiberg wrote:
>> Interesting, you mean by speculatively setting local_rcv = true and
>> postpone the decsion to br_pass_frame_up(), right? Yeah that would
>> indeed be a bit more work.
> Yes, I was thinking maybe local_rcv can become an enum with an exact reason for the
> local_rcv, so if it's > 0 do the local_rcv and br_pass_frame_up() then
> can make a proper decision without passing all of the vars. I haven't tried it,
> so not sure if it's feasible. :)
Ah, yeah that could definitely work. Thanks, I'll keep that in mind! :)
next prev parent reply other threads:[~2022-04-13 10:09 UTC|newest]
Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-11 13:38 [Bridge] [PATCH RFC net-next 00/13] net: bridge: forwarding of unknown IPv4/IPv6/MAC BUM traffic Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 01/13] net: bridge: add control of bum flooding to bridge itself Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-12 18:27 ` [Bridge] " Nikolay Aleksandrov
2022-04-12 18:27 ` Nikolay Aleksandrov
2022-04-12 20:29 ` [Bridge] " Nikolay Aleksandrov
2022-04-12 20:29 ` Nikolay Aleksandrov
2022-04-13 9:51 ` [Bridge] " Joachim Wiberg
2022-04-13 9:51 ` Joachim Wiberg
2022-04-13 9:58 ` [Bridge] " Nikolay Aleksandrov
2022-04-13 9:58 ` Nikolay Aleksandrov
2022-04-13 10:09 ` Joachim Wiberg [this message]
2022-04-13 10:09 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 02/13] net: bridge: rename br_switchdev_set_port_flag() to .._dev_flag() Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 03/13] net: bridge: minor refactor of br_setlink() for readability Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-12 18:36 ` [Bridge] " Nikolay Aleksandrov
2022-04-12 18:36 ` Nikolay Aleksandrov
2022-04-13 9:22 ` [Bridge] " Joachim Wiberg
2022-04-13 9:22 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 04/13] net: bridge: netlink support for controlling BUM flooding to bridge Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-12 18:24 ` [Bridge] " Nikolay Aleksandrov
2022-04-12 18:24 ` Nikolay Aleksandrov
2022-04-13 10:04 ` [Bridge] " Joachim Wiberg
2022-04-13 10:04 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 05/13] selftests: forwarding: add TCPDUMP_EXTRA_FLAGS to lib.sh Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 17:20 ` [Bridge] " Vladimir Oltean
2022-04-11 17:20 ` Vladimir Oltean
2022-04-12 7:39 ` [Bridge] " Joachim Wiberg
2022-04-12 7:39 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 06/13] selftests: forwarding: multiple instances in tcpdump helper Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 17:26 ` [Bridge] " Vladimir Oltean
2022-04-11 17:26 ` Vladimir Oltean
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 07/13] selftests: forwarding: new test, verify bridge flood flags Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 20:21 ` [Bridge] " Vladimir Oltean
2022-04-11 20:21 ` Vladimir Oltean
2022-04-12 7:55 ` [Bridge] " Joachim Wiberg
2022-04-12 7:55 ` Joachim Wiberg
2022-04-12 13:40 ` [Bridge] " Vladimir Oltean
2022-04-12 13:40 ` Vladimir Oltean
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 08/13] net: bridge: avoid classifying unknown multicast as mrouters_only Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-12 13:59 ` [Bridge] " Nikolay Aleksandrov
2022-04-12 13:59 ` Nikolay Aleksandrov
2022-04-12 17:27 ` [Bridge] " Joachim Wiberg
2022-04-12 17:27 ` Joachim Wiberg
2022-04-12 17:37 ` [Bridge] " Nikolay Aleksandrov
2022-04-12 17:37 ` Nikolay Aleksandrov
2022-04-13 8:51 ` [Bridge] " Joachim Wiberg
2022-04-13 8:51 ` Joachim Wiberg
2022-04-13 8:55 ` [Bridge] " Nikolay Aleksandrov
2022-04-13 8:55 ` Nikolay Aleksandrov
2022-04-13 9:00 ` [Bridge] " Nikolay Aleksandrov
2022-04-13 9:00 ` Nikolay Aleksandrov
2022-04-13 10:12 ` [Bridge] " Joachim Wiberg
2022-04-13 10:12 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 09/13] selftests: forwarding: rename test groups for next bridge mdb tests Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 20:23 ` [Bridge] " Vladimir Oltean
2022-04-11 20:23 ` Vladimir Oltean
2022-04-12 7:57 ` [Bridge] " Joachim Wiberg
2022-04-12 7:57 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 10/13] selftests: forwarding: verify flooding of unknown multicast Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 11/13] selftests: forwarding: verify strict mdb fwd of known multicast Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 12/13] selftests: forwarding: verify strict filtering doesn't leak Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
2022-04-11 13:38 ` [Bridge] [PATCH RFC net-next 13/13] selftests: forwarding: verify flood of known mc on mcast_router port Joachim Wiberg
2022-04-11 13:38 ` Joachim Wiberg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87czhl9twn.fsf@gmail.com \
--to=troglobit@gmail.com \
--cc=bridge@lists.linux-foundation.org \
--cc=davem@davemloft.net \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=razor@blackwall.org \
--cc=roopa@nvidia.com \
--cc=tobias@waldekranz.com \
--cc=vladimir.oltean@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.