* Re: [PATCH] ipr: Remove SATA support
[not found] <CABa-fKRfE8B2TLVJASB9xQaOXDiYH3YCw0YEEg1UcGu2Le8xWw@mail.gmail.com>
@ 2024-02-29 22:04 ` Damien Le Moal
0 siblings, 0 replies; only message in thread
From: Damien Le Moal @ 2024-02-29 22:04 UTC (permalink / raw
To: Elektrokinesis DJ, brking, linux-ide@vger.kernel.org
Cc: jejb, john.g.garry, linux-scsi, martin.petersen, wenxiong,
Niklas Cassel
On 2024/02/29 9:31, Elektrokinesis DJ wrote:
> I am dissatisfied with the decision to remove SATA from the ipr driver, I
> have multiple machines in my homelab that now require manual patching every
> time I would like to update the kernel and initrd image. Bladecenter PS700
> uses the ipr SCSI driver if you install a common SATA SSD into the Planar
> SAS slots. This limits me from being able to use newer kernels without
> patching, unless I would like to procure expensive SAS SSDs.
The ipr driver was the only libsas/ATA driver that had not been converted to the
new libata error handling, and was thus preventing necessary cleanups in libata.
When it was removed, it seemed that no-one was using ATA devices with IPR beside
SATA DVDs, and very few people had this hardware to test anything.
So it seems that devices beside DVDs were/are actually used.
We can try to reintroduce the support for ATA in ipr, but that will not be a
simple revert as the new EH handling needs to be supported.
Brian,
Any comment ? Is this feasible ?
I do not have the hardware to work on this, so patches will have to be done by
you or someone with access.
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2024-02-29 22:05 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CABa-fKRfE8B2TLVJASB9xQaOXDiYH3YCw0YEEg1UcGu2Le8xWw@mail.gmail.com>
2024-02-29 22:04 ` [PATCH] ipr: Remove SATA support Damien Le Moal
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 read-only IMAP folder(s) and NNTP newsgroup(s).