[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <cf0cfd01-43e3-141e-01ca-867a8eb9a982@huawei.com>
Date: Mon, 11 Mar 2024 09:46:34 +0800
From: Jason Yan <yanaijie@...wei.com>
To: John Garry <john.g.garry@...cle.com>, <jejb@...ux.ibm.com>,
<martin.petersen@...cle.com>, <chenxiang66@...ilicon.com>,
<jinpu.wang@...ud.ionos.com>, <artur.paszkiewicz@...el.com>,
<dlemoal@...nel.org>, <ipylypiv@...gle.com>, <cassel@...nel.org>
CC: <linux-scsi@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/6] Add LIBSAS_SHT_BASE for libsas
Hi John,
On 2024/3/8 19:43, John Garry wrote:
> There is much duplication in the scsi_host_template structure for the
> drivers which use libsas.
>
> Similar to how a standard template is used in libata with __ATA_BASE_SHT,
> create a standard template in LIBSAS_SHT_BASE.
>
> Based on following:
> b914227e4215 (tag: mkp-scsi-staging, mkp-scsi/staging, mkp-scsi/for-next, mkp-scsi/6.9/scsi-staging) Merge patch series "Pass data lifetime information to SCSI disk devices"
>
> Differences to v1:
> - tidy libsas.h change (Jason)
> - Don't set eh_abort_handler in LIBSAS_SHT_BASE (Jason)
> - Remove sg_tablesize in LIBSAS_SHT_BASE, as W=1 build dislikes it
I cannot find sg_tablesize in LIBSAS_SHT_BASE in v1, did I misssed anything?
Thanks,
Jason
> - Add some RB tags (Thanks)
>
> John Garry (6):
> scsi: libsas: Add LIBSAS_SHT_BASE
> scsi: pm8001: Use LIBSAS_SHT_BASE
> scsi: hisi_sas: Use LIBSAS_SHT_BASE_NO_SLAVE_INIT
> scsi: aic94xx: Use LIBSAS_SHT_BASE
> scsi: mvsas: Use LIBSAS_SHT_BASE
> scsi: isci: Use LIBSAS_SHT_BASE
>
> drivers/scsi/aic94xx/aic94xx_init.c | 21 ++-----------------
> drivers/scsi/hisi_sas/hisi_sas_v1_hw.c | 18 +---------------
> drivers/scsi/hisi_sas/hisi_sas_v2_hw.c | 18 +---------------
> drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | 18 +---------------
> drivers/scsi/isci/init.c | 23 ++------------------
> drivers/scsi/mvsas/mv_init.c | 19 +----------------
> drivers/scsi/pm8001/pm8001_init.c | 20 +-----------------
> include/scsi/libsas.h | 29 ++++++++++++++++++++++++++
> 8 files changed, 38 insertions(+), 128 deletions(-)
>
Powered by blists - more mailing lists