When KCOV is enabled all functions get instrumented, unless the __no_sanitize_coverage attribute is used. To prepare for __no_sanitize_coverage being applied to __init functions[1], we have to handle differences in how GCC's inline optimizations get resolved. For thinkpad_acpi routines, this means forcing two functions to be inline with __always_inline.
Link: https://lore.kernel.org/lkml/20250523043935.2009972-11-k...@kernel.org/ [1] Signed-off-by: Kees Cook <k...@kernel.org> --- This is split out of https://lore.kernel.org/lkml/20250523043935.2009972-4-k...@kernel.org/ at Ilpo's request. --- Cc: "Ilpo Järvinen" <ilpo.jarvi...@linux.intel.com> Cc: Henrique de Moraes Holschuh <h...@hmh.eng.br> Cc: Hans de Goede <hdego...@redhat.com> Cc: <ibm-acpi-devel@lists.sourceforge.net> Cc: <platform-driver-...@vger.kernel.org> --- drivers/platform/x86/thinkpad_acpi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index 657625dd60a0..dc1fc069fed9 100644 --- a/drivers/platform/x86/thinkpad_acpi.c +++ b/drivers/platform/x86/thinkpad_acpi.c @@ -558,12 +558,12 @@ static unsigned long __init tpacpi_check_quirks( return 0; } -static inline bool __pure __init tpacpi_is_lenovo(void) +static __always_inline bool __pure __init tpacpi_is_lenovo(void) { return thinkpad_id.vendor == PCI_VENDOR_ID_LENOVO; } -static inline bool __pure __init tpacpi_is_ibm(void) +static __always_inline bool __pure __init tpacpi_is_ibm(void) { return thinkpad_id.vendor == PCI_VENDOR_ID_IBM; } -- 2.34.1
