Hi Tao,
On Tue, Sep 10, 2024 at 3:20 PM <qiwu.chen(a)transsion.com> wrote:
Can we remove this one? I see the value has been changed into a new
one since v6.11-rc1.
PAGE_TYPE_BASE = 0x80000000
Is keeping the value hard coded here necessary?
I don't think so, since the value can be got from vmcoreinfo after kernel
commit ff202303c398e,
keeping the value hard before kernel commit ff202303c398e:
#define PAGE_TYPE_BASE 0xf0000000
So just keeping the original code will get the correct value for any kernel version.
The variable name "pp" doesn't make sense. No one knows what pp is
until you read its context.
Okay, how about renaming "pp" to "page"?
Thanks
Qiwu