From: Linus Torvalds <torvalds@linux-foundation.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Hugh Dickins <hughd@google.com>,
Johannes Weiner <hannes@cmpxchg.org>,
"Kirill A. Shutemov" <kirill@shutemov.name>,
Matthew Wilcox <willy@infradead.org>,
David Hildenbrand <david@redhat.com>,
Vlastimil Babka <vbabka@suse.cz>, Peter Xu <peterx@redhat.com>,
Yang Shi <shy828301@gmail.com>,
John Hubbard <jhubbard@nvidia.com>,
Mike Kravetz <mike.kravetz@oracle.com>,
Sidhartha Kumar <sidhartha.kumar@oracle.com>,
Muchun Song <songmuchun@bytedance.com>,
Miaohe Lin <linmiaohe@huawei.com>,
Naoya Horiguchi <naoya.horiguchi@linux.dev>,
Mina Almasry <almasrymina@google.com>,
James Houghton <jthoughton@google.com>,
"Zach O'Keefe" <zokeefe@google.com>,
linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [PATCH 0/3] mm,thp,rmap: rework the use of subpages_mapcount
Date: Fri, 18 Nov 2022 14:07:35 -0800 [thread overview]
Message-ID: <CAHk-=whrA5F=cJ4yk6C5uyg2VniwFMHp9U_DLBB-9uZbqdd25w@mail.gmail.com> (raw)
In-Reply-To: <20221118140346.b9026301b4ba03e43e15aeca@linux-foundation.org>
On Fri, Nov 18, 2022 at 2:03 PM Andrew Morton <akpm@linux-foundation.org> wrote:
>
> I'd prefer that approach.
The "that approach" is a bit ambiguous here, particularly considering
how you quoted things.
But I think from the context you meant "keep them as two separate
series, even if the second undoes part of the first and does it
differently".
And that's fine. Even if it's maybe a bit odd to introduce that
locking that then goes away, I can't argue with "the first series was
already reviewed and has gone through a fair amount of testing".
Linus
next prev parent reply other threads:[~2022-11-18 22:15 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-03 1:44 [PATCH 0/3] mm,huge,rmap: unify and speed up compound mapcounts Hugh Dickins
2022-11-03 1:48 ` [PATCH 1/3] mm,hugetlb: use folio fields in second tail page Hugh Dickins
2022-11-03 21:18 ` Sidhartha Kumar
2022-11-04 4:29 ` Hugh Dickins
2022-11-10 0:11 ` Sidhartha Kumar
2022-11-10 2:10 ` Hugh Dickins
2022-11-10 2:13 ` [PATCH 1/3 fix] mm,hugetlb: use folio fields in second tail page: fix Hugh Dickins
2022-11-05 19:13 ` [PATCH 1/3] mm,hugetlb: use folio fields in second tail page Kirill A. Shutemov
2022-11-10 1:58 ` Hugh Dickins
2022-11-03 1:51 ` [PATCH 2/3] mm,thp,rmap: simplify compound page mapcount handling Hugh Dickins
2022-11-05 19:51 ` Kirill A. Shutemov
2022-11-10 2:49 ` Hugh Dickins
2022-11-03 1:53 ` [PATCH 3/3] mm,thp,rmap: lock_compound_mapcounts() on THP mapcounts Hugh Dickins
2022-11-05 20:06 ` Kirill A. Shutemov
2022-11-10 3:31 ` Hugh Dickins
2022-11-10 2:18 ` [PATCH 4/3] mm,thp,rmap: handle the normal !PageCompound case first Hugh Dickins
2022-11-10 3:23 ` Linus Torvalds
2022-11-10 4:21 ` Hugh Dickins
2022-11-10 16:31 ` Matthew Wilcox
2022-11-10 16:58 ` Linus Torvalds
2022-11-18 9:08 ` [PATCH 0/3] mm,thp,rmap: rework the use of subpages_mapcount Hugh Dickins
2022-11-18 9:12 ` [PATCH 1/3] mm,thp,rmap: subpages_mapcount of PTE-mapped subpages Hugh Dickins
2022-11-19 0:12 ` Yu Zhao
2022-11-19 0:37 ` Hugh Dickins
2022-11-19 1:35 ` [PATCH 1/3 fix] mm,thp,rmap: subpages_mapcount of PTE-mapped subpages: fix Hugh Dickins
2022-11-21 12:38 ` Kirill A. Shutemov
2022-11-22 9:13 ` Hugh Dickins
2022-11-21 12:36 ` [PATCH 1/3] mm,thp,rmap: subpages_mapcount of PTE-mapped subpages Kirill A. Shutemov
2022-11-22 9:03 ` Hugh Dickins
2022-11-18 9:14 ` [PATCH 2/3] mm,thp,rmap: subpages_mapcount COMPOUND_MAPPED if PMD-mapped Hugh Dickins
2022-11-21 13:09 ` Kirill A. Shutemov
2022-11-22 9:33 ` Hugh Dickins
2022-11-18 9:16 ` [PATCH 3/3] mm,thp,rmap: clean up the end of __split_huge_pmd_locked() Hugh Dickins
2022-11-21 13:24 ` Kirill A. Shutemov
2022-11-18 20:18 ` [PATCH 0/3] mm,thp,rmap: rework the use of subpages_mapcount Linus Torvalds
2022-11-18 20:42 ` Johannes Weiner
2022-11-18 20:51 ` Hugh Dickins
2022-11-18 22:03 ` Andrew Morton
2022-11-18 22:07 ` Linus Torvalds [this message]
2022-11-18 22:10 ` Hugh Dickins
2022-11-18 22:23 ` Andrew Morton
2022-11-21 16:59 ` Shakeel Butt
2022-11-21 17:16 ` Linus Torvalds
2022-11-22 16:27 ` Shakeel Butt
2022-11-21 18:52 ` Johannes Weiner
2022-11-22 1:32 ` Hugh Dickins
2022-11-22 5:57 ` Matthew Wilcox
2022-11-22 6:55 ` Johannes Weiner
2022-11-22 16:30 ` Shakeel Butt
2022-11-22 9:38 ` [PATCH v2 " Hugh Dickins
2022-11-22 9:42 ` [PATCH v2 1/3] mm,thp,rmap: subpages_mapcount of PTE-mapped subpages Hugh Dickins
2022-11-22 9:49 ` [PATCH v2 2/3] mm,thp,rmap: subpages_mapcount COMPOUND_MAPPED if PMD-mapped Hugh Dickins
2022-11-22 9:51 ` [PATCH v2 3/3] mm,thp,rmap: clean up the end of __split_huge_pmd_locked() Hugh Dickins
2022-12-05 1:38 ` Hugh Dickins
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAHk-=whrA5F=cJ4yk6C5uyg2VniwFMHp9U_DLBB-9uZbqdd25w@mail.gmail.com' \
--to=torvalds@linux-foundation.org \
--cc=akpm@linux-foundation.org \
--cc=almasrymina@google.com \
--cc=david@redhat.com \
--cc=hannes@cmpxchg.org \
--cc=hughd@google.com \
--cc=jhubbard@nvidia.com \
--cc=jthoughton@google.com \
--cc=kirill@shutemov.name \
--cc=linmiaohe@huawei.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mike.kravetz@oracle.com \
--cc=naoya.horiguchi@linux.dev \
--cc=peterx@redhat.com \
--cc=shy828301@gmail.com \
--cc=sidhartha.kumar@oracle.com \
--cc=songmuchun@bytedance.com \
--cc=vbabka@suse.cz \
--cc=willy@infradead.org \
--cc=zokeefe@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox