lkml.org 
[lkml]   [2005]   [Mar]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Subject[patch] Fix e1000 driver disable interrupts bug for realtime-preempt-2.6.11-rc4-V0.7.39-02
From
Date
Hi ,Ingo

this patch fixes e1000 driver disable interrupt bug when enabling
"Complete Preemption (Realtime)".

Type: Defect Fix
Disposition: submitted to LKML
Signed-off-by: Yi Yang <yyang@ch.mvista.com>
Description: When enabling Complete Real-time Preemption, e1000 driver
always disables interrupts while calling e1000_xmit_frame, this will
lead to some serious problem, for example, the time will be skewed
because timer interrupt is also disabled, it also leads to network
packet missed, netperf's result indicates that thing is very very bad.
As a matter of fact, the reason is that
spin_unlock_irqrestore(&adapter->tx_lock) won't restore flags under the
Complete Preemption (Realtime) case, according to Real-time Preemption
regular, spin_lock_irqsave(&adapter->tx_lock) also dosen't disable
interrupts, so, local_irq_save and local_irq_restore should be changed
into local_irq_save_nort and local_irq_restore_nort, respectively.

--- a/drivers/net/e1000/e1000_main.c 2005-03-01 11:04:53.000000000
+0800
+++ b/drivers/net/e1000/e1000_main.c 2005-03-01 13:46:40.000000000
+0800
@@ -1802,10 +1802,10 @@ e1000_xmit_frame(struct sk_buff *skb, st
if(adapter->pcix_82544)
count += nr_frags;

- local_irq_save(flags);
+ local_irq_save_nort(flags);
if (!spin_trylock(&adapter->tx_lock)) {
/* Collision - tell upper layer to requeue */
- local_irq_restore(flags);
+ local_irq_restore_nort(flags);
return NETDEV_TX_LOCKED;
}

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 14:10    [W:0.046 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site