On 2/7/23 16:57, Lai Jiangshan wrote:
From: Lai Jiangshan<jiangshan.ljs@xxxxxxxxxxxx> mmu->sync_page for direct paging is never called. And both mmu->sync_page and mm->invlpg only make sense in shadowpaging. Setting mmu->sync_page as NULL for direct paging makes it consistent with mm->invlpg which is set NULL for the case. Signed-off-by: Lai Jiangshan<jiangshan.ljs@xxxxxxxxxxxx>
I'd rather have a WARN_ON_ONCE in kvm_sync_page(), otherwise looks good. Paolo