All of lore.kernel.org
 help / color / mirror / Atom feed
From: John Garry <john.g.garry@oracle.com>
To: Bart Van Assche <bvanassche@acm.org>,
	James.Bottomley@HansenPartnership.com,
	martin.petersen@oracle.com
Cc: 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: Tue, 4 Aug 2026 20:42:56 +0100	[thread overview]
Message-ID: <7ab3dca8-83f3-4ec7-8bb4-e9c3508a5c59@oracle.com> (raw)
In-Reply-To: <efff8e1d-0eb9-47eb-ba20-6bdbeeea15e1@acm.org>

On 04/08/2026 18:53, Bart Van Assche wrote:
> On 8/4/26 4:34 AM, John Garry wrote:
>> Getting the address of embedded host_lock structure just requires adding
>> a fixed offset value to the shost pointer. However, getting the value of
>> the host_lock pointer requires loading from a fixed offset to the shost
>> pointer. The latter should be very slightly slower, which is relevant as
>> this lock is used a lot throughout the core code and drivers.
> 
> I like the idea behind this patch. However, I think this patch is
> incomplete.

For sure, it's a bit of work to cover them all.

> The changes shown below are needed to make all SCSI drivers
> build again with the script available at
> https://urldefense.com/v3/__https://github.com/bvanassche/build-scsi-
> drivers__;!!ACWV5N9M2RV99hQ!
> M6E9mlAub1UcWra37zJgh_viQQpt0iL5URAr5QPhRm9PfF4eaqXeGyG1MOWRzdyojhZ3zFekfZ53Dro49iIFtQ$<https://urldefense.com/v3/__https://github.com/bvanassche/build-scsi- 
> drivers__;!!ACWV5N9M2RV99hQ! 
> M6E9mlAub1UcWra37zJgh_viQQpt0iL5URAr5QPhRm9PfF4eaqXeGyG1MOWRzdyojhZ3zFekfZ53Dro49iIFtQ$>:

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.

> 
> diff --git a/drivers/s390/scsi/zfcp_sysfs.c b/drivers/s390/scsi/zfcp_sysfs.c
> index 4f23d585d062..b1ed0cb30a72 100644
> --- a/drivers/s390/scsi/zfcp_sysfs.c
> +++ b/drivers/s390/scsi/zfcp_sysfs.c

Thanks!

  reply	other threads:[~2026-08-04 19:43 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 [this message]
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

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=7ab3dca8-83f3-4ec7-8bb4-e9c3508a5c59@oracle.com \
    --to=john.g.garry@oracle.com \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=bvanassche@acm.org \
    --cc=hch@lst.de \
    --cc=john.garry@linux.dev \
    --cc=linux-scsi@vger.kernel.org \
    --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.