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



      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 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.