Linus Torvalds writes: (Summary) wrote:
Thanks for doing this.
Thanks for doing this.
Ok, those two patches are a bit more complex than I was hoping for, but not *too* bad.
but not *too* bad.
And the final end result certainly looks nice:
And the final end result certainly looks nice:
for a nice patch.
And I actually think you missed some more lines that can now be removed: kvm_arch_mmu_notifier_invalidate_page() should no longer be needed either, so you can remove all of those too (most of them are empty inline functions, but x86 has one that actually does something.
[...]
r[...]
API.Thanks for doing this.
Thanks for doing this.
[...]
mmu_notifier_invalidate_range_start()/end().Ok, those two patches are a bit more complex than I was hoping for, but not *too* bad.
but not *too* bad.
And the final end result certainly looks nice:
And the final end result certainly looks nice:
[...]
16 files changed, 74 insertions(+), 214 deletions(-) Yeah, removing all those invalidate_page() notifiers certainly makes for a nice patch.for a nice patch.
And I actually think you missed some more lines that can now be removed: kvm_arch_mmu_notifier_invalidate_page() should no longer be needed either, so you can remove all of those too (most of them are empty inline functions, but x86 has one that actually does something.