From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Tomlin Subject: Re: [PATCH RFC 4/5] mm/ksm, proc: introduce remote merge Date: Thu, 16 May 2019 17:29:46 +0100 Message-ID: <20190516162946.zwzhxkft342b25pd@atomlin.usersys.com> References: <20190516094234.9116-1-oleksandr@redhat.com> <20190516094234.9116-5-oleksandr@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Jann Horn Cc: Oleksandr Natalenko , kernel list , Kirill Tkhai , Hugh Dickins , Alexey Dobriyan , Vlastimil Babka , Michal Hocko , Matthew Wilcox , Pavel Tatashin , Greg KH , Suren Baghdasaryan , Minchan Kim , Timofey Titovets , Grzegorz Halat , Linux-MM , Linux API List-Id: linux-api@vger.kernel.org On Thu 2019-05-16 12:00 +0200, Jann Horn wrote: > On Thu, May 16, 2019 at 11:43 AM Oleksandr Natalenko > wrote: [ ... ] > > + } > > + > > + down_write(&mm->mmap_sem); > > Should a check for mmget_still_valid(mm) be inserted here? See commit > 04f5866e41fb70690e28397487d8bd8eea7d712a. Yes - I'd say this is required here. Thanks, -- Aaron Tomlin