[lkml]   [2008]   [Mar]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] Improvev netconsole support for RTL8139 NIC driver
yshi wrote:
> David Miller 写道:
>> From: Jeff Garzik <>
>> Date: Tue, 25 Mar 2008 22:23:24 -0400
>>> This is bogus -- you should never need to slow down the hot path in
>>> such a way.
>> Slow down in what way? Even on x86 saving the flags is just
>> about as expensive as a plain sti/cli.
>> I would in fact prefer to see drivers unconditionally use
>> spin_lock_irqsave() et al. in the interrupt handler, for
>> consistency.
> Yes, I agree. Many NIC drivers do the same thing, like Gianfar, E1000, etc.

No, I just looked. Neither gianfar nor e1000 nor e1000e does this.

In fact, gfar_transmit() is precisely an example of what I'm talking
about: you only need to use spin_lock() there.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2008-03-26 04:17    [W:0.054 / U:0.352 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site