Thread: [zd1211-devs] [PATCH] Fix races on preempt / multiprocessor
Status: Beta
Brought to you by:
mayne
From: Guennadi L. <g.l...@gm...> - 2005-11-30 07:44:27
Attachments:
races-20051129.diff
|
Hello, list This patch removes wrong locking in disconnect method, around functions, that call schedule(), and attempts to fix races there. It also fixes bugs in timer handling, which produce Oopses on SMP. I am using it for 2 days now on a UP machine with preemption and preemption debugging. Will also try to test it more extensively on an SMP, however, there my test chances are limited. Please test / consider for inclusion. Thanks Guennadi --- Guennadi Liakhovetski |
From: Mayne <zd1...@co...> - 2005-12-04 00:14:23
|
On Tue, 29 Nov 2005, Guennadi Liakhovetski wrote: > This patch removes wrong locking in disconnect method, around functions, > that call schedule(), and attempts to fix races there. It also fixes bugs > in timer handling, which produce Oopses on SMP. This is in the http://zd1211.ath.cx/ driver, changesets 40 and 41. Thanks a lot! Regards |
From: Guennadi L. <g.l...@gm...> - 2005-12-04 08:45:15
|
On Tue, 29 Nov 2005, Guennadi Liakhovetski wrote: > This patch removes wrong locking in disconnect method, around functions, > that call schedule(), and attempts to fix races there. It also fixes bugs > in timer handling, which produce Oopses on SMP. > > I am using it for 2 days now on a UP machine with preemption and > preemption debugging. Will also try to test it more extensively on > an SMP, however, there my test chances are limited. Tested last night on an SMP - every 30 minutes during a download and a ping bring the network down, unload the driver, load the driver start downloading again, sleep 30 minutes not to irritate my provider too much:-) This for about 8 hours. Also tried unplugging the dongle during a transfer (unfortunately, cannot do this in a script:-)) - no problem. Of course, this is no 100% guarantee, but it looks good so far. Thanks Guennadi --- Guennadi Liakhovetski |