Re: [Resend PATCH V5 7/10] KVM: Make kvm_set_spte_hva() return int

From: Paul Mackerras
Date: Wed Dec 12 2018 - 00:04:00 EST


On Thu, Dec 06, 2018 at 09:21:10PM +0800, lantianyu1986@xxxxxxxxx wrote:
> From: Lan Tianyu <Tianyu.Lan@xxxxxxxxxxxxx>
>
> The patch is to make kvm_set_spte_hva() return int and caller can
> check return value to determine flush tlb or not.

It would be helpful if the patch description told the reader which
return value(s) mean that the caller should flush the tlb. I would
guess that non-zero means to do the flush, but you should make that
explicit.

> Signed-off-by: Lan Tianyu <Tianyu.Lan@xxxxxxxxxxxxx>

For the powerpc bits:

Acked-by: Paul Mackerras <paulus@xxxxxxxxxx>