Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1626416
| From | Johannes Weiner <hannes@cmpxchg.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH -mm -v9 1/3] mm, THP, swap: Delay splitting THP during swap out |
| Date | 2017-04-19 18:00 +0200 |
| Message-ID | <ty0bo-5gm-1@gated-at.bofh.it> (permalink) |
| References | <txRUu-lm-9@gated-at.bofh.it> <txRUv-lm-33@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Wed, Apr 19, 2017 at 03:06:23PM +0800, Huang, Ying wrote:
> @@ -206,17 +212,34 @@ int add_to_swap(struct page *page, struct list_head *list)
> */
> err = add_to_swap_cache(page, entry,
> __GFP_HIGH|__GFP_NOMEMALLOC|__GFP_NOWARN);
> -
> - if (!err) {
> - return 1;
> - } else { /* -ENOMEM radix-tree allocation failure */
> + /* -ENOMEM radix-tree allocation failure */
> + if (err)
> /*
> * add_to_swap_cache() doesn't return -EEXIST, so we can safely
> * clear SWAP_HAS_CACHE flag.
> */
> - swapcache_free(entry);
> - return 0;
> + goto fail_free;
> +
> + if (unlikely(PageTransHuge(page))) {
> + err = split_huge_page_to_list(page, list);
> + if (err) {
> + delete_from_swap_cache(page);
> + return 0;
> + }
> }
> +
> + return 1;
> +
> +fail_free:
> + if (unlikely(PageTransHuge(page)))
> + swapcache_free_cluster(entry);
> + else
> + swapcache_free(entry);
> +fail:
> + if (unlikely(PageTransHuge(page)) &&
> + !split_huge_page_to_list(page, list))
> + goto retry;
May I ask why you added the unlikelies there? Can you generally say
THPs are unlikely in this path? Is the swap-out path so hot that
branch layout is critical? I doubt either is true.
Also please mention changes like these in the changelog next time.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH -mm -v9 0/3] THP swap: Delay splitting THP during swapping out "Huang, Ying" <ying.huang@intel.com> - 2017-04-19 09:10 +0200
[PATCH -mm -v9 1/3] mm, THP, swap: Delay splitting THP during swap out "Huang, Ying" <ying.huang@intel.com> - 2017-04-19 09:10 +0200
Re: [PATCH -mm -v9 1/3] mm, THP, swap: Delay splitting THP during swap out Johannes Weiner <hannes@cmpxchg.org> - 2017-04-19 18:00 +0200
Re: [PATCH -mm -v9 1/3] mm, THP, swap: Delay splitting THP during swap out "Huang\, Ying" <ying.huang@intel.com> - 2017-04-20 02:50 +0200
Re: [PATCH -mm -v9 1/3] mm, THP, swap: Delay splitting THP during swap out Johannes Weiner <hannes@cmpxchg.org> - 2017-04-21 16:10 +0200
Re: [PATCH -mm -v9 1/3] mm, THP, swap: Delay splitting THP during swap out "Huang\, Ying" <ying.huang@intel.com> - 2017-04-22 02:10 +0200
csiph-web