Re: [PATCH] switch EH thread startup to the kthread API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Christoph Hellwig wrote:

>@@ -1600,8 +1593,6 @@
> 	SCSI_LOG_ERROR_RECOVERY(3, printk("Wake up parent of"
> 					  " scsi_eh_%d\n",shost->host_no));
>
>-	complete(shost->eh_notify);
>-
> 	while (1) {
> 		/*
> 		 * If we get a signal, it means we are supposed to go
>@@ -1622,7 +1613,7 @@
> 		 * semaphores isn't unreasonable.
> 		 */
> 		down_interruptible(&sem);
>-		if (shost->eh_kill)
>+		if (kthread_should_stop())
> 			break;
>
> 		SCSI_LOG_ERROR_RECOVERY(1, printk("Error handler"

We used nearly the same code in cpqfcTS yesterday for testing. On module 
unload down_interruptible() does not get interrupted and the kthread_stop() 
will wait forever and hang rmmod.

Eike

Attachment: pgpD5CKZAvvPs.pgp
Description: PGP signature


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux