Messages in this thread | ![/](/images/icornerl.gif) | | Subject | Re: stmmac ethernet in kernel 4.9-rc6: coalescing related pauses. | From | Lino Sanfilippo <> | Date | Mon, 5 Dec 2016 23:54:30 +0100 |
| |
On 05.12.2016 23:40, Pavel Machek wrote: > On Mon 2016-12-05 23:37:09, Lino Sanfilippo wrote: >> Hi Pavel, >> >> On 05.12.2016 23:02, Pavel Machek wrote: >> > >> > we need spin_lock_bh at minimum, as we are locking user context >> > against timer. >> > >> > Best regards, >> > Pavel >> > >> >> I was referring to stmmac_tx_clean() which AFAICS is only called from softirq context, >> (one time in the timer handler and one time in napi poll handler) so a spin_lock() should >> be sufficient. I cant see how this is called from userspace. If it were, a spin_lock_bh() had >> to be used, of course. > > stmmac_tx_clean() shares lock with stmmac_tx() -- and that's process > context as far as I can tell. So... spin_lock_bh() at > minimum... right? > > Pavel >
You mean stmmac_xmit()? Thats also softirq AFAICT, its the TX softirq....
Regards, Lino
| ![\](/images/icornerr.gif) |