* [PATCHv3 RFC net-next 0/4] sunvnet: NAPIfy sunvnet
@ 2014-10-15 16:42 Sowmini Varadhan
2014-10-22 18:57 ` Sowmini Varadhan
0 siblings, 1 reply; 3+ messages in thread
From: Sowmini Varadhan @ 2014-10-15 16:42 UTC (permalink / raw)
To: davem, bob.picco, sowmini.varadhan, dwight.engen,
raghuram.kothakota
Cc: netdev, sparclinux
This patchset converts the sunvnet driver to use the NAPI framework.
Patch 1 in the series addresses the packet-receive path- all
the vnet_event() processing is moved into NAPI context.
This patch is dependant on the sparc-next commit:
"sparc64: Add vio_set_intr() to enable/disable Rx interrupts"
(Please cherry-pick sparc commit id ca605b7dd740c8909408d67911d8ddd272c2b320)
Patch 2 uses RCU to fix race conditions between vnet_port_remove and
paths that access/modify port-related state, such as vnet_start_xmit.
Patch 3 and Patch 4 in the series leverage from the NAPIfied Rx path,
dropping superfluous usage of the irqsave/irqrestores on the vio.lock
where possible.
Note: Patch 3 contains changes that target sparc-next, Patch 4 targets
net-next.
Sowmini Varadhan (4):
NAPIfy sunvnet
Use RCU to synchronize port usage with vnet_port_remove()
Avoid irqsave/restore on vio.lock if in_softirq()
Remove irqsave/irqrestore on vio.lock
arch/sparc/kernel/viohs.c | 8 +-
drivers/net/ethernet/sun/sunvnet.c | 261 +++++++++++++++++++++++--------------
drivers/net/ethernet/sun/sunvnet.h | 6 +-
3 files changed, 176 insertions(+), 99 deletions(-)
--
1.8.4.2
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCHv3 RFC net-next 0/4] sunvnet: NAPIfy sunvnet
2014-10-15 16:42 [PATCHv3 RFC net-next 0/4] sunvnet: NAPIfy sunvnet Sowmini Varadhan
@ 2014-10-22 18:57 ` Sowmini Varadhan
2014-10-22 19:39 ` David Miller
0 siblings, 1 reply; 3+ messages in thread
From: Sowmini Varadhan @ 2014-10-22 18:57 UTC (permalink / raw)
To: davem, bob.picco, dwight.engen, raghuram.kothakota; +Cc: netdev, sparclinux
Please hold off on this patch-set for the moment.
I just discovered a subtle bug that can be triggered when multiple
ldc events (e.g., UP and DATA_READY) are passed up from
ldc_rx -> send_events in one shot- we need to treat the vnet_port's
(new field) rx_event as a bit mask, and need to unroll send_events()
into vnet_event_napi.
the result of this bug is that inter-vnet handshake can sometimes
fail incorrectly causing things to needlessly go thorugh the
switchport. I'm working on the fix for this, I'll send out
PATCHv5 for this series shortly.
Apologies for the confusion.
--Sowmini
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCHv3 RFC net-next 0/4] sunvnet: NAPIfy sunvnet
2014-10-22 18:57 ` Sowmini Varadhan
@ 2014-10-22 19:39 ` David Miller
0 siblings, 0 replies; 3+ messages in thread
From: David Miller @ 2014-10-22 19:39 UTC (permalink / raw)
To: sowmini.varadhan
Cc: bob.picco, dwight.engen, raghuram.kothakota, netdev, sparclinux
From: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Date: Wed, 22 Oct 2014 14:57:57 -0400
> Please hold off on this patch-set for the moment.
Ok, no problem.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-10-22 19:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-15 16:42 [PATCHv3 RFC net-next 0/4] sunvnet: NAPIfy sunvnet Sowmini Varadhan
2014-10-22 18:57 ` Sowmini Varadhan
2014-10-22 19:39 ` David Miller
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).