lkml.org 
[lkml]   [2017]   [Jul]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] mm/mremap: Document MREMAP_FIXED dependency on MREMAP_MAYMOVE
From
Date
On 07/10/2017 11:01 PM, Mike Kravetz wrote:
> On 07/10/2017 06:41 AM, Michal Hocko wrote:
>> On Mon 10-07-17 17:02:11, Anshuman Khandual wrote:
>>> In the header file, just specify the dependency of MREMAP_FIXED
>>> on MREMAP_MAYMOVE and make it explicit for the user space.
>> I really fail to see a point of this patch. The depency belongs to the
>> code and it seems that we already enforce it
>> if (flags & MREMAP_FIXED && !(flags & MREMAP_MAYMOVE))
>> return ret;
>>
>> So what is the point here?
> Agree, I am not sure of your reasoning.
>
> If to assist the programmer, there is no need as this is clearly specified
> in the man page:
>
> "If MREMAP_FIXED is specified, then MREMAP_MAYMOVE must also be
> specified."

Yeah the idea was to assist the programmer and I missed the man page's
details on this. My bad.

\
 
 \ /
  Last update: 2017-07-11 04:17    [W:0.047 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site