From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerome Glisse Subject: Re: [PATCH 2/3] mm/mmu_notifier: use structure for invalidate_range_start/end calls Date: Mon, 3 Dec 2018 19:09:12 -0500 Message-ID: <20181204000911.GB20742@redhat.com> References: <20181203201817.10759-1-jglisse@redhat.com> <20181203201817.10759-3-jglisse@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <20181203201817.10759-3-jglisse@redhat.com> Sender: linux-kernel-owner@vger.kernel.org To: Andrew Morton , linux-mm@kvack.org Cc: linux-kernel@vger.kernel.org, Matthew Wilcox , Ross Zwisler , Jan Kara , Dan Williams , Paolo Bonzini , Radim =?utf-8?B?S3LEjW3DocWZ?= , Michal Hocko , Christian Koenig , Felix Kuehling , Ralph Campbell , John Hubbard , kvm@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-rdma@vger.kernel.org, linux-fsdevel@vger.kernel.org List-Id: linux-rdma@vger.kernel.org On Mon, Dec 03, 2018 at 03:18:16PM -0500, jglisse@redhat.com wrote: > diff --git a/mm/migrate.c b/mm/migrate.c > index f7e4bfdc13b7..4896dd9d8b28 100644 > --- a/mm/migrate.c > +++ b/mm/migrate.c > @@ -2303,8 +2303,13 @@ static int migrate_vma_collect_pmd(pmd_t *pmdp, > */ > static void migrate_vma_collect(struct migrate_vma *migrate) > { > + struct mmu_notifier_range range; > struct mm_walk mm_walk; > > + range.start = migrate->start; > + range.end = migrate->end; > + range.mm = mm_walk.mm; Andrew can you replace above line by: + range.mm = migrate->vma->vm_mm; I made a mistake here when i was rebasing before posting. I checked the patchset again and i believe this is the only mistake i made. Do you want me to repost ? Sorry for my stupid mistake. Cheers, Jérôme