[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aXORiA7_Oy60MSDR@casper.infradead.org>
Date: Fri, 23 Jan 2026 15:19:36 +0000
From: Matthew Wilcox <willy@...radead.org>
To: Vernon Yang <vernon2gm@...il.com>
Cc: akpm@...ux-foundation.org, david@...nel.org, lorenzo.stoakes@...cle.com,
ziy@...dia.com, dev.jain@....com, baohua@...nel.org,
lance.yang@...ux.dev, linux-mm@...ck.org,
linux-kernel@...r.kernel.org, Vernon Yang <yanglincheng@...inos.cn>
Subject: Re: [PATCH mm-new v5 2/5] mm: khugepaged: refine scan progress number
On Fri, Jan 23, 2026 at 04:22:29PM +0800, Vernon Yang wrote:
> + if (cur_progress) {
> + unsigned long idx = xas_get_index(&xas) - start;
I agree with Dev. This is less readable than:
unsigned long idx = xas.xa_index - start;
Powered by blists - more mailing lists