* [REGRESSION] iwlwifi: suspend failure, bisected
@ 2025-05-31 14:09 Chris Bainbridge
2025-05-31 18:35 ` Korenblit, Miriam Rachel
2025-07-16 18:50 ` Bjorn Helgaas
0 siblings, 2 replies; 7+ messages in thread
From: Chris Bainbridge @ 2025-05-31 14:09 UTC (permalink / raw)
To: Gustavo A. R. Silva
Cc: linux-wireless@vger.kernel.org, kvalo@kernel.org,
Korenblit, Miriam Rachel, Berg, Johannes,
benjamin@sipsolutions.net, regressions@lists.linux.dev
Hi,
There is a recently introduced iwlwifi regression on suspend. It appears
the bisected commit was only supposed to fix a warning, but reverting
the commit (with a small change for revert conflict) does fix the error.
7438843df8cf7a484303caee7540a00f0ef972b2 is the first bad commit
Gustavo A. R. Silva <gustavoars@kernel.org>commit 7438843df8cf7a484303caee7540a00f0ef972b2
Author: Gustavo A. R. Silva <gustavoars@kernel.org>
Date: Wed Mar 26 18:04:02 2025 -0600
wifi: iwlwifi: mvm: Avoid -Wflex-array-member-not-at-end warning
-Wflex-array-member-not-at-end was introduced in GCC-14, and we are
getting ready to enable it, globally.
Use the `DEFINE_RAW_FLEX()` helper for an on-stack definition of
a flexible structure where the size of the flexible-array member
is known at compile-time, and refactor the rest of the code,
accordingly.
So, with these changes, fix the following warning:
drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c:6430:41: warning: structure containing a flexible array member is not at the end of another structure [-Wflex-array-member-not-at-end]
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Acked-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Link: https://patch.msgid.link/Z-SV8gb6MuZJmmhe@kspp
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
[ 88.149152] PM: suspend entry (s2idle)
[ 88.203279] Filesystems sync: 0.054 seconds
[ 88.207029] Freezing user space processes
[ 88.209603] Freezing user space processes completed (elapsed 0.002 seconds)
[ 88.209646] OOM killer disabled.
[ 88.209649] Freezing remaining freezable tasks
[ 88.210556] Freezing remaining freezable tasks completed (elapsed 0.000 seconds)
[ 88.210703] printk: Suspending console(s) (use no_console_suspend to debug)
[ 88.797416] wlp1s0: deauthenticating from d6:92:5e:eb:ee:15 by local choice (Reason: 3=DEAUTH_LEAVING)
[ 88.811761] iwlwifi 0000:01:00.0: Microcode SW error detected. Restarting 0x0.
[ 88.811990] iwlwifi 0000:01:00.0: Start IWL Error Log Dump:
[ 88.811993] iwlwifi 0000:01:00.0: Transport status: 0x0000004B, valid: 6
[ 88.811995] iwlwifi 0000:01:00.0: Loaded firmware version: 89.202a2f7b.0 ty-a0-gf-a0-89.ucode
[ 88.811997] iwlwifi 0000:01:00.0: 0x00000071 | NMI_INTERRUPT_UMAC_FATAL
[ 88.812000] iwlwifi 0000:01:00.0: 0x000002F0 | trm_hw_status0
[ 88.812001] iwlwifi 0000:01:00.0: 0x00000000 | trm_hw_status1
[ 88.812003] iwlwifi 0000:01:00.0: 0x004DAD6C | branchlink2
[ 88.812004] iwlwifi 0000:01:00.0: 0x004D0176 | interruptlink1
[ 88.812006] iwlwifi 0000:01:00.0: 0x004D0176 | interruptlink2
[ 88.812007] iwlwifi 0000:01:00.0: 0x00016AD2 | data1
[ 88.812008] iwlwifi 0000:01:00.0: 0x00000010 | data2
[ 88.812010] iwlwifi 0000:01:00.0: 0x00000000 | data3
[ 88.812011] iwlwifi 0000:01:00.0: 0xB100EB58 | beacon time
[ 88.812013] iwlwifi 0000:01:00.0: 0x926B5499 | tsf low
[ 88.812014] iwlwifi 0000:01:00.0: 0x000009F8 | tsf hi
[ 88.812015] iwlwifi 0000:01:00.0: 0x00000000 | time gp1
[ 88.812017] iwlwifi 0000:01:00.0: 0x048BA562 | time gp2
[ 88.812018] iwlwifi 0000:01:00.0: 0x00000001 | uCode revision type
[ 88.812019] iwlwifi 0000:01:00.0: 0x00000059 | uCode version major
[ 88.812021] iwlwifi 0000:01:00.0: 0x202A2F7B | uCode version minor
[ 88.812022] iwlwifi 0000:01:00.0: 0x00000420 | hw version
[ 88.812023] iwlwifi 0000:01:00.0: 0x58C80002 | board version
[ 88.812025] iwlwifi 0000:01:00.0: 0x80B5FC4E | hcmd
[ 88.812026] iwlwifi 0000:01:00.0: 0x00020000 | isr0
[ 88.812027] iwlwifi 0000:01:00.0: 0x00000000 | isr1
[ 88.812028] iwlwifi 0000:01:00.0: 0x48F04802 | isr2
[ 88.812030] iwlwifi 0000:01:00.0: 0x00C3000C | isr3
[ 88.812031] iwlwifi 0000:01:00.0: 0x00000000 | isr4
[ 88.812032] iwlwifi 0000:01:00.0: 0x00860103 | last cmd Id
[ 88.812033] iwlwifi 0000:01:00.0: 0x00016AD2 | wait_event
[ 88.812035] iwlwifi 0000:01:00.0: 0x000000D4 | l2p_control
[ 88.812036] iwlwifi 0000:01:00.0: 0x00000000 | l2p_duration
[ 88.812037] iwlwifi 0000:01:00.0: 0x00000007 | l2p_mhvalid
[ 88.812039] iwlwifi 0000:01:00.0: 0x00000048 | l2p_addr_match
[ 88.812040] iwlwifi 0000:01:00.0: 0x00000008 | lmpm_pmg_sel
[ 88.812041] iwlwifi 0000:01:00.0: 0x00000000 | timestamp
[ 88.812043] iwlwifi 0000:01:00.0: 0x0000A81C | flow_handler
[ 88.812158] iwlwifi 0000:01:00.0: Start IWL Error Log Dump:
[ 88.812159] iwlwifi 0000:01:00.0: Transport status: 0x0000004B, valid: 7
[ 88.812161] iwlwifi 0000:01:00.0: 0x20101D04 | ADVANCED_SYSASSERT
[ 88.812162] iwlwifi 0000:01:00.0: 0x00000000 | umac branchlink1
[ 88.812164] iwlwifi 0000:01:00.0: 0x804838B2 | umac branchlink2
[ 88.812165] iwlwifi 0000:01:00.0: 0xC00814E0 | umac interruptlink1
[ 88.812166] iwlwifi 0000:01:00.0: 0x00000000 | umac interruptlink2
[ 88.812168] iwlwifi 0000:01:00.0: 0x00000000 | umac data1
[ 88.812169] iwlwifi 0000:01:00.0: 0x00000000 | umac data2
[ 88.812170] iwlwifi 0000:01:00.0: 0x00000004 | umac data3
[ 88.812171] iwlwifi 0000:01:00.0: 0x00000059 | umac major
[ 88.812173] iwlwifi 0000:01:00.0: 0x202A2F7B | umac minor
[ 88.812174] iwlwifi 0000:01:00.0: 0x048BA55D | frame pointer
[ 88.812175] iwlwifi 0000:01:00.0: 0xC0886BE8 | stack pointer
[ 88.812177] iwlwifi 0000:01:00.0: 0x008D0502 | last host cmd
[ 88.812178] iwlwifi 0000:01:00.0: 0x00000000 | isr status reg
[ 88.812273] iwlwifi 0000:01:00.0: IML/ROM dump:
[ 88.812274] iwlwifi 0000:01:00.0: 0x00000B03 | IML/ROM error/state
[ 88.812379] iwlwifi 0000:01:00.0: 0x00008D5E | IML/ROM data1
[ 88.812436] iwlwifi 0000:01:00.0: 0x00000090 | IML/ROM WFPM_AUTH_KEY_0
[ 88.812539] iwlwifi 0000:01:00.0: Fseq Registers:
[ 88.812594] iwlwifi 0000:01:00.0: 0x60000000 | FSEQ_ERROR_CODE
[ 88.812641] iwlwifi 0000:01:00.0: 0x80440007 | FSEQ_TOP_INIT_VERSION
[ 88.812691] iwlwifi 0000:01:00.0: 0x00080009 | FSEQ_CNVIO_INIT_VERSION
[ 88.812742] iwlwifi 0000:01:00.0: 0x0000A652 | FSEQ_OTP_VERSION
[ 88.812787] iwlwifi 0000:01:00.0: 0x00000002 | FSEQ_TOP_CONTENT_VERSION
[ 88.812893] iwlwifi 0000:01:00.0: 0x4552414E | FSEQ_ALIVE_TOKEN
[ 88.812943] iwlwifi 0000:01:00.0: 0x00400410 | FSEQ_CNVI_ID
[ 88.812989] iwlwifi 0000:01:00.0: 0x00400410 | FSEQ_CNVR_ID
[ 88.813040] iwlwifi 0000:01:00.0: 0x00400410 | CNVI_AUX_MISC_CHIP
[ 88.813088] iwlwifi 0000:01:00.0: 0x00400410 | CNVR_AUX_MISC_CHIP
[ 88.813136] iwlwifi 0000:01:00.0: 0x00009061 | CNVR_SCU_SD_REGS_SD_REG_DIG_DCDC_VTRIM
[ 88.813141] iwlwifi 0000:01:00.0: 0x00000061 | CNVR_SCU_SD_REGS_SD_REG_ACTIVE_VDIG_MIRROR
[ 88.813144] iwlwifi 0000:01:00.0: 0x00080009 | FSEQ_PREV_CNVIO_INIT_VERSION
[ 88.813147] iwlwifi 0000:01:00.0: 0x00440007 | FSEQ_WIFI_FSEQ_VERSION
[ 88.813149] iwlwifi 0000:01:00.0: 0x00440007 | FSEQ_BT_FSEQ_VERSION
[ 88.813152] iwlwifi 0000:01:00.0: 0x000000E6 | FSEQ_CLASS_TP_VERSION
[ 88.813209] iwlwifi 0000:01:00.0: UMAC CURRENT PC: 0x804a5578
[ 88.813211] iwlwifi 0000:01:00.0: LMAC1 CURRENT PC: 0xd0
[ 88.813333] iwlwifi 0000:01:00.0: FW error in SYNC CMD TRIGGER_RX_QUEUES_NOTIF_CMD
[ 88.813384] CPU: 7 UID: 0 PID: 2409 Comm: kworker/u64:41 Not tainted 6.15.0-09113-g8477ab143069 #364 PREEMPT(voluntary)
[ 88.813386] Hardware name: HP HP Pavilion Aero Laptop 13-be0xxx/8916, BIOS F.17 12/18/2024
[ 88.813387] Workqueue: async async_run_entry_fn
[ 88.813392] Call Trace:
[ 88.813393] <TASK>
[ 88.813395] dump_stack_lvl+0x6e/0x90
[ 88.813402] iwl_trans_pcie_send_hcmd+0x446/0x450 [iwlwifi]
[ 88.813416] ? prepare_to_wait_exclusive+0xf0/0xf0
[ 88.813421] iwl_trans_send_cmd+0x6c/0x150 [iwlwifi]
[ 88.813433] iwl_mvm_send_cmd+0x2d/0x70 [iwlmvm]
[ 88.813446] iwl_mvm_sync_rx_queues_internal+0x123/0x2e0 [iwlmvm]
[ 88.813462] iwl_mvm_sta_rx_agg+0x30f/0xa60 [iwlmvm]
[ 88.813478] ? iwl_mvm_mac_ampdu_action+0x20c/0x400 [iwlmvm]
[ 88.813486] iwl_mvm_mac_ampdu_action+0x20c/0x400 [iwlmvm]
[ 88.813494] ? lock_is_held_type+0x96/0x100
[ 88.813498] drv_ampdu_action+0x108/0x250 [mac80211]
[ 88.813530] __ieee80211_stop_rx_ba_session+0xde/0x200 [mac80211]
[ 88.813559] ieee80211_sta_tear_down_BA_sessions+0x47/0x150 [mac80211]
[ 88.813582] __sta_info_destroy_part1+0x5c/0x7e0 [mac80211]
[ 88.813606] __sta_info_flush+0x13f/0x280 [mac80211]
[ 88.813631] ieee80211_set_disassoc+0x68b/0x940 [mac80211]
[ 88.813660] ieee80211_mgd_deauth.cold+0x51/0x224 [mac80211]
[ 88.813685] cfg80211_mlme_deauth+0xe4/0x2b0 [cfg80211]
[ 88.813720] cfg80211_mlme_down+0x93/0xd0 [cfg80211]
[ 88.813742] cfg80211_disconnect+0x1b9/0x2a0 [cfg80211]
[ 88.813766] wiphy_suspend+0x22d/0x3d0 [cfg80211]
[ 88.813793] ? wiphy_resume+0x240/0x240 [cfg80211]
[ 88.813815] dpm_run_callback+0x5f/0x1e0
[ 88.813818] device_suspend+0x18d/0x5f0
[ 88.813822] async_suspend+0x1d/0x30
[ 88.813824] async_run_entry_fn+0x2e/0x130
[ 88.813827] process_one_work+0x22b/0x5b0
[ 88.813833] worker_thread+0x1da/0x3d0
[ 88.813835] ? bh_worker+0x260/0x260
[ 88.813837] kthread+0x10a/0x250
[ 88.813839] ? kthreads_online_cpu+0x130/0x130
[ 88.813841] ret_from_fork+0x20c/0x270
[ 88.813843] ? kthreads_online_cpu+0x130/0x130
[ 88.813844] ret_from_fork_asm+0x11/0x20
[ 88.813852] </TASK>
[ 88.813853] iwlwifi 0000:01:00.0: Failed to trigger RX queues sync (-5)
[ 88.817949] wlp1s0: HW problem - can not stop rx aggregation for d6:92:5e:eb:ee:15 tid 3
[ 88.817951] iwlwifi 0000:01:00.0: WRT: Collecting data: ini trigger 4 fired (delay=0ms).
[ 88.963365] sd 0:0:0:0: [sda] Synchronizing SCSI cache
[ 89.429653] iwlwifi 0000:01:00.0: Failed to trigger RX queues sync (-5)
[ 89.429655] iwlwifi 0000:01:00.0: Device error - SW reset
[ 89.429897] ieee80211 phy0: Hardware restart was requested
[ 89.429981] iwlwifi 0000:01:00.0: Failed to send flush command (-5)
[ 89.429984] iwlwifi 0000:01:00.0: flush request fail
[ 89.429997] iwlwifi 0000:01:00.0: Failed to send rate scale config (-5)
[ 89.430058] wlp1s0: failed to remove key (0, d6:92:5e:eb:ee:15) from hardware (-5)
[ 89.430086] iwlwifi 0000:01:00.0: Failed to send flush command (-5)
[ 89.430194] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:2): -5
[ 89.430199] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:3): -5
[ 89.430200] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:1): -5
[ 89.430204] iwlwifi 0000:01:00.0: PHY ctxt cmd error. ret=-5
[ 89.430222] wlp1s0: failed to remove key (1, ff:ff:ff:ff:ff:ff) from hardware (-5)
[ 89.430262] wlp1s0: failed to remove key (4, ff:ff:ff:ff:ff:ff) from hardware (-5)
[ 89.431375] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:3): -5
[ 89.431379] iwlwifi 0000:01:00.0: Failed to send MAC_CONFIG_CMD (action:3): -5
[ 89.431386] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:3): -5
[ 89.431387] iwlwifi 0000:01:00.0: Failed to send MAC_CONFIG_CMD (action:3): -5
[ 89.431391] iwlwifi 0000:01:00.0: LED command failed: -5
[ 89.431421] ------------[ cut here ]------------
[ 89.431423] WARNING: CPU: 1 PID: 2409 at drivers/net/wireless/intel/iwlwifi/mvm/d3.c:3673 iwl_mvm_fast_suspend+0x102/0x110 [iwlmvm]
[ 89.431441] Modules linked in: snd_seq_dummy snd_hrtimer snd_seq xt_conntrack nft_chain_nat xt_MASQUERADE nf_nat nf_conntrack_netlink nf_conntrack nf_defrag_ipv6 nf_defrag_ipv4 xfrm_user xfrm_algo xt_addrtype nft_compat nf_tables br_netfilter bridge stp llc ccm overlay qrtr rfcomm cmac algif_hash algif_skcipher af_alg bnep binfmt_misc snd_acp3x_rn snd_soc_dmic snd_acp3x_pdm_dma snd_sof_amd_rembrandt snd_sof_amd_acp snd_sof_pci snd_sof_xtensa_dsp snd_ctl_led snd_sof snd_hda_codec_realtek nls_ascii snd_sof_utils snd_soc_core nls_cp437 snd_hda_codec_generic snd_compress snd_hda_codec_hdmi vfat snd_hda_scodec_component uvcvideo snd_pci_ps fat intel_rapl_msr snd_hda_intel snd_soc_acpi_amd_match intel_rapl_common videobuf2_vmalloc snd_usb_audio snd_rpl_pci_acp6x iwlmvm snd_intel_dspcfg videobuf2_memops snd_acp_pci snd_amd_acpi_mach btusb uvc snd_hda_codec snd_usbmidi_lib kvm_amd mac80211 snd_acp_legacy_common btrtl videobuf2_v4l2 snd_hwdep btintel snd_rawmidi snd_pci_acp6x kvm snd_hda_core videodev libarc4 btbcm
[ 89.431514] snd_pci_acp5x snd_seq_device hp_wmi btmtk snd_rn_pci_acp3x iwlwifi snd_pcm irqbypass videobuf2_common ucsi_acpi platform_profile ee1004 snd_acp_config snd_timer bluetooth rapl mc pcspkr sg cfg80211 typec_ucsi sparse_keymap wmi_bmof snd_soc_acpi snd sp5100_tco roles snd_pci_acp3x k10temp watchdog ccp soundcore rfkill typec battery ac acpi_tad joydev amd_pmc serio_raw evdev msr parport_pc ppdev dm_mod lp parport nvme_fabrics efi_pstore configfs nfnetlink efivarfs ip_tables x_tables autofs4 crc32c_cryptoapi btrfs blake2b_generic sd_mod xor raid6_pq hid_microsoft ff_memless r8153_ecm cdc_ether usbnet uas usb_storage scsi_mod scsi_common r8152 mii libphy mdio_bus usbhid amdgpu drm_client_lib i2c_algo_bit drm_ttm_helper ttm drm_panel_backlight_quirks drm_exec drm_suballoc_helper amdxcp drm_buddy hid_multitouch gpu_sched hid_generic drm_display_helper xhci_pci drm_kms_helper nvme i2c_hid_acpi xhci_hcd cec nvme_core ghash_clmulni_intel i2c_hid amd_sfh usbcore rc_core nvme_keyring i2c_piix4 video sha512_ssse3 hid
[ 89.431604] usb_common crc16 nvme_auth i2c_smbus fan button wmi drm aesni_intel
[ 89.431614] CPU: 1 UID: 0 PID: 2409 Comm: kworker/u64:41 Not tainted 6.15.0-09113-g8477ab143069 #364 PREEMPT(voluntary)
[ 89.431617] Hardware name: HP HP Pavilion Aero Laptop 13-be0xxx/8916, BIOS F.17 12/18/2024
[ 89.431619] Workqueue: async async_run_entry_fn
[ 89.431624] RIP: 0010:iwl_mvm_fast_suspend+0x102/0x110 [iwlmvm]
[ 89.431635] Code: 48 8b 3b 89 c1 48 c7 c2 d0 51 2d c2 31 f6 e8 e5 b3 9b ff eb b6 48 8b 3b 89 c1 48 c7 c2 a0 51 2d c2 31 f6 e8 d0 b3 9b ff eb 90 <0f> 0b e9 6c ff ff ff e8 52 0a 23 ef 66 90 66 0f 1f 00 0f 1f 44 00
[ 89.431637] RSP: 0018:ffffb013c619fc90 EFLAGS: 00010282
[ 89.431639] RAX: 00000000fffffffb RBX: ffff974e86d2b0a8 RCX: 0000000000000000
[ 89.431641] RDX: 0000000000000000 RSI: ffffb013c619fc10 RDI: ffff974e91d10028
[ 89.431642] RBP: 0000000000000001 R08: 0000000000000001 R09: 0000000000000077
[ 89.431643] R10: ffffb013c619fcd0 R11: 0000000000000005 R12: ffff974e86d2b0d8
[ 89.431644] R13: ffff974e86d2b200 R14: ffff974e86d2b1b8 R15: ffff974e86d2b0a8
[ 89.431646] FS: 0000000000000000(0000) GS:ffff9751da7c6000(0000) knlGS:0000000000000000
[ 89.431647] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 89.431649] CR2: 00007f5f7dd08a20 CR3: 000000037303b000 CR4: 0000000000f50ef0
[ 89.431650] PKRU: 55555554
[ 89.431651] Call Trace:
[ 89.431653] <TASK>
[ 89.431656] __iwl_mvm_mac_stop+0xf4/0x170 [iwlmvm]
[ 89.431673] iwl_mvm_mac_stop+0xe6/0x150 [iwlmvm]
[ 89.431686] drv_stop+0x6e/0x1e0 [mac80211]
[ 89.431730] __ieee80211_suspend+0x38c/0x4e0 [mac80211]
[ 89.431767] wiphy_suspend+0xbf/0x3d0 [cfg80211]
[ 89.431811] ? wiphy_resume+0x240/0x240 [cfg80211]
[ 89.431845] dpm_run_callback+0x5f/0x1e0
[ 89.431849] device_suspend+0x18d/0x5f0
[ 89.431854] async_suspend+0x1d/0x30
[ 89.431856] async_run_entry_fn+0x2e/0x130
[ 89.431860] process_one_work+0x22b/0x5b0
[ 89.431867] worker_thread+0x1da/0x3d0
[ 89.431871] ? bh_worker+0x260/0x260
[ 89.431873] kthread+0x10a/0x250
[ 89.431875] ? kthreads_online_cpu+0x130/0x130
[ 89.431878] ret_from_fork+0x20c/0x270
[ 89.431881] ? kthreads_online_cpu+0x130/0x130
[ 89.431883] ret_from_fork_asm+0x11/0x20
[ 89.431893] </TASK>
[ 89.431894] irq event stamp: 3177
[ 89.431895] hardirqs last enabled at (3183): [<ffffffffb0ba339e>] __up_console_sem+0x5e/0x70
[ 89.431899] hardirqs last disabled at (3188): [<ffffffffb0ba3383>] __up_console_sem+0x43/0x70
[ 89.431901] softirqs last enabled at (1708): [<ffffffffc245fdee>] ieee80211_stop_device+0x1e/0x90 [mac80211]
[ 89.431939] softirqs last disabled at (1702): [<ffffffffc245fdee>] ieee80211_stop_device+0x1e/0x90 [mac80211]
[ 89.431969] ---[ end trace 0000000000000000 ]---
[ 89.431970] iwlwifi 0000:01:00.0: fast suspend: couldn't send D3_CONFIG_CMD -5
[ 91.450917] iwlwifi 0000:01:00.0: Timeout entering D3
[ 91.450931] iwlwifi 0000:01:00.0: fast suspend: trans_d3_suspend failed -110
[ 91.451335] PM: suspend of devices complete after 2660.094 msecs
[ 91.451349] PM: start suspend of devices complete after 3239.729 msecs
[ 91.454951] Disabling GPIO #9 interrupt for suspend.
[ 91.455586] PM: late suspend of devices complete after 4.232 msecs
[ 91.460716] ACPI: EC: interrupt blocked
[ 91.488965] PM: noirq suspend of devices complete after 32.848 msecs
#regzbot introduced: 7438843df8cf7a484303caee7540a00f0ef972b2
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [REGRESSION] iwlwifi: suspend failure, bisected
2025-05-31 14:09 [REGRESSION] iwlwifi: suspend failure, bisected Chris Bainbridge
@ 2025-05-31 18:35 ` Korenblit, Miriam Rachel
2025-05-31 19:48 ` Chris Bainbridge
2025-07-16 18:50 ` Bjorn Helgaas
1 sibling, 1 reply; 7+ messages in thread
From: Korenblit, Miriam Rachel @ 2025-05-31 18:35 UTC (permalink / raw)
To: Chris Bainbridge, Gustavo A. R. Silva
Cc: linux-wireless@vger.kernel.org, kvalo@kernel.org, Berg, Johannes,
benjamin@sipsolutions.net, regressions@lists.linux.dev
[-- Attachment #1: Type: text/plain, Size: 955 bytes --]
> -----Original Message-----
> From: Chris Bainbridge <chris.bainbridge@gmail.com>
> Sent: Saturday, 31 May 2025 17:10
> To: Gustavo A. R. Silva <gustavoars@kernel.org>
> Cc: linux-wireless@vger.kernel.org; kvalo@kernel.org; Korenblit, Miriam
> Rachel <miriam.rachel.korenblit@intel.com>; Berg, Johannes
> <johannes.berg@intel.com>; benjamin@sipsolutions.net;
> regressions@lists.linux.dev
> Subject: [REGRESSION] iwlwifi: suspend failure, bisected
>
> Hi,
>
> There is a recently introduced iwlwifi regression on suspend. It appears the
> bisected commit was only supposed to fix a warning, but reverting the
> commit (with a small change for revert conflict) does fix the error.
>
> 7438843df8cf7a484303caee7540a00f0ef972b2 is the first bad commit
> Gustavo A. R. Silva <gustavoars@kernel.org>commit
Hi Chris, thanks for the report.
Could you please try the attached patch and see if that solves the issue?
Thanks,
Miri
[-- Attachment #2: 0001-wifi-iwlwifi-mvm-fix-assert-on-suspend.patch --]
[-- Type: application/octet-stream, Size: 1261 bytes --]
From 9bb936752c1338f465ef5ea3e7698afbdc6218df Mon Sep 17 00:00:00 2001
From: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Date: Sat, 31 May 2025 21:27:39 +0300
Subject: [PATCH] wifi: iwlwifi: mvm: fix assert on suspend
Organization: Intel Israel (74) Limited
After using DEFINE_RAW_FLEX, cmd is a pointer to iwl_rxq_sync_cmd, where
the size should be sizeof(struct iwl_rxq_sync_cmd) + sizof(struct
iwl_mvm_internal_rxq_notif). Fix this.
Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
---
drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
index f99dc8624bd1..f695fa9177bb 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
@@ -6362,7 +6362,7 @@ void iwl_mvm_sync_rx_queues_internal(struct iwl_mvm *mvm,
struct iwl_host_cmd hcmd = {
.id = WIDE_ID(DATA_PATH_GROUP, TRIGGER_RX_QUEUES_NOTIF_CMD),
.data[0] = &cmd,
- .len[0] = sizeof(cmd),
+ .len[0] = sizeof(*cmd) + sizeof(*notif),
.data[1] = data,
.len[1] = size,
.flags = CMD_SEND_IN_RFKILL | (sync ? 0 : CMD_ASYNC),
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [REGRESSION] iwlwifi: suspend failure, bisected
2025-05-31 18:35 ` Korenblit, Miriam Rachel
@ 2025-05-31 19:48 ` Chris Bainbridge
2025-05-31 20:07 ` Korenblit, Miriam Rachel
0 siblings, 1 reply; 7+ messages in thread
From: Chris Bainbridge @ 2025-05-31 19:48 UTC (permalink / raw)
To: Korenblit, Miriam Rachel
Cc: Gustavo A. R. Silva, linux-wireless@vger.kernel.org,
kvalo@kernel.org, Berg, Johannes, benjamin@sipsolutions.net,
regressions@lists.linux.dev
On Sat, May 31, 2025 at 06:35:33PM +0000, Korenblit, Miriam Rachel wrote:
>
>
> > -----Original Message-----
> > From: Chris Bainbridge <chris.bainbridge@gmail.com>
> > Sent: Saturday, 31 May 2025 17:10
> > To: Gustavo A. R. Silva <gustavoars@kernel.org>
> > Cc: linux-wireless@vger.kernel.org; kvalo@kernel.org; Korenblit, Miriam
> > Rachel <miriam.rachel.korenblit@intel.com>; Berg, Johannes
> > <johannes.berg@intel.com>; benjamin@sipsolutions.net;
> > regressions@lists.linux.dev
> > Subject: [REGRESSION] iwlwifi: suspend failure, bisected
> >
> > Hi,
> >
> > There is a recently introduced iwlwifi regression on suspend. It appears the
> > bisected commit was only supposed to fix a warning, but reverting the
> > commit (with a small change for revert conflict) does fix the error.
> >
> > 7438843df8cf7a484303caee7540a00f0ef972b2 is the first bad commit
> > Gustavo A. R. Silva <gustavoars@kernel.org>commit
>
> Hi Chris, thanks for the report.
> Could you please try the attached patch and see if that solves the issue?
It doesn't look like it made a difference:
[ 67.779335] PM: suspend entry (s2idle)
[ 68.040688] Filesystems sync: 0.260 seconds
[ 68.044182] Freezing user space processes
[ 68.047319] Freezing user space processes completed (elapsed 0.003 seconds)
[ 68.047367] OOM killer disabled.
[ 68.047370] Freezing remaining freezable tasks
[ 68.048775] Freezing remaining freezable tasks completed (elapsed 0.001 seconds)
[ 68.048925] printk: Suspending console(s) (use no_console_suspend to debug)
[ 68.642563] wlp1s0: deauthenticating from d6:92:5e:eb:ee:15 by local choice (Reason: 3=DEAUTH_LEAVING)
[ 68.655494] iwlwifi 0000:01:00.0: Microcode SW error detected. Restarting 0x0.
[ 68.655747] iwlwifi 0000:01:00.0: Start IWL Error Log Dump:
[ 68.655750] iwlwifi 0000:01:00.0: Transport status: 0x0000004B, valid: 6
[ 68.655752] iwlwifi 0000:01:00.0: Loaded firmware version: 89.202a2f7b.0 ty-a0-gf-a0-89.ucode
[ 68.655755] iwlwifi 0000:01:00.0: 0x00000071 | NMI_INTERRUPT_UMAC_FATAL
[ 68.655757] iwlwifi 0000:01:00.0: 0x000002F0 | trm_hw_status0
[ 68.655759] iwlwifi 0000:01:00.0: 0x00000000 | trm_hw_status1
[ 68.655760] iwlwifi 0000:01:00.0: 0x004DAD6C | branchlink2
[ 68.655762] iwlwifi 0000:01:00.0: 0x000146B2 | interruptlink1
[ 68.655763] iwlwifi 0000:01:00.0: 0x000146B2 | interruptlink2
[ 68.655764] iwlwifi 0000:01:00.0: 0x00016AD2 | data1
[ 68.655766] iwlwifi 0000:01:00.0: 0x00000010 | data2
[ 68.655767] iwlwifi 0000:01:00.0: 0x00000000 | data3
[ 68.655768] iwlwifi 0000:01:00.0: 0x79C17988 | beacon time
[ 68.655770] iwlwifi 0000:01:00.0: 0x6356E654 | tsf low
[ 68.655771] iwlwifi 0000:01:00.0: 0x000009FD | tsf hi
[ 68.655772] iwlwifi 0000:01:00.0: 0x00000000 | time gp1
[ 68.655774] iwlwifi 0000:01:00.0: 0x033059C3 | time gp2
[ 68.655775] iwlwifi 0000:01:00.0: 0x00000001 | uCode revision type
[ 68.655776] iwlwifi 0000:01:00.0: 0x00000059 | uCode version major
[ 68.655778] iwlwifi 0000:01:00.0: 0x202A2F7B | uCode version minor
[ 68.655779] iwlwifi 0000:01:00.0: 0x00000420 | hw version
[ 68.655780] iwlwifi 0000:01:00.0: 0x58C80002 | board version
[ 68.655782] iwlwifi 0000:01:00.0: 0x8026FC4E | hcmd
[ 68.655783] iwlwifi 0000:01:00.0: 0x00020000 | isr0
[ 68.655784] iwlwifi 0000:01:00.0: 0x00000000 | isr1
[ 68.655786] iwlwifi 0000:01:00.0: 0x48F04802 | isr2
[ 68.655787] iwlwifi 0000:01:00.0: 0x00C0000C | isr3
[ 68.655788] iwlwifi 0000:01:00.0: 0x00000000 | isr4
[ 68.655789] iwlwifi 0000:01:00.0: 0x00B90103 | last cmd Id
[ 68.655791] iwlwifi 0000:01:00.0: 0x00016AD2 | wait_event
[ 68.655792] iwlwifi 0000:01:00.0: 0x000000D4 | l2p_control
[ 68.655793] iwlwifi 0000:01:00.0: 0x00000000 | l2p_duration
[ 68.655795] iwlwifi 0000:01:00.0: 0x00000007 | l2p_mhvalid
[ 68.655796] iwlwifi 0000:01:00.0: 0x00000048 | l2p_addr_match
[ 68.655797] iwlwifi 0000:01:00.0: 0x00000008 | lmpm_pmg_sel
[ 68.655799] iwlwifi 0000:01:00.0: 0x00000000 | timestamp
[ 68.655800] iwlwifi 0000:01:00.0: 0x0000B810 | flow_handler
[ 68.655917] iwlwifi 0000:01:00.0: Start IWL Error Log Dump:
[ 68.655918] iwlwifi 0000:01:00.0: Transport status: 0x0000004B, valid: 7
[ 68.655920] iwlwifi 0000:01:00.0: 0x2000209A | ADVANCED_SYSASSERT
[ 68.655923] iwlwifi 0000:01:00.0: 0x00000000 | umac branchlink1
[ 68.655924] iwlwifi 0000:01:00.0: 0x804838B2 | umac branchlink2
[ 68.655926] iwlwifi 0000:01:00.0: 0xC00814E0 | umac interruptlink1
[ 68.655928] iwlwifi 0000:01:00.0: 0x00000000 | umac interruptlink2
[ 68.655929] iwlwifi 0000:01:00.0: 0xDEADBEEF | umac data1
[ 68.655930] iwlwifi 0000:01:00.0: 0xDEADBEEF | umac data2
[ 68.655931] iwlwifi 0000:01:00.0: 0xDEADBEEF | umac data3
[ 68.655932] iwlwifi 0000:01:00.0: 0x00000059 | umac major
[ 68.655932] iwlwifi 0000:01:00.0: 0x202A2F7B | umac minor
[ 68.655933] iwlwifi 0000:01:00.0: 0x033059BE | frame pointer
[ 68.655934] iwlwifi 0000:01:00.0: 0xC0886C20 | stack pointer
[ 68.655935] iwlwifi 0000:01:00.0: 0x00C00502 | last host cmd
[ 68.655936] iwlwifi 0000:01:00.0: 0x00000000 | isr status reg
[ 68.656032] iwlwifi 0000:01:00.0: IML/ROM dump:
[ 68.656032] iwlwifi 0000:01:00.0: 0x00000B03 | IML/ROM error/state
[ 68.656130] iwlwifi 0000:01:00.0: 0x00008D5F | IML/ROM data1
[ 68.656188] iwlwifi 0000:01:00.0: 0x00000090 | IML/ROM WFPM_AUTH_KEY_0
[ 68.656260] iwlwifi 0000:01:00.0: Fseq Registers:
[ 68.656306] iwlwifi 0000:01:00.0: 0x60000000 | FSEQ_ERROR_CODE
[ 68.656352] iwlwifi 0000:01:00.0: 0x80440007 | FSEQ_TOP_INIT_VERSION
[ 68.656355] iwlwifi 0000:01:00.0: 0x00080009 | FSEQ_CNVIO_INIT_VERSION
[ 68.656358] iwlwifi 0000:01:00.0: 0x0000A652 | FSEQ_OTP_VERSION
[ 68.656361] iwlwifi 0000:01:00.0: 0x00000002 | FSEQ_TOP_CONTENT_VERSION
[ 68.656364] iwlwifi 0000:01:00.0: 0x4552414E | FSEQ_ALIVE_TOKEN
[ 68.656367] iwlwifi 0000:01:00.0: 0x00400410 | FSEQ_CNVI_ID
[ 68.656370] iwlwifi 0000:01:00.0: 0x00400410 | FSEQ_CNVR_ID
[ 68.656373] iwlwifi 0000:01:00.0: 0x00400410 | CNVI_AUX_MISC_CHIP
[ 68.656378] iwlwifi 0000:01:00.0: 0x00400410 | CNVR_AUX_MISC_CHIP
[ 68.656383] iwlwifi 0000:01:00.0: 0x00009061 | CNVR_SCU_SD_REGS_SD_REG_DIG_DCDC_VTRIM
[ 68.656388] iwlwifi 0000:01:00.0: 0x00000061 | CNVR_SCU_SD_REGS_SD_REG_ACTIVE_VDIG_MIRROR
[ 68.656391] iwlwifi 0000:01:00.0: 0x00080009 | FSEQ_PREV_CNVIO_INIT_VERSION
[ 68.656394] iwlwifi 0000:01:00.0: 0x00440007 | FSEQ_WIFI_FSEQ_VERSION
[ 68.656397] iwlwifi 0000:01:00.0: 0x00440007 | FSEQ_BT_FSEQ_VERSION
[ 68.656400] iwlwifi 0000:01:00.0: 0x000000E6 | FSEQ_CLASS_TP_VERSION
[ 68.656507] iwlwifi 0000:01:00.0: UMAC CURRENT PC: 0x804a5578
[ 68.656554] iwlwifi 0000:01:00.0: LMAC1 CURRENT PC: 0xd0
[ 68.656672] iwlwifi 0000:01:00.0: FW error in SYNC CMD TRIGGER_RX_QUEUES_NOTIF_CMD
[ 68.656687] CPU: 13 UID: 0 PID: 2449 Comm: kworker/u64:38 Not tainted 6.15.0-09114-g351e969adca9 #380 PREEMPT(voluntary)
[ 68.656689] Hardware name: HP HP Pavilion Aero Laptop 13-be0xxx/8916, BIOS F.17 12/18/2024
[ 68.656690] Workqueue: async async_run_entry_fn
[ 68.656694] Call Trace:
[ 68.656695] <TASK>
[ 68.656697] dump_stack_lvl+0x6e/0x90
[ 68.656702] iwl_trans_pcie_send_hcmd+0x446/0x450 [iwlwifi]
[ 68.656715] ? prepare_to_wait_exclusive+0xf0/0xf0
[ 68.656719] iwl_trans_send_cmd+0x6c/0x150 [iwlwifi]
[ 68.656728] iwl_mvm_send_cmd+0x2d/0x70 [iwlmvm]
[ 68.656738] iwl_mvm_sync_rx_queues_internal+0x123/0x2e0 [iwlmvm]
[ 68.656750] iwl_mvm_sta_rx_agg+0x30f/0xa60 [iwlmvm]
[ 68.656764] ? iwl_mvm_mac_ampdu_action+0x20c/0x400 [iwlmvm]
[ 68.656770] iwl_mvm_mac_ampdu_action+0x20c/0x400 [iwlmvm]
[ 68.656776] ? lock_is_held_type+0x96/0x100
[ 68.656780] drv_ampdu_action+0x108/0x250 [mac80211]
[ 68.656801] __ieee80211_stop_rx_ba_session+0xde/0x200 [mac80211]
[ 68.656822] ieee80211_sta_tear_down_BA_sessions+0x47/0x150 [mac80211]
[ 68.656840] __sta_info_destroy_part1+0x5c/0x7e0 [mac80211]
[ 68.656862] __sta_info_flush+0x13f/0x280 [mac80211]
[ 68.656886] ieee80211_set_disassoc+0x68b/0x940 [mac80211]
[ 68.656918] ieee80211_mgd_deauth.cold+0x51/0x224 [mac80211]
[ 68.656952] cfg80211_mlme_deauth+0xe4/0x2b0 [cfg80211]
[ 68.656987] cfg80211_mlme_down+0x93/0xd0 [cfg80211]
[ 68.657009] cfg80211_disconnect+0x1b9/0x2a0 [cfg80211]
[ 68.657031] wiphy_suspend+0x22d/0x3d0 [cfg80211]
[ 68.659918] ? wiphy_resume+0x240/0x240 [cfg80211]
[ 68.659953] dpm_run_callback+0x5f/0x1e0
[ 68.659958] device_suspend+0x18d/0x5f0
[ 68.659964] async_suspend+0x1d/0x30
[ 68.659966] async_run_entry_fn+0x2e/0x130
[ 68.659970] process_one_work+0x22b/0x5b0
[ 68.659976] worker_thread+0x1da/0x3d0
[ 68.659978] ? bh_worker+0x260/0x260
[ 68.659980] kthread+0x10a/0x250
[ 68.659982] ? kthreads_online_cpu+0x130/0x130
[ 68.659984] ret_from_fork+0x20c/0x270
[ 68.659986] ? kthreads_online_cpu+0x130/0x130
[ 68.659987] ret_from_fork_asm+0x11/0x20
[ 68.659996] </TASK>
[ 68.660026] iwlwifi 0000:01:00.0: Failed to trigger RX queues sync (-5)
[ 68.660358] iwlwifi 0000:01:00.0: WRT: Collecting data: ini trigger 4 fired (delay=0ms).
[ 68.661681] sd 0:0:0:0: [sda] Synchronizing SCSI cache
[ 69.276552] wlp1s0: HW problem - can not stop rx aggregation for d6:92:5e:eb:ee:15 tid 3
[ 69.276554] iwlwifi 0000:01:00.0: Device error - SW reset
[ 69.276559] wlp1s0: HW problem - can not stop rx aggregation for d6:92:5e:eb:ee:15 tid 4
[ 69.276571] iwlwifi 0000:01:00.0: Failed to trigger RX queues sync (-5)
[ 69.276802] ieee80211 phy0: Hardware restart was requested
[ 69.276884] iwlwifi 0000:01:00.0: Failed to send flush command (-5)
[ 69.276887] iwlwifi 0000:01:00.0: flush request fail
[ 69.276901] iwlwifi 0000:01:00.0: Failed to send rate scale config (-5)
[ 69.276963] wlp1s0: failed to remove key (0, d6:92:5e:eb:ee:15) from hardware (-5)
[ 69.276995] iwlwifi 0000:01:00.0: Failed to send flush command (-5)
[ 69.277100] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:2): -5
[ 69.277104] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:3): -5
[ 69.277106] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:1): -5
[ 69.277111] iwlwifi 0000:01:00.0: PHY ctxt cmd error. ret=-5
[ 69.277128] wlp1s0: failed to remove key (1, ff:ff:ff:ff:ff:ff) from hardware (-5)
[ 69.277172] wlp1s0: failed to remove key (4, ff:ff:ff:ff:ff:ff) from hardware (-5)
[ 69.277917] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:3): -5
[ 69.277919] iwlwifi 0000:01:00.0: Failed to send MAC_CONFIG_CMD (action:3): -5
[ 69.277924] iwlwifi 0000:01:00.0: Failed to send LINK_CONFIG_CMD (action:3): -5
[ 69.277925] iwlwifi 0000:01:00.0: Failed to send MAC_CONFIG_CMD (action:3): -5
[ 69.277929] iwlwifi 0000:01:00.0: LED command failed: -5
[ 69.277952] ------------[ cut here ]------------
[ 69.277953] WARNING: CPU: 13 PID: 2449 at drivers/net/wireless/intel/iwlwifi/mvm/d3.c:3673 iwl_mvm_fast_suspend+0x102/0x110 [iwlmvm]
[ 69.277966] Modules linked in: snd_seq_dummy snd_hrtimer snd_seq xt_conntrack nft_chain_nat xt_MASQUERADE nf_nat nf_conntrack_netlink nf_conntrack nf_defrag_ipv6 nf_defrag_ipv4 xfrm_user xfrm_algo xt_addrtype nft_compat nf_tables br_netfilter bridge stp llc ccm overlay qrtr rfcomm cmac algif_hash algif_skcipher af_alg bnep binfmt_misc snd_acp3x_rn snd_soc_dmic snd_acp3x_pdm_dma nls_ascii snd_sof_amd_rembrandt nls_cp437 snd_sof_amd_acp snd_sof_pci snd_sof_xtensa_dsp vfat snd_sof snd_ctl_led fat snd_hda_codec_realtek snd_sof_utils snd_hda_codec_generic snd_soc_core iwlmvm snd_hda_codec_hdmi snd_compress snd_hda_scodec_component uvcvideo btusb snd_pci_ps intel_rapl_msr snd_hda_intel snd_soc_acpi_amd_match intel_rapl_common videobuf2_vmalloc snd_usb_audio snd_rpl_pci_acp6x mac80211 btrtl snd_intel_dspcfg videobuf2_memops snd_acp_pci kvm_amd btintel snd_usbmidi_lib snd_hda_codec uvc snd_amd_acpi_mach libarc4 snd_acp_legacy_common videobuf2_v4l2 btbcm snd_hwdep kvm snd_pci_acp6x videodev snd_rawmidi snd_pci_acp5x hp_wmi
[ 69.278021] btmtk snd_hda_core iwlwifi snd_seq_device irqbypass ucsi_acpi snd_rn_pci_acp3x bluetooth videobuf2_common platform_profile ee1004 snd_pcm sg snd_acp_config rapl typec_ucsi cfg80211 pcspkr mc sparse_keymap wmi_bmof snd_timer snd_soc_acpi sp5100_tco roles snd k10temp watchdog rfkill ccp snd_pci_acp3x typec soundcore ac battery acpi_tad amd_pmc serio_raw joydev evdev msr parport_pc ppdev dm_mod lp parport nvme_fabrics efi_pstore configfs nfnetlink efivarfs ip_tables x_tables autofs4 crc32c_cryptoapi sd_mod hid_microsoft ff_memless btrfs blake2b_generic xor raid6_pq uas usb_storage scsi_mod scsi_common r8153_ecm cdc_ether usbnet r8152 mii libphy mdio_bus usbhid amdgpu drm_client_lib i2c_algo_bit drm_ttm_helper ttm drm_panel_backlight_quirks drm_exec drm_suballoc_helper amdxcp drm_buddy gpu_sched hid_multitouch hid_generic drm_display_helper nvme xhci_pci drm_kms_helper nvme_core i2c_hid_acpi xhci_hcd cec ghash_clmulni_intel nvme_keyring i2c_hid amd_sfh usbcore rc_core i2c_piix4 video sha512_ssse3 nvme_auth
[ 69.278088] hid usb_common crc16 i2c_smbus fan button wmi drm aesni_intel
[ 69.278096] CPU: 13 UID: 0 PID: 2449 Comm: kworker/u64:38 Not tainted 6.15.0-09114-g351e969adca9 #380 PREEMPT(voluntary)
[ 69.278098] Hardware name: HP HP Pavilion Aero Laptop 13-be0xxx/8916, BIOS F.17 12/18/2024
[ 69.278099] Workqueue: async async_run_entry_fn
[ 69.278104] RIP: 0010:iwl_mvm_fast_suspend+0x102/0x110 [iwlmvm]
[ 69.278111] Code: 48 8b 3b 89 c1 48 c7 c2 d0 11 92 c2 31 f6 e8 e5 c3 96 ff eb b6 48 8b 3b 89 c1 48 c7 c2 a0 11 92 c2 31 f6 e8 d0 c3 96 ff eb 90 <0f> 0b e9 6c ff ff ff e8 52 1a 5e ee 66 90 66 0f 1f 00 0f 1f 44 00
[ 69.278112] RSP: 0018:ffffb858061abc90 EFLAGS: 00010282
[ 69.278114] RAX: 00000000fffffffb RBX: ffff9dced082b0a8 RCX: 0000000000000000
[ 69.278115] RDX: 0000000000000000 RSI: ffffb858061abc10 RDI: ffff9dceced20028
[ 69.278116] RBP: 0000000000000001 R08: 0000000000000001 R09: 0000000000000077
[ 69.278117] R10: ffffb858061abcd0 R11: 0000000000000005 R12: ffff9dced082b0d8
[ 69.278118] R13: ffff9dced082b200 R14: ffff9dced082b1b8 R15: ffff9dced082b0a8
[ 69.278119] FS: 0000000000000000(0000) GS:ffff9dd21b7c6000(0000) knlGS:0000000000000000
[ 69.278120] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 69.278121] CR2: 00007f26f400e038 CR3: 00000003abe3b000 CR4: 0000000000f50ef0
[ 69.278122] PKRU: 55555554
[ 69.278123] Call Trace:
[ 69.278124] <TASK>
[ 69.278127] __iwl_mvm_mac_stop+0xf4/0x170 [iwlmvm]
[ 69.278135] iwl_mvm_mac_stop+0xe6/0x150 [iwlmvm]
[ 69.278143] drv_stop+0x6e/0x1e0 [mac80211]
[ 69.278165] __ieee80211_suspend+0x38c/0x4e0 [mac80211]
[ 69.278189] wiphy_suspend+0xbf/0x3d0 [cfg80211]
[ 69.278212] ? wiphy_resume+0x240/0x240 [cfg80211]
[ 69.278228] dpm_run_callback+0x5f/0x1e0
[ 69.278231] device_suspend+0x18d/0x5f0
[ 69.278235] async_suspend+0x1d/0x30
[ 69.278236] async_run_entry_fn+0x2e/0x130
[ 69.278239] process_one_work+0x22b/0x5b0
[ 69.278244] worker_thread+0x1da/0x3d0
[ 69.278247] ? bh_worker+0x260/0x260
[ 69.278248] kthread+0x10a/0x250
[ 69.278250] ? kthreads_online_cpu+0x130/0x130
[ 69.278253] ret_from_fork+0x20c/0x270
[ 69.278255] ? kthreads_online_cpu+0x130/0x130
[ 69.278256] ret_from_fork_asm+0x11/0x20
[ 69.278263] </TASK>
[ 69.278264] irq event stamp: 3307
[ 69.278265] hardirqs last enabled at (3313): [<ffffffffb01a339e>] __up_console_sem+0x5e/0x70
[ 69.278268] hardirqs last disabled at (3318): [<ffffffffb01a3383>] __up_console_sem+0x43/0x70
[ 69.278270] softirqs last enabled at (1838): [<ffffffffc2687dee>] ieee80211_stop_device+0x1e/0x90 [mac80211]
[ 69.278289] softirqs last disabled at (1832): [<ffffffffc2687dee>] ieee80211_stop_device+0x1e/0x90 [mac80211]
[ 69.278305] ---[ end trace 0000000000000000 ]---
[ 69.278306] iwlwifi 0000:01:00.0: fast suspend: couldn't send D3_CONFIG_CMD -5
[ 71.288600] iwlwifi 0000:01:00.0: Timeout entering D3
[ 71.288615] iwlwifi 0000:01:00.0: fast suspend: trans_d3_suspend failed -110
[ 71.289046] PM: suspend of devices complete after 2649.981 msecs
[ 71.289056] PM: start suspend of devices complete after 3233.752 msecs
[ 71.293998] Disabling GPIO #9 interrupt for suspend.
[ 71.294615] PM: late suspend of devices complete after 5.547 msecs
[ 71.297174] ACPI: EC: interrupt blocked
[ 71.327771] PM: noirq suspend of devices complete after 32.591 msecs
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [REGRESSION] iwlwifi: suspend failure, bisected
2025-05-31 19:48 ` Chris Bainbridge
@ 2025-05-31 20:07 ` Korenblit, Miriam Rachel
2025-05-31 21:16 ` Chris Bainbridge
0 siblings, 1 reply; 7+ messages in thread
From: Korenblit, Miriam Rachel @ 2025-05-31 20:07 UTC (permalink / raw)
To: Chris Bainbridge
Cc: Gustavo A. R. Silva, linux-wireless@vger.kernel.org,
kvalo@kernel.org, Berg, Johannes, benjamin@sipsolutions.net,
regressions@lists.linux.dev
[-- Attachment #1: Type: text/plain, Size: 145 bytes --]
>
> It doesn't look like it made a difference:
It actually did, that's a different assert 😊
Please try the new version of the patch.
[-- Attachment #2: 0001-wifi-iwlwifi-mvm-fix-assert-on-suspend.patch --]
[-- Type: application/octet-stream, Size: 1339 bytes --]
From 7087e3db3bf4d2f73476e9d0bfd425db52e008ae Mon Sep 17 00:00:00 2001
From: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Date: Sat, 31 May 2025 21:27:39 +0300
Subject: [PATCH] wifi: iwlwifi: mvm: fix assert on suspend
Organization: Intel Israel (74) Limited
After using DEFINE_RAW_FLEX, cmd is a pointer to iwl_rxq_sync_cmd, where
the size should be sizeof(struct iwl_rxq_sync_cmd) + sizof(struct
iwl_mvm_internal_rxq_notif). Fix this.
Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
---
drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
index f99dc8624bd1..e860eea77cb6 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
@@ -6361,8 +6361,8 @@ void iwl_mvm_sync_rx_queues_internal(struct iwl_mvm *mvm,
(struct iwl_mvm_internal_rxq_notif *)cmd->payload;
struct iwl_host_cmd hcmd = {
.id = WIDE_ID(DATA_PATH_GROUP, TRIGGER_RX_QUEUES_NOTIF_CMD),
- .data[0] = &cmd,
- .len[0] = sizeof(cmd),
+ .data[0] = cmd,
+ .len[0] = sizeof(*cmd) + sizeof(*notif),
.data[1] = data,
.len[1] = size,
.flags = CMD_SEND_IN_RFKILL | (sync ? 0 : CMD_ASYNC),
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [REGRESSION] iwlwifi: suspend failure, bisected
2025-05-31 20:07 ` Korenblit, Miriam Rachel
@ 2025-05-31 21:16 ` Chris Bainbridge
0 siblings, 0 replies; 7+ messages in thread
From: Chris Bainbridge @ 2025-05-31 21:16 UTC (permalink / raw)
To: Korenblit, Miriam Rachel
Cc: Gustavo A. R. Silva, linux-wireless@vger.kernel.org,
kvalo@kernel.org, Berg, Johannes, benjamin@sipsolutions.net,
regressions@lists.linux.dev
On Sat, May 31, 2025 at 08:07:31PM +0000, Korenblit, Miriam Rachel wrote:
> >
> > It doesn't look like it made a difference:
> It actually did, that's a different assert 😊
>
> Please try the new version of the patch.
>
Fix looks good to me, tested on two laptops and the errors are gone.
Thank you for the quick response.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [REGRESSION] iwlwifi: suspend failure, bisected
2025-05-31 14:09 [REGRESSION] iwlwifi: suspend failure, bisected Chris Bainbridge
2025-05-31 18:35 ` Korenblit, Miriam Rachel
@ 2025-07-16 18:50 ` Bjorn Helgaas
2025-07-22 11:23 ` Chris Bainbridge
1 sibling, 1 reply; 7+ messages in thread
From: Bjorn Helgaas @ 2025-07-16 18:50 UTC (permalink / raw)
To: Chris Bainbridge
Cc: Gustavo A. R. Silva, linux-wireless@vger.kernel.org,
kvalo@kernel.org, Korenblit, Miriam Rachel, Berg, Johannes,
benjamin@sipsolutions.net, regressions@lists.linux.dev
On Sat, May 31, 2025 at 03:09:35PM +0100, Chris Bainbridge wrote:
> Hi,
>
> There is a recently introduced iwlwifi regression on suspend. It appears
> the bisected commit was only supposed to fix a warning, but reverting
> the commit (with a small change for revert conflict) does fix the error.
>
> 7438843df8cf7a484303caee7540a00f0ef972b2 is the first bad commit
> Gustavo A. R. Silva <gustavoars@kernel.org>commit 7438843df8cf7a484303caee7540a00f0ef972b2
> Author: Gustavo A. R. Silva <gustavoars@kernel.org>
> Date: Wed Mar 26 18:04:02 2025 -0600
>
> wifi: iwlwifi: mvm: Avoid -Wflex-array-member-not-at-end warning
https://git.kernel.org/linus/264c844abb29 ("wifi: iwlwifi: mvm: fix
assert on suspend") appeared in v6.16-rc1 and includes a Fixes: tag
for 7438843df8cf.
264c844abb29 is not exactly the same as the patch Miri posted at
https://lore.kernel.org/r/DM3PPF63A6024A9ECC64ACBA27CCABBAB0FA360A@DM3PPF63A6024A9.namprd11.prod.outlook.com,
but maybe it still fixes this regression?
If so, can somebody send the appropriate regzbot fix command to remove
this regression from the tracker
(https://linux-regtracking.leemhuis.info/regzbot/mainline/)?
(See https://linux-regtracking.leemhuis.info/about/#tldr)
I feel like I might be missing something because there are several
regressions in the tracker that *look* like they should have been
resolved months ago.
Bjorn
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [REGRESSION] iwlwifi: suspend failure, bisected
2025-07-16 18:50 ` Bjorn Helgaas
@ 2025-07-22 11:23 ` Chris Bainbridge
0 siblings, 0 replies; 7+ messages in thread
From: Chris Bainbridge @ 2025-07-22 11:23 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Gustavo A. R. Silva, linux-wireless@vger.kernel.org,
kvalo@kernel.org, Korenblit, Miriam Rachel, Berg, Johannes,
benjamin@sipsolutions.net, regressions@lists.linux.dev
On Wed, Jul 16, 2025 at 01:50:57PM -0500, Bjorn Helgaas wrote:
> On Sat, May 31, 2025 at 03:09:35PM +0100, Chris Bainbridge wrote:
> > Hi,
> >
> > There is a recently introduced iwlwifi regression on suspend. It appears
> > the bisected commit was only supposed to fix a warning, but reverting
> > the commit (with a small change for revert conflict) does fix the error.
> >
> > 7438843df8cf7a484303caee7540a00f0ef972b2 is the first bad commit
> > Gustavo A. R. Silva <gustavoars@kernel.org>commit 7438843df8cf7a484303caee7540a00f0ef972b2
> > Author: Gustavo A. R. Silva <gustavoars@kernel.org>
> > Date: Wed Mar 26 18:04:02 2025 -0600
> >
> > wifi: iwlwifi: mvm: Avoid -Wflex-array-member-not-at-end warning
>
> https://git.kernel.org/linus/264c844abb29 ("wifi: iwlwifi: mvm: fix
> assert on suspend") appeared in v6.16-rc1 and includes a Fixes: tag
> for 7438843df8cf.
>
> 264c844abb29 is not exactly the same as the patch Miri posted at
> https://lore.kernel.org/r/DM3PPF63A6024A9ECC64ACBA27CCABBAB0FA360A@DM3PPF63A6024A9.namprd11.prod.outlook.com,
> but maybe it still fixes this regression?
>
> If so, can somebody send the appropriate regzbot fix command to remove
> this regression from the tracker
> (https://linux-regtracking.leemhuis.info/regzbot/mainline/)?
>
> (See https://linux-regtracking.leemhuis.info/about/#tldr)
>
> I feel like I might be missing something because there are several
> regressions in the tracker that *look* like they should have been
> resolved months ago.
>
> Bjorn
#regzbot fix: 264c844abb29
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2025-07-22 11:23 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-31 14:09 [REGRESSION] iwlwifi: suspend failure, bisected Chris Bainbridge
2025-05-31 18:35 ` Korenblit, Miriam Rachel
2025-05-31 19:48 ` Chris Bainbridge
2025-05-31 20:07 ` Korenblit, Miriam Rachel
2025-05-31 21:16 ` Chris Bainbridge
2025-07-16 18:50 ` Bjorn Helgaas
2025-07-22 11:23 ` Chris Bainbridge
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox