Bart, We have these patches internally reviewed, so is it alright to keep the reviewed-by tag for internal reviewer, Arun Easi. Thanks, Nilesh > -----Original Message----- > From: Nilesh Javali <njavali@xxxxxxxxxxx> > Sent: Tuesday, June 9, 2020 11:31 AM > To: Bart Van Assche <bvanassche@xxxxxxx>; martin.petersen@xxxxxxxxxx > Cc: linux-scsi@xxxxxxxxxxxxxxx; GR-QLogic-Storage-Upstream <GR-QLogic- > Storage-Upstream@xxxxxxxxxxx> > Subject: RE: [EXT] Re: [PATCH v2 1/2] qla2xxx: Change in PUREX to handle > FPIN ELS requests. > > Bart, > > Sure, I would remove the tags and re-post the patch set. > > Thanks, > Nilesh > > > -----Original Message----- > > From: Bart Van Assche <bvanassche@xxxxxxx> > > Sent: Tuesday, June 9, 2020 1:07 AM > > To: Nilesh Javali <njavali@xxxxxxxxxxx>; martin.petersen@xxxxxxxxxx > > Cc: linux-scsi@xxxxxxxxxxxxxxx; GR-QLogic-Storage-Upstream <GR-QLogic- > > Storage-Upstream@xxxxxxxxxxx> > > Subject: [EXT] Re: [PATCH v2 1/2] qla2xxx: Change in PUREX to handle FPIN > > ELS requests. > > > > External Email > > > > ---------------------------------------------------------------------- > > On 2020-06-08 11:46, Nilesh Javali wrote: > > > From: Shyam Sundar <ssundar@xxxxxxxxxxx> > > > > > > SAN Congestion Management generates ELS pkts whose size > > > can vary, and be > 64 bytes. Change the purex > > > handling code to support non standard ELS pkt size. > > > > > > Signed-off-by: Shyam Sundar <ssundar@xxxxxxxxxxx> > > > Reviewed-by: Arun Easi <aeasi@xxxxxxxxxxx> > > > Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx> > > > Reviewed-by: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx> > > > Reviewed-by: James Smart <james.smart@xxxxxxxxxxxx> > > > Signed-off-by: Nilesh Javali <njavali@xxxxxxxxxxx> > > > > Has anyone other than Himanshu ever posted a positive review for this > > patch? I can't find the other review tags here: > > https://urldefense.proofpoint.com/v2/url?u=https- > > 3A__lore.kernel.org_linux-2Dscsi_20200514101026.10040-2D1-2Dnjavali- > > > 40marvell.com_&d=DwICaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=FAW9wuzbtH > > IZL7SV63sr8rG59Hctu-eGu0G9pxwOXgQ&m=WrKsjn8ilzVn32q- > > > 9eMHbnUUvl1cpj06n4fKq8KfBhc&s=gm2J0Gh3hotgenLae9uFvfCiD9MK7iYGn > > aYhYrnSwzs&e= > > > > Bart.
