From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5ECDBC433F5 for ; Fri, 4 Mar 2022 19:01:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id D9F778D0002; Fri, 4 Mar 2022 14:01:15 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id D4D8E8D0001; Fri, 4 Mar 2022 14:01:15 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C3C3E8D0002; Fri, 4 Mar 2022 14:01:15 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (relay.hostedemail.com [64.99.140.25]) by kanga.kvack.org (Postfix) with ESMTP id B41C98D0001 for ; Fri, 4 Mar 2022 14:01:15 -0500 (EST) Received: from smtpin08.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 7B1732292B for ; Fri, 4 Mar 2022 19:01:15 +0000 (UTC) X-FDA: 79207621710.08.187169A Received: from mail-pj1-f53.google.com (mail-pj1-f53.google.com [209.85.216.53]) by imf23.hostedemail.com (Postfix) with ESMTP id B0F18140043 for ; Fri, 4 Mar 2022 19:01:14 +0000 (UTC) Received: by mail-pj1-f53.google.com with SMTP id p3-20020a17090a680300b001bbfb9d760eso11341942pjj.2 for ; Fri, 04 Mar 2022 11:01:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=QQXB5eKyYpG5vQk9wBbiRIz8N7tmcSgj6RBiMhN2WaY=; b=nJt0PulthXit3IgmiSHjfSEjOW/u0S00+vxPPPieNMTCXKaijg5Cjk6aIBOfoSgjnE CppbYHQ6Qonw6HCrBJ63uEqPfCeucSOUpQpsDy9RbUpsfao6YrrpHqWzo/zPV+uki9bh UBEdcpzj3x0rC6uEAQss1L4h0Tq0Nmw9eBkcn69d0lBctvsm8hR7MGxvn8xAtQIna4A9 dDSYq6FR9TJ8WBO/x/VBUuJxlilnneuNYlQh4IVL4v2QWp40SnUkCxCjKYvqijRDKXGM KgwTlnNAbDA/Ri0lXMuB4Ul+r+bmVwbgJ5NA9tuFQexQMC0mshoKFdzntubnb8k6ZYmj E1oQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=QQXB5eKyYpG5vQk9wBbiRIz8N7tmcSgj6RBiMhN2WaY=; b=sYOfaSBQmR1pjLrAmtRXCEVq/yV70YuelFxmKGMsX0wTdb1/sleBEz5raZfAMA/O7Q +hPdM4+JL3nxVryKGlcRyI3c65DFlKb92sgTJPhGN67QZbxWKA1GC+Dv46KTAJfBFZ4D GWF1Q+AYSb/slz6hpLFjkGh77aUfDPNaEx2Ypu1UCbak0MQWvy7rUsd77Dp2NEVOcjzr Z4z66i7AYOula4MWAK9Ji3apiu5LKsPqz/M09NozBmgtwYM6v3i5FKOCxWUHINIouQuc fhWitqGXSNwWHrndNmvy7M/K/KmxOZxXRntMuYz2d3EIrdt4dxWaSWxb8564kad8uR2P p+qw== X-Gm-Message-State: AOAM531NGj+C0yvODgMbfc9Y7VF2Bo/wCV1hl3z0Zy5sGiW2CNQaYCzl +Cs1NTgv7B5ncnHxtA/W6eneNWWkG+dL2NSguqA= X-Google-Smtp-Source: ABdhPJxsOWT5hTiInL2/NAyxR64UevNatuCAu8IOGJEZeUOLuf+ewhvvLVLVMZlr9xTKy8Ot9pjAGlHQSEFafgIShuQ= X-Received: by 2002:a17:90b:1b0e:b0:1bf:1c4f:2bc4 with SMTP id nu14-20020a17090b1b0e00b001bf1c4f2bc4mr87820pjb.200.1646420473648; Fri, 04 Mar 2022 11:01:13 -0800 (PST) MIME-Version: 1.0 References: <20220303222014.517033-1-shy828301@gmail.com> <13ad4ba1-2a88-9459-3995-70af36aba33e@redhat.com> In-Reply-To: <13ad4ba1-2a88-9459-3995-70af36aba33e@redhat.com> From: Yang Shi Date: Fri, 4 Mar 2022 11:01:02 -0800 Message-ID: Subject: Re: [PATCH] mm: thp: don't have to lock page anymore when splitting PMD To: David Hildenbrand Cc: Andrea Arcangeli , Andrew Morton , Hugh Dickins , "Kirill A. Shutemov" , Linux Kernel Mailing List , Linux MM , Matthew Wilcox Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: B0F18140043 X-Stat-Signature: wyouiadonbq14rpnr8jertxczhgk1j5q Authentication-Results: imf23.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=nJt0Pult; spf=pass (imf23.hostedemail.com: domain of shy828301@gmail.com designates 209.85.216.53 as permitted sender) smtp.mailfrom=shy828301@gmail.com; dmarc=pass (policy=none) header.from=gmail.com X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1646420474-937668 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Fri, Mar 4, 2022 at 10:50 AM David Hildenbrand wrote: > > On 04.03.22 19:30, Yang Shi wrote: > > On Thu, Mar 3, 2022 at 9:06 PM David Hildenbrand wrote: > >> > >> Hi, > >> > >> This probably bounces on the list due to html junk from the gmail app. > >> > >> What happened to > >> > >> https://lore.kernel.org/linux-mm/20220131162940.210846-10-david@redhat.com/ > >> > >> Included in the very series mentioned below? > >> > >> Was this silently dropped due to folio conversion collisions? :/ > > > > I really didn't notice you already proposed this. Maybe folio > > conversion, maybe mlock cleanup, I can't tell. But anyway this patch > > needs to get rebased. I will submit v2 to solve the comment, will add > > your signed-off-by. > > > > Why a rebase? The folio change comes via another tree (unfortunately not > Andrews tree, I wish we would have a single MM tree for MM patches). > > @Andrew, the last mail I received was > > + mm-huge_memory-remove-stale-locking-logic-from-__split_huge_pmd.patch > added to -mm tree > > The patch shows up in mmotm as > > #[merged]mm-huge_memory-remove-stale-locking-logic-from-__split_huge_pmd.patch > > ... which shouldn't be true. Yes, otherwise my patch can't be applied. What I saw on linux-next for mm/huge_memory.c looks like: bfede97b8d6d mm/huge_memory: remove stale page_trans_huge_mapcount() 6c127ac2ff1d mm/huge_memory: streamline COW logic in do_huge_pmd_wp_page() cb63c1eb2a3c mm/migration: add trace events for THP migrations e65f964148fe Merge branch 'akpm-current/current' 61435e1e267c mm/readahead: Align file mappings for non-DAX 80ef527bf929 mm: Turn can_split_huge_page() into can_split_folio() 8339af1d0a18 mm/rmap: Convert rmap_walk() to take a folio 16f06327291e mm/migrate: Convert remove_migration_ptes() to folios bd23d3f12232 memory tiering: skip to scan fast memory 47a3e10abb78 mm: thp: fix wrong cache flush in remove_migration_pmd() 1de8566cca0a mm/rmap: Convert try_to_migrate() to folios 5a470d51cb2b mm/rmap: Convert try_to_unmap() to take a folio 1c760ad73a13 mm/huge_memory: Convert __split_huge_pmd() to take a folio 82865a9e1187 mm: Add folio_mapcount() 07ca76067308 mm/munlock: maintain page->mlock_count while unevictable cea86fe246b6 mm/munlock: rmap call mlock_vma_page() munlock_vma_page() b67bf49ce7aa mm/munlock: delete FOLL_MLOCK and FOLL_POPULATE f56caedaf94f Merge branch 'akpm' (patches from Andrew) It looks like your series were rebased on top of mlock cleanup and folio. Anyway I will wait for Andrew. If he thought a new patch rebased on top of both mlock cleanup and folio would make his life easier, I will submit v2. > > -- > Thanks, > > David / dhildenb >