netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] sky2: Resolve sky2_down/transmit races, lockless transmit
@ 2009-08-03 11:07 Mike McCormack
  2009-08-03 15:52 ` Stephen Hemminger
  0 siblings, 1 reply; 2+ messages in thread
From: Mike McCormack @ 2009-08-03 11:07 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: netdev

Hi Stephen,

Here is an alternate (and more intrusive) patch set to solve the
transmit during restart issues.

These patches avoid introducing a new state variable by

* reworking the transmit queue to work without locks
* tweaking sky2_down() to avoid invalid tx queue positions being
received in the status queue

Please let me know if this looks better,

thanks,

Mike

^ permalink raw reply	[flat|nested] 2+ messages in thread
* Re: [PATCH 0/5] sky2: Resolve sky2_down/transmit races, lockless transmit
  2009-08-03 11:07 [PATCH 0/5] sky2: Resolve sky2_down/transmit races, lockless transmit Mike McCormack
@ 2009-08-03 15:52 ` Stephen Hemminger
  0 siblings, 0 replies; 2+ messages in thread
From: Stephen Hemminger @ 2009-08-03 15:52 UTC (permalink / raw)
  To: Mike McCormack; +Cc: netdev

On Mon, 3 Aug 2009 20:07:49 +0900
Mike McCormack <mikem@ring3k.org> wrote:

> Hi Stephen,
> 
> Here is an alternate (and more intrusive) patch set to solve the
> transmit during restart issues.
> 
> These patches avoid introducing a new state variable by
> 
> * reworking the transmit queue to work without locks
> * tweaking sky2_down() to avoid invalid tx queue positions being
> received in the status queue
> 
> Please let me know if this looks better,

Looks good, I will checkout on a couple of (older) boards as well.


^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-08-03 15:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-08-03 11:07 [PATCH 0/5] sky2: Resolve sky2_down/transmit races, lockless transmit Mike McCormack
2009-08-03 15:52 ` Stephen Hemminger

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).