Re: [PATCH] net: Adjust softirq raising in __napi_schedule
- To: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
- Subject: Re: [PATCH] net: Adjust softirq raising in __napi_schedule
- From: Jarek Poplawski <jarkao2@xxxxxxxxx>
- Date: Mon, 26 Oct 2009 07:41:26 +0000
- Cc: Tilman Schmidt <tilman@xxxxxxx>, David Miller <davem@xxxxxxxxxxxxx>, hidave.darkstar@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, tglx@xxxxxxxxxxxxx, linux-wireless@xxxxxxxxxxxxxxx, linux-ppp@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, paulus@xxxxxxxxx, Michael Buesch <mb@xxxxxxxxx>, Oliver Hartkopp <oliver@xxxxxxxxxxxx>
- In-reply-to: <1256309191.12174.51.camel@xxxxxxxxxxxxxx>
- User-agent: Mutt/1.5.18 (2008-05-17)
On Fri, Oct 23, 2009 at 04:46:31PM +0200, Johannes Berg wrote:
> On Fri, 2009-10-23 at 16:39 +0200, Tilman Schmidt wrote:
>
> > Strange. Then what are the two separate functions netif_rx() and
> > netif_rx_ni() for?
>
> netif_rx_ni() disables preemption.
You wrote earlier:
> [...] the networking layer needs to have
> packets handed to it with softirqs disabled.
How disabling preemption can fix something which needs softirqs
disabled? Could you be more precise?
> > > This really should be obvious. You're fixing the warning at the source
> > > of the warning, rather than the source of the problem.
> >
> > Good idea. So please do tell us where the source of the problem is.
>
> You use netif_rx_ni() instead of netif_rx() at whatever place that
> causes this problem.
This isn't a very precise description either.
Jarek P.
--
To unsubscribe from this list: send the line "unsubscribe linux-ppp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Linux Audio Users]
[Hams]
[Kernel Newbies]
[Security]
[Netfilter]
[Bugtraq]
[Photo]
[Yosemite Photos]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Samba]
[Video 4 Linux]
[Linux Resources]
[Fedora Users]