# Re: \[PATCH 3/9\] sections: Move and rename core\_kernel\_data() to is\_kernel\_data()

- From: Kefeng Wang \<hidden\>
- Date: 2021-06-29 06:36:38 
- List: linux-arch
- Page: https://ratatoskr.run/linux-arch/2021/06/1428669
- Thread: https://ratatoskr.run/linux-arch/2021/06/1426532/t.md

```text

On 2021/6/29 9:05, Steven Rostedt wrote:
> On Sat, 26 Jun 2021 15:34:33 +0800
> Kefeng Wang [ref] wrote:
>
>> Move core_kernel_data() into sections.h and rename it to
>> is_kernel_data(), also make it return bool value, then
>> update all the callers.
> Removing the "core" part of "core_kernel_data()" is misleading. As
> modules can have kernel data, but this will return false on module data
> (as it should). This is similar to core_kernel_text() which this series
> doesn't seem to touch.

Yes, The series only collect and move the basic part(aka, the 'core' part, which only

contains the start/end address check of sections) into include/asm-generic/sections.h,

the core_kernel_text() contains system_state validation, it's not inappropriate into sections.h,

so it won't be modified and moved.

>
> I'd like to keep the "core" in the name which makes it obvious this is
> not about module data, and if someone were to make it about module
> data, it will break ftrace synchronization.

After this series, we have 5 APIs, only use core_kernel_data() is a little strange,

 * is_kernel_data()       --- come from core_kernel_data() in kernel.h
 * is_kernel_rodata()    --- already in sections.h
 * is_kernel_text()        --- come from kallsyms.h
 * is_kernel_inittext()   --- come from kernel.h and kallsyms.h
 * is_kernel()                --- come from kallsyms.h

Could we rename is_kernel_data() to is_kernel_core_data(), and add some comment your mentioned,

or any better naming, thanks.

>
> -- Steve
> .
>
```