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.
