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>