Commit 1dc01aba authored by Borislav Petkov's avatar Borislav Petkov Committed by Peter Zijlstra
Browse files

cpumask: Always inline helpers which use bit manipulation functions 



Former are always inlined so do that for the latter too, for
consistency.

Size impact is a whopping 5 bytes increase! :-)

   text    data     bss     dec     hex filename
22350551        8213184 1917164 32480899        1ef9e83 vmlinux.x86-64.defconfig.before
22350556        8213152 1917164 32480872        1ef9e68 vmlinux.x86-64.defconfig.after

Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Acked-by: default avatarMarco Elver <elver@google.com>
Link: https://lore.kernel.org/r/20220113155357.4706-3-bp@alien8.de
parent acb13ea0

include/linux/cpumask.h

+7 −7
Original line number Diff line number Diff line
@@ -341,12 +341,12 @@ extern int cpumask_next_wrap(int n, const struct cpumask *mask, int start, bool 
 * @cpu: cpu number (< nr_cpu_ids)

 * @dstp: the cpumask pointer

 */

static inline void cpumask_set_cpu(unsigned int cpu, struct cpumask *dstp)

static __always_inline void cpumask_set_cpu(unsigned int cpu, struct cpumask *dstp)

{

	set_bit(cpumask_check(cpu), cpumask_bits(dstp));

}



static inline void __cpumask_set_cpu(unsigned int cpu, struct cpumask *dstp)

static __always_inline void __cpumask_set_cpu(unsigned int cpu, struct cpumask *dstp)

{

	__set_bit(cpumask_check(cpu), cpumask_bits(dstp));

}
@@ -357,12 +357,12 @@ static inline void __cpumask_set_cpu(unsigned int cpu, struct cpumask *dstp) 
 * @cpu: cpu number (< nr_cpu_ids)

 * @dstp: the cpumask pointer

 */

static inline void cpumask_clear_cpu(int cpu, struct cpumask *dstp)

static __always_inline void cpumask_clear_cpu(int cpu, struct cpumask *dstp)

{

	clear_bit(cpumask_check(cpu), cpumask_bits(dstp));

}



static inline void __cpumask_clear_cpu(int cpu, struct cpumask *dstp)

static __always_inline void __cpumask_clear_cpu(int cpu, struct cpumask *dstp)

{

	__clear_bit(cpumask_check(cpu), cpumask_bits(dstp));

}
@@ -374,7 +374,7 @@ static inline void __cpumask_clear_cpu(int cpu, struct cpumask *dstp) 
 *

 * Returns 1 if @cpu is set in @cpumask, else returns 0

 */

static inline int cpumask_test_cpu(int cpu, const struct cpumask *cpumask)

static __always_inline int cpumask_test_cpu(int cpu, const struct cpumask *cpumask)

{

	return test_bit(cpumask_check(cpu), cpumask_bits((cpumask)));

}
@@ -388,7 +388,7 @@ static inline int cpumask_test_cpu(int cpu, const struct cpumask *cpumask) 
 *

 * test_and_set_bit wrapper for cpumasks.

 */

static inline int cpumask_test_and_set_cpu(int cpu, struct cpumask *cpumask)

static __always_inline int cpumask_test_and_set_cpu(int cpu, struct cpumask *cpumask)

{

	return test_and_set_bit(cpumask_check(cpu), cpumask_bits(cpumask));

}
@@ -402,7 +402,7 @@ static inline int cpumask_test_and_set_cpu(int cpu, struct cpumask *cpumask) 
 *

 * test_and_clear_bit wrapper for cpumasks.

 */

static inline int cpumask_test_and_clear_cpu(int cpu, struct cpumask *cpumask)

static __always_inline int cpumask_test_and_clear_cpu(int cpu, struct cpumask *cpumask)

{

	return test_and_clear_bit(cpumask_check(cpu), cpumask_bits(cpumask));

}