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;
From:   Avri Altman <Avri.Altman@wdc.com>
To:     Bean Huo <beanhuo@iokpp.de>,
        "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "stanley.chu@mediatek.com" <stanley.chu@mediatek.com>,
        "beanhuo@micron.com" <beanhuo@micron.com>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "tomas.winkler@intel.com" <tomas.winkler@intel.com>,
        "daejun7.park@samsung.com" <daejun7.park@samsung.com>,
        "quic_cang@quicinc.com" <quic_cang@quicinc.com>,
        "quic_nguyenb@quicinc.com" <quic_nguyenb@quicinc.com>,
        "quic_xiaosenh@quicinc.com" <quic_xiaosenh@quicinc.com>,
        "quic_richardp@quicinc.com" <quic_richardp@quicinc.com>,
        "quic_asutoshd@quicinc.com" <quic_asutoshd@quicinc.com>,
        "hare@suse.de" <hare@suse.de>
CC:     "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v2 6/6] ufs: core: Add advanced RPMB support in ufs_bsg
Thread-Topic: [PATCH v2 6/6] ufs: core: Add advanced RPMB support in ufs_bsg
Thread-Index: AQHY/S6fGNUnGplQxEOfi3uFlr5Rnq5KyPtw
Date:   Tue, 22 Nov 2022 11:05:15 +0000
Message-ID: <DM6PR04MB6575FE2AEA80828348604AF0FC0D9@DM6PR04MB6575.namprd04.prod.outlook.com>
References: <20221120222217.108492-1-beanhuo@iokpp.de>
 <20221120222217.108492-7-beanhuo@iokpp.de>
In-Reply-To: <20221120222217.108492-7-beanhuo@iokpp.de>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?MFXABLtR1a4oRNRGmO9YGuQCXCVKhfetbC/uVOMVnYMcba/+zSwYV2CM6HMG?=
 =?us-ascii?Q?Ad4jNULDTO8eBkQSvg7jXBqFfHsOnmtxJpKzYhTgM/gmjrv/TNBi2c2Fo4UP?=
 =?us-ascii?Q?6RlrSeuptdY2Bd+s62kexVRZdwgbgtG3CTMCjF4X5Br9QB+Z1dV0E5rsVvZo?=
 =?us-ascii?Q?DgqVQByP4ayy9l6KPe0XZGbx9b3rgeVK3V9LIWexW56RBueqc5YxkqwzVhYk?=
 =?us-ascii?Q?AM6Nc5Tv+jHQlyfZbbNJzcFxuaQI7IB4w46wiUTqc4qyMmfaOi3s+wwnM0NK?=
 =?us-ascii?Q?feH3DpmvRKFXwF1u1LqOYKk8BCtsMlo/vGcy/mOEQoCtjBxcEdGP9OtGyx8L?=
 =?us-ascii?Q?Jo6GeBB/J/ZLLUJ0AtRBUlO1C+gRRJDlMSZHcrckkmGipYIl4wjVuimAHWF0?=
 =?us-ascii?Q?+lz8GkxNfwJCQ8KgzvzZOjuKf2PEHT5flXI3mTzLI/fq36Ppvae6cIy2zKhU?=
 =?us-ascii?Q?ny2qeM3+djNMTxqz9giGUr1f6rQxrvEBsi9wkH2xvz+zGzUf6AaRREb5Ynx0?=
 =?us-ascii?Q?jJzqXvaX6cBIxIlb9CJmAU4/VZk6sNI+j3Hr1wD1C+TJdGY0VZaJnX+u7ZkR?=
 =?us-ascii?Q?3TuMnm8NFQQ6l932IG5vj9mPhDKTzqOaj3evt4R8/gOJkObNZ4T1cfMrtPuD?=
 =?us-ascii?Q?oKsgp1k+4xN6jltbDlQVtklwfj7blVzUQ1GKcRHXFHvQI6+jVdRbCOssMtmF?=
 =?us-ascii?Q?lz0j9CoHKg9zob56Y+KWDyVd97Qpa7mDiTVfcga/JWNGmJs9ww3OAwicNbg7?=
 =?us-ascii?Q?/rgI4vItyJTfDwGCYS1vQWc0PnC8zySx6hS3LD0yECmSCiNVoA6FiVNCqczO?=
 =?us-ascii?Q?3nCIN5OZIBOdISbS8bUD1CSO0ruLaPPQ+xXUD11360pZXmS1ScKnFVLkDuvk?=
 =?us-ascii?Q?0+kVuU1WAFBvzJmEX2zuHeuv6Fb08MgQcozSkpZpoB/6HhiRdLuRM2S/PF7W?=
 =?us-ascii?Q?gl2of3ZDESUED1eVsHfr39mf/JT3kWgvpQuBR7xyn+ULveuMWHg1XwxVyc5P?=
 =?us-ascii?Q?Ykb3d5x5rCkCG22pr//xFYzHBQ+fxitg+RmXjySw6ospsNZgbG+N7ThwNRbv?=
 =?us-ascii?Q?Qh/nYi/lx1OasDqGpWYN85Nn1ElUaWuP/LEJm4Oa/MzGlGV639sZ3xmW2Ct+?=
 =?us-ascii?Q?H/5y+y8g7mDZnifQKKvAXnKNkO0YdhUSpsMy2WbDp7X5BgCxE0fSse+tcB0u?=
 =?us-ascii?Q?dEwPHc9ouoz9mryxQw6MS9d9Kd59F0eteJ1x65B4x+/CWuPJguRUuFDK/q04?=
 =?us-ascii?Q?MtpYdFemF//06RsTiPh6+Ojw6mR0JZEmMj5B7hlIiyP9Z6FTlNGyOaFpY0H7?=
 =?us-ascii?Q?QOJHSgU8+BtZX6F1XE8Z585cnqL8/F7NVIK5neFQ8IqYEW85tIzme7rR6u4Y?=
 =?us-ascii?Q?2CXTJU7HBHz7d0PFmgd4E4RNeUud6uTW8M4uOjHc5JXcUytRj9j7fE4vD7lq?=
 =?us-ascii?Q?gdt3HANm+pFMvf0nhEz/3coQr7y2XZ81tcEVvGLL0jJLNOk0QdDmjM4HJVD1?=
 =?us-ascii?Q?NhGV7q52/SqQCxJlDoWar5XODOzj5igUpek6zQye9r8NxuAT+sA1nOkKnCq0?=
 =?us-ascii?Q?R3clPqI8xTpv1E9sLjb+bVb1+DeTNUo5Gft1/8o9?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?OBtaObcD8+A4NFvkDdC2qpdFlrS2sLg5A4OV7g0oUv2sTqx3PHsGpDnr+p0G?=
 =?us-ascii?Q?GQsbAqPpifWndcDEpACK0uzIpCbDNppoRlWLt+wXb3iT0r7Iq4g6kw+Yf5Kc?=
 =?us-ascii?Q?LQcs5xzie8gGAAcICtx2OS9hXhkLHLorrPaApSSQ53Xew7uzRveIZmxeruyO?=
 =?us-ascii?Q?TXWQoS+xMZ3P1W4DDa41iuOkR+G9I6hyWh4A2aHZaBPNrxY+Lg5s9DfBX70U?=
 =?us-ascii?Q?Bccm8HsOiBuLzbt+gox7nwg3BpZn3zfFwXMicXdvoIrFYy3q9YM/nJ1EfGl2?=
 =?us-ascii?Q?5dCGjlU+wMhnucWBJ+qvqVykE5xzjhIjAbD1DgNKHjxfEyZxe61bJUfPiRfF?=
 =?us-ascii?Q?wf23aKI1vldRepbOgKABcjVISULYY472qAxGMQUrQPs1AEF5WWefpYHh/R6t?=
 =?us-ascii?Q?i4g1BzUG3iSJYmhT83zaixzRhW7h245JtMqDonVr5w4ZpENHFPRy7H1muCUf?=
 =?us-ascii?Q?yIj185cNV+WxvvYyvvv6zcWpWUXAmBTJjhnA8qyhhmkOaH8t4MaBf8E2k0Gj?=
 =?us-ascii?Q?/kkuFVTQGsi9Ug2hytZlz5ihZS8vvv6S/WJZSTYdWahy7KClKh4uLXa7eMOi?=
 =?us-ascii?Q?Ybd5IjtSqogL9mGOxoONyRuVYdgvSuoSBmFR4CCq1ti9xp+CJlGLMLeOqN9/?=
 =?us-ascii?Q?ulaSqPxirGuLK2rS1AuY+IQ4+i/GMN1OPaILRthGUeaqEyKgp+iZ7gel6/+9?=
 =?us-ascii?Q?IYQRVBwi80AVcVZE0keF5oq2wAOQWOj9KmDTvUFb5VLAsZi4446NyxdMhd5g?=
 =?us-ascii?Q?HqufhqoAxe1cgWgo0dMCNqkh2fjM3Pv7Q0uZciH+wUfSFKQVk/c8JhBq7d2e?=
 =?us-ascii?Q?dCPjPrvLXGcVKkYnOFP2iga6SWECacbKawQA+TuhVsKjdVlB1GoivkRk942m?=
 =?us-ascii?Q?GYRhGtWzxuJp8rabM2FrirJaXMAuL9Lfz1fqpLilVSgP/vZbQWpEriqIr/yV?=
 =?us-ascii?Q?7Rfj26cTYDkwZbTPRM5J9bSbf+fe7qBeCmISuDf07IgIH98VFRL3ZDUdFAGm?=
 =?us-ascii?Q?0qcVR9wojviin1BKB6I6tqni+Wc3BS/uMIOw8XgL/0za2tYlwXU4sRpQz4Z1?=
 =?us-ascii?Q?mK8Rxt+21P2rq61KzrbVM8dYowm+oLndO8hHk7Al3sUbiTE1xm0rMqkQ4PVu?=
 =?us-ascii?Q?KiqPtGNI+1qmmuVc1GdE+zC5P8YfUuoIgR6vfv1t8loLNd6o1FNgvsdrfk1D?=
 =?us-ascii?Q?+KRUmkrGOb23+5ENeyKWgqAUyEazuLmJ0UbCsUjbD+d04c37oSjnl4VHM5XP?=
 =?us-ascii?Q?eXTTjLBVwG6ypadz9TmEJS718PCtKeuEygKKTIM9tA=3D=3D?=
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB6575.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5f9fd6a4-65e1-439a-daee-08dacc796f78
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Nov 2022 11:05:15.9178
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: SXeLqR+gWyaAJHluBw8tB+N69mH8HlFAnfEytM28DvCYE3WRN5SxDq5Ts/lzsJV8lYmXh3Elhn0di0GXa2xCKw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR04MB5602
X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,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

> +               sg_cnt =3D dma_map_sg(hba->host->dma_dev, payload->sg_lis=
t,
> payload->sg_cnt, dir);
> +               if (unlikely(!sg_cnt))
> +                       return -ENOMEM;
> +               sg_list =3D payload->sg_list;
> +               data_len =3D payload->payload_len;
> +       }
Isn't bsg_map_buffer does that for you already?
For both request & reply?

Thanks,
Avri