[5.15,030/124] igc: Fix BUG: scheduling while atomic

From: Sasha Neftin <sasha.neftin@intel.com>

From: Sasha Neftin <sasha.neftin@intel.com>

[ Upstream commit c80a29f0fe9b6f5457e0788e27d1110577eba99b ]

Replace usleep_range() method with udelay() method to allow atomic contexts
in low-level MDIO access functions.

The following issue can be seen by doing the following:
$ modprobe -r bonding
$ modprobe -v bonding max_bonds=1 mode=1 miimon=100 use_carrier=0
$ ip link set bond0 up
$ ifenslave bond0 eth0 eth1

[  982.357308] BUG: scheduling while atomic: kworker/u64:0/9/0x00000002
[  982.364431] INFO: lockdep is turned off.
[  982.368824] Modules linked in: bonding sctp ip6_udp_tunnel udp_tunnel mlx4_ib ib_uverbs ib_core mlx4_en mlx4_core nfp tls sunrpc intel_rapl_msr iTCO_wdt iTCO_vendor_support mxm_wmi dcdbas intel_rapl_common sb_edac x86_pkg_temp_thermal intel_powerclamp coretemp kvm_intel kvm irqbypass crct10dif_pclmul crc32_pclmul ghash_clmulni_intel rapl intel_cstate intel_uncore pcspkr lpc_ich mei_me ipmi_ssif mei ipmi_si ipmi_devintf ipmi_msghandler wmi acpi_power_meter xfs libcrc32c sr_mod cdrom sd_mod t10_pi sg mgag200 drm_kms_helper syscopyarea sysfillrect sysimgblt fb_sys_fops drm ahci libahci crc32c_intel libata i2c_algo_bit tg3 megaraid_sas igc dm_mirror dm_region_hash dm_log dm_mod [last unloaded: bonding]
[  982.437941] CPU: 25 PID: 9 Comm: kworker/u64:0 Kdump: loaded Tainted: G        W        --------- -  - 4.18.0-348.el8.x86_64+debug #1
[  982.451333] Hardware name: Dell Inc. PowerEdge R730/0H21J3, BIOS 2.7.0 12/005/2017
[  982.459791] Workqueue: bond0 bond_mii_monitor [bonding]
[  982.465622] Call Trace:
[  982.468355]  dump_stack+0x8e/0xd0
[  982.472056]  __schedule_bug.cold.60+0x3a/0x60
[  982.476919]  __schedule+0x147b/0x1bc0
[  982.481007]  ? firmware_map_remove+0x16b/0x16b
[  982.485967]  ? hrtimer_fixup_init+0x40/0x40
[  982.490625]  schedule+0xd9/0x250
[  982.494227]  schedule_hrtimeout_range_clock+0x10d/0x2c0
[  982.500058]  ? hrtimer_nanosleep_restart+0x130/0x130
[  982.505598]  ? hrtimer_init_sleeper_on_stack+0x90/0x90
[  982.511332]  ? usleep_range+0x88/0x130
[  982.515514]  ? recalibrate_cpu_khz+0x10/0x10
[  982.520279]  ? ktime_get+0xab/0x1c0
[  982.524175]  ? usleep_range+0x88/0x130
[  982.528355]  usleep_range+0xdd/0x130
[  982.532344]  ? console_conditional_schedule+0x30/0x30
[  982.537987]  ? igc_put_hw_semaphore+0x17/0x60 [igc]
[  982.543432]  igc_read_phy_reg_gpy+0x111/0x2b0 [igc]
[  982.548887]  igc_phy_has_link+0xfa/0x260 [igc]
[  982.553847]  ? igc_get_phy_id+0x210/0x210 [igc]
[  982.558894]  ? lock_acquire+0x34d/0x890
[  982.563187]  ? lock_downgrade+0x710/0x710
[  982.567659]  ? rcu_read_unlock+0x50/0x50
[  982.572039]  igc_check_for_copper_link+0x106/0x210 [igc]
[  982.577970]  ? igc_config_fc_after_link_up+0x840/0x840 [igc]
[  982.584286]  ? rcu_read_unlock+0x50/0x50
[  982.588661]  ? lock_release+0x591/0xb80
[  982.592939]  ? lock_release+0x591/0xb80
[  982.597220]  igc_has_link+0x113/0x330 [igc]
[  982.601887]  ? lock_downgrade+0x710/0x710
[  982.606362]  igc_ethtool_get_link+0x6d/0x90 [igc]
[  982.611614]  bond_check_dev_link+0x131/0x2c0 [bonding]
[  982.617350]  ? bond_time_in_interval+0xd0/0xd0 [bonding]
[  982.623277]  ? rcu_read_lock_held+0x62/0xc0
[  982.627944]  ? rcu_read_lock_sched_held+0xe0/0xe0
[  982.633198]  bond_mii_monitor+0x314/0x2500 [bonding]
[  982.638738]  ? lock_contended+0x880/0x880
[  982.643214]  ? bond_miimon_link_change+0xa0/0xa0 [bonding]
[  982.649336]  ? lock_acquire+0x34d/0x890
[  982.653615]  ? lock_downgrade+0x710/0x710
[  982.658089]  ? debug_object_deactivate+0x221/0x340
[  982.663436]  ? rcu_read_unlock+0x50/0x50
[  982.667811]  ? debug_print_object+0x2b0/0x2b0
[  982.672672]  ? __switch_to_asm+0x41/0x70
[  982.677049]  ? __switch_to_asm+0x35/0x70
[  982.681426]  ? _raw_spin_unlock_irq+0x24/0x40
[  982.686288]  ? trace_hardirqs_on+0x20/0x195
[  982.690956]  ? _raw_spin_unlock_irq+0x24/0x40
[  982.695818]  process_one_work+0x8f0/0x1770
[  982.700390]  ? pwq_dec_nr_in_flight+0x320/0x320
[  982.705443]  ? debug_show_held_locks+0x50/0x50
[  982.710403]  worker_thread+0x87/0xb40
[  982.714489]  ? process_one_work+0x1770/0x1770
[  982.719349]  kthread+0x344/0x410
[  982.722950]  ? kthread_insert_work_sanity_check+0xd0/0xd0
[  982.728975]  ret_from_fork+0x3a/0x50

Fixes: 5586838fe9ce ("igc: Add code for PHY support")
Reported-by: Corinna Vinschen <vinschen@redhat.com>
Suggested-by: Dima Ruinskiy <dima.ruinskiy@intel.com>
Signed-off-by: Sasha Neftin <sasha.neftin@intel.com>
Tested-by: Corinna Vinschen <vinschen@redhat.com>
Tested-by: Naama Meir <naamax.meir@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 drivers/net/ethernet/intel/igc/igc_phy.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Message ID	20220426081748.159788232@linuxfoundation.org
State	Superseded
Headers	show Return-Path: <stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, Corinna Vinschen <vinschen@redhat.com>, Dima Ruinskiy <dima.ruinskiy@intel.com>, Sasha Neftin <sasha.neftin@intel.com>, Naama Meir <naamax.meir@linux.intel.com>, Tony Nguyen <anthony.l.nguyen@intel.com>, Sasha Levin <sashal@kernel.org> Subject: [PATCH 5.15 030/124] igc: Fix BUG: scheduling while atomic Date: Tue, 26 Apr 2022 10:20:31 +0200 Message-Id: <20220426081748.159788232@linuxfoundation.org> In-Reply-To: <20220426081747.286685339@linuxfoundation.org> References: <20220426081747.286685339@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	None \| expand [5.15,002/124] block: remove __sync_blockdev [5.15,003/124] block: simplify the block device syncing code [5.15,004/124] vfs: make sync_filesystem return errors from ->sync_fs [5.15,005/124] xfs: return errors in xfs_fs_sync_fs [5.15,006/124] dma-mapping: remove bogus test for pfn_valid from dma_map_resource [5.15,007/124] arm64/mm: drop HAVE_ARCH_PFN_VALID [5.15,008/124] etherdevice: Adjust ether_addr* prototypes to silence -Wstringop-overead [5.15,009/124] mm: page_alloc: fix building error on -Werror=array-compare [5.15,010/124] perf tools: Fix segfault accessing sample_id xyarray [5.15,011/124] mm, kfence: support kmem_dump_obj() for KFENCE objects [5.15,012/124] gfs2: assign rgrp glock before compute_bitstructs [5.15,013/124] scsi: ufs: core: scsi_get_lba() error fix [5.15,014/124] net/sched: cls_u32: fix netns refcount changes in u32_change() [5.15,015/124] ALSA: usb-audio: Clear MIDI port active flag after draining [5.15,016/124] ALSA: hda/realtek: Add quirk for Clevo NP70PNP [5.15,017/124] ASoC: atmel: Remove system clock tree configuration for at91sam9g20ek [5.15,018/124] ASoC: topology: Correct error handling in soc_tplg_dapm_widget_create() [5.15,019/124] ASoC: rk817: Use devm_clk_get() in rk817_platform_probe [5.15,020/124] ASoC: msm8916-wcd-digital: Check failure for devm_snd_soc_register_component [5.15,021/124] ASoC: codecs: wcd934x: do not switch off SIDO Buck when codec is in use [5.15,022/124] dmaengine: idxd: fix device cleanup on disable [5.15,023/124] dmaengine: imx-sdma: Fix error checking in sdma_event_remap [5.15,024/124] dmaengine: mediatek:Fix PM usage reference leak of mtk_uart_apdma_alloc_chan_resou... [5.15,025/124] dmaengine: dw-edma: Fix unaligned 64bit access [5.15,026/124] spi: spi-mtk-nor: initialize spi controller after resume [5.15,027/124] esp: limit skb_page_frag_refill use to a single page [5.15,028/124] spi: cadence-quadspi: fix incorrect supports_op() return value [5.15,029/124] igc: Fix infinite loop in release_swfw_sync [5.15,030/124] igc: Fix BUG: scheduling while atomic [5.15,031/124] igc: Fix suspending when PTM is active [5.15,032/124] ALSA: hda/hdmi: fix warning about PCM count when used with SOF [5.15,033/124] rxrpc: Restore removed timer deletion [5.15,034/124] net/smc: Fix sock leak when release after smc_shutdown() [5.15,035/124] net/packet: fix packet_sock xmit return value checking [5.15,036/124] ip6_gre: Avoid updating tunnel->tun_hlen in __gre6_xmit() [5.15,037/124] ip6_gre: Fix skb_under_panic in __gre6_xmit() [5.15,038/124] net: restore alpha order to Ethernet devices in config [5.15,039/124] net/sched: cls_u32: fix possible leak in u32_init_knode() [5.15,040/124] l3mdev: l3mdev_master_upper_ifindex_by_index_rcu should be using netdev_master_upp... [5.15,041/124] ipv6: make ip6_rt_gc_expire an atomic_t [5.15,042/124] can: isotp: stop timeout monitoring when no first frame was sent [5.15,043/124] net: dsa: hellcreek: Calculate checksums in tagger [5.15,044/124] net: mscc: ocelot: fix broken IP multicast flooding [5.15,045/124] netlink: reset network and mac headers in netlink_dump() [5.15,046/124] drm/i915/display/psr: Unset enable_psr2_sel_fetch if other checks in intel_psr2_co... [5.15,047/124] net: stmmac: Use readl_poll_timeout_atomic() in atomic state [5.15,048/124] dmaengine: idxd: add RO check for wq max_batch_size write [5.15,049/124] dmaengine: idxd: add RO check for wq max_transfer_size write [5.15,050/124] dmaengine: idxd: skip clearing device context when device is read-only [5.15,051/124] selftests: mlxsw: vxlan_flooding: Prevent flooding of unwanted packets [5.15,052/124] arm64: mm: fix p?d_leaf() [5.15,053/124] ARM: vexpress/spc: Avoid negative array index when !SMP [5.15,054/124] reset: renesas: Check return value of reset_control_deassert() [5.15,055/124] reset: tegra-bpmp: Restore Handle errors in BPMP response [5.15,056/124] platform/x86: samsung-laptop: Fix an unsigned comparison which can never be negative [5.15,057/124] ALSA: usb-audio: Fix undefined behavior due to shift overflowing the constant [5.15,058/124] drm/msm/disp: check the return value of kzalloc() [5.15,059/124] arm64: dts: imx: Fix imx8*-var-som touchscreen property sizes [5.15,060/124] vxlan: fix error return code in vxlan_fdb_append [5.15,061/124] cifs: Check the IOCB_DIRECT flag, not O_DIRECT [5.15,062/124] net: atlantic: Avoid out-of-bounds indexing [5.15,063/124] mt76: Fix undefined behavior due to shift overflowing the constant [5.15,064/124] brcmfmac: sdio: Fix undefined behavior due to shift overflowing the constant [5.15,065/124] dpaa_eth: Fix missing of_node_put in dpaa_get_ts_info() [5.15,066/124] drm/msm/mdp5: check the return of kzalloc() [5.15,067/124] net: macb: Restart tx only if queue pointer is lagging [5.15,068/124] scsi: iscsi: Release endpoint ID when its freed [5.15,069/124] scsi: iscsi: Merge suspend fields [5.15,070/124] scsi: iscsi: Fix NOP handling during conn recovery [5.15,071/124] scsi: qedi: Fix failed disconnect handling [5.15,072/124] stat: fix inconsistency between struct stat and struct compat_stat [5.15,073/124] VFS: filename_create(): fix incorrect intent. [5.15,074/124] nvme: add a quirk to disable namespace identifiers [5.15,075/124] nvme-pci: disable namespace identifiers for the MAXIO MAP1002/1202 [5.15,076/124] nvme-pci: disable namespace identifiers for Qemu controllers [5.15,077/124] EDAC/synopsys: Read the error count from the correct register [5.15,078/124] mm/memory-failure.c: skip huge_zero_page in memory_failure() [5.15,079/124] memcg: sync flush only if periodic flush is delayed [5.15,080/124] mm, hugetlb: allow for "high" userspace addresses [5.15,081/124] oom_kill.c: futex: delay the OOM reaper to allow time for proper futex cleanup [5.15,082/124] mm/mmu_notifier.c: fix race in mmu_interval_notifier_remove() [5.15,083/124] ata: pata_marvell: Check the bmdma_addr beforing reading [5.15,084/124] dma: at_xdmac: fix a missing check on list iterator [5.15,085/124] dmaengine: imx-sdma: fix init of uart scripts [5.15,086/124] net: atlantic: invert deep par in pm functions, preventing null derefs [5.15,087/124] Input: omap4-keypad - fix pm_runtime_get_sync() error checking [5.15,088/124] scsi: sr: Do not leak information in ioctl [5.15,089/124] sched/pelt: Fix attach_entity_load_avg() corner case [5.15,090/124] perf/core: Fix perf_mmap fail when CONFIG_PERF_USE_VMALLOC enabled [5.15,091/124] drm/panel/raspberrypi-touchscreen: Avoid NULL deref if not initialised [5.15,092/124] drm/panel/raspberrypi-touchscreen: Initialise the bridge in prepare [5.15,093/124] KVM: PPC: Fix TCE handling for VFIO [5.15,094/124] drm/vc4: Use pm_runtime_resume_and_get to fix pm_runtime_get_sync() usage [5.15,095/124] powerpc/perf: Fix power9 event alternatives [5.15,096/124] powerpc/perf: Fix power10 event alternatives [5.15,097/124] perf script: Always allow field data_src for auxtrace [5.15,098/124] perf report: Set PERF_SAMPLE_DATA_SRC bit for Arm SPE event [5.15,099/124] xtensa: patch_text: Fixup last cpu should be master [5.15,100/124] xtensa: fix a7 clobbering in coprocessor context load/store [5.15,101/124] openvswitch: fix OOB access in reserve_sfa_size() [5.15,102/124] gpio: Request interrupts after IRQ is initialized [5.15,103/124] ASoC: soc-dapm: fix two incorrect uses of list iterator [5.15,104/124] e1000e: Fix possible overflow in LTR decoding [5.15,105/124] ARC: entry: fix syscall_trace_exit argument [5.15,106/124] arm_pmu: Validate single/group leader events [5.15,107/124] KVM: x86/pmu: Update AMD PMC sample period to fix guest NMI-watchdog [5.15,108/124] KVM: x86: Pend KVM_REQ_APICV_UPDATE during vCPU creation to fix a race [5.15,109/124] KVM: nVMX: Defer APICv updates while L2 is active until L1 is active [5.15,110/124] KVM: SVM: Flush when freeing encrypted pages even on SME_COHERENT CPUs [5.15,111/124] netfilter: conntrack: convert to refcount_t api [5.15,112/124] netfilter: conntrack: avoid useless indirection during conntrack destruction [5.15,113/124] ext4: fix fallocate to use file_modified to update permissions consistently [5.15,114/124] ext4: fix symlink file size not match to file content [5.15,115/124] ext4: fix use-after-free in ext4_search_dir [5.15,116/124] ext4: limit length to bitmap_maxbytes - blocksize in punch_hole [5.15,117/124] ext4, doc: fix incorrect h_reserved size [5.15,118/124] ext4: fix overhead calculation to account for the reserved gdt blocks [5.15,119/124] ext4: force overhead calculation if the s_overhead_cluster makes no sense [5.15,120/124] netfilter: nft_ct: fix use after free when attaching zone template [5.15,121/124] jbd2: fix a potential race while discarding reserved buffers after an abort [5.15,122/124] spi: atmel-quadspi: Fix the buswidth adjustment between spi-mem and controller [5.15,123/124] block/compat_ioctl: fix range check in BLKGETSIZE [5.15,124/124] arm64: dts: qcom: add IPA qcom,qmp property

[5.15,030/124] igc: Fix BUG: scheduling while atomic

Commit Message

Patch