From: Lukasz Kosewski <lkosewsk@gmail.com>
To: Jeff Garzik <jgarzik@pobox.com>
Cc: linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
linux-scsi@vger.kernel.org
Subject: [PATCH 0/3] Add disk hotswap support to libata RESEND #5
Date: Mon, 26 Sep 2005 21:00:48 -0400 [thread overview]
Message-ID: <355e5e5e0509261800271c39b7@mail.gmail.com> (raw)
Hello Jeff, all,
Once again I besiege you with what I hope is the last necessary fix to
my hotswap patches!
For those caught with their pants down, the purpose of these patches
is as follows:
patch1: Mask out the correct hotswap interrupts on the Promise
SATAII150 line of controllers.
patch2: Add a disk hotswap API to libata, a general purpose
infrastructure which all libata drivers can use to play with disk
swapping.
patch3: A reference and working implementation of a driver using this
architecture with the sata_promise module. Tested on SATA150 and
SATAII150 Tx4/Tx2 Plus controllers.
The differences between this patchset and previous ones:
- no longer memory leaking struct pdc_host_priv all over the place,
just in one (there are other leaks there too... I even added a comment
saying that this should be taken care of!).
- Slight difference in the order of instructions executed on disk
removal to ensure that we don't have instructions bottled up in the
ata workqueue waiting on a disk that's been removed.
These patches will apply against seemingly any incarnation of the
2.6.14-rc2 kernel, HOWEVER (BIG HOWEVER!) in my testing (with
2.6.14-rc2 and 2.6.14-rc2-git5), the original -rc2 release has
interesting problems that were rectified between 2.6.14-rc2-git5,
which would cause kernel panics on disk removal during heavy I/O
(thanks to Thomas Lustig for pointing out something was awry!). So if
you plan on sending me "hey, it crashed!" messages, please use
2.6.15-git5 or later.
The remaining vestiges of issues that I see might be related to
swapping strange combinations of disks... I"ve tried to properly reset
flags and such to allow swapping and switching arbitrary disks, but I
only have a few disks and so only observed a few errors. If you get
strange errors trying to go from some disk of type A to another disk
of type B (for instance, LBA48 to non-LBA48 drives, vice versa,
different capacities being picked up wrong, etc), let me know!
That's about it. Thanks to Net Integration Technologies for giving me
the freedom to do this kind of work and do the initial submission
while working for them, and thanks to the testers; keep it coming!
Luke Kosewski
next reply other threads:[~2005-09-27 1:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-09-27 1:00 Lukasz Kosewski [this message]
2005-09-27 2:30 ` [PATCH 0/3] Add disk hotswap support to libata RESEND #5 Lukasz Kosewski
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=355e5e5e0509261800271c39b7@mail.gmail.com \
--to=lkosewsk@gmail.com \
--cc=jgarzik@pobox.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).