[PATCH stable] Separate multicast configuration for mesh and wlan interfaces.

David Woodhouse dwmw2 at infradead.org
Tue May 13 17:59:26 EDT 2008


On Tue, 2008-05-13 at 12:30 -0700, Andrew Morton wrote:
> On Tue, 13 May 2008 13:20:19 -0400
> Andres Salomon <dilinger at queued.net> wrote:
> 
> > On Tue, 13 May 2008 15:45:39 +0100
> > David Woodhouse <dwmw2 at infradead.org> wrote:
> > 
> > > On Tue, 2008-05-13 at 15:38 +0100, David Woodhouse wrote:
> > > > 
> > > > And even without that, it doesn't seem to do the right thing. Set
> > > > IFF_PROMISC mode on one interface, then on the other, then clear it
> > > > on the first.... it should remain set in hardware. And AFAICT it
> > > > doesn't.
> > > > 
> > > > I'll see if I can make it work....
> > > 
> > > Hm, a single cup of tea mostly achieves that; sorry :)
> > > 
> > > I was missing the fact that priv->packetfilter[] is now an array.
> > > It got a bit clearer after I reformatted it to stop trying to fit in
> > > 80 columns. Sometimes the code just doesn't fit; it's painful to try
> > > to make it.
> > > 
> > 
> > Gosh, I sure wish you, Andrew, checkpatch.pl, and Ingo[0] were all on the same page
> > regarding that.. it would sure make my life easier.
> > 
> 
> David is off in his own little world on this and can be safely ignored.
> 
> Meanwhile the rest of us are forced to stare at crocks of shit like
> http://userweb.kernel.org/~akpm/x.jpg, wondering who hates us and why.

I think the large amount of whitespace in the screenshot at
http://david.woodhou.se/narrow.png shows quite effectively why I think
you're talking nonsense on this particular topic.

-- 
dwmw2




More information about the Devel mailing list