Thread (6 messages) flat view 6 messages, 3 authors, 2013-01-09

Re: [PATCH] 8139cp: Prevent dev_close/cp_interrupt race on MTU change

From: David Miller <davem@davemloft.net>
Date: 2012-12-19 20:40:15

From: John Greene <redacted>
Date: Wed, 19 Dec 2012 14:47:48 -0500
commit:  cb64edb6b89491edfdbae52ba7db9a8b8391d339 upstream

Above commit may introduce a race between cp_interrupt and dev_close
/ change MTU / dev_open up state. Changes cp_interrupt to tolerate
this.  Change spin_locking in cp_interrupt to avoid possible
but unobserved race.

Reported-by: "Francois Romieu" <romieu@fr.zoreil.com>

Tested on virtual hardware, Tx MTU size up to 4096, max tx payload
    was ping -s 4068 for MTU of 4096. No real hardware, need test
    assist.

Signed-off-by: "John Greene" <redacted>
You sent this as a "request for testing" last week, but I saw
no testing on real hardware whatsoever.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help