Re: [PATCH 3/9] sections: Move and rename core_kernel_data() to is_kernel_data()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 2021/6/29 9:05, Steven Rostedt wrote:
On Sat, 26 Jun 2021 15:34:33 +0800
Kefeng Wang <wangkefeng.wang@xxxxxxxxxx> 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
.




[Index of Archives]     [Linux Kernel]     [Kernel Newbies]     [x86 Platform Driver]     [Netdev]     [Linux Wireless]     [Netfilter]     [Bugtraq]     [Linux Filesystems]     [Yosemite Discussion]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]

  Powered by Linux