[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080714212620.GA8503@orion.carnet.hr>
Date: Mon, 14 Jul 2008 23:26:20 +0200
From: Josip Rodin <joy@...uzijast.net>
To: Jarek Poplawski <jarkao2@...il.com>
Cc: Patrick McHardy <kaber@...sh.net>,
David Miller <davem@...emloft.net>, mchan@...adcom.com,
billfink@...dspring.com, bhutchings@...arflare.com,
netdev@...r.kernel.org, mirrors@...ian.org, devik@....cz
Subject: Re: bnx2_poll panicking kernel
On Mon, Jul 14, 2008 at 11:22:23PM +0200, Jarek Poplawski wrote:
> On Mon, Jul 14, 2008 at 10:21:51PM +0200, Josip Rodin wrote:
> > On Mon, Jul 14, 2008 at 07:20:55PM +0200, Jarek Poplawski wrote:
> ...
> > > BTW, I wonder how Josip's testing?
> >
> > Do you want me to try out David's patch? I was hoping to get a definite okay
> > from someone before applying it, given that this is a production machine
> > for us and we do need it in *some* kind of an operational state.
>
> Hmm... if you trust me more than David?! As a matter of fact, I thought
> it's under testing already, but since I've some doubts, I attach below
> my version of David's patch. IMHO, it looks safe, but never knows...
No (sorry :) but he did call his patch an "idiotic attempt", and asked you
guys for input, so I held back :)
> > I've got many more tens of kilobytes of logs from the previous debugging
> > patches, if you want I can send them over.
>
> This debugging, I guess, shows corrupted data, but the reason is hard
> to find. David found one of possible reasons and it needs checking. I
> don't think this patch in any version can do more damage than doing
> nothing - unless you prefer to use Michael's patch to next kernels
> (BTW, don't remove this patch yet).
OK, I'll give it a whirl.
(Should I update to .26 at the same time, now that it went live in the
meantime?)
--
2. That which causes joy or happiness.
--
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