On 12/11/2015 12:48, Takuya Yoshikawa wrote: > v1->v2: > Patch 5 and 7 are added based on Paolo's suggestions. > Patch 8-10 are new. > > Patch 1/2/3/4: no change. > Patch 5: Needed a bit more work than I had expected. > Patch 6: Removed extra comment of v1 (patch 5 made it inappropriate). > Patch 7: As expected, many places needed to be converted. > Patch 8: This is new, but only a small change. > > Patch 9: Kind of an RFC (though I have checked it to some extent). > Following two places need to be carefully checked: > - in kvm_mmu_get_page: "if (!direct)" block after kvm_mmu_alloc_page() > - in FNAME(fetch): "if (FNAME(gpte_changed)(vcpu, gw, it.level - 1))" case > Patch 10: Trivial cleanup, assuming that patch 9 is correct. > > > In summary: patch 1-7 is the result of updating v1 based on the suggestions. > Although patch 5 does not look so nice than expected, this is the most > conservative approach, and patch 8-10 try to alleviate the sadness. If it works, it's actually better than what we have now. I'll review it in a few days. Marcelo, can you look at this as well? You are still king of MMU. :) Thanks, Paolo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html