> Hi Andrew, > > Yes, I wasn't aware of the 100 lines limit, and yes, this is not a bug fix, > but an improvement (optimisation). > > I think by this I can join to consensus, this patch is not a candidate for > backporting. :-/ > > However, I am concerned about the possibility of two kernel threads accessing > the RTL NIC intermittently attempting to program the NIC over the RTL_(R|W)(8|16|32) > instructions (which are expanded to readl/writel and assembly). Most calls into the driver are protected by the RTNL lock. There are a few exceptions. probe() obviously does not hold RTNL. Actually sending packets, and interrupt handlers don't hold RTNL. Please look at the code and see if you can see any paths which might do parallel access without holding RTNL. You could also do some testing. Add ASSERT_RTNL() in the code you are worried about. If the lock is not held, you will get a stack trace. Andrew
