From: Michael Schmitz <schmitzmic@gmail.com>
To: Finn Thain <fthain@linux-m68k.org>,
Russell King <linux@armlinux.org.uk>,
John Garry <john.g.garry@oracle.com>
Cc: Bart Van Assche <bvanassche@acm.org>,
James.Bottomley@HansenPartnership.com,
martin.petersen@oracle.com, linux-scsi@vger.kernel.org,
hch@lst.de, John Garry <john.garry@linux.dev>
Subject: Re: [RFC PATCH] scsi: core: Drop Scsi_Host.default_lock
Date: Sat, 8 Aug 2026 13:53:18 +1200 [thread overview]
Message-ID: <0631ac45-1908-4ffc-842c-6a6c26096397@gmail.com> (raw)
In-Reply-To: <095e7779-6c40-5257-a684-63cc054c0e20@linux-m68k.org>
Hi Finn,
On 06/08/2026 12:35 PM, Finn Thain wrote:
> On Wed, 5 Aug 2026, John Garry wrote:
>
>> On 04/08/2026 21:08, Bart Van Assche wrote:
>>>> Cool, I'll go through this. But how does it handle the arm v3-related
>>>> drivers, like fas216? At a glance, the script requires gnu-
>>>> a/10.3-2021.07/binrel/gcc-arm-10.3-2021.07 - but I thought that HW like
>>>> this requires an even earlier toolchain version.
>>> Correct. I think that gcc-8 or older is required to generate correct
>>> code. But generating correct code is not required to do build testing.
>>> Hence the following hack in my script for 32-bit ARM architectures that
>>> deletes a GCC version check from a Kconfig file:
>>>
>>> if [ "${arch%-*}" = "arm" ]; then
>>> # gcc-9 and later do not support armv3m. Hence the sed commands below.
>>> local f
>>> for f in arch/arm/Kconfig arch/arm/mach-rpc/Kconfig; do
>>> if [ -e "$f" ]; then
>>> sed -i 's/GCC_VERSION < 90100 && //' "$f"
>>> fi
>>> done
>>> sed -i 's/select CPU_32v3/select CPU_32v4/' arch/arm/mm/Kconfig
>>> fi
>> ok, I'll try it, thanks!
>>
>> I do have to question why we maintain support on modern kernels for such
>> obsolete HW. The maintenance burden is not zero, as we see here ...
>>
> The non-zero maintenance burden is a two-way street. That's just the
> downside of the "open" part of "open source", of course.
>
> The maintainer for the relevant hardware port is Russell King whom you
> forgot to Cc. But Linux on obsolete hardware is quite normal in my
> experience, and the issue here is actually the compiler.
>
> I'm not going to attempt to fix the compiler. I don't have the talent,
> time, hardware etc. And yet the MAINTAINERS file lists Michael and I as
> maintainers for two RISCPC drivers:
>
> F: drivers/scsi/arm/cumana_1.c
> F: drivers/scsi/arm/oak.c
>
> Given the compiler problem, which I too have long had to work around when
> build-testing NCR5380 drivers, and given that I have no ability to test
> this hardware, I'm thinking of sending a patch to shift the responsibility
> for these files from the NCR5380 entry to Russell's RISCPC entry in
> MAINTAINERS.
>
> Russell and Michael -- do you have any thoughts about this issue?
Not wanting to set a precedent, but seeing as Russell may be the only
maintainer still in a position to build and hopefully test these
drivers, moving responsibility for driver code to the arch maintainer
may be justified here.
I must admit I've been dreadfully slack in testing recent code myself,
ever since I started running out of memory at boot ...
Cheers,
Michael
prev parent reply other threads:[~2026-08-08 1:53 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-04 11:34 [RFC PATCH] scsi: core: Drop Scsi_Host.default_lock John Garry
2026-08-04 17:53 ` Bart Van Assche
2026-08-04 19:42 ` John Garry
2026-08-04 20:08 ` Bart Van Assche
2026-08-05 13:45 ` John Garry
2026-08-05 21:47 ` Bart Van Assche
2026-08-05 22:04 ` Arnd Bergmann
2026-08-06 0:35 ` Finn Thain
2026-08-08 1:53 ` Michael Schmitz [this message]
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=0631ac45-1908-4ffc-842c-6a6c26096397@gmail.com \
--to=schmitzmic@gmail.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=bvanassche@acm.org \
--cc=fthain@linux-m68k.org \
--cc=hch@lst.de \
--cc=john.g.garry@oracle.com \
--cc=john.garry@linux.dev \
--cc=linux-scsi@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=martin.petersen@oracle.com \
/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