Re: netpoll/netconsole minor tweaks

From: Chris Wright
Date: Thu Sep 18 2003 - 00:18:37 EST


* Matt Mackall (mpm@xxxxxxxxxxx) wrote:
> On Wed, Sep 17, 2003 at 11:24:47AM -0700, Chris Wright wrote:
> > Here's a couple small tweaks. The first is to netpoll_setup. The settle
> > time was too short for my e100, and the system would hang.
>
> This probably ought to be a command line arg, my tg3 apparently
> doesn't need it at all. One second is what's in the nfs-root stuff,
> perhaps they need to share an arg, I changed it to two because my tlan
> was dropping stuff (but not hanging - that may be a driver bug).
> Ideally, I'd like to find a way to wait until the damn thing is up
> before sending.

Yeah, I agree, that seems best.

> net/ipv4/ipconfig.c:
> /* Define the friendly delay before and after opening net devices */
> #define CONF_PRE_OPEN (HZ/2) /* Before opening: 1/2 second */
> #define CONF_POST_OPEN (1*HZ) /* After opening: 1 second */
>
> Anyway, I'm still struggling with getting stuff working on my Opteron
> box, care to take a stab at it?

Sure, I'll poke at it tomorrow, unless you get to it first.

> > The second
> > is to netconsole so that it registers a console with CON_PRINTBUFFER.
> > This helps debugging early bootup issues where you want to capture data
> > from before netconsole is initialized. Perhaps it should be a param
> > to netconsole?
>
> I think this can probably be unconditional. Merged.

Ok, cool.
thanks,
-chris
--
Linux Security Modules http://lsm.immunix.org http://lsm.bkbits.net
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/