From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg0-f71.google.com (mail-pg0-f71.google.com [74.125.83.71]) by kanga.kvack.org (Postfix) with ESMTP id 69E0B6B0389 for ; Tue, 14 Mar 2017 20:54:48 -0400 (EDT) Received: by mail-pg0-f71.google.com with SMTP id b2so5825288pgc.6 for ; Tue, 14 Mar 2017 17:54:48 -0700 (PDT) Received: from mga06.intel.com (mga06.intel.com. [134.134.136.31]) by mx.google.com with ESMTPS id w70si288483pgw.402.2017.03.14.17.54.47 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 14 Mar 2017 17:54:47 -0700 (PDT) From: "Huang\, Ying" Subject: Re: [PATCH -mm -v6 5/9] mm, THP, swap: Support to clear SWAP_HAS_CACHE for huge page References: <20170308072613.17634-1-ying.huang@intel.com> <20170308072613.17634-6-ying.huang@intel.com> <1489536859.2733.53.camel@linux.intel.com> Date: Wed, 15 Mar 2017 08:54:44 +0800 In-Reply-To: <1489536859.2733.53.camel@linux.intel.com> (Tim Chen's message of "Tue, 14 Mar 2017 17:14:19 -0700") Message-ID: <87a88ne4vf.fsf@yhuang-dev.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org List-ID: To: Tim Chen Cc: "Huang, Ying" , Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrea Arcangeli , "Kirill A . Shutemov" , Hugh Dickins , Shaohua Li , Minchan Kim , Rik van Riel Tim Chen writes: > On Wed, 2017-03-08 at 15:26 +0800, Huang, Ying wrote: >> From: Huang Ying >> >> __swapcache_free() is added to support to clear the SWAP_HAS_CACHE flag >> for the huge page.A A This will free the specified swap cluster now. >> Because now this function will be called only in the error path to free >> the swap cluster just allocated.A A So the corresponding swap_map[i] == >> SWAP_HAS_CACHE, that is, the swap count is 0.A A This makes the >> implementation simpler than that of the ordinary swap entry. >> >> This will be used for delaying splitting THP (Transparent Huge Page) >> during swapping out.A A Where for one THP to swap out, we will allocate a >> swap cluster, add the THP into the swap cache, then split the THP.A A If >> anything fails after allocating the swap cluster and before splitting >> the THP successfully, the swapcache_free_trans_huge() will be used to >> free the swap space allocated. >> >> Cc: Andrea Arcangeli >> Cc: Kirill A. Shutemov >> Cc: Hugh Dickins >> Cc: Shaohua Li >> Cc: Minchan Kim >> Cc: Rik van Riel >> Signed-off-by: "Huang, Ying" >> --- >> A include/linux/swap.h |A A 9 +++++++-- >> A mm/swapfile.cA A A A A A A A | 34 ++++++++++++++++++++++++++++++++-- >> A 2 files changed, 39 insertions(+), 4 deletions(-) >> >> diff --git a/include/linux/swap.h b/include/linux/swap.h >> index e3a7609a8989..2f2a6c0363aa 100644 >> --- a/include/linux/swap.h >> +++ b/include/linux/swap.h >> @@ -394,7 +394,7 @@ extern void swap_shmem_alloc(swp_entry_t); >> A extern int swap_duplicate(swp_entry_t); >> A extern int swapcache_prepare(swp_entry_t); >> A extern void swap_free(swp_entry_t); >> -extern void swapcache_free(swp_entry_t); >> +extern void __swapcache_free(swp_entry_t entry, bool huge); >> A extern void swapcache_free_entries(swp_entry_t *entries, int n); >> A extern int free_swap_and_cache(swp_entry_t); >> A extern int swap_type_of(dev_t, sector_t, struct block_device **); >> @@ -456,7 +456,7 @@ static inline void swap_free(swp_entry_t swp) >> A { >> A } >> A >> -static inline void swapcache_free(swp_entry_t swp) >> +static inline void __swapcache_free(swp_entry_t swp, bool huge) >> A { >> A } >> A >> @@ -544,6 +544,11 @@ static inline swp_entry_t get_huge_swap_page(void) >> A } >> A #endif >> A >> +static inline void swapcache_free(swp_entry_t entry) >> +{ >> + __swapcache_free(entry, false); >> +} >> + >> A #ifdef CONFIG_MEMCG >> A static inline int mem_cgroup_swappiness(struct mem_cgroup *memcg) >> A { >> diff --git a/mm/swapfile.c b/mm/swapfile.c >> index 7241c937e52b..6019f94afbaf 100644 >> --- a/mm/swapfile.c >> +++ b/mm/swapfile.c >> @@ -855,6 +855,29 @@ static void swap_free_huge_cluster(struct swap_info_struct *si, >> A _swap_entry_free(si, offset, true); >> A } >> A >> +static void swapcache_free_trans_huge(struct swap_info_struct *si, >> + A A A A A A swp_entry_t entry) >> +{ >> + unsigned long offset = swp_offset(entry); >> + unsigned long idx = offset / SWAPFILE_CLUSTER; >> + struct swap_cluster_info *ci; >> + unsigned char *map; >> + unsigned int i; >> + >> + spin_lock(&si->lock); >> + ci = lock_cluster(si, offset); >> + map = si->swap_map + offset; >> + for (i = 0; i < SWAPFILE_CLUSTER; i++) { >> + VM_BUG_ON(map[i] != SWAP_HAS_CACHE); >> + map[i] &= ~SWAP_HAS_CACHE; > > Nitpicking a bit: > map[i] = 0 A is more readable if map[i] == SWAP_HAS_CACHE here. OK. I will change this. Best Regards, Huang, Ying -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org