I sent you a patch yesterday , however looks like it was not delivered.
-----Original Message----- From: Dave, Tushar N Sent: Friday, February 10, 2012 12:36 PM To: Dave, Tushar N; [email protected] Cc: [email protected] Subject: [PATCH] e1000: test/debug patch for things like netconsole If things like netconsole is running and napi is enabled, tx ring should be clean explicitly to prevent it from being full of done descriptor. This patch is meant for test/debug issue [E1000-devel] x86_64 arch+SMP ON+linux-2.6.32.46+E1000+KGDBOE = Timed out. Ignoring packet error, continuing... Signed-off-by: Tushar Dave <[email protected]> --- drivers/net/e1000/e1000_main.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c index 4079a33..7b71419 100644 --- a/drivers/net/e1000/e1000_main.c +++ b/drivers/net/e1000/e1000_main.c @@ -3383,6 +3383,7 @@ static irqreturn_t e1000_intr(int irq, void *data) } else { /* this really should not happen! if it does it is basically a * bug, but not a hard error, so enable ints and continue */ + printk(KERN_INFO "napi not scheduled!!!\n"); if (!test_bit(__E1000_DOWN, &adapter->flags)) e1000_irq_enable(adapter); } @@ -4669,6 +4670,7 @@ static void e1000_netpoll(struct net_device *netdev) disable_irq(adapter->pdev->irq); e1000_intr(adapter->pdev->irq, netdev); + e1000_clean_tx_irq(adapter, &adapter->tx_ring[0]); enable_irq(adapter->pdev->irq); } #endif ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
