Message-ID: <86968600-34ba-490a-9543-1a6958a93faf@redhat.com>
Date: Tue, 20 Feb 2024 20:15:36 +0100
From: Hans de Goede <hdegoede@...hat.com>
To: Paul Menzel <pmenzel@...gen.mpg.de>,
 Wolfram Sang <wsa+renesas@...g-engineering.com>, Pali Rohár
 <pali@...nel.org>, Andi Shyti <andi.shyti@...nel.org>
Cc: linux-i2c@...r.kernel.org, Liam Girdwood <lgirdwood@...il.com>,
 Mark Brown <broonie@...nel.org>, LKML <linux-kernel@...r.kernel.org>,
 regressions@...ts.linux.dev
Subject: Re: [REGRESSION] `lis3lv02d_i2c_suspend()` causes `unbalanced
 disables for regulator-dummy` and `Failed to disable Vdd_IO: -EIO`

Hi Paul,

On 2/2/24 12:38, Paul Menzel wrote:
> #regzbot introduced 2f189493ae32
> 
> 
> Dear Linux folks,
> 
> 
> Testing commit 2f189493ae32 (i2c: i801: Add lis3lv02d for Dell XPS 15 7590) [1], it’s very likely this commit, it turns out, that Linux logs the warning below during ACPI S3 suspend:

This is actually happening on all models in the DMI match list for creating
an lis3lv02d i2c-client if a SM88xx ACPI device is present.

This is caused by:
b1b9f7a49440 ("misc: lis3lv02d_i2c: Add missing setting of the reg_ctrl callback")

Which fixed lis3lv02d_poweroff() never turning off the regulators.

But that exposed a bug in the suspend/resume handling.

I have reproduced this on a Latitude E6430 and I'veposted a fix for
the suspend resume handling upstream:
https://lore.kernel.org/regressions/20240220190035.53402-1-hdegoede@redhat.com/

Regards,

Hans






> ```
> [    0.000000] Linux version 6.8.0-rc2 (pmenzel@...lette) (gcc (Debian 13.2.0-12) 13.2.0, GNU ld (GNU Binutils for Debian) 2.41.90.20240122) #9 SMP PREEMPT_DYNAMIC Mon Jan 29 15:49:07 CET 2024
> [    0.000000] Command line: BOOT_IMAGE=/vmlinuz-6.8.0-rc2 root=UUID=9fa41e21-7a5f-479e-afdc-9a5503368d8e ro cryptomgr.notests quiet rd.luks=1 rd.auto=1 initcall_debug log_buf_len=8M
> […]
> [    0.000000] DMI: Dell Inc. XPS 15 7590/0VYV0G, BIOS 1.24.0 09/11/2023
> […]
> [ 1528.577767] lis3lv02d_i2c 0-0029: PM: calling lis3lv02d_i2c_suspend+0x0/0x30 [lis3lv02d_i2c] @ 92, parent: i2c-0
> […]
> [ 1528.578506] ------------[ cut here ]------------
> [ 1528.578508] unbalanced disables for regulator-dummy
> [ 1528.578522] WARNING: CPU: 1 PID: 92 at drivers/regulator/core.c:2999 _regulator_disable+0xbb/0x190
> [ 1528.578532] Modules linked in: snd_seq_dummy(E) snd_hrtimer(E) snd_seq(E) snd_seq_device(E) rfcomm(E) snd_sof_probes(E) snd_hda_codec_hdmi(E) snd_hda_codec_realtek(E) snd_hda_codec_generic(E) l2tp_ppp(E) l2tp_netlink(E) l2tp_core(E) ip6_udp_tunnel(E) udp_tunnel(E) pppox(E) ppp_generic(E) slhc(E) xfrm_interface(E) xfrm6_tunnel(E) tunnel6(E) xfrm_user(E) xfrm_algo(E) qrtr(E) cmac(E) algif_hash(E) algif_skcipher(E) af_alg(E) bnep(E) snd_sof_pci_intel_cnl(E) snd_sof_intel_hda_common(E) snd_sof_pci(E) snd_sof_xtensa_dsp(E) snd_sof_intel_hda(E) snd_sof(E) snd_sof_utils(E) snd_soc_acpi_intel_match(E) snd_soc_acpi(E) snd_soc_hdac_hda(E) binfmt_misc(E) iwlmvm(E) snd_soc_core(E) snd_compress(E) snd_sof_intel_hda_mlink(E) x86_pkg_temp_thermal(E) wacom(E) snd_hda_ext_core(E) intel_powerclamp(E) i915(E) mac80211(E) joydev(E) coretemp(E) iTCO_wdt(E) usbhid(E) hid_multitouch(E) kvm_intel(E) intel_pmc_bxt(E) lis3lv02d_i2c(E) hid_generic(E) iTCO_vendor_support(E) dell_laptop(E)
> snd_hda_intel(E) lis3lv02d(E) dell_wmi(E) mei_wdt(E)
> [ 1528.578601]  nls_ascii(E) ledtrig_audio(E) ee1004(E) mei_hdcp(E) libarc4(E) mei_pxp(E) watchdog(E) nls_cp437(E) snd_intel_dspcfg(E) kvm(E)
> [ 1528.578609] idma64 idma64.1: PM: platform_pm_suspend+0x0/0x50 returned 0 after 742 usecs
> [ 1528.578612]  snd_hda_codec(E) btusb(E) snd_hwdep(E) vfat(E) i2c_algo_bit(E) processor_thermal_device_pci_legacy(E) btrtl(E) uvcvideo(E) snd_hda_core(E) dell_smbios(E) btintel(E) fat(E)
> [ 1528.578623] platform regulatory.0: PM: calling platform_pm_suspend+0x0/0x50 @ 70543, parent: platform
> [ 1528.578625]  videobuf2_vmalloc(E) dcdbas(E) intel_rapl_msr(E) snd_pcm(E) irqbypass(E)
> [ 1528.578628] platform regulatory.0: PM: platform_pm_suspend+0x0/0x50 returned 0 after 0 usecs
> [ 1528.578632]  drm_buddy(E) intel_soc_dts_iosf(E) dell_wmi_sysman(E) videobuf2_memops(E) btbcm(E) rapl(E) processor_thermal_device(E) uvc(E) iwlwifi(E)
> [ 1528.578639] iTCO_wdt iTCO_wdt: PM: calling platform_pm_suspend+0x0/0x50 @ 70543, parent: 0000:00:1f.4
> [ 1528.578641]  dell_smm_hwmon(E) ttm(E) videobuf2_v4l2(E) mxm_wmi(E)
> [ 1528.578644] iTCO_wdt iTCO_wdt: PM: platform_pm_suspend+0x0/0x50 returned 0 after 0 usecs
> [ 1528.578646]  snd_timer(E) firmware_attributes_class(E) wmi_bmof(E) intel_cstate(E) btmtk(E) dell_wmi_descriptor(E) intel_wmi_thunderbolt(E) processor_thermal_wt_hint(E) processor_thermal_rfim(E) intel_uncore(E) psmouse(E) bluetooth(E) pcspkr(E) drm_display_helper(E) i2c_i801(E) snd(E)
> [ 1528.578661] intel_rapl_msr intel_rapl_msr.0: PM: calling platform_pm_suspend+0x0/0x50 @ 70543, parent: platform
> [ 1528.578663]  processor_thermal_rapl(E) video(E) videodev(E) drm_kms_helper(E) cfg80211(E) soundcore(E)
> [ 1528.578667] intel_rapl_msr intel_rapl_msr.0: PM: platform_pm_suspend+0x0/0x50 returned 0 after 0 usecs
> [ 1528.578670]  i2c_smbus(E) intel_rapl_common(E)
> [ 1528.578667] i2c_hid_acpi i2c-WCOM490B:00: PM: calling acpi_subsys_suspend+0x0/0x60 @ 88, parent: i2c-1
> [ 1528.578672]  intel_lpss_pci(E) videobuf2_common(E) cec(E) mei_me(E) processor_thermal_wt_req(E) intel_lpss(E) processor_thermal_power_floor(E) ecdh_generic(E) thunderbolt(E)
> [ 1528.578679] idma64 idma64.0: PM: calling platform_pm_suspend+0x0/0x50 @ 70543, parent: 0000:00:15.0
> [ 1528.578683]  mc(E) cdc_acm(E) mei(E) rfkill(E) rc_core(E) idma64(E) processor_thermal_mbox(E) intel_pch_thermal(E) int3403_thermal(E) int340x_thermal_zone(E) dell_smo8800(E) wmi(E) button(E) intel_hid(E) evdev(E) battery(E) int3400_thermal(E) sparse_keymap(E) acpi_thermal_rel(E) acpi_pad(E) ac(E) parport_pc(E) ppdev(E) lp(E) parport(E) fuse(E) loop(E) efi_pstore(E) configfs(E) nfnetlink(E) ip_tables(E) x_tables(E) ext4(E) crc32c_generic(E) crc16(E) mbcache(E) jbd2(E) dm_crypt(E) nvme(E) rtsx_pci_sdmmc(E) xhci_pci(E) i2c_hid_acpi(E) nvme_core(E) i2c_hid(E) mmc_core(E) ahci(E) xhci_hcd(E) hid(E) t10_pi(E) libahci(E) crc64_rocksoft_generic(E) crc32_pclmul(E) ucsi_acpi(E) crc64_rocksoft(E) crc32c_intel(E) libata(E) typec_ucsi(E) crc_t10dif(E) ghash_clmulni_intel(E) crct10dif_generic(E) drm(E) roles(E) sha512_ssse3(E) usbcore(E) crct10dif_pclmul(E) sha256_ssse3(E) crc64(E) scsi_mod(E) sha1_ssse3(E) rtsx_pci(E) typec(E) crct10dif_common(E) scsi_common(E) usb_common(E)
> serio_raw(E) dm_mirror(E) dm_region_hash(E) dm_log(E)
> [ 1528.578760]  dm_mod(E) msr(E) efivarfs(E) autofs4(E) aesni_intel(E) crypto_simd(E) cryptd(E)
> [ 1528.578768] CPU: 1 PID: 92 Comm: kworker/u24:5 Tainted: G E      6.8.0-rc2 #9
> [ 1528.578773] Hardware name: Dell Inc. XPS 15 7590/0VYV0G, BIOS 1.24.0 09/11/2023
> [ 1528.578775] Workqueue: events_unbound async_run_entry_fn
> [ 1528.578783] RIP: 0010:_regulator_disable+0xbb/0x190
> [ 1528.578789] Code: 56 c7 45 0c 00 00 00 00 eb a8 48 8b 85 a8 03 00 00 48 85 c0 74 2a 48 8b 30 48 85 f6 74 22 48 c7 c7 6f a1 45 b9 e8 55 86 8b ff <0f> 0b 41 bc fb ff ff ff 5b 5d 44 89 e0 41 5c 41 5d c3 cc cc cc cc
> [ 1528.578793] RSP: 0018:ffffa2a3403e3d48 EFLAGS: 00010282
> [ 1528.578796] RAX: 0000000000000000 RBX: ffff8bd0456ee300 RCX: 0000000000000027
> [ 1528.578799] RDX: ffff8bd79a061708 RSI: 0000000000000001 RDI: ffff8bd79a061700
> [ 1528.578801] RBP: ffff8bd041304000 R08: 0000000000000000 R09: ffff8bd8bc03adff
> [ 1528.578803] R10: ffffffffffffffff R11: 20726f662073656c R12: ffffffffc10692c8
> [ 1528.578805] R13: 0000000000000002 R14: ffffffffc10692a8 R15: 0000000000000001
> [ 1528.578808] FS:  0000000000000000(0000) GS:ffff8bd79a040000(0000) knlGS:0000000000000000
> [ 1528.578811] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [ 1528.578813] CR2: 000055742d2f3e06 CR3: 0000000466020004 CR4: 00000000003706f0
> [ 1528.578816] Call Trace:
> [ 1528.578819]  <TASK>
> [ 1528.578820]  ? _regulator_disable+0xbb/0x190
> [ 1528.578825]  ? __warn+0x81/0x130
> [ 1528.578832]  ? _regulator_disable+0xbb/0x190
> [ 1528.578837]  ? report_bug+0x171/0x1a0
> [ 1528.578847]  ? handle_bug+0x3c/0x80
> [ 1528.578852]  ? exc_invalid_op+0x17/0x70
> [ 1528.578856]  ? asm_exc_invalid_op+0x1a/0x20
> [ 1528.578863]  ? _regulator_disable+0xbb/0x190
> [ 1528.578869]  ? _regulator_disable+0xbb/0x190
> [ 1528.578874]  regulator_bulk_disable+0x8e/0x180
> [ 1528.578880]  ? __pfx_lis3lv02d_i2c_suspend+0x10/0x10 [lis3lv02d_i2c]
> [ 1528.578890]  lis3lv02d_i2c_suspend+0x24/0x30 [lis3lv02d_i2c]
> [ 1528.578898]  dpm_run_callback+0x89/0x1e0
> [ 1528.578906]  __device_suspend+0xf1/0x4f0
> [ 1528.578912]  async_suspend+0x1e/0x70
> [ 1528.578918]  async_run_entry_fn+0x31/0x130
> [ 1528.578924]  process_one_work+0x171/0x340
> [ 1528.578930]  worker_thread+0x27b/0x3a0
> [ 1528.578936]  ? __pfx_worker_thread+0x10/0x10
> [ 1528.578940]  kthread+0xe5/0x120
> [ 1528.578944]  ? __pfx_kthread+0x10/0x10
> [ 1528.578948]  ret_from_fork+0x31/0x50
> [ 1528.578952]  ? __pfx_kthread+0x10/0x10
> [ 1528.578955]  ret_from_fork_asm+0x1b/0x30
> [ 1528.578965]  </TASK>
> [ 1528.578967] ---[ end trace 0000000000000000 ]---
> [ 1528.578970] Failed to disable Vdd_IO: -EIO
> [ 1528.578974] lis3lv02d_i2c 0-0029: PM: lis3lv02d_i2c_suspend+0x0/0x30 [lis3lv02d_i2c] returned 0 after 1190 usecs
> ```
> 
> Unfortunately, this is out of my expertise, so any help is appreciated.
> 
> 
> Kind regards,
> 
> Paul
> 
> 
> [1]: https://git.kernel.org/linus/2f189493ae32be9768b27072c9388e62b38d2dda
> 

Powered by blists - more mailing lists