riscv: add compile-time test into is_compat_task()

Browse source

Currently several places will test for CONFIG_COMPAT before testing
is_compat_task(), probably in order to avoid a run-time test into the task
structure.

Since is_compat_task() is an inlined function, it would be helpful to add a
compile-time test of CONFIG_COMPAT, making sure it always returns zero when
the option is not enabled during the kernel build.

With this, the compiler is able to understand in build-time that
is_compat_task() will always return 0, and optimize-out some of the extra
code introduced by the option.

This will also allow removing a lot #ifdefs that were introduced, and make
the code more clean.

Signed-off-by: Leonardo Bras <leobras@redhat.com>
Reviewed-by: Guo Ren <guoren@kernel.org>
Reviewed-by: Andy Chiu <andy.chiu@sifive.com>
Link: https://lore.kernel.org/r/20240103160024.70305-5-leobras@redhat.com
Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>

...

This commit is contained in:

Leonardo Bras 2024-01-03 13:00:21 -03:00 • committed by Palmer Dabbelt

parent 9dc3041924

commit 4c0b5a4516

No known key found for this signature in database

GPG key ID: 2E1319F35FBB1889

4 changed files with 5 additions and 12 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

3

arch/riscv/include/asm/compat.h

View file

4

arch/riscv/include/asm/elf.h

View file

6

arch/riscv/include/asm/pgtable.h

View file

4

arch/riscv/include/asm/processor.h

View file