lkml.org 
[lkml]   [2017]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC PATCH 1/3] mm, numa: rework do_pages_move
On Thu 14-12-17 18:35:58, Kirill A. Shutemov wrote:
> On Wed, Dec 13, 2017 at 03:39:48PM +0100, Michal Hocko wrote:
[...]
> > + err = 0;
> > + if (page_to_nid(page) == node)
> > + goto out_putpage;
> > +
> > + err = -EACCES;
> > + if (page_mapcount(page) > 1 &&
> > + !migrate_all)
>
> Non-sensible line break.

fixed

> > + goto out_putpage;
> > +
> > + if (PageHuge(page)) {
> > + if (PageHead(page)) {
> > + isolate_huge_page(page, pagelist);
> > + err = 0;
> > + }
> > + } else {
>
> Hm. I think if the page is PageTail() we have to split the huge page.
> If an user asks to migrate part of THP, we shouldn't migrate the whole page,
> otherwise it's not transparent anymore.

Well, as I've said in the cover letter. There are more things which are
worth considering but I've tried to keep the original semantic so
further changes should be done in separete patches. I will work on those
but I would prefer this to stay smaller if you do not mind.

> Otherwise, the patch looks good to me.

Thanks for the review
--
Michal Hocko
SUSE Labs

\
 
 \ /
  Last update: 2017-12-15 10:29    [W:0.095 / U:0.656 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site