From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755178AbeDCIex (ORCPT ); Tue, 3 Apr 2018 04:34:53 -0400 Received: from mx2.suse.de ([195.135.220.15]:60880 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754969AbeDCIew (ORCPT ); Tue, 3 Apr 2018 04:34:52 -0400 Date: Tue, 3 Apr 2018 10:34:51 +0200 From: Michal Hocko To: Naoya Horiguchi Cc: "linux-mm@kvack.org" , Andrew Morton , Vlastimil Babka , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v1] mm: consider non-anonymous thp as unmovable page Message-ID: <20180403083451.GG5501@dhcp22.suse.cz> References: <1522730788-24530-1-git-send-email-n-horiguchi@ah.jp.nec.com> <20180403075928.GC5501@dhcp22.suse.cz> <20180403082405.GA23809@hori1.linux.bs1.fc.nec.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180403082405.GA23809@hori1.linux.bs1.fc.nec.co.jp> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue 03-04-18 08:24:06, Naoya Horiguchi wrote: > On Tue, Apr 03, 2018 at 09:59:28AM +0200, Michal Hocko wrote: > > On Tue 03-04-18 13:46:28, Naoya Horiguchi wrote: > > > My testing for the latest kernel supporting thp migration found out an > > > infinite loop in offlining the memory block that is filled with shmem > > > thps. We can get out of the loop with a signal, but kernel should > > > return with failure in this case. > > > > > > What happens in the loop is that scan_movable_pages() repeats returning > > > the same pfn without any progress. That's because page migration always > > > fails for shmem thps. > > > > Why does it fail? Shmem pages should be movable without any issues. > > .. because try_to_unmap_one() explicitly skips unmapping for migration. > > #ifdef CONFIG_ARCH_ENABLE_THP_MIGRATION > /* PMD-mapped THP migration entry */ > if (!pvmw.pte && (flags & TTU_MIGRATION)) { > VM_BUG_ON_PAGE(PageHuge(page) || !PageTransCompound(page), page); > > if (!PageAnon(page)) > continue; > > set_pmd_migration_entry(&pvmw, page); > continue; > } > #endif > > When I implemented this code, I felt hard to work on both of anon thp > and shmem thp at one time, so I separated the proposal into smaller steps. > Shmem uses pagecache so we need some non-trivial effort (including testing) > to extend thp migration for shmem. But I think it's a reasonable next step. OK, I see. I have forgot about this part. Please be explicit about that in the changelog. Also the proper fix is to not use movable zone for shmem page THP rather than hack around it in the hotplug specific code IMHO. -- Michal Hocko SUSE Labs