From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl0-f71.google.com (mail-pl0-f71.google.com [209.85.160.71]) by kanga.kvack.org (Postfix) with ESMTP id BCA436B0007 for ; Thu, 28 Jun 2018 01:31:20 -0400 (EDT) Received: by mail-pl0-f71.google.com with SMTP id e39-v6so2485538plb.10 for ; Wed, 27 Jun 2018 22:31:20 -0700 (PDT) Received: from mail.linuxfoundation.org (mail.linuxfoundation.org. [140.211.169.12]) by mx.google.com with ESMTPS id q4-v6si5766451plb.312.2018.06.27.22.31.19 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 27 Jun 2018 22:31:19 -0700 (PDT) Date: Wed, 27 Jun 2018 22:31:18 -0700 From: Andrew Morton Subject: Re: [PATCH -mm -v4 00/21] mm, THP, swap: Swapout/swapin THP in one piece Message-Id: <20180627223118.dd2f52d87f53e7e002ed0153@linux-foundation.org> In-Reply-To: <87r2krfpi2.fsf@yhuang-dev.intel.com> References: <20180622035151.6676-1-ying.huang@intel.com> <20180627215144.73e98b01099191da59bff28c@linux-foundation.org> <87r2krfpi2.fsf@yhuang-dev.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: "Huang, Ying" Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Hugh Dickins , "Kirill A. Shutemov" , Daniel Jordan On Thu, 28 Jun 2018 13:29:09 +0800 "Huang\, Ying" wrote: > Andrew Morton writes: > > > On Fri, 22 Jun 2018 11:51:30 +0800 "Huang, Ying" wrote: > > > >> This is the final step of THP (Transparent Huge Page) swap > >> optimization. After the first and second step, the splitting huge > >> page is delayed from almost the first step of swapout to after swapout > >> has been finished. In this step, we avoid splitting THP for swapout > >> and swapout/swapin the THP in one piece. > > > > It's a tremendously good performance improvement. It's also a > > tremendously large patchset :( > > > > And it depends upon your > > mm-swap-fix-race-between-swapoff-and-some-swap-operations.patch and > > mm-fix-race-between-swapoff-and-mincore.patch, the first of which has > > been floating about since February without adequate review. > > > > I'll give this patchset a spin in -mm to see what happens and will come > > back later to take a closer look. But the best I can do at this time > > is to hopefully cc some possible reviewers :) > > Thanks a lot for your help! Hope more people can review it! I took it out of -mm again, temporarily. Due to a huge tangle with the xarray conversions in linux-next.