scsi: qedf: Do not retry ELS request if qedf_alloc_cmd fails
[ Upstream commit
f1c43590365bac054d753d808dbbd207d09e088d ]
If we cannot allocate an ELS middlepath request, simply fail instead of
trying to delay and then reallocate. This delay logic is causing soft
lockup messages:
NMI watchdog: BUG: soft lockup - CPU#2 stuck for 22s! [kworker/2:1:7639]
Modules linked in: xt_CHECKSUM ipt_MASQUERADE nf_nat_masquerade_ipv4 tun devlink ip6t_rpfilter ipt_REJECT nf_reject_ipv4 ip6t_REJECT nf_reject_ipv6 xt_conntrack ip_set nfnetlink ebtable_nat ebtable_broute bridge stp llc ip6table_nat nf_conntrack_ipv6 nf_defrag_ipv6 nf_nat_ipv6 ip6table_mangle ip6table_security ip6table_raw iptable_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_nat_ipv4 nf_nat nf_conntrack iptable_mangle iptable_security iptable_raw ebtable_filter ebtables ip6table_filter ip6_tables iptable_filter dm_service_time vfat fat rpcrdma sunrpc ib_isert iscsi_target_mod ib_iser libiscsi scsi_transport_iscsi ib_srpt target_core_mod ib_srp scsi_transport_srp ib_ipoib rdma_ucm ib_ucm ib_uverbs ib_umad rdma_cm ib_cm iw_cm sb_edac intel_powerclamp coretemp intel_rapl iosf_mbi kvm_intel kvm
irqbypass crc32_pclmul ghash_clmulni_intel aesni_intel lrw gf128mul glue_helper ablk_helper cryptd iTCO_wdt iTCO_vendor_support qedr(OE) ib_core joydev ipmi_ssif pcspkr hpilo hpwdt sg ipmi_si ipmi_devintf ipmi_msghandler ioatdma shpchp lpc_ich wmi dca acpi_power_meter dm_multipath ip_tables xfs libcrc32c sd_mod crc_t10dif crct10dif_generic qedf(OE) libfcoe mgag200 libfc i2c_algo_bit drm_kms_helper scsi_transport_fc qede(OE) syscopyarea sysfillrect sysimgblt fb_sys_fops ttm qed(OE) drm crct10dif_pclmul e1000e crct10dif_common crc32c_intel scsi_tgt hpsa i2c_core ptp scsi_transport_sas pps_core dm_mirror dm_region_hash dm_log dm_mod
CPU: 2 PID: 7639 Comm: kworker/2:1 Kdump: loaded Tainted: G OEL ------------ 3.10.0-861.el7.x86_64 #1
Hardware name: HP ProLiant DL580 Gen9/ProLiant DL580 Gen9, BIOS U17 07/21/2016
Workqueue: qedf_2_dpc qedf_handle_rrq [qedf]
task:
ffff959edd628fd0 ti:
ffff959ed6f08000 task.ti:
ffff959ed6f08000
RIP: 0010:[<
ffffffff8355913a>] [<
ffffffff8355913a>] delay_tsc+0x3a/0x60
RSP: 0018:
ffff959ed6f0bd30 EFLAGS:
00000246
RAX:
000000008ef5f791 RBX:
5f646d635f666465 RCX:
0000025b8ededa2f
RDX:
000000000000025b RSI:
0000000000000002 RDI:
0000000000217d1e
RBP:
ffff959ed6f0bd30 R08:
ffffffffc079aae8 R09:
0000000000000200
R10:
ffffffffc07952c6 R11:
0000000000000000 R12:
6c6c615f66646571
R13:
ffff959ed6f0bcc8 R14:
ffff959ed6f0bd08 R15:
ffff959e00000028
FS:
0000000000000000(0000) GS:
ffff959eff480000(0000) knlGS:
0000000000000000
CS: 0010 DS: 0000 ES: 0000 CR0:
0000000080050033
CR2:
00007f4117fa1eb0 CR3:
0000002039e66000 CR4:
00000000003607e0
DR0:
0000000000000000 DR1:
0000000000000000 DR2:
0000000000000000
DR3:
0000000000000000 DR6:
00000000fffe0ff0 DR7:
0000000000000400
Call Trace:
[<
ffffffff8355907d>] __const_udelay+0x2d/0x30
[<
ffffffffc079444a>] qedf_initiate_els+0x13a/0x450 [qedf]
[<
ffffffffc0794210>] ? qedf_srr_compl+0x2a0/0x2a0 [qedf]
[<
ffffffffc0795337>] qedf_send_rrq+0x127/0x230 [qedf]
[<
ffffffffc078ed55>] qedf_handle_rrq+0x15/0x20 [qedf]
[<
ffffffff832b2dff>] process_one_work+0x17f/0x440
[<
ffffffff832b3ac6>] worker_thread+0x126/0x3c0
[<
ffffffff832b39a0>] ? manage_workers.isra.24+0x2a0/0x2a0
[<
ffffffff832bae31>] kthread+0xd1/0xe0
[<
ffffffff832bad60>] ? insert_kthread_work+0x40/0x40
[<
ffffffff8391f637>] ret_from_fork_nospec_begin+0x21/0x21
[<
ffffffff832bad60>] ? insert_kthread_work+0x40/0x40
Signed-off-by: Chad Dupuis <cdupuis@marvell.com>
Signed-off-by: Saurav Kashyap <skashyap@marvell.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>