common/softirq: address violation of MISRA C Rule 13.6

In the file common/softirq macro set_bit is called with argument
smp_processor_id.
Once expanded this set_bit's argument is used in sizeof operations
and thus 'smp_processor_id', being a macro that may expand to a
function call with potential side effects, generates a violation.

To address this violation the value of smp_processor_id is therefore
stored in a variable called 'cpu' before passing it to macro set_bit.

No functional change.

Signed-off-by: Alessandro Zucchelli <alessandro.zucchelli@bugseng.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
This commit is contained in:
Alessandro Zucchelli authored and Jan Beulich committed 2024-07-23 14:00:27 +02:00
1 parent d51b2f5ea1
commit daff80607a
1 file changed
+3 -1
+3 -1
View File
@@ -139,7 +139,9 @@ void cpu_raise_softirq_batch_finish(void)
void raise_softirq(unsigned int nr)
{
set_bit(nr, &softirq_pending(smp_processor_id()));
unsigned int cpu = smp_processor_id();
set_bit(nr, &softirq_pending(cpu));
}
/*
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.