All of lore.kernel.org
 help / color / mirror / Atom feed
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.

  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.