Re: [PATCH 8/10] Mempolicy man pages: misc cleanup of mbind(2)

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

 



Lee,

On Thu, Aug 7, 2008 at 6:21 PM, Lee Schermerhorn
<Lee.Schermerhorn@xxxxxx> wrote:
> On Thu, 2008-08-07 at 15:35 +0200, Michael Kerrisk wrote:
>> Lee,
>>
>> Lee Schermerhorn wrote:
>> > Attempt to claify discussion of MPOL_DEFAULT.
>> >
>> > Signed-off-by:  Lee Schermerhorn <lee.schermerhorn@xxxxxx>
>>
>> These look like good changes.
>>
>> One question: is this just improving the description, or has some behavior
>> changed in a recent kernel version, and the text is being adjusted for that
>> change?
>
> I'd like to think it improves the description :).  Already discussed the
> change in implementation != change in behavior in response to the same
> question on [one of the] patch #4.
>
> I see that here I used "explicit \"local allocation\"" again, but with
> the quotes in what I think are the right places.

I've applied tis for man-page-3.07.

Thanks!

Michael

>
>>
>> Cheers,
>>
>> Michael
>>
>> >
>> >  man2/mbind.2 |   18 +++++++++++++-----
>> >  1 files changed, 13 insertions(+), 5 deletions(-)
>> >
>> > Index: man-pages-3.05/man2/mbind.2
>> > ===================================================================
>> > --- man-pages-3.05.orig/man2/mbind.2        2008-07-29 16:53:48.000000000 -0400
>> > +++ man-pages-3.05/man2/mbind.2     2008-07-29 16:54:13.000000000 -0400
>> > @@ -184,7 +184,8 @@ the memory policy reverts to
>> >
>> >  The
>> >  .B MPOL_DEFAULT
>> > -mode specifies that the default policy should be used.
>> > +mode requests that any non-default policy be removed,
>> > +restoring default behavior.
>> >  When applied to a range of memory via
>> >  .BR mbind (),
>> >  this means to use the process policy,
>> > @@ -193,7 +194,7 @@ which may have been set with
>> >  If the mode of the process policy is also
>> >  .BR MPOL_DEFAULT ,
>> >  the system-wide default policy will be used.
>> > -The system-wide default policy will allocate
>> > +The system-wide default policy allocates
>> >  pages on the node of the CPU that triggers the allocation.
>> >  For
>> >  .BR MPOL_DEFAULT ,
>> > @@ -399,20 +400,27 @@ is ignored on huge page mappings.
>> >
>> >  The
>> >  .BR MPOL_DEFAULT ,
>> > -mode has different effects for
>> > +mode can have different effects for
>> >  .BR mbind ()
>> >  and
>> >  .BR set_mempolicy (2).
>> >  When
>> >  .B MPOL_DEFAULT
>> > +is specified for
>> > +.BR set_mempolicy (2),
>> > +the process' policy reverts to system default policy
>> > +or local allocation.
>> > +When
>> > +.B MPOL_DEFAULT
>> >  is specified for a range of memory using
>> >  .BR mbind (),
>> >  any pages subsequently allocated for that range will use
>> >  the process's policy, as set by
>> >  .BR set_mempolicy (2).
>> >  This effectively removes the explicit policy from the
>> > -specified range.
>> > -To select "local allocation" for a memory range,
>> > +specified range, "falling back" to a possibly non-default
>> > +policy.
>> > +To select explicit "local allocation" for a memory range,
>> >  specify a
>> >  .I mode
>> >  of
>> >
>>
>
>



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
man-pages online: http://www.kernel.org/doc/man-pages/online_pages.html
Found a bug? http://www.kernel.org/doc/man-pages/reporting_bugs.html
--
To unsubscribe from this list: send the line "unsubscribe linux-numa" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]     [Devices]

  Powered by Linux