From: Quinn Tran Date: Sat, 24 Dec 2016 02:06:14 +0000 (-0800) Subject: qla2xxx: Disable out-of-order processing by default in firmware X-Git-Tag: v4.1.12-117.0_27200813_3~80 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=fb707b8b5cbd9145a42e80ab59aa17966ea680c5;p=users%2Fjedix%2Flinux-maple.git qla2xxx: Disable out-of-order processing by default in firmware Orabug: 26844197, 26923029 Out of order(OOO) processing requires initiator, switch and target to support OOO. In today's environment, none of the switches support OOO. OOO requires extra buffer space which affect performance. By turning ON this feature in QLogic's FW, it delays error recovery because dropped frame is treated as out of order frame. We're turning OFF this option of speed up error recovery. Signed-off-by: Quinn Tran Signed-off-by: Himanshu Madhani Reviewed-by: Christoph Hellwig [ bvanassche: Fixed spelling in patch description ] Signed-off-by: Bart Van Assche Signed-off-by: Brian Maly Reviewed-by: Jack Vogel --- diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c index b7ad8f6066fe..7681a0364748 100644 --- a/drivers/scsi/qla2xxx/qla_target.c +++ b/drivers/scsi/qla2xxx/qla_target.c @@ -6566,9 +6566,6 @@ qlt_24xx_config_nvram_stage1(struct scsi_qla_host *vha, struct nvram_24xx *nv) return; } - /* out-of-order frames reassembly */ - nv->firmware_options_3 |= BIT_6|BIT_9; - if (ha->tgt.enable_class_2) { if (vha->flags.init_done) fc_host_supported_classes(vha->host) = @@ -6661,9 +6658,6 @@ qlt_81xx_config_nvram_stage1(struct scsi_qla_host *vha, struct nvram_81xx *nv) return; } - /* out-of-order frames reassembly */ - nv->firmware_options_3 |= BIT_6|BIT_9; - if (ha->tgt.enable_class_2) { if (vha->flags.init_done) fc_host_supported_classes(vha->host) =