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 X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 88892C433DB for ; Wed, 24 Mar 2021 01:16:25 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 1534F619CC for ; Wed, 24 Mar 2021 01:16:24 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1534F619CC Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 7E80A6B01F4; Tue, 23 Mar 2021 21:16:24 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 796C76B01F5; Tue, 23 Mar 2021 21:16:24 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 610456B01F6; Tue, 23 Mar 2021 21:16:24 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0030.hostedemail.com [216.40.44.30]) by kanga.kvack.org (Postfix) with ESMTP id 40EA96B01F4 for ; Tue, 23 Mar 2021 21:16:24 -0400 (EDT) Received: from smtpin39.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id 039DD6C1A for ; Wed, 24 Mar 2021 01:16:23 +0000 (UTC) X-FDA: 77953002288.39.935215E Received: from mail-ej1-f41.google.com (mail-ej1-f41.google.com [209.85.218.41]) by imf10.hostedemail.com (Postfix) with ESMTP id 72542407F8F6 for ; Wed, 24 Mar 2021 01:16:22 +0000 (UTC) Received: by mail-ej1-f41.google.com with SMTP id k10so30188356ejg.0 for ; Tue, 23 Mar 2021 18:16:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=lmZtlOkzXKW+6Uymw8Z89AWwFd+oGjy+eP5xv5XjORQ=; b=jEZFAO6PtEX45puN+a6V92NL3n6JxqQ68ZVmiZ595EgF3s7VxmKYxN/ehbcb/Ap9Uw EsFnNdRsCPNW8kYTcmq5oTHWKW5kOEnEQULsmHpa/oc0QumzpRSmz4ll4FbAzUqI84RL Zu1UDyjg5QlIpR2hTOBdUFXDzaFr3MhaGBWoiyUfjb+P1aKZeZxN+CNYqb3AVQRskTPr Cn8bCmSg/Jc4AK5mt/ZIHBbTmDyPxOv4wfJAdFIUyBQRq+GJFErWR3O2tDLjZjJ5zfEo BDG0Y0FoXiBZAy3UT8QTkstQW7COTJzd9QYzk5e/t5vpRqoRXZWeOXwT7l1q0OoJPM+L 08Wg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=lmZtlOkzXKW+6Uymw8Z89AWwFd+oGjy+eP5xv5XjORQ=; b=AZY/RyJI6wU7LiRExHy7EghVxg9qX7tgqMTRcCP5RmH8vNC983wSnNnui7QCRxCkGD /fNFj5ssgd7RNHoBGy+AlFo9ejv4YTcvrBLDhwAvP1vmgiNQygvLctn4zNcCZM5ELOaD hoj47dJCFbJ349oaVTzUu1i8SxC6yvC3dmf/HXKkfAoWDcwQaMZoY2p7OotUGMT/T1iC l7+DCJHBFH5Lm/FO6c3lGSNG4fyKCmtyP1LvLIgsvJS+jindLtBUlRkH1hJbTDQeyc8O qmzHHsXGOM1Sfud6iA5Gt6MP//P1KHuhKYOJ1m6xmqBa7gRQbNe5oMbtWH1xV3F3lL3U Zyuw== X-Gm-Message-State: AOAM530mpzjXYGJFKP+AZ/2y1lrXmu99090pvQaKLQh+yW37uzcYhk2x 6BecxD8CrYUGibz4Iw4f5lETIXZY1Zo5chSZCFM= X-Google-Smtp-Source: ABdhPJzPlT93p22g0xEGPzonusBjb1xcTgpziEnVjBCeHKb30p1gDkf5S4cljaPUYc4jyqF75YlZken7f67/QV7OCI4= X-Received: by 2002:a17:906:bcfc:: with SMTP id op28mr990221ejb.238.1616548582221; Tue, 23 Mar 2021 18:16:22 -0700 (PDT) MIME-Version: 1.0 References: <20210323135405.65059-1-linmiaohe@huawei.com> <20210323135405.65059-6-linmiaohe@huawei.com> In-Reply-To: From: Yang Shi Date: Tue, 23 Mar 2021 18:16:10 -0700 Message-ID: Subject: Re: [PATCH v2 5/5] mm/migrate.c: fix potential deadlock in NUMA balancing shared exec THP case To: Miaohe Lin Cc: Andrew Morton , Jerome Glisse , Rafael Aquini , David Hildenbrand , Alistair Popple , Linux Kernel Mailing List , Linux MM Content-Type: text/plain; charset="UTF-8" X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 72542407F8F6 X-Stat-Signature: sbqje1h3bp1i1zzq4ugjfnx7138qhw6e Received-SPF: none (gmail.com>: No applicable sender policy available) receiver=imf10; identity=mailfrom; envelope-from=""; helo=mail-ej1-f41.google.com; client-ip=209.85.218.41 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1616548582-429289 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 Tue, Mar 23, 2021 at 10:17 AM Yang Shi wrote: > > On Tue, Mar 23, 2021 at 6:55 AM Miaohe Lin wrote: > > > > Since commit c77c5cbafe54 ("mm: migrate: skip shared exec THP for NUMA > > balancing"), the NUMA balancing would skip shared exec transhuge page. > > But this enhancement is not suitable for transhuge page. Because it's > > required that page_mapcount() must be 1 due to no migration pte dance > > is done here. On the other hand, the shared exec transhuge page will > > leave the migrate_misplaced_page() with pte entry untouched and page > > locked. Thus pagefault for NUMA will be triggered again and deadlock > > occurs when we start waiting for the page lock held by ourselves. > > Thanks for catching this. By relooking the code I think the other > important reason for removing this is > migrate_misplaced_transhuge_page() actually can't see shared exec file > THP at all since page_lock_anon_vma_read() is called before and if > page is not anonymous page it will just restore the PMD without > migrating anything. > > The pages for private mapped file vma may be anonymous pages due to > COW but they can't be THP so it won't trigger THP numa fault at all. I > think this is why no bug was reported. I overlooked this in the first > place. > > Your fix is correct, and please add the above justification to your commit log. BTW, I think you can just undo or revert commit c77c5cbafe54 ("mm: migrate: skip shared exec THP for NUMA balancing"). Thanks, Yang > > Reviewed-by: Yang Shi > > > > > Fixes: c77c5cbafe54 ("mm: migrate: skip shared exec THP for NUMA balancing") > > Signed-off-by: Miaohe Lin > > --- > > mm/migrate.c | 4 ---- > > 1 file changed, 4 deletions(-) > > > > diff --git a/mm/migrate.c b/mm/migrate.c > > index 5357a8527ca2..68bfa1625898 100644 > > --- a/mm/migrate.c > > +++ b/mm/migrate.c > > @@ -2192,9 +2192,6 @@ int migrate_misplaced_transhuge_page(struct mm_struct *mm, > > int page_lru = page_is_file_lru(page); > > unsigned long start = address & HPAGE_PMD_MASK; > > > > - if (is_shared_exec_page(vma, page)) > > - goto out; > > - > > new_page = alloc_pages_node(node, > > (GFP_TRANSHUGE_LIGHT | __GFP_THISNODE), > > HPAGE_PMD_ORDER); > > @@ -2306,7 +2303,6 @@ int migrate_misplaced_transhuge_page(struct mm_struct *mm, > > > > out_unlock: > > unlock_page(page); > > -out: > > put_page(page); > > return 0; > > } > > -- > > 2.19.1 > >