From: "Linus Lüssing" <linus.luessing@web.de>
To: David Miller <davem@davemloft.net>
Cc: b.a.t.m.a.n@lists.open-mesh.org, herbert@gondor.apana.org.au,
netdev@vger.kernel.org, bridge@lists.linux-foundation.org,
linux-kernel@vger.kernel.org, stephen@networkplumber.org
Subject: Re: [B.A.T.M.A.N.] [PATCHv3 net-next 0/4] bridge: multicast snooping patches / exports
Date: Thu, 5 Jun 2014 23:20:16 +0200 [thread overview]
Message-ID: <20140605212016.GE2544@Linus-Debian> (raw)
In-Reply-To: <20140605.004311.19890338095820543.davem@davemloft.net>
On Thu, Jun 05, 2014 at 12:43:11AM -0700, David Miller wrote:
> From: Linus Lüssing <linus.luessing@web.de>
> Date: Mon, 2 Jun 2014 20:42:15 +0200
>
> > Changes in v3:
> >
> > * use EXPORT_SYMBOL_GPL() instead of EXPORT_SYMBOL()
>
> This is not a sufficient header posting.
>
> You must also describe at a high level what this patch series is
> doing, and why you are doing it.
Oh, I'm sorry, I thought it'd only be necessary for the v1
patchset. Going to change that from now on. Anyways, here it is
(copied 'n' pasted):
-----
The first patch is simply a cosmetic patch. So far I (and maybe others
too?) have been regularly confusing these two structs, therefore I'd
suggest renaming them and therefore making the follow-up patches easier
to understand and nicer to fit in.
The second patch fixes a minor issue, but probably not worth for stable.
On the other hand the first two patches are also preparations for the
third and fourth patch:
These two patches are exporting functionality needed to marry the bridge
multicast snooping with the batman-adv multicast optimizations recently
added for the 3.15 kernel, allowing to use these optimzations in common
setups having a bridge on top of e.g. bat0, too. So far these bridged
setups would fall back to simple flooding through the batman-adv mesh
network for any multicast packet entering bat0.
More information about the batman-adv multicast optimizations currently
implemented can be found here:
http://www.open-mesh.org/projects/batman-adv/wiki/Basic-multicast-optimizations
The integration on the batman-adv side could afterwards look like this,
for instance:
http://git.open-mesh.org/batman-adv.git/commitdiff/576b59dd3e34737c702e548b21fa72059262f796?hp=f95ce7131746c65fbcdffcf2089cab59e2c2f7ac
-----
WARNING: multiple messages have this Message-ID (diff)
From: "Linus Lüssing" <linus.luessing@web.de>
To: David Miller <davem@davemloft.net>
Cc: b.a.t.m.a.n@lists.open-mesh.org, herbert@gondor.apana.org.au,
netdev@vger.kernel.org, bridge@lists.linux-foundation.org,
linux-kernel@vger.kernel.org, stephen@networkplumber.org
Subject: Re: [Bridge] [PATCHv3 net-next 0/4] bridge: multicast snooping patches / exports
Date: Thu, 5 Jun 2014 23:20:16 +0200 [thread overview]
Message-ID: <20140605212016.GE2544@Linus-Debian> (raw)
In-Reply-To: <20140605.004311.19890338095820543.davem@davemloft.net>
On Thu, Jun 05, 2014 at 12:43:11AM -0700, David Miller wrote:
> From: Linus Lüssing <linus.luessing@web.de>
> Date: Mon, 2 Jun 2014 20:42:15 +0200
>
> > Changes in v3:
> >
> > * use EXPORT_SYMBOL_GPL() instead of EXPORT_SYMBOL()
>
> This is not a sufficient header posting.
>
> You must also describe at a high level what this patch series is
> doing, and why you are doing it.
Oh, I'm sorry, I thought it'd only be necessary for the v1
patchset. Going to change that from now on. Anyways, here it is
(copied 'n' pasted):
-----
The first patch is simply a cosmetic patch. So far I (and maybe others
too?) have been regularly confusing these two structs, therefore I'd
suggest renaming them and therefore making the follow-up patches easier
to understand and nicer to fit in.
The second patch fixes a minor issue, but probably not worth for stable.
On the other hand the first two patches are also preparations for the
third and fourth patch:
These two patches are exporting functionality needed to marry the bridge
multicast snooping with the batman-adv multicast optimizations recently
added for the 3.15 kernel, allowing to use these optimzations in common
setups having a bridge on top of e.g. bat0, too. So far these bridged
setups would fall back to simple flooding through the batman-adv mesh
network for any multicast packet entering bat0.
More information about the batman-adv multicast optimizations currently
implemented can be found here:
http://www.open-mesh.org/projects/batman-adv/wiki/Basic-multicast-optimizations
The integration on the batman-adv side could afterwards look like this,
for instance:
http://git.open-mesh.org/batman-adv.git/commitdiff/576b59dd3e34737c702e548b21fa72059262f796?hp=f95ce7131746c65fbcdffcf2089cab59e2c2f7ac
-----
WARNING: multiple messages have this Message-ID (diff)
From: "Linus Lüssing" <linus.luessing@web.de>
To: David Miller <davem@davemloft.net>
Cc: netdev@vger.kernel.org, bridge@lists.linux-foundation.org,
stephen@networkplumber.org, herbert@gondor.apana.org.au,
linux-kernel@vger.kernel.org, b.a.t.m.a.n@lists.open-mesh.org
Subject: Re: [PATCHv3 net-next 0/4] bridge: multicast snooping patches / exports
Date: Thu, 5 Jun 2014 23:20:16 +0200 [thread overview]
Message-ID: <20140605212016.GE2544@Linus-Debian> (raw)
In-Reply-To: <20140605.004311.19890338095820543.davem@davemloft.net>
On Thu, Jun 05, 2014 at 12:43:11AM -0700, David Miller wrote:
> From: Linus Lüssing <linus.luessing@web.de>
> Date: Mon, 2 Jun 2014 20:42:15 +0200
>
> > Changes in v3:
> >
> > * use EXPORT_SYMBOL_GPL() instead of EXPORT_SYMBOL()
>
> This is not a sufficient header posting.
>
> You must also describe at a high level what this patch series is
> doing, and why you are doing it.
Oh, I'm sorry, I thought it'd only be necessary for the v1
patchset. Going to change that from now on. Anyways, here it is
(copied 'n' pasted):
-----
The first patch is simply a cosmetic patch. So far I (and maybe others
too?) have been regularly confusing these two structs, therefore I'd
suggest renaming them and therefore making the follow-up patches easier
to understand and nicer to fit in.
The second patch fixes a minor issue, but probably not worth for stable.
On the other hand the first two patches are also preparations for the
third and fourth patch:
These two patches are exporting functionality needed to marry the bridge
multicast snooping with the batman-adv multicast optimizations recently
added for the 3.15 kernel, allowing to use these optimzations in common
setups having a bridge on top of e.g. bat0, too. So far these bridged
setups would fall back to simple flooding through the batman-adv mesh
network for any multicast packet entering bat0.
More information about the batman-adv multicast optimizations currently
implemented can be found here:
http://www.open-mesh.org/projects/batman-adv/wiki/Basic-multicast-optimizations
The integration on the batman-adv side could afterwards look like this,
for instance:
http://git.open-mesh.org/batman-adv.git/commitdiff/576b59dd3e34737c702e548b21fa72059262f796?hp=f95ce7131746c65fbcdffcf2089cab59e2c2f7ac
-----
WARNING: multiple messages have this Message-ID (diff)
From: "Linus Lüssing" <linus.luessing@web.de>
To: David Miller <davem@davemloft.net>
Cc: b.a.t.m.a.n@lists.open-mesh.org, herbert@gondor.apana.org.au,
netdev@vger.kernel.org, bridge@lists.linux-foundation.org,
linux-kernel@vger.kernel.org, stephen@networkplumber.org
Subject: Re: [PATCHv3 net-next 0/4] bridge: multicast snooping patches / exports
Date: Thu, 5 Jun 2014 23:20:16 +0200 [thread overview]
Message-ID: <20140605212016.GE2544@Linus-Debian> (raw)
In-Reply-To: <20140605.004311.19890338095820543.davem@davemloft.net>
On Thu, Jun 05, 2014 at 12:43:11AM -0700, David Miller wrote:
> From: Linus Lüssing <linus.luessing@web.de>
> Date: Mon, 2 Jun 2014 20:42:15 +0200
>
> > Changes in v3:
> >
> > * use EXPORT_SYMBOL_GPL() instead of EXPORT_SYMBOL()
>
> This is not a sufficient header posting.
>
> You must also describe at a high level what this patch series is
> doing, and why you are doing it.
Oh, I'm sorry, I thought it'd only be necessary for the v1
patchset. Going to change that from now on. Anyways, here it is
(copied 'n' pasted):
-----
The first patch is simply a cosmetic patch. So far I (and maybe others
too?) have been regularly confusing these two structs, therefore I'd
suggest renaming them and therefore making the follow-up patches easier
to understand and nicer to fit in.
The second patch fixes a minor issue, but probably not worth for stable.
On the other hand the first two patches are also preparations for the
third and fourth patch:
These two patches are exporting functionality needed to marry the bridge
multicast snooping with the batman-adv multicast optimizations recently
added for the 3.15 kernel, allowing to use these optimzations in common
setups having a bridge on top of e.g. bat0, too. So far these bridged
setups would fall back to simple flooding through the batman-adv mesh
network for any multicast packet entering bat0.
More information about the batman-adv multicast optimizations currently
implemented can be found here:
http://www.open-mesh.org/projects/batman-adv/wiki/Basic-multicast-optimizations
The integration on the batman-adv side could afterwards look like this,
for instance:
http://git.open-mesh.org/batman-adv.git/commitdiff/576b59dd3e34737c702e548b21fa72059262f796?hp=f95ce7131746c65fbcdffcf2089cab59e2c2f7ac
-----
next prev parent reply other threads:[~2014-06-05 21:20 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-02 18:07 [B.A.T.M.A.N.] [PATCHv3 net-next 1/4] bridge: rename struct bridge_mcast_query/querier Linus Lüssing
2014-06-02 18:07 ` Linus Lüssing
2014-06-02 18:07 ` Linus Lüssing
2014-06-02 18:07 ` [Bridge] " Linus Lüssing
2014-06-02 18:07 ` [B.A.T.M.A.N.] [PATCHv3 net-next 2/4] bridge: adhere to querier election mechanism specified by RFCs Linus Lüssing
2014-06-02 18:07 ` Linus Lüssing
2014-06-02 18:07 ` Linus Lüssing
2014-06-02 18:07 ` [Bridge] " Linus Lüssing
2014-06-02 18:07 ` [B.A.T.M.A.N.] [PATCHv3 net-next 3/4] bridge: add export of multicast database adjacent to net_dev Linus Lüssing
2014-06-02 18:07 ` Linus Lüssing
2014-06-02 18:07 ` [Bridge] " Linus Lüssing
2014-06-02 18:07 ` [B.A.T.M.A.N.] [PATCHv3 net-next 4/4] bridge: memorize and export selected IGMP/MLD querier port Linus Lüssing
2014-06-02 18:07 ` Linus Lüssing
2014-06-02 18:07 ` [Bridge] " Linus Lüssing
2014-06-02 18:42 ` [B.A.T.M.A.N.] [PATCHv3 net-next 0/4] bridge: multicast snooping patches / exports Linus Lüssing
2014-06-02 18:42 ` Linus Lüssing
2014-06-02 18:42 ` Linus Lüssing
2014-06-02 18:42 ` [Bridge] " Linus Lüssing
2014-06-05 7:43 ` [B.A.T.M.A.N.] " David Miller
2014-06-05 7:43 ` David Miller
2014-06-05 7:43 ` David Miller
2014-06-05 7:43 ` [Bridge] " David Miller
2014-06-05 21:20 ` Linus Lüssing [this message]
2014-06-05 21:20 ` Linus Lüssing
2014-06-05 21:20 ` Linus Lüssing
2014-06-05 21:20 ` [Bridge] " Linus Lüssing
2014-06-05 21:50 ` [B.A.T.M.A.N.] " David Miller
2014-06-05 21:50 ` David Miller
2014-06-05 21:50 ` David Miller
2014-06-05 21:50 ` [Bridge] " David Miller
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=20140605212016.GE2544@Linus-Debian \
--to=linus.luessing@web.de \
--cc=b.a.t.m.a.n@lists.open-mesh.org \
--cc=bridge@lists.linux-foundation.org \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=stephen@networkplumber.org \
/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.