Re: [PATCH mm-new v6 2/5] mm: khugepaged: refine scan progress number

From: David Hildenbrand (Arm)

Date: Fri Feb 06 2026 - 04:08:18 EST


On 2/5/26 15:25, Dev Jain wrote:

On 05/02/26 5:41 pm, David Hildenbrand (arm) wrote:
On 2/5/26 07:08, Vernon Yang wrote:
On Thu, Feb 5, 2026 at 5:35 AM David Hildenbrand (arm)
<david@xxxxxxxxxx> wrote:

I guess, your meaning is "min(_pte - pte + 1, HPAGE_PMD_NR)", not max().

Yes!



I'm also worried that the compiler can't optimize this since the body of
the loop is complex, as with Dev's opinion [1].

Why do we even have to optimize this? :)

Premature ... ? :)


I mean .... we don't, but the alternate is a one liner using max().

I'm fine with the max(), but it still seems like adding complexity to optimize something that is nowhere prove to really be a problem.

--
Cheers,

David