Received: by 2002:a05:6a10:d5a5:0:0:0:0 with SMTP id gn37csp4070299pxb; Mon, 4 Oct 2021 16:45:24 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy7c4GCi/EE2tgsJQBAJ72Zh433yawCkriRKNqP6B0KTUXlhUhiJKo0Knf0wbnV6P4YNGX/ X-Received: by 2002:a17:90b:1d0d:: with SMTP id on13mr5246476pjb.118.1633391124000; Mon, 04 Oct 2021 16:45:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633391123; cv=none; d=google.com; s=arc-20160816; b=Q9/bjR3IJ/psJ97AgFB6ffXSSanpo1UGHwS/B4e66oO84YSgcTmLbCzYGBE8gzvMMo bG7gUnXMDI3qR5IQDAi4GYMyz7ui7qKEVlwy/Tql6bXpcUunwQtz9+SXvu4jMWliO63t PzP98U/rELoenFRJQiDaFX/5uhdaVKb/JpPFteObn1EwLA3LBAndsXue3t9QvFBF2MZR H4eRIpdiOPlxnWZ5KNrT1tmWrCXNvTfC8uqMO/WYZh1/6X+1+NwdPD2PX2BPa6GuAyKN NwoDeIuQz4QnESFb+OhdqEtrWpBVNaFeossGHRp7ko39IyeR6fPm19HJmjMJALHacv6v +/uA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=MTC71zy1g9PYXKu+Q17R6YOob819VdI6fXgddq3gR7E=; b=qPX6CogVKok8S4Hi3mAkAVU+vBVssVZzLlnthqZsNYWQAx3suYs/S4UGCTF9FUnHm+ VYQYGCAHt6fdl2eRMX3x/pGu0wc+z5e2EZJjih14n2by5kpIsIfC8NbE9M6oS+kJTCj/ EooB6+uaMjWrhvq1AqmT57vL3DZms+o7dMcuoVjytxZIr1KqL9yuPtX5fMkAYYSDztPQ VYNX5fYJAqRs4s8iX+hYXQCxJph9PElcrjNtAcAWvs5GZUjOiWPw8sz5YIO/Imy8ZGp0 kNvyJBkqOE3Ro934iK0CLg+Jd956wYizo4sqZ+dzJrfBJ+R0kcrXQaBFbzzCXkh16m1B u3hA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=ZCksl3VX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id t8si4358669plg.3.2021.10.04.16.45.11; Mon, 04 Oct 2021 16:45:23 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=ZCksl3VX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236006AbhJDU2a (ORCPT + 99 others); Mon, 4 Oct 2021 16:28:30 -0400 Received: from mail.kernel.org ([198.145.29.99]:54980 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235444AbhJDU2a (ORCPT ); Mon, 4 Oct 2021 16:28:30 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id D9D9B611F0 for ; Mon, 4 Oct 2021 20:26:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1633379200; bh=+j6iW1HJ6VNM1YovE/pabuUcnQL+weiXJ3J12Bf98GM=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=ZCksl3VXuTapt4dY5+83zKtOxoamasb/PEfHx+bjJnkX/cQd7vAJ8a/lC30xnukEi MZwFS3SXf1Ex1g3HMUb/9IqYuG/WWj5Xh5hxmwBNRNnY16lTTVgINO0rR7/b6zFsoA KUpyVgyJ7+G+41+GGCaSmw22b5kxNYjj6/MYzbnMBWatmBu6KO1rUuyjVNf5ZP5RtS 26a4Z67naja839ozfzTGjgzAL7UjBztU7Zum4rXhQT58aHE35G1AZ+U2zFWG2mKLIz Pvk6E1DUC29XZb/d+Hkeu7rCBMUXBs7IKTgi1EV8ct7YAaKWFCp+LDszmyr9ak5Ewr V3XhccxwJd1eA== Received: by mail-lf1-f44.google.com with SMTP id i24so27211686lfj.13 for ; Mon, 04 Oct 2021 13:26:40 -0700 (PDT) X-Gm-Message-State: AOAM530NP7UXeRZvCRgn0OP7NNy0z+dhlkfwhdFg/RgGvv5QwRajhQ8l L+op3Gk6qWrgrDfzkOwYQk5TOARXW8y08/KqXE8= X-Received: by 2002:a05:6512:39c4:: with SMTP id k4mr16294730lfu.14.1633379197973; Mon, 04 Oct 2021 13:26:37 -0700 (PDT) MIME-Version: 1.0 References: <20210923194343.ca0f29e1c4d361170343a6f2@linux-foundation.org> <9e41661d-9919-d556-8c49-610dae157553@linux.alibaba.com> <68737431-01d2-e6e3-5131-7d7c731e49ae@linux.alibaba.com> <8d8fb192-bd8d-8a08-498d-ca7204d4a716@linux.alibaba.com> In-Reply-To: <8d8fb192-bd8d-8a08-498d-ca7204d4a716@linux.alibaba.com> From: Song Liu Date: Mon, 4 Oct 2021 13:26:26 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 1/2] mm, thp: check page mapping when truncating page cache To: Rongwei Wang Cc: Matthew Wilcox , Hugh Dickins , Andrew Morton , Linux MM , Linux Kernel Mailing List , William Kucharski Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Oct 4, 2021 at 10:26 AM Rongwei Wang wrote: > > Hi, > I have run our cases these two days to stress test new Patch #1. The new > Patch #1 mainly add filemap_invalidate_{un}lock before and after > truncate_pagecache(), basing on original Patch #1. And the crash has not > happened. > > Now, I keep the original Patch #1, then adding the code below which > suggested by liu song (I'm not sure which one I should add in the next > version, Suggested-by or Signed-off-by? If you know, please remind me). > > - if (filemap_nr_thps(inode->i_mapping)) > + if (filemap_nr_thps(inode->i_mapping)) { > + filemap_invalidate_lock(inode->i_mapping); > truncate_pagecache(inode, 0); > + filemap_invalidate_unlock(inode->i_mapping); > + } It is mostly suggested by Matthew. If the patch goes that way, you can add Tested-by: Song Liu