On Thu, 24 May 2012, Steven Rostedt wrote: > On Thu, 2012-05-24 at 04:28 +0000, Jain Priyanka-B32167 wrote: > > Waiting for review comments on this. > > > > > diff --git a/net/core/dev.c b/net/core/dev.c index 452db70..4017820 100644 > > --- a/net/core/dev.c > > +++ b/net/core/dev.c > > @@ -2940,7 +2940,7 @@ int netif_rx(struct sk_buff *skb) > > struct rps_dev_flow voidflow, *rflow = &voidflow; > > int cpu; > > > > - preempt_disable(); > > + migrate_disable(); > > I really want to avoid placing open coded migrate_disable() around the > kernel. Perhaps we should use "get_cpu_light()" here too. No. get_cpu_light() and migrate_disable() are different. Following your argument we would have to replace preempt_disable() with get_cpu() all over the place. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
