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

From: Kay Sievers
Date: Wed Aug 22 2007 - 16:52:22 EST


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

-
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/