Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Michal Hocko <mhocko@xxxxxxxx>
Subject: Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
From: Muchun Song <songmuchun@xxxxxxxxxxxxx>
Date: Thu, 11 Mar 2021 21:00:01 +0800
Cc: Jonathan Corbet <corbet@xxxxxxx>, Mike Kravetz <mike.kravetz@xxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, bp@xxxxxxxxx, x86@xxxxxxxxxx, hpa@xxxxxxxxx, dave.hansen@xxxxxxxxxxxxxxx, luto@xxxxxxxxxx, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, paulmck@xxxxxxxxxx, mchehab+huawei@xxxxxxxxxx, pawan.kumar.gupta@xxxxxxxxxxxxxxx, Randy Dunlap <rdunlap@xxxxxxxxxxxxx>, oneukum@xxxxxxxx, anshuman.khandual@xxxxxxx, jroedel@xxxxxxx, Mina Almasry <almasrymina@xxxxxxxxxx>, David Rientjes <rientjes@xxxxxxxxxx>, Matthew Wilcox <willy@xxxxxxxxxxxxx>, Oscar Salvador <osalvador@xxxxxxx>, "Song Bao Hua (Barry Song)" <song.bao.hua@xxxxxxxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, HORIGUCHI NAOYA(堀口直也) <naoya.horiguchi@xxxxxxx>, Joao Martins <joao.m.martins@xxxxxxxxxx>, Xiongchun duan <duanxiongchun@xxxxxxxxxxxxx>, linux-doc@xxxxxxxxxxxxxxx, LKML <linux-kernel@xxxxxxxxxxxxxxx>, Linux Memory Management List <linux-mm@xxxxxxxxx>, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, Miaohe Lin <linmiaohe@xxxxxxxxxx>, Chen Huang <chenhuang5@xxxxxxxxxx>, Bodeddula Balasubramaniam <bodeddub@xxxxxxxxxx>
In-reply-to: <YEoKJYzP8//qVebC@dhcp22.suse.cz>

On Thu, Mar 11, 2021 at 8:16 PM Michal Hocko <mhocko@xxxxxxxx> wrote:
>
> On Thu 11-03-21 18:00:09, Muchun Song wrote:
> [...]
> > Sorry. I am confused why you disagree with this change.
> > It does not bring any disadvantages.
>
> Because it is adding a code which is not really necessary and which will
> have to be maintained. Think of future changes which would need to grow
> more of these. Hugetlb code paths shouldn't really think about size of
> the struct page.

Got it. I will drop this patch.

> --
> Michal Hocko
> SUSE Labs

References:
- [PATCH v18 0/9] Free some vmemmap pages of HugeTLB page
  - From: Muchun Song
- [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Muchun Song
- Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Michal Hocko
- Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Muchun Song
- Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Michal Hocko
- Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Muchun Song
- Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Michal Hocko
- Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Muchun Song
- Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
  - From: Michal Hocko

Prev by Date: Re: [RESEND PATCH v2.1 07/10] iomap: Introduce iomap_apply2() for operations on two files
Next by Date: Re: [PATCHv2] iomap: Fix negative assignment to unsigned sis->pages in iomap_swapfile_activate
Previous by thread: Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
Next by thread: Re: [External] Re: [PATCH v18 9/9] mm: hugetlb: optimize the code with the help of the compiler
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [Samba] [Device Mapper] [CEPH Development]