From: Willem Riede <wrlk@riede.org>
To: James Bottomley <James.Bottomley@steeleye.com>
Cc: linux-scsi@vger.kernel.org
Subject: Re: [PATCH] Eliminate signals entirely from the mid-layer
Date: Sat, 11 Jan 2003 09:23:11 -0500 [thread overview]
Message-ID: <20030111142311.GQ1378@linnie.riede.org> (raw)
In-Reply-To: <200301102119.h0ALJUf05888@localhost.localdomain>; from James.Bottomley@steeleye.com on Fri, Jan 10, 2003 at 16:19:30 -0500
On 2003.01.10 16:19 James Bottomley wrote:
> the attached patch removes SCSI's use of signals for killing the error handler
> thread. It also:
>
> - changes the eh_notify semaphore to a completion
> - adds complete_and_exit() to the thread
> - uses a host structure flag to signal thread death instead of the signal
>
> I'm currently travelling, so could someone test it out for me? Thanks,
>
Sorry, James,
When I apply your patch, rmmod ide-scsi hangs. This is my log:
Jan 11 09:08:06 fallguy kernel: Error handler scsi_eh_2 waking up
Jan 11 09:08:06 fallguy kernel: Total of 0 commands on 0 devices require eh work
Jan 11 09:08:06 fallguy kernel: scsi_eh_get_sense: checking to see if we need to request sense
Jan 11 09:08:06 fallguy kernel: scsi_restart_operations: waking up host to restart
Jan 11 09:08:06 fallguy kernel: Error handler scsi_eh_2 sleeping
and ps shows:
...
040 0 236 1 15 0 0 0 down_i SW ? 0:00 [scsi_eh_2]
...
100 0 946 887 16 0 1324 380 scsi_u D pts/0 0:00 rmmod ide-scsi
044 0 947 946 16 0 0 0 do_exi Z pts/0 0:00 [kmodule0? <defunct>]
...
Regards, Willem Riede.
next prev parent reply other threads:[~2003-01-11 14:23 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-01-10 21:19 [PATCH] Eliminate signals entirely from the mid-layer James Bottomley
2003-01-10 22:01 ` Mike Anderson
2003-01-10 22:05 ` James Bottomley
2003-01-11 14:23 ` Willem Riede [this message]
2003-01-11 15:44 ` James Bottomley
2003-01-11 16:33 ` Mike Anderson
2003-01-11 16:47 ` Willem Riede
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20030111142311.GQ1378@linnie.riede.org \
--to=wrlk@riede.org \
--cc=James.Bottomley@steeleye.com \
--cc=linux-scsi@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.