Received-SPF: pass (google.com: domain of linux-kernel+bounces-117894-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
To: John Garry <john.g.garry@oracle.com>
Cc: jejb@linux.ibm.com, martin.petersen@oracle.com, chenxiang66@hisilicon.com,
        jinpu.wang@cloud.ionos.com, artur.paszkiewicz@intel.com,
        yanaijie@huawei.com, dlemoal@kernel.org, ipylypiv@google.com,
        cassel@kernel.org, linux-scsi@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/6] Add LIBSAS_SHT_BASE for libsas
From: "Martin K. Petersen" <martin.petersen@oracle.com>
In-Reply-To: <20240308114339.1340549-1-john.g.garry@oracle.com> (John Garry's
	message of "Fri, 8 Mar 2024 11:43:33 +0000")
Organization: Oracle Corporation
Message-ID: <yq1wmpq9ink.fsf@ca-mkp.ca.oracle.com>
References: <20240308114339.1340549-1-john.g.garry@oracle.com>
Date: Mon, 25 Mar 2024 16:11:05 -0400
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?us-ascii?Q?G4VcB9Eua8qxSK9HKdKE6p+ZTUpVh86FYkCf2aIaf7xRlAoXJf9YBX0dIerw?=
 =?us-ascii?Q?ZxmIDnifTy2mSs7qP4EGiTOF91y3r8a9Vp/LFgugb8TGc+/orK4YQ2kFzSaz?=
 =?us-ascii?Q?5iWzFfRADYUHKRjbUfgo6ORF3axniyew/lg0W4Ac4nAcI01i2wkvV25BrZK+?=
 =?us-ascii?Q?WTXQEirmgvh74UHqTZitsIlWvFh0X95e3iYWm3W/C2fCU8bxDmP0VR1jr2yk?=
 =?us-ascii?Q?+uBFOXzLAy7qeaF4Ku+PTjUcea+CgGa1mcOdBxtRjgaYoONwXbfC2HIBtL0X?=
 =?us-ascii?Q?w//HFUWamG6s0riBcfL0o6H74IUSHUI4VfukbmfHO8ofH48KjfOCamG8+McN?=
 =?us-ascii?Q?eo/VOBx7XlBnliKgiPKAYQIGncGHlGKX4o6995v7G/AsNBRsiSNbK/5x9LMO?=
 =?us-ascii?Q?lMe9AadCWoQ/Y2f7CPOPNV+xmw/bszRYMjHiMq2KIWwbI59TPUzGj9RkNVd5?=
 =?us-ascii?Q?lH3RDk2MJGLdOVtAU0/zDwFAemF10qVp6lw3GLW8FjYKP4XGWTrgmP20x4Ym?=
 =?us-ascii?Q?QOOMj3lSLdCF0lBfbb1xYKkAZzojHFG60l6bWTMoSBLuji+HlyXIBWLFwPnl?=
 =?us-ascii?Q?U6QayErLl305XSCrAt/Gq92erxBBWfWuoOy6b76BvCMLlz/qsmyUr43CzpXz?=
 =?us-ascii?Q?GKStPRocd15ztBpmVGfcbkQUwTC0x584MZ6kEBHcQpCOyfNHomaV+j4NPY4G?=
 =?us-ascii?Q?48ehveiC0HTjwmTDu8Pdc5wtfei1cUM32H4cy1618nQk3MSfemqSqvP6owPp?=
 =?us-ascii?Q?sZYFcJd+G2+OXJwVQvTaN6HqPY5/Iljg2McenEhruN2/Lk56Ww2zkZMbx8QD?=
 =?us-ascii?Q?nc4VOny/jB84oNQ02k5ZKD8hFLWG1bcJ9BjoRQfZ/PVnkLAnA+MTRs+RTsXS?=
 =?us-ascii?Q?tmLFGrCkP57xttE0AMTVcV1UvGy+XZqArV4i3dzE4ayNVAtEQv9vJBUohUNX?=
 =?us-ascii?Q?UusqxOaKgi0YQJXJrsy9b54/SD/CzQXW1+VuQSkJESZUQJ7Em81vPRckLyKz?=
 =?us-ascii?Q?VzgcljKGA3Z7/wwnP8pbJziGfSrMAlvC7j273ci16NtaRxt4zp3MieMXyC16?=
 =?us-ascii?Q?NgPz9tmoWp7XUAa4cNO2KszZk75vNVXwvFz6Pa1ihj9yxCQYA2BGv1tADnYn?=
 =?us-ascii?Q?p4a1s9+T0yOj0z0cGAwuG8JkvC8XGLcT/q4WdFXKSwYFVpCKOyAjApwWAX+g?=
 =?us-ascii?Q?6RWgvW4as3qw0+eauqF8uOw/nhJgYPDP7rA3XDsWLlJg/tpgJdRF873OKsNw?=
 =?us-ascii?Q?WDmCKzn2kExMZB4MOLkxPRJEan3P/A6Y52Ps2x4qyqeqHUYwJA1Ri7cn1v67?=
 =?us-ascii?Q?e8sa6MKaJt1Fyg5KK+SV3q3n3DTILfA0Afs9M8tCz4xESTk2kICy/fbkUQPE?=
 =?us-ascii?Q?WaOElPEy0yu0dPE+tJ0e/9U6+7DqIj4achDza/clY6bpsbB5JNkBka3M5ViE?=
 =?us-ascii?Q?ZiNDVg/U5CwzXXJJOUWi4OKN8mM47TlgzAr0qStRBS8s1xRmvk1m25+JXFWV?=
 =?us-ascii?Q?FoYngEBmgCeF5jTXseuI38bGegx305UgAwslD7Ph4PfXO8TP2EbD/Zkmf4TB?=
 =?us-ascii?Q?TZTX+R0CryEKTDl5fBvEM0VYDQrWKOZuGtdo32UkKKXwmXCL+LdOk9x3ua1i?=
 =?us-ascii?Q?bg=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	eRuIO57a2q3QrCsT182AK//Gwa6PHdth/S00SQF+dmt+6Sn/cHZYTB5en5xKYcuw3+joSMEgwMx3ZmHlwLtUejC6Zb0lVRKIPjswkqrUVq2mEpG3S+RQqOot9hBmNWiXJW5/c1iKVw0AzVUaT1iMgYmyTD1ZZlmCyIz+QsA2zmYqBwEj+M8ZmzCmZgHFxYGnoM+cB9067Y6p0acNQBIVKryt8A1QGuBVQmp2sHt5gZZc9k7PufQMpLSr7M+rVLsiOwY3QkxxjMAUGsVl9bEsLADdDrxtGWy6PwqyqqiDQBwJf0NGVwyvO0bMi7sAiQknB9eggQ8Kkr/xgzfzblNpWzWqRELeW6u64LM1IFOAZ7w1KPqeIKZ3B8MR98mNQwBTXStKHbFNyBHa9E1HVcFJcqVw8r1a9kKSp1Rmv0peK2pqkJ0PerLAlX9XQ+9GXUg8M09Ylxr0MBNlWTtOZ+4yLdM5l6vtxFfy2yIH0NMTdQmYWvjlTe8YXGw9mP+YayLY2C99dsDmWwr7NeqniR6t/GIkkFGZWihHjRF1n8LGn4IkuzniDtsvl4sFGFFKOjgnMZlx0lP+erK96HE5wfiHnY7q6Os0AehwP3v1ex0/4Gs=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3206648f-939c-4a20-1db1-08dc4d07b4a7
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Mar 2024 20:11:07.2290
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: J3YeB27aSymSgwq87z2iGog3ukEWnHqJ7NjsO4STIT/WlkI4CP7KiBzNMbH/zTp4E2yGlw96XRbr7Y2b9wDSiMD8YCqglLZQ16r8Z7khxb8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR10MB4124
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-03-25_19,2024-03-21_02,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=912 phishscore=0
 bulkscore=0 spamscore=0 adultscore=0 mlxscore=0 suspectscore=0
 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2403210000 definitions=main-2403250123
X-Proofpoint-ORIG-GUID: jJH6Guae0zHdvGx-PBhxQfyX_U9nOyg5
X-Proofpoint-GUID: jJH6Guae0zHdvGx-PBhxQfyX_U9nOyg5


John,

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

Applied to 6.10/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering