Bug 210221 - KASAN: turn CONFIG_KASAN_STACK into bool
Summary: KASAN: turn CONFIG_KASAN_STACK into bool
Status: RESOLVED CODE_FIX
Alias: None
Product: Memory Management
Classification: Unclassified
Component: Sanitizers (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: MM/Sanitizers virtual assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-16 13:50 UTC by Andrey Konovalov
Modified: 2021-02-03 11:05 UTC (History)
3 users (show)

See Also:
Kernel Version: upstream
Subsystem:
Regression: No
Bisected commit-id:


Attachments

Description Andrey Konovalov 2020-11-16 13:50:19 UTC
CONFIG_KASAN_STACK config option is currently defined as an int, while it's logically a bool. This is a bit confusing and is worth fixing.
Comment 1 Dmitry Vyukov 2020-11-16 13:59:51 UTC
FWIW we could also have just 1 config instead of 2 as well (CONFIG_KASAN_STACK/CONFIG_KASAN_STACK_ENABLE).
For gcc we could do no prompt and default value y, and for clang --
prompt and default value n. I think it should do what we need.
Comment 2 Walter Wu 2021-01-07 06:22:25 UTC
Hi Dmitry/Andrey,

I already sent one patch(kasan: remove redundant config option) to fix it.
Thank you for your suggestion.

Walter
Comment 3 Dmitry Vyukov 2021-02-03 11:05:56 UTC
"kasan: remove redundant config option" is now in linux-next.

Note You need to log in before you can comment on or make changes to this bug.