[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4F336B52.90000@profihost.ag>
Date: Thu, 09 Feb 2012 07:44:34 +0100
From: Stefan Priebe - Profihost AG <s.priebe@...fihost.ag>
To: David Miller <davem@...emloft.net>
CC: eric.dumazet@...il.com, gregkh@...uxfoundation.org,
jwboyer@...il.com, hch@...radead.org, netdev@...r.kernel.org,
david@...morbit.com, stable@...r.kernel.org
Subject: Re: BUG: unable to handle kernel NULL pointer dereference at 000000000000002c
Am 09.02.2012 02:26, schrieb David Miller:
> From: Stefan Priebe - Profihost AG <s.priebe@...fihost.ag>
> Date: Fri, 03 Feb 2012 09:09:35 +0100
>
>> Hi,
>>
>> attached you find the patch files applying cleanly to 3.0.X.
>>
>>>> I've made my own backport of the patch and removed at least 1-2
>>>> dependencies. Anybody interested?
>>>>
>>>
>>> If you did the work, post it for review.
>>
>> Thanks!
>
> I think you backported way too much, the ->lookup_neigh() dst ops
> abstraction is unnecessary, all of the neighbour hash function
> changes are also completely unnecessary.
>
> All you needed to backport is that bit that abstracts dst->neighbour()
> behind helper routines.
>
> The point of this exercise is not to keep backporting changes until
> the subsequent patch applies cleanly, it's to backport the minimal
> amount necessary to get the effect of the patch you're ultimately
> interested in adding.
>
> I'll work on coming up with something much more sensible tonight.
Thank you very much. As im pinted out i'm not a C dev nor a kernel
hacker. So i was pretty happy that it was working.
Stefan
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists