Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
To:     "Fabio M. De Francesco" <fmdefrancesco@gmail.com>
Cc:     Adaptec OEM Raid Solutions <aacraid@microsemi.com>,
        "James E.J. Bottomley" <jejb@linux.ibm.com>,
        "Martin K. Petersen" <martin.petersen@oracle.com>,
        linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org,
        Ira Weiny <ira.weiny@intel.com>
Subject: Re: [PATCH] scsi: ips: Replace kmap_atomic() with kmap_local_page()
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq18rhzpfzw.fsf@ca-mkp.ca.oracle.com>
References: <20230103173131.21259-1-fmdefrancesco@gmail.com>
Date:   Wed, 18 Jan 2023 18:42:15 -0500
In-Reply-To: <20230103173131.21259-1-fmdefrancesco@gmail.com> (Fabio M. De
        Francesco's message of "Tue, 3 Jan 2023 18:31:31 +0100")
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tpumKHuh1EXwoD2Q3DpAJF23JFXj5fGfhN50XjTctno3Ak0SLs/H5fNxskl9?=
 =?us-ascii?Q?fMrQKRotiMvhQ73h4tJ5qsRkkmn2E+tWhAh87SHLbur+8rjdrXDIvGGIMY7V?=
 =?us-ascii?Q?w19TDwXwYik+XBNMP0PxACvCu0K60qeOwqvQthHIPUEAJ2dmcN0kmmODqOeS?=
 =?us-ascii?Q?rCvA54EDqQj87SNvXLRmsDNSDNKq1fIi7ObWadtnB/4wW61sv20LE7W/ivUH?=
 =?us-ascii?Q?nIa/OiSmxsXsxXGSaxdQLCXMG6VdPMYApToo6cDN9sCAXpBo8wmpsrCLv5Hc?=
 =?us-ascii?Q?aY6CKnTY+CtFplEPVZKxzmMVy+GVaOBtKNBxCFnWPi24DiY3LRJVQaPoBThZ?=
 =?us-ascii?Q?jM28//BMCcmbYnJ33ed3ZFKi4ZqWKijbd4T2ySLbNDN7oOggLIssI8Sijreq?=
 =?us-ascii?Q?2deDoAWUl5XzKtIENRw4W2dGvUL724CDOXka3sSHofaD9TYNdy+568vNHblK?=
 =?us-ascii?Q?pZzXfbTJ9Z0jQoCCGE0YS2I947lMvrnU95/+W4pB8lb/uzCq+DF6otn7URF6?=
 =?us-ascii?Q?y0VclIB2KISeF1iUtS5VI6JFcvu8DJUWWurvWajSZSm3EqxEM0BmovI5V534?=
 =?us-ascii?Q?vJSCH/V7f2drln9w39JqTyU8iuydPBofJP1fkn8+pN5c/H/OW+M2egnqkIgp?=
 =?us-ascii?Q?5+beeSi7B7oSoz7vowOFRqMskAcS6ZGJFH9hu0jeGr1HQqe2MT6Uba5dle7/?=
 =?us-ascii?Q?8bGvjRSc3k8Vd8MvIWSqrvJo76aoMlyHpJmABTvyRzGgXTI2VemygO47Qf5K?=
 =?us-ascii?Q?S1X4edsaTojAk3R8xa5zntWdWSWoGpJN67NXIzJvNg3nTcGMHjQ8mZaReiAA?=
 =?us-ascii?Q?xLKIQfndbenJ6hU2Tx5fP6Nd8kIUVUKz7msRCgg5yh4xgciEOPWgrTEpt1+d?=
 =?us-ascii?Q?D08himhAcp4LkSNwSD25enTsZC0F9raNJArlrYZ/GHr/RVi/vqrnpMo7BPlc?=
 =?us-ascii?Q?rAXmGPVLH9J8RBBQJWyYygn34OPrc/NPsGo0pzk/zNzg1nKjqRoJO79OJQ2a?=
 =?us-ascii?Q?BgZrBibZX8qDQnzYeeuap+eIH/OjXvvietkwi6F1gixGNKMmohBCaxSg3+ez?=
 =?us-ascii?Q?nTIpFVqI4MC9IaoXXLwi9792AzprC2urJMdSMjr6i9J/NtcLbTD7k0GP5+4D?=
 =?us-ascii?Q?HTJfzqVRzoQOAUbYd/65XkIqQrbSx2OFHH6aj7F3I6RpFAjNXM+U1OHZ7oL+?=
 =?us-ascii?Q?IskbqI48TUal+ISxSeIHoE17UdhPmE+TB1is0GEukV5OdMEzBsmo4ZJKRU02?=
 =?us-ascii?Q?XrIYg4iwwICcOyZEIfbvpgioEMlNQ+4+t6G/hM6eOyvghI32k7TH7F9LXVxy?=
 =?us-ascii?Q?LUKNaKh+w9iornQpHqsvNGImL788EYoZdx4dTtWbtctZ5uYzgxoBD2ZweBcg?=
 =?us-ascii?Q?Qk5D+LAdYVIxyBFYcNV56T7szZkEd+fS2rsYtCZNQuisc/FwnqmkGMz+CpPR?=
 =?us-ascii?Q?xrx9wI+HcSyKSGgiCywcvbfTDrWobuqCp9Dvww2sx2il7fU571MVST3TQZEF?=
 =?us-ascii?Q?PH433DRlKnqj9t36Loq9jqTTBVMO3OciXuas/nBAFhb2Ki5nWvcDuEn+swxK?=
 =?us-ascii?Q?mWMWEu4y5q7EtY8J8OidCirCWjybKqmImzply9PySTqHWw5TAjhppAOi0fmv?=
 =?us-ascii?Q?UA=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: ohz9pXRvIDgQ/h2kYKInIRm4qGjUK4qtzI9auTiy2mHsCtZO7313TxOU0L535V4j34L3ngHchCh7d0ZwbKkeW+1bsOj7alqQ7k97JnZKoewBLhXoSVzbz4uV/1uRAaacVAEBBMNlse+Ic/as6LkYlPrg/igAhxg+sP1OIFIE3ibQA3dDmn41HjkHVNqfnbbm2G6rXI3wok7jdfLjMc6xRONuZLJjaDai2AR+pRDjkx01VtbNLmx5a94+EufcG/swx89DwF1dUNB5N7svP3b9EssiOYWEQr7qfgBZMp/g9QxcwdgzO3E6de5mAUCnl5l4VJalm6H2OkDkHVTQrya4jfiH1Z9oOMadNpnsNENLmu4rRjZx5qAHwJIJ2EsPsI+rMbXtKQbteT77w4pSneEs2NYMujR61COwsdE5ZBpKZoc0tVTgDuiCdDECIu7vkFEJdHT6LWFo+JLGL4RFWIYSdPzgm/Uk8uZiwl59RuZTDbr2BVB+XLwrFIdbn4WO2zaJ3LufROXfUKhSjcWgc60OJzkSQcLd2MJYz+Dd0wbdOKxL8ahj0nWxk/xO2h1pgV+939jXZYzpgOe2IMjcGQPxOMbqnxDjIkIOdcr/eXHNMLDJVEgypEOyL8a+AuKfzq4PXNJHrqCV1pSvyhZOLuL6EAwRqWCsweLjjGUX7r2kCVB28mzsG0urW0sQz0CoqanloaCjahwQhagQMDAMSrq0KSpiq+1DTZZGHMXeEzVZAj6PEW0cjp3vmu789ZG2Uf85NMVYftZM2FekUGbmgwgggPRQzNoHPcnxXygigU/9aQLZIMUJHYFuFFVCo66xptoAsxNNsWFFGFpoqYGqeEGr3FGlbsEn7IGDsYW3KNgpQPpZ43c1mgKocAPRRSrCoiH5DBxwQaW0asED7Y8xbkoJxw==
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 062dace2-01ee-425f-6682-08daf9ada3a3
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jan 2023 23:42:19.8458
 (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: XtmiQtocljRbTi2YK6em0aeWKkWzwuZaKN6DuXaMpa5UzRTNZ0kEYlkXEz8jBONDe0EB06Oo6wbaci7TYmV/74NU93be+pC1Ui0HGOHAJo0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR10MB5821
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.923,Hydra:6.0.562,FMLib:17.11.122.1
 definitions=2023-01-18_05,2023-01-18_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 bulkscore=0 suspectscore=0
 spamscore=0 malwarescore=0 mlxscore=0 mlxlogscore=740 adultscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000
 definitions=main-2301180196
X-Proofpoint-GUID: 9xghnK0O7CfCCZWSVHtoDxbdPZczBWzx
X-Proofpoint-ORIG-GUID: 9xghnK0O7CfCCZWSVHtoDxbdPZczBWzx
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


Fabio,

> Therefore, a mere replacement of the old API with the new one is all
> it is required (i.e., there is no need to explicitly add any calls to
> pagefault_disable() and/or preempt_disable()).

Applied to 6.3/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering