Re: [PATCH v3 6/7] KVM: arm64: Participate in bitmap-based PTE aging
From: Marc Zyngier
Date: Tue Apr 02 2024 - 03:33:48 EST
On Tue, 02 Apr 2024 05:06:56 +0100,
Yu Zhao <yuzhao@xxxxxxxxxx> wrote:
>
> On Mon, Apr 1, 2024 at 7:30 PM James Houghton <jthoughton@xxxxxxxxxx> wrote:
> >
> > Participate in bitmap-based aging while grabbing the KVM MMU lock for
> > reading. Ideally we wouldn't need to grab this lock at all, but that
> > would require a more intrustive and risky change.
> ^^^^^^^^^^ intrusive
> This sounds subjective -- I'd just present the challenges and let
> reviewers make their own judgements.
Quite the opposite.
This sort of comment actually indicates that the author has at least
understood some of the complexity behind the proposed changes. It is a
qualitative comment that conveys useful information to reviewers, and
even more to the maintainers of this code.
That's the difference between a human developer and a bot, and I'm not
overly fond of bots.
M.
--
Without deviation from the norm, progress is not possible.