Skip to content

Commit e638081

Browse files
sean-jcbp3tk0v
authored andcommitted
x86/lib: Drop the unused return value from wbinvd_on_all_cpus()
Drop wbinvd_on_all_cpus()'s return value; both the "real" version and the stub always return '0', and none of the callers check the return. Signed-off-by: Sean Christopherson <[email protected]> Signed-off-by: Borislav Petkov (AMD) <[email protected]> Link: https://lore.kernel.org/[email protected]
1 parent 1d738db commit e638081

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

arch/x86/include/asm/smp.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ void __noreturn hlt_play_dead(void);
112112
void native_play_dead(void);
113113
void play_dead_common(void);
114114
void wbinvd_on_cpu(int cpu);
115-
int wbinvd_on_all_cpus(void);
115+
void wbinvd_on_all_cpus(void);
116116

117117
void smp_kick_mwait_play_dead(void);
118118
void __noreturn mwait_play_dead(unsigned int eax_hint);
@@ -148,10 +148,9 @@ static inline struct cpumask *cpu_l2c_shared_mask(int cpu)
148148

149149
#else /* !CONFIG_SMP */
150150
#define wbinvd_on_cpu(cpu) wbinvd()
151-
static inline int wbinvd_on_all_cpus(void)
151+
static inline void wbinvd_on_all_cpus(void)
152152
{
153153
wbinvd();
154-
return 0;
155154
}
156155

157156
static inline struct cpumask *cpu_llc_shared_mask(int cpu)

arch/x86/lib/cache-smp.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,8 @@ void wbinvd_on_cpu(int cpu)
1414
}
1515
EXPORT_SYMBOL(wbinvd_on_cpu);
1616

17-
int wbinvd_on_all_cpus(void)
17+
void wbinvd_on_all_cpus(void)
1818
{
1919
on_each_cpu(__wbinvd, NULL, 1);
20-
return 0;
2120
}
2221
EXPORT_SYMBOL(wbinvd_on_all_cpus);

0 commit comments

Comments
 (0)