All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tim Wright <timw@splhi.com>
To: linux-kernel@vger.kernel.org
Subject: rmmod st "hangs" - bad interaction with sg
Date: Sat, 10 Jul 2004 08:31:00 -0700	[thread overview]
Message-ID: <1089473460.1473.17.camel@tp-timw.internal.splhi.com> (raw)

Hi,
I was working on the qlogicisp/isp1020 driver in 2.6, as I still have
one of these antiques and the driver is a bit out of date (a patch is
forthcoming). In the process of testing my changes, I came across the
following:

I have a single DDS-2 tape drive attached to the SCSI bus. qlogicisp
loads, and pulls and the tape is found. However, when I tried to unload
st, the unload "hung" unkillable. I use the quotes for a reason. After
much searching, I found that there is still a reference count on the
sysfs scsi_device and that is because when the driver gets loaded, not
only does st get loaded, but sg also gets pulled in and increments the
refcount on the sysfs scsi_device. Running 'rmmod sg' in another window
allows the original 'rmmod st' to complete.

This seems bad to me - either the original rmmod should fail with EBUSY,
or it should complete. However, for it to do so, it seems that st needs
to know that sg has its hooks into the device it controls, and it needs
to be able to make it let go. My workaround is impractical if sg is in
use on other devices too.

So....
what do interested parties think? Would this be considered a bug? Should
I chase and see if I can find a way to make it behave, or does someone
already know what's wrong?

Regards,

Tim

BTW, if anyone is interested in the fixed-up qlogicisp, I can make it
available. Error-handling is in and seems to be working (the main
complaint that 2.6 has), and interfaces have been updated to 2.6
standards.


             reply	other threads:[~2004-07-10 15:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-10 15:31 Tim Wright [this message]
2004-07-10 15:41 ` rmmod st "hangs" - bad interaction with sg Christoph Hellwig
2004-07-10 16:03   ` Tim Wright

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=1089473460.1473.17.camel@tp-timw.internal.splhi.com \
    --to=timw@splhi.com \
    --cc=linux-kernel@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.