Re: [BUG] 2.6.23-rc3-mm1 - kernel BUG at net/core/skbuff.c:95!

[Kay Sievers <kay.sievers@vrfy.org>]

On Thu, 2007-08-23 at 00:34 +0530, Balbir Singh wrote:
> Kay Sievers wrote:
> >> gargh, sorry, that's probably due to my screwed up attempt to fix Kay's
> >> screwed up
> >> gregkh-driver-driver-core-change-add_uevent_var-to-use-a-struct.patch.
> >>
> >> Kay sent an update patch but it didn't arrive in time.
> >>
> >> Greg, if you haven't yet merged that, please do so asap?
> >>
> >> So what _should_ this:
> >>
> >> --- a/arch/powerpc/kernel/vio.c~fix-4-gregkh-driver-driver-core-change-add_uevent_var-to-use-a-struct
> >> +++ a/arch/powerpc/kernel/vio.c
> >> @@ -373,7 +373,7 @@ static int vio_hotplug(struct device *de
> >>  	dn = dev->archdata.of_node;
> >>  	if (!dn)
> >>  		return -ENODEV;
> >> -	cp = of_get_property(dn, "compatible", &length);
> >> +	cp = of_get_property(dn, "compatible", &env->buflen);
> >>  	if (!cp)
> >>  		return -ENODEV;
> >>  
> >> _
> >>
> >> have done?
> > 
> > Does replacing "&length" with "NULL" work? That's what's in the updated
> > patch.

> replacing &length with NULL does not work for me. I get a message saying that
> init terminated with signal 7.

Hi Balbir,
ugh, I can't see what's going wrong here.

Care to just "return 0" for the whole function, and try again? Just to
rule out that this is the cause of the problem.

Thanks,
Kay