On Sun, Nov 08, 2009 at 01:08:48AM -0800, David Miller wrote: > From: Anton Vorontsov <avoront...@ru.mvista.com> > Date: Thu, 5 Nov 2009 19:57:38 +0300 > > > But that basically means that with skb recycling we can't safely > > use KGDBoE, though we can add something like this: > > Please stop adding special logic only to your driver to handle these > things. > > Either it's a non-issue, or it's going to potentially be an issue for > everyone using skb_recycle_check() in a NAPI driver, right? > > So why not add the "in_interrupt()" or whatever check to > skb_recycle_check() and if the context is unsuitable return false (0) > ok?
I think the check is needed, yes. But if we add just that check, then we'll never do skb recycling in the tx path (since gianfar always grabs irqsave spinlock during tx ring cleanup, so the check will always return false). So we must add the check to keep polling with disabled IRQs safe, but we also should get rid of irqsave spinlock in the gianfar driver (to make the skb recycling actually work in most cases). Thanks! -- Anton Vorontsov email: cbouatmai...@gmail.com irc://irc.freenode.net/bd2 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev