Skip to content

Commit 291c401

Browse files
anadavIngo Molnar
authored andcommitted
cpumask: Mark functions as pure
cpumask_next_and() and cpumask_any_but() are pure, and marking them as such seems to generate different and presumably better code for native_flush_tlb_multi(). Signed-off-by: Nadav Amit <[email protected]> Signed-off-by: Ingo Molnar <[email protected]> Reviewed-by: Dave Hansen <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent 09c5272 commit 291c401

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

include/linux/cpumask.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ static inline unsigned int cpumask_last(const struct cpumask *srcp)
235235
return find_last_bit(cpumask_bits(srcp), nr_cpumask_bits);
236236
}
237237

238-
unsigned int cpumask_next(int n, const struct cpumask *srcp);
238+
unsigned int __pure cpumask_next(int n, const struct cpumask *srcp);
239239

240240
/**
241241
* cpumask_next_zero - get the next unset cpu in a cpumask
@@ -252,8 +252,8 @@ static inline unsigned int cpumask_next_zero(int n, const struct cpumask *srcp)
252252
return find_next_zero_bit(cpumask_bits(srcp), nr_cpumask_bits, n+1);
253253
}
254254

255-
int cpumask_next_and(int n, const struct cpumask *, const struct cpumask *);
256-
int cpumask_any_but(const struct cpumask *mask, unsigned int cpu);
255+
int __pure cpumask_next_and(int n, const struct cpumask *, const struct cpumask *);
256+
int __pure cpumask_any_but(const struct cpumask *mask, unsigned int cpu);
257257
unsigned int cpumask_local_spread(unsigned int i, int node);
258258
int cpumask_any_and_distribute(const struct cpumask *src1p,
259259
const struct cpumask *src2p);

0 commit comments

Comments
 (0)