Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   "Bean Huo (beanhuo)" <beanhuo@micron.com>
To:     Avri Altman <Avri.Altman@wdc.com>,
        "huobean@gmail.com" <huobean@gmail.com>,
        "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        "asutoshd@codeaurora.org" <asutoshd@codeaurora.org>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "stanley.chu@mediatek.com" <stanley.chu@mediatek.com>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "tomas.winkler@intel.com" <tomas.winkler@intel.com>,
        "cang@codeaurora.org" <cang@codeaurora.org>
CC:     "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [EXT] RE: [PATCH 0/1] Revert "scsi: ufs: Let the SCSI core
 allocate per-command UFS data"
Thread-Topic: [EXT] RE: [PATCH 0/1] Revert "scsi: ufs: Let the SCSI core
 allocate per-command UFS data"
Thread-Index: AQHV96A3H5vu8h64PUWShi5i74/3jahDVomg
Date:   Wed, 11 Mar 2020 12:53:07 +0000
Message-ID: <BN7PR08MB5684DBAD57C95A40CB62B24ADBFC0@BN7PR08MB5684.namprd08.prod.outlook.com>
References: <20200311112921.29031-1-beanhuo@micron.com>
 <SN6PR04MB46404175998962B4FA575824FCFC0@SN6PR04MB4640.namprd04.prod.outlook.com>
In-Reply-To: <SN6PR04MB46404175998962B4FA575824FCFC0@SN6PR04MB4640.namprd04.prod.outlook.com>
Accept-Language: en-150, en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: micron.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 404d9a00-1133-407d-fe75-08d7c5bb257a
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Mar 2020 12:53:07.2289
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: f38a5ecd-2813-4862-b11b-ac1d563c806f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: j+Crf06BNVMz0taRzQ0xjwt+e/qrBH4cliKd1W+Lid5+xRu24zF1QHm8NeMC1WBoP6uoau6rd6sxk06sOk5lPQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7PR08MB4769
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi, Avri
Do you want to revert all or two (scsi: ufs: Let the SCSI core allocate per=
-command UFS data,   scsi: core: Introduce {init,exit}_cmd_priv() )?
Because the patches "scsi: ufs: Simplify two tests" and  "scsi: ufs: Introd=
uce ufshcd_init_lrb()" are ok to me.
No problem keeping them. Just this one "scsi: core: Introduce {init,exit}_c=
md_priv()" is not necessary, since no drivers it now.=20

//Bean


>=20
> Hi,
>=20
> >
> > From: Bean Huo <beanhuo@micron.com>
> >
> > Hi, Martin
> >
> > Based on Bart's feedack, the less risky way is to revert commit:
> >
> > 34656dda81ac "scsi: ufs: Let the SCSI core allocate per-command UFS dat=
a"
> >
> > Bean Huo (1):
> >   Revert "scsi: ufs: Let the SCSI core allocate per-command UFS data"
> Maybe it's safer to revert the entire series?
>=20
> Thanks,
> Avri
>=20
>=20
> >
> >  drivers/scsi/ufs/ufshcd.c | 198 ++++++++++++++------------------------
> >  drivers/scsi/ufs/ufshcd.h |   5 +
> >  2 files changed, 75 insertions(+), 128 deletions(-)
> >
> > --
> > 2.17.1