Re: [Lhms-devel] [patch 4/6] memory hotplug for hugetlbpages

[Hirokazu Takahashi]

Hello,

> > @@ -1667,6 +1670,7 @@ int handle_mm_fault(struct mm_struct *mm
> >  pmd_t fastcall *__pmd_alloc(struct mm_struct *mm, pgd_t *pgd, unsigned long address)
> >  {
> >  	pmd_t *new;
> > +	struct page *page;
> >  
> >  	spin_unlock(&mm->page_table_lock);
> >  	new = pmd_alloc_one(mm, address);
> > @@ -1682,6 +1686,8 @@ pmd_t fastcall *__pmd_alloc(struct mm_st
> >  		pmd_free(new);
> >  		goto out;
> >  	}
> > +	page = virt_to_page(new);
> > +	pmd_add_rmap(new, mm, address);
> 
> Doesn't this want to be:
> 
> 	pmd_add_rmap(page, mm, address);
> 
> ?
> 
> And how about collapsing this down to:
> 
> 	pmd_add_rmap(virt_to_page(new), mm, address);

Yes, it can be. I'll fix it. 

But I guess these code would be replaced with objrmap in no distant
future.

Thank you,
Hirokazu Takahashi.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/