On Wed, 20 Jun 2012 21:21:07 +0800 Xiao Guangrong <xiaoguangrong@xxxxxxxxxxxxxxxxxx> wrote: > Again, "rmap" does not break the logic, the spte we handle in this function must > be in rmap. I'm not saying whether this breaks some logic or not. rmap_printk("rmap_write_protect: spte %p %llx\n", sptep, *sptep); helps developers by saying "rmap_write_protect() protected ...." For another function, the message should be "another_func_name: spte ..." But OK, I won't argue about this anymore. We can change the debug message later if needed. Thanks, Takuya -- 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