Hi, you forgot to drop a spinlock before you report an error. A deadlock will occur.
Regards Oliver You can import this changeset into BK by piping this whole message to: '| bk receive [path to repository]' or apply the patch as usual. =================================================================== [EMAIL PROTECTED], 2004-03-08 03:45:19+01:00, [EMAIL PROTECTED] - fix locking in error case pid.c | 1 + 1 files changed, 1 insertion(+) diff -Nru a/drivers/usb/input/pid.c b/drivers/usb/input/pid.c --- a/drivers/usb/input/pid.c Mon Mar 8 03:52:14 2004 +++ b/drivers/usb/input/pid.c Mon Mar 8 03:52:14 2004 @@ -200,6 +200,7 @@ break; if ( id == FF_EFFECTS_MAX) { + spin_unlock_irqrestore(&pid_private->lock,flags); // TEMP - We need to get ff_effects_max correctly first: || id >= dev->ff_effects_max) { dev_dbg(&pid_private->hid->dev->dev, "Not enough device memory\n"); return -ENOMEM; =================================================================== This BitKeeper patch contains the following changesets: 1.1169 ## Wrapped with gzip_uu ## begin 664 bkpatch14064 M'XL(`-[?2T```\64[XO3,!C'7S=_Q0,'HIQK\_3'NE5V3,]#1<%Q>J]$CBQ] MUM;KDIFD-Y7^\:8;W"&Z@:+8M!#R)-]^G^?YD!.XLF2*0+?-+1EV`B^U=47@ MY^NN)!4JZFZZ=:A-Y6.76OM85.LU1?L#T7M#9*.V49T=Q6'&_*Z%<+(&'[1% M@&%RM^*^;J@(+B]>7+UY>LG8;`;GM5`5O2,'LQESVMR*MK1SX>I6J]`9H>R: MG`BE7O=W6_N8\]B/#/.$9^,>QSS->XDEHDB12AZGDW'*]O;FO\CC1ZF4)SQ% MGO$DZQ$G6<*>`X:(XRGP-.))Q"?`DR+-"IR><BPXA\/2<(HPXNP9_-U<SIF$ M$:R:+]!J>=.H"AH%9(PV((4E]AHPYCAFB_N"LM%O/HQQP=D9W.I/CF0]=]NF M;:K:A9W<AO);7YHA:1MU=ADU:M.Y:-.4H=QGD"-BAC%F?>PKE_9Y0D+&JW*) M93F>)-,C)3LF//1FXFN0\;S'/,?)#IH#!P:$_HEYMI2Z]8#;>4G+1JC!]8?* M4/7QF*[O9\+]EPUMC0>9@:N?H.+Y_X6J-'JSHPJ6M-*&P)[EMAIL PROTECTED] MOV9U9R39@;1=']["R&QWKR=G<:@E?P#A*^\<D`5!8#>-NN[4X.VZ,9^]!Y\_ M/[EMAIL PROTECTED]:\W_G_"T>ALB#Y>M:*RCY[<7SVR)GECN_6,IR+.Y)38=T=U\5SF!``` ` end ------------------------------------------------------- This SF.Net email is sponsored by: IBM Linux Tutorials Free Linux tutorial presented by Daniel Robbins, President and CEO of GenToo technologies. Learn everything from fundamentals to system administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel