lkml.org 
[lkml]   [2017]   [Oct]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 1/4] kaslr: parse the extended movable_node=nn[KMG]@ss[KMG]
From
Date
[...]
>>>>>
>>>>> +#ifdef CONFIG_MEMORY_HOTPLUG
>>>>> +static void mem_mark_immovable(char *str)
>>>>> +{
>>>>> + int i = 0;
>>>>> +
>>>>
>>>> you have use num_immovable_region, 'i' is useless. just remove it.
>>>
>>> Using num_immovable_region makes code too long. Using i will be
>>> clear and make sure shoter than 80 characters.
>>
>> Oh, God, that's horrific. Did you find that your code is wrong?
>>
>> num_immovable_region will be reset each time.
>
> Did you test?
>

you can try with more than one movable_node, eg:

"...movable_node=128G@128G movable_node=128G@256G..."

then, you will find the problem of you code.

Thanks,
dou


\
 
 \ /
  Last update: 2017-10-22 17:34    [W:0.073 / U:0.708 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site