All of lore.kernel.org
 help / color / mirror / Atom feed
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

-----

  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.