From: Jonathan Cameron <jic23@kernel.org>
To: kernel test robot <oliver.sang@intel.com>
Cc: Nuno Sa <nuno.sa@analog.com>, <oe-lkp@lists.linux.dev>,
<lkp@intel.com>,
Linux Memory Management List <linux-mm@kvack.org>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
<linux-iio@vger.kernel.org>,
"Borislav Petkov (AMD)" <bp@alien8.de>,
Paul Menzel <pmenzel@molgen.mpg.de>,
Marco Elver <elver@google.com>,
Masahiro Yamada <masahiroy@kernel.org>
Subject: Re: [linux-next:master] [iio] 8d61d01cde: WARNING:at_arch/x86/kernel/cpu/bugs.c:#__warn_thunk
Date: Tue, 16 Jul 2024 17:37:22 +0100 [thread overview]
Message-ID: <20240716173722.5e582581@jic23-huawei> (raw)
In-Reply-To: <202407161520.ee3d9347-lkp@intel.com>
On Tue, 16 Jul 2024 15:59:12 +0800
kernel test robot <oliver.sang@intel.com> wrote:
> Hello,
>
> kernel test robot noticed "WARNING:at_arch/x86/kernel/cpu/bugs.c:#__warn_thunk" on:
>
> commit: 8d61d01cdec275719b70295e98e4d45c5f284f29 ("iio: gyro: adis16260: make use of the new lock helpers")
> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
>
> [test failed on linux-next/master 91e3b24eb7d297d9d99030800ed96944b8652eaf]
>
> in testcase: rcutorture
> version:
> with following parameters:
>
> runtime: 300s
> test: cpuhotplug
> torture_type: trivial
>
>
>
> compiler: clang-18
> test machine: qemu-system-x86_64 -enable-kvm -cpu SandyBridge -smp 2 -m 16G
>
> (please refer to attached dmesg/kmsg for entire log/backtrace)
>
>
> +-----------------------------------------------------+------------+------------+
> | | ccd52641f9 | 8d61d01cde |
> +-----------------------------------------------------+------------+------------+
> | WARNING:at_arch/x86/kernel/cpu/bugs.c:#__warn_thunk | 0 | 30 |
> | RIP:__warn_thunk | 0 | 30 |
> +-----------------------------------------------------+------------+------------+
Hi All.
I'm struggling to have any idea what this is reporting.
The code is question must be the use of adis_dev_autoscoped_lock()
which is very straight forward.
adis_dev_auto_scoped_lock(adis) {
if (t >= 0x0A)
adis->spi->max_speed_hz = ADIS16260_SPI_SLOW;
else
adis->spi->max_speed_hz = ADIS16260_SPI_FAST;
return __adis_write_reg_8(adis, ADIS16260_SMPL_PRD, t);
}
unreachable();
//needed because not all compilers seem to be able to tell
//that a scoped_guard() always runs.
where the definition is simply
#define adis_dev_auto_scoped_lock(adis) \
scoped_guard(mutex, &(adis)->state_lock)
Looks like there was some fun with ksan earlier in the year that smells superficially
similar but that is long resolved? (+CC some of the folk involved in solving that).
Any thoughts?
Jonathan
>
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <oliver.sang@intel.com>
> | Closes: https://lore.kernel.org/oe-lkp/202407161520.ee3d9347-lkp@intel.com
>
>
> [ 6.698254][ T1] ------------[ cut here ]------------
> [ 6.698774][ T1] Unpatched return thunk in use. This should not happen!
> [ 6.699377][ T1] WARNING: CPU: 0 PID: 1 at arch/x86/kernel/cpu/bugs.c:3023 __warn_thunk (arch/x86/kernel/cpu/bugs.c:3023)
> [ 6.699604][ T1] Modules linked in:
> [ 6.699937][ T1] CPU: 0 PID: 1 Comm: swapper Not tainted 6.10.0-rc4-00198-g8d61d01cdec2 #1
> [ 6.700668][ T1] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.16.2-debian-1.16.2-1 04/01/2014
> [ 6.701548][ T1] RIP: 0010:__warn_thunk (arch/x86/kernel/cpu/bugs.c:3023)
> [ 6.701983][ T1] Code: 1f 44 00 00 80 3d dd fb e3 03 00 74 08 31 ff c3 cc cc cc cc cc c6 05 cc fb e3 03 01 90 48 c7 c7 40 b0 a1 83 e8 76 c6 13 00 90 <0f> 0b 90 90 31 ff c3 cc cc cc cc cc cc cc cc cc cc cc cc cc cc b8
> All code
> ========
> 0: 1f (bad)
> 1: 44 00 00 add %r8b,(%rax)
> 4: 80 3d dd fb e3 03 00 cmpb $0x0,0x3e3fbdd(%rip) # 0x3e3fbe8
> b: 74 08 je 0x15
> d: 31 ff xor %edi,%edi
> f: c3 ret
> 10: cc int3
> 11: cc int3
> 12: cc int3
> 13: cc int3
> 14: cc int3
> 15: c6 05 cc fb e3 03 01 movb $0x1,0x3e3fbcc(%rip) # 0x3e3fbe8
> 1c: 90 nop
> 1d: 48 c7 c7 40 b0 a1 83 mov $0xffffffff83a1b040,%rdi
> 24: e8 76 c6 13 00 call 0x13c69f
> 29: 90 nop
> 2a:* 0f 0b ud2 <-- trapping instruction
> 2c: 90 nop
> 2d: 90 nop
> 2e: 31 ff xor %edi,%edi
> 30: c3 ret
> 31: cc int3
> 32: cc int3
> 33: cc int3
> 34: cc int3
> 35: cc int3
> 36: cc int3
> 37: cc int3
> 38: cc int3
> 39: cc int3
> 3a: cc int3
> 3b: cc int3
> 3c: cc int3
> 3d: cc int3
> 3e: cc int3
> 3f: b8 .byte 0xb8
>
> Code starting with the faulting instruction
> ===========================================
> 0: 0f 0b ud2
> 2: 90 nop
> 3: 90 nop
> 4: 31 ff xor %edi,%edi
> 6: c3 ret
> 7: cc int3
> 8: cc int3
> 9: cc int3
> a: cc int3
> b: cc int3
> c: cc int3
> d: cc int3
> e: cc int3
> f: cc int3
> 10: cc int3
> 11: cc int3
> 12: cc int3
> 13: cc int3
> 14: cc int3
> 15: b8 .byte 0xb8
> [ 6.703602][ T1] RSP: 0000:ffffc9000001fe88 EFLAGS: 00010246
> [ 6.704116][ T1] RAX: 0000000000000000 RBX: ffffffff856a96a0 RCX: 0000000000000000
> [ 6.704787][ T1] RDX: 0000000000000000 RSI: 0000000000000000 RDI: 0000000000000000
> [ 6.705462][ T1] RBP: ffffc9000001fed8 R08: 0000000000000000 R09: 0000000000000000
> [ 6.706134][ T1] R10: 0000000000000000 R11: 0000000000000000 R12: 0000000000000000
> [ 6.706804][ T1] R13: 0000000000000000 R14: dffffc0000000000 R15: ffffffff856a96a0
> [ 6.707476][ T1] FS: 0000000000000000(0000) GS:ffffffff844a7000(0000) knlGS:0000000000000000
> [ 6.707601][ T1] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [ 6.708157][ T1] CR2: ffff88843ffff000 CR3: 000000000446c000 CR4: 00000000000406f0
> [ 6.708834][ T1] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [ 6.709505][ T1] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> [ 6.710174][ T1] Call Trace:
> [ 6.710452][ T1] <TASK>
> [ 6.710701][ T1] ? __warn (kernel/panic.c:239 kernel/panic.c:693)
> [ 6.711054][ T1] ? __warn_thunk (arch/x86/kernel/cpu/bugs.c:3023)
> [ 6.711434][ T1] ? __warn_thunk (arch/x86/kernel/cpu/bugs.c:3023)
> [ 6.711604][ T1] ? report_bug (lib/bug.c:?)
> [ 6.711991][ T1] ? handle_bug (arch/x86/kernel/traps.c:239)
> [ 6.712357][ T1] ? exc_invalid_op (arch/x86/kernel/traps.c:260)
> [ 6.712759][ T1] ? asm_exc_invalid_op (arch/x86/include/asm/idtentry.h:621)
> [ 6.713193][ T1] ? __warn_thunk (arch/x86/kernel/cpu/bugs.c:3023)
> [ 6.713575][ T1] ? __warn_thunk (arch/x86/kernel/cpu/bugs.c:3023)
> [ 6.713956][ T1] warn_thunk_thunk (arch/x86/entry/entry.S:48)
> [ 6.714352][ T1] ? __cfi_asan.module_ctor (main.c:?)
> [ 6.714810][ T1] do_basic_setup (init/main.c:1126 init/main.c:1363)
> [ 6.715194][ T1] kernel_init_freeable (init/main.c:1582)
> [ 6.715602][ T1] ? __cfi_kernel_init (init/main.c:1459)
> [ 6.716022][ T1] kernel_init (init/main.c:1469)
> [ 6.716389][ T1] ret_from_fork (arch/x86/kernel/process.c:153)
> [ 6.716769][ T1] ? __cfi_kernel_init (init/main.c:1459)
> [ 6.717187][ T1] ret_from_fork_asm (arch/x86/entry/entry_64.S:257)
> [ 6.717600][ T1] </TASK>
> [ 6.717856][ T1] irq event stamp: 14123
> [ 6.718213][ T1] hardirqs last enabled at (14131): console_unlock (arch/x86/include/asm/irqflags.h:42 arch/x86/include/asm/irqflags.h:77 arch/x86/include/asm/irqflags.h:135 kernel/printk/printk.c:341 kernel/printk/printk.c:2746 kernel/printk/printk.c:3065)
> [ 6.719011][ T1] hardirqs last disabled at (14138): console_unlock (kernel/printk/printk.c:339)
> [ 6.719601][ T1] softirqs last enabled at (2244): irq_exit_rcu (kernel/softirq.c:651)
> [ 6.720366][ T1] softirqs last disabled at (2235): irq_exit_rcu (kernel/softirq.c:651)
> [ 6.721120][ T1] ---[ end trace 0000000000000000 ]---
>
>
>
> The kernel config and materials to reproduce are available at:
> https://download.01.org/0day-ci/archive/20240716/202407161520.ee3d9347-lkp@intel.com
>
>
>
next prev parent reply other threads:[~2024-07-16 16:37 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-16 7:59 [linux-next:master] [iio] 8d61d01cde: WARNING:at_arch/x86/kernel/cpu/bugs.c:#__warn_thunk kernel test robot
2024-07-16 16:37 ` Jonathan Cameron [this message]
2024-08-05 5:40 ` Oliver Sang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240716173722.5e582581@jic23-huawei \
--to=jic23@kernel.org \
--cc=Jonathan.Cameron@huawei.com \
--cc=bp@alien8.de \
--cc=elver@google.com \
--cc=linux-iio@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lkp@intel.com \
--cc=masahiroy@kernel.org \
--cc=nuno.sa@analog.com \
--cc=oe-lkp@lists.linux.dev \
--cc=oliver.sang@intel.com \
--cc=pmenzel@molgen.mpg.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).