Re: [tip: x86/core] x86: Fix CPUIDLE_FLAG_IRQ_ENABLE leaking timer reprogram
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [tip: x86/core] x86: Fix CPUIDLE_FLAG_IRQ_ENABLE leaking timer reprogram
- From: Frederic Weisbecker <frederic@xxxxxxxxxx>
- Date: Tue, 12 Dec 2023 14:46:58 +0100
- Cc: linux-kernel@xxxxxxxxxxxxxxx, Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>, linux-tip-commits@xxxxxxxxxxxxxxx, "Rafael J. Wysocki" <rafael@xxxxxxxxxx>, x86@xxxxxxxxxx
- In-reply-to: <20231130111519.GA20153@noisy.programming.kicks-ass.net>
- References: <20231115151325.6262-3-frederic@kernel.org> <170126975511.398.12493947150541739641.tip-bot2@tip-bot2> <20231130111519.GA20153@noisy.programming.kicks-ass.net>
On Thu, Nov 30, 2023 at 12:15:19PM +0100, Peter Zijlstra wrote:
> This is so, because all mwait users should be in __cpuidle section,
> which itself is part of the noinstr section and as such
> kprobes/hw-breakpoints etc.. are disallowed.
>
> Notable vmlinux.lds.h has:
>
> #define NOINSTR_TEXT \
> ALIGN_FUNCTION(); \
> __noinstr_text_start = .; \
> *(.noinstr.text) \
> __cpuidle_text_start = .; \
> *(.cpuidle.text) \
> __cpuidle_text_end = .; \
> __noinstr_text_end = .;
So #DB aren't supposed to happen then, right? Or you noticed an mwait
user that doesn't have __cpuidle?
Thanks.
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]