From: Stephen Hemminger <[EMAIL PROTECTED]>
Date: Sat, 03 Nov 2007 11:43:20 -0700
> The device name was only in npinfo for netconsole target
> configuration, so move it to netconsole. Netconsole only
> needs the value during config, so no need to do all
> the device name tracking etc..
>
> Make functions for common code for instantiation and
> start up.
>
> Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]>
Sigh...
return -EUNTESTED;
In netpoll_parse_options() np->dev isn't setup yet, so if you had
tested this patch you would have gotten an immediate OOPS.
That's why it needs the npinfo->dev_name in the first place, because
it has to be parsed in order to even know what device to attach to.
The np->dev usually isn't setup until netpoll_setup() is called.
I appreciate all the work you are doing trying to clean up this beast
but you have to start testing this stuff and audit the transformations
you are making, instead of always relying on me or someone else to do
it for you.
That's why I let these particular patches sit in my inbox for two
weeks, I knew half of them would have bugs and most if not all of them
were totally untested, so I dreaded reviewing them.
I might have to toss some of the rest of these netconsole things
if there are dependencies on this bogus change.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
