Messages in this thread | | | From | Mina Almasry <> | Date | Wed, 7 Dec 2022 15:16:20 -0800 | Subject | Re: [RFC PATCH v2 03/47] hugetlb: remove redundant pte_mkhuge in migration path |
| |
On Fri, Oct 21, 2022 at 9:37 AM James Houghton <jthoughton@google.com> wrote: > > arch_make_huge_pte, which is called immediately following pte_mkhuge, > already makes the necessary changes to the PTE that pte_mkhuge would > have. The generic implementation of arch_make_huge_pte simply calls > pte_mkhuge. > > Signed-off-by: James Houghton <jthoughton@google.com>
Acked-by: Mina Almasry <almasrymina@google.com>
> --- > mm/migrate.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/mm/migrate.c b/mm/migrate.c > index 8e5eb6ed9da2..1457cdbb7828 100644 > --- a/mm/migrate.c > +++ b/mm/migrate.c > @@ -237,7 +237,6 @@ static bool remove_migration_pte(struct folio *folio, > if (folio_test_hugetlb(folio)) { > unsigned int shift = huge_page_shift(hstate_vma(vma)); > > - pte = pte_mkhuge(pte); > pte = arch_make_huge_pte(pte, shift, vma->vm_flags); > if (folio_test_anon(folio)) > hugepage_add_anon_rmap(new, vma, pvmw.address, > -- > 2.38.0.135.g90850a2211-goog >
| |