Subject: Re: [RFC v2 3/3] mm: make swapin readahead to improve thp collapse
 rate
To: Rik van Riel <riel@redhat.com>,
        "Kirill A. Shutemov" <kirill@shutemov.name>,
        Ebru Akagunduz <ebru.akagunduz@gmail.com>
References: <1434799686-7929-1-git-send-email-ebru.akagunduz@gmail.com>
 <1434799686-7929-4-git-send-email-ebru.akagunduz@gmail.com>
 <20150621181131.GA6710@node.dhcp.inet.fi> <558766E4.5020801@redhat.com>
Cc: linux-mm@kvack.org, akpm@linux-foundation.org,
        kirill.shutemov@linux.intel.com, n-horiguchi@ah.jp.nec.com,
        aarcange@redhat.com, iamjoonsoo.kim@lge.com, xiexiuqi@huawei.com,
        gorcunov@openvz.org, linux-kernel@vger.kernel.org, mgorman@suse.de,
        rientjes@google.com, aneesh.kumar@linux.vnet.ibm.com, hughd@google.com,
        hannes@cmpxchg.org, mhocko@suse.cz, boaz@plexistor.com,
        raindel@mellanox.com
From: Vlastimil Babka <vbabka@suse.cz>
Message-ID: <558AA37E.20106@suse.cz>
Date: Wed, 24 Jun 2015 14:33:02 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.0.1
MIME-Version: 1.0
In-Reply-To: <558766E4.5020801@redhat.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1286
Lines: 29

On 06/22/2015 03:37 AM, Rik van Riel wrote:
> On 06/21/2015 02:11 PM, Kirill A. Shutemov wrote:
>> On Sat, Jun 20, 2015 at 02:28:06PM +0300, Ebru Akagunduz wrote:
>>> +	__collapse_huge_page_swapin(mm, vma, address, pmd, pte);
>>> +
>> 
>> And now the pages we swapped in are not isolated, right?
>> What prevents them from being swapped out again or whatever?
> 
> Nothing, but __collapse_huge_page_isolate is run with the
> appropriate locks to ensure that once we actually collapse
> the THP, things are present.
> 
> The way do_swap_page is called, khugepaged does not even
> wait for pages to be brought in from swap. It just maps
> in pages that are in the swap cache, and which can be
> immediately locked (without waiting).
> 
> It will also start IO on pages that are not in memory
> yet, and will hopefully get those next round.

Hm so what if the process is slightly larger than available memory and really
doesn't touch the swapped out pages that much? Won't that just be thrashing and
next round you find them swapped out again?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/