On Wed, May 11, 2022 at 04:19:06AM +0300, Kirill A. Shutemov wrote:
> JFYI, I've reworked it as
>
> accepted = try_accept_one(start, len, PG_LEVEL_1G);
> if (!accepted)
> accepted = try_accept_one(start, len, PG_LEVEL_2M);
> if (!accepted)
> accepted = try_accept_one(start, len, PG_LEVEL_4K);
> if (!accepted)
> return false;
> start += accepted;
s/accepted/accpt_size/
and then it is perfectly clear what that variable contains.
But it seems you're preparing a new version so I'll continue looking at
there.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
