From: Tom Rix <[email protected]>
Attributing the function allows the compiler to more thoroughly
check the use of the function with -Wformat and similar flags.
Signed-off-by: Tom Rix <[email protected]>
---
arch/x86/kernel/cpu/resctrl/internal.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h
index ee71c47844cb..c4d320d02fd5 100644
--- a/arch/x86/kernel/cpu/resctrl/internal.h
+++ b/arch/x86/kernel/cpu/resctrl/internal.h
@@ -572,6 +572,7 @@ union cpuid_0x10_x_edx {
void rdt_last_cmd_clear(void);
void rdt_last_cmd_puts(const char *s);
+__printf(1, 2)
void rdt_last_cmd_printf(const char *fmt, ...);
void rdt_ctrl_update(void *arg);
--
2.27.0
Hi Tom,
On 12/21/2020 8:00 AM, [email protected] wrote:
> From: Tom Rix <[email protected]>
>
> Attributing the function allows the compiler to more thoroughly
> check the use of the function with -Wformat and similar flags.
>
> Signed-off-by: Tom Rix <[email protected]>
> ---
> arch/x86/kernel/cpu/resctrl/internal.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h
> index ee71c47844cb..c4d320d02fd5 100644
> --- a/arch/x86/kernel/cpu/resctrl/internal.h
> +++ b/arch/x86/kernel/cpu/resctrl/internal.h
> @@ -572,6 +572,7 @@ union cpuid_0x10_x_edx {
>
> void rdt_last_cmd_clear(void);
> void rdt_last_cmd_puts(const char *s);
> +__printf(1, 2)
> void rdt_last_cmd_printf(const char *fmt, ...);
>
> void rdt_ctrl_update(void *arg);
>
This is a valuable addition, thank you.
Acked-by: Reinette Chatre <[email protected]>
Reinette
The following commit has been merged into the x86/cache branch of tip:
Commit-ID: 91031e096e1fa0216027bfb7fdca931225aebbf0
Gitweb: https://git.kernel.org/tip/91031e096e1fa0216027bfb7fdca931225aebbf0
Author: Tom Rix <[email protected]>
AuthorDate: Mon, 21 Dec 2020 08:00:09 -08:00
Committer: Borislav Petkov <[email protected]>
CommitterDate: Fri, 08 Jan 2021 08:55:02 +01:00
x86/resctrl: Add printf attribute to log function
Mark the function with the __printf attribute to allow the compiler to
more thoroughly typecheck its arguments against a format string with
-Wformat and similar flags.
[ bp: Massage commit message. ]
Signed-off-by: Tom Rix <[email protected]>
Signed-off-by: Borislav Petkov <[email protected]>
Acked-by: Reinette Chatre <[email protected]>
Link: https://lkml.kernel.org/r/[email protected]
---
arch/x86/kernel/cpu/resctrl/internal.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h
index ee71c47..c4d320d 100644
--- a/arch/x86/kernel/cpu/resctrl/internal.h
+++ b/arch/x86/kernel/cpu/resctrl/internal.h
@@ -572,6 +572,7 @@ union cpuid_0x10_x_edx {
void rdt_last_cmd_clear(void);
void rdt_last_cmd_puts(const char *s);
+__printf(1, 2)
void rdt_last_cmd_printf(const char *fmt, ...);
void rdt_ctrl_update(void *arg);
The following commit has been merged into the x86/cache branch of tip:
Commit-ID: 3ff4ec0e281d0b234917e6e3033dd3067a5ea945
Gitweb: https://git.kernel.org/tip/3ff4ec0e281d0b234917e6e3033dd3067a5ea945
Author: Tom Rix <[email protected]>
AuthorDate: Mon, 21 Dec 2020 08:00:09 -08:00
Committer: Borislav Petkov <[email protected]>
CommitterDate: Mon, 11 Jan 2021 11:20:36 +01:00
x86/resctrl: Add printf attribute to log function
Mark the function with the __printf attribute to allow the compiler to
more thoroughly typecheck its arguments against a format string with
-Wformat and similar flags.
[ bp: Massage commit message. ]
Signed-off-by: Tom Rix <[email protected]>
Signed-off-by: Borislav Petkov <[email protected]>
Acked-by: Reinette Chatre <[email protected]>
Link: https://lkml.kernel.org/r/[email protected]
---
arch/x86/kernel/cpu/resctrl/internal.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h
index ee71c47..c4d320d 100644
--- a/arch/x86/kernel/cpu/resctrl/internal.h
+++ b/arch/x86/kernel/cpu/resctrl/internal.h
@@ -572,6 +572,7 @@ union cpuid_0x10_x_edx {
void rdt_last_cmd_clear(void);
void rdt_last_cmd_puts(const char *s);
+__printf(1, 2)
void rdt_last_cmd_printf(const char *fmt, ...);
void rdt_ctrl_update(void *arg);