On 2015/05/20 2:25, Paolo Bonzini wrote: > Architecture-specific helpers are not supposed to muck with > struct kvm_userspace_memory_region contents. Add const to > enforce this. Nice, this is what I wanted to do, but could not do, when I cleaned up these functions a few years ago. > > In order to eliminate the only write in __kvm_set_memory_region, > the cleaning of deleted slots is pulled up from update_memslots > to __kvm_set_memory_region. - if (!npages) - mem->flags &= ~KVM_MEM_LOG_DIRTY_PAGES; Especially, removing this hard-to-understand lines is really nice. > > Signed-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx> Reviewed-by: Takuya Yoshikawa <yoshikawa_takuya_b1@xxxxxxxxxxxxx> 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