Re: [PATCH 0/2] KVM: async_pf: use_mm/mm_users fixes

2014-04-28 Thread Andrea Arcangeli
On Mon, Apr 28, 2014 at 01:06:05PM +0200, Paolo Bonzini wrote: > Patch 1 will be for 3.16 only, I'd like a review from Marcelo or Andrea > though (that's "KVM: async_pf: kill the unnecessary use_mm/unuse_mm > async_pf_execute()" for easier googling). Patch 1: Reviewed-by: Andrea Arcangeli I t

Re: [PATCH 0/2] KVM: async_pf: use_mm/mm_users fixes

2014-04-28 Thread Paolo Bonzini
Il 21/04/2014 15:25, Oleg Nesterov ha scritto: Hello. Completely untested and I know nothing about kvm ;) Please review. But use_mm() really looks misleading, and the usage of mm_users looks "obviously wrong". I already sent this change while we were discussing vmacache, but it was ignored. Sin

Re: [PATCH 0/2] KVM: async_pf: use_mm/mm_users fixes

2014-04-24 Thread Oleg Nesterov
On 04/24, Christian Borntraeger wrote: > > On 21/04/14 15:25, Oleg Nesterov wrote: > > Hello. > > > > Completely untested and I know nothing about kvm ;) Please review. > > > > But use_mm() really looks misleading, and the usage of mm_users looks > > "obviously wrong". I already sent this change wh

Re: [PATCH 0/2] KVM: async_pf: use_mm/mm_users fixes

2014-04-24 Thread Christian Borntraeger
On 21/04/14 15:25, Oleg Nesterov wrote: > Hello. > > Completely untested and I know nothing about kvm ;) Please review. > > But use_mm() really looks misleading, and the usage of mm_users looks > "obviously wrong". I already sent this change while we were discussing > vmacache, but it was ignored

[PATCH 0/2] KVM: async_pf: use_mm/mm_users fixes

2014-04-21 Thread Oleg Nesterov
Hello. Completely untested and I know nothing about kvm ;) Please review. But use_mm() really looks misleading, and the usage of mm_users looks "obviously wrong". I already sent this change while we were discussing vmacache, but it was ignored. Since then kvm_async_page_present_sync() was added i