[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YLpmCBhmQcAayzCF@casper.infradead.org>
Date: Fri, 4 Jun 2021 18:42:32 +0100
From: Matthew Wilcox <willy@...radead.org>
To: "Kirill A. Shutemov" <kirill@...temov.name>
Cc: Hugh Dickins <hughd@...gle.com>,
Andrew Morton <akpm@...ux-foundation.org>,
"Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
Yang Shi <shy828301@...il.com>,
Wang Yugui <wangyugui@...-tech.com>,
Naoya Horiguchi <naoya.horiguchi@....com>,
Alistair Popple <apopple@...dia.com>,
Ralph Campbell <rcampbell@...dia.com>, Zi Yan <ziy@...dia.com>,
Miaohe Lin <linmiaohe@...wei.com>,
Minchan Kim <minchan@...nel.org>, Jue Wang <juew@...gle.com>,
Peter Xu <peterx@...hat.com>, Jan Kara <jack@...e.cz>,
linux-mm@...ck.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 5/7] mm/thp: fix page_vma_mapped_walk() if huge page
mapped by ptes
On Fri, Jun 04, 2021 at 07:24:02PM +0300, Kirill A. Shutemov wrote:
> > /* when pud is not present, pte will be NULL */
> > - pvmw->pte = huge_pte_offset(mm, pvmw->address, page_size(page));
> > + pvmw->pte = huge_pte_offset(mm, pvmw->address,
> > + page_size(pvmw->page));
>
> AFAICS, it exactly fits into 80-column.
.. not after putting the 'pvmw->' before 'page'.
> > if (!pvmw->pte)
> > return false;
> >
> > - pvmw->ptl = huge_pte_lockptr(page_hstate(page), mm, pvmw->pte);
> > + pvmw->ptl = huge_pte_lockptr(page_hstate(pvmw->page),
> > + mm, pvmw->pte);
>
> And this one end on 79.
likewise. I mean, the 'mm' could go on the previous line, but that's
beyond my level of code format caring.
Powered by blists - more mailing lists