netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] netxen: fix poll implementation
       [not found] <AANLkTikHt2tY7qPqfeXaRUqMudV1vZ3Wo46D=mxcuD2f@mail.gmail.com>
@ 2010-08-23  1:43 ` David Miller
  2010-08-23  9:04   ` Yinglin Luan
  0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2010-08-23  1:43 UTC (permalink / raw)
  To: synmyth; +Cc: netdev, monstr


Both of your patches have been severely corrupted by your email
client.

For example, tab characters have been changed into spaces.

This makes your patches unusable.

Please read Documentation/email-clients.txt to learn how to correct
this, try sending a patch to yourself and verify that it arrives
to you correctly and can be applied properly, then resubmit your
patches.

Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] netxen: fix poll implementation
@ 2010-08-23  7:56 Yinglin Luan
  2010-08-24  3:29 ` David Miller
  0 siblings, 1 reply; 4+ messages in thread
From: Yinglin Luan @ 2010-08-23  7:56 UTC (permalink / raw)
  To: netdev; +Cc: davem, monstr

Function netxen_intr has pointer to nx_host_sds_ring
as second parameter not pointer to netxen_adapter.

Signed-off-by: Yinglin Luan <synmyth@gmail.com>
---
 drivers/net/netxen/netxen_nic_main.c |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/drivers/net/netxen/netxen_nic_main.c b/drivers/net/netxen/netxen_nic_main.c
index fd86e18..4147be5 100644
--- a/drivers/net/netxen/netxen_nic_main.c
+++ b/drivers/net/netxen/netxen_nic_main.c
@@ -2133,9 +2133,16 @@ static int netxen_nic_poll(struct napi_struct *napi, int budget)
 #ifdef CONFIG_NET_POLL_CONTROLLER
 static void netxen_nic_poll_controller(struct net_device *netdev)
 {
+	int ring;
+	struct nx_host_sds_ring *sds_ring;
 	struct netxen_adapter *adapter = netdev_priv(netdev);
+	struct netxen_recv_context *recv_ctx = &adapter->recv_ctx;
+
 	disable_irq(adapter->irq);
-	netxen_intr(adapter->irq, adapter);
+	for (ring = 0; ring < adapter->max_sds_rings; ring++) {
+		sds_ring = &recv_ctx->sds_rings[ring];
+		netxen_intr(adapter->irq, sds_ring);
+	}
 	enable_irq(adapter->irq);
 }
 #endif
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] netxen: fix poll implementation
  2010-08-23  1:43 ` David Miller
@ 2010-08-23  9:04   ` Yinglin Luan
  0 siblings, 0 replies; 4+ messages in thread
From: Yinglin Luan @ 2010-08-23  9:04 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, monstr

> 
> Both of your patches have been severely corrupted by your email
> client.
> 
> For example, tab characters have been changed into spaces.
> 
> This makes your patches unusable.
> 
> Please read Documentation/email-clients.txt to learn how to correct
> this, try sending a patch to yourself and verify that it arrives
> to you correctly and can be applied properly, then resubmit your
> patches.
> 
> Thanks.
Sorry, it's my first time to submit patches to kernel community. I
use Gmail web client, and it automatically converts tabs to spaces.
I change my e-mail client to Thunderbird and configure it as
Documentation/email-clients.txt says. It shouled be applied properly, 
I have resubmit it just now, please check it again. Thanks a lot!


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] netxen: fix poll implementation
  2010-08-23  7:56 [PATCH] netxen: fix poll implementation Yinglin Luan
@ 2010-08-24  3:29 ` David Miller
  0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2010-08-24  3:29 UTC (permalink / raw)
  To: synmyth; +Cc: netdev, monstr

From: Yinglin Luan <synmyth@gmail.com>
Date: Mon, 23 Aug 2010 15:56:19 +0800

> Function netxen_intr has pointer to nx_host_sds_ring
> as second parameter not pointer to netxen_adapter.
> 
> Signed-off-by: Yinglin Luan <synmyth@gmail.com>

Applied.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-08-24  3:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-08-23  7:56 [PATCH] netxen: fix poll implementation Yinglin Luan
2010-08-24  3:29 ` David Miller
     [not found] <AANLkTikHt2tY7qPqfeXaRUqMudV1vZ3Wo46D=mxcuD2f@mail.gmail.com>
2010-08-23  1:43 ` David Miller
2010-08-23  9:04   ` Yinglin Luan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).