Messages in this thread | | | Date | Wed, 22 Dec 2004 09:18:36 -0800 | From | Matt Mackall <> | Subject | Re: Lockup with 2.6.9-ac15 related to netconsole |
| |
On Wed, Dec 22, 2004 at 03:57:57PM +0100, Patrick McHardy wrote: > >Of course the patch is completely ugly and violates any layering > >principle one could think of. It was not submitted for inclusion :o) > > Sure, but I think we should have a short-term workaround until > a better solution has been invented. Maybe dropping the packets > would be best for now, it only affects printks issued in paths > starting at qdisc_restart (-> hard_start_xmit -> ...). Queueing > the packets might also cause reordering since not all packets > are queued.
When I mentioned queueing, I was thinking of a netpoll-private queue that would be hooked to a softirq or some such so that it would be pushed out as soon as possible. Dropping may be the better approach as queueing throws away netpoll's immediacy and ordering properties. And getting netpoll _more_ tangled in the net stack mechanics is definitely a step in the wrong direction.
More generally, I'm tempted to add some warn_on style functionality so that printks in such troublesome paths can be lifted out.
-- Mathematics is the supreme nostalgia of our time. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |