Re: [RFT][PATCH] cleanup net/atm/br2684.c

[Mitchell Blank Jr <mitch@xxxxxxxxxx>]

Stephen Hemminger wrote:
> > Mmmm.. I see this snippet in your diff in multiplce places and it looks
> > wrong.  Where does the "brdev" structure get freed now?
> > 
> The brdev structure is contained inside the allocation of net_device,
> this is how all the ether and other devices do it.

OK, sorry... that should have been obvious.  The reason I was confused is
that almost always the net_dev is referenced as brdev->net_dev which makes
look like brdev merely has a reference to net_dev.  I think it would be
cleaner to change the code to pass around "net_dev" instead of "brdev"
references (and just get the brdev via net_dev->priv as needed) and just
get rid of the brdev->net_dev pointer entirely.

Are you interested in doing that?  Otherwise I could take a crack at it
when I get the chance (I've done some surgery on this file before)

-Mitch