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 62BAFC433EF for ; Thu, 3 Mar 2022 18:39:27 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9AB8B8D0002; Thu, 3 Mar 2022 13:39:26 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 959CD8D0001; Thu, 3 Mar 2022 13:39:26 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8700A8D0002; Thu, 3 Mar 2022 13:39:26 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0234.hostedemail.com [216.40.44.234]) by kanga.kvack.org (Postfix) with ESMTP id 776448D0001 for ; Thu, 3 Mar 2022 13:39:26 -0500 (EST) Received: from smtpin28.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 2590A181AC550 for ; Thu, 3 Mar 2022 18:39:26 +0000 (UTC) X-FDA: 79203937932.28.9076857 Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [46.235.227.227]) by imf10.hostedemail.com (Postfix) with ESMTP id 7E5EBC0007 for ; Thu, 3 Mar 2022 18:39:25 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: krisman) with ESMTPSA id DB6361F45EF7 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1646332764; bh=fqL4sRiIZuSw4D7lvMNNLB/CnfsldiP6MMuPIpLaqTY=; h=From:To:Cc:Subject:References:Date:In-Reply-To:From; b=mzY1bNcNUw40RKsLg4UG8YXhJNN8dB3x5zuv2XO8p4fy6MqbgjuWMG/qZxPIgqM0+ 6lMfCTL6UWRhwEMhpI+B7PObV7DsUlyOR4rPnXoOa3vqOxCaJwSa1IJjoaBPQSj7Td ST5WHcS820F9XWEhMsEVw2oWAyWIq517fEJeM9/PWmptNfyWu8kTr08mUKpBZFjYCH /1bCnuat6ZY3xGa7emC82dsVqe+22jaZnAczwAlVnttkRc6sX6lX/UyLOBUUXcdEt/ zE5wztNuR9OToiuR19n0L4I52QdnatmaMZSfcDGKNoGnHomdpvLTOZqyDrz4N0fp++ 1BV04XeTUkf2A== From: Gabriel Krisman Bertazi To: Shuah Khan Cc: David Hildenbrand , Muhammad Usama Anjum , Andrew Morton , Shuah Khan , kernel@collabora.com, kernelci@groups.io, Will Deacon , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-kselftest@vger.kernel.org Subject: Re: [PATCH V3] selftests: vm: Add test for Soft-Dirty PTE bit Organization: Collabora References: <20220224212335.3045905-1-usama.anjum@collabora.com> <3b7c068b-ac7e-62fc-f0cd-a8dbf8642876@redhat.com> <6133317f-4da0-3aae-f352-b75f0f94dbd4@linuxfoundation.org> Date: Thu, 03 Mar 2022 13:39:20 -0500 In-Reply-To: <6133317f-4da0-3aae-f352-b75f0f94dbd4@linuxfoundation.org> (Shuah Khan's message of "Thu, 3 Mar 2022 11:19:35 -0700") Message-ID: <87o82mkhif.fsf@collabora.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Rspamd-Queue-Id: 7E5EBC0007 X-Stat-Signature: p4ww7m8nmuoau86r3ut71bnx19kdrcj8 X-Rspam-User: Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=mzY1bNcN; spf=pass (imf10.hostedemail.com: domain of krisman@collabora.com designates 46.235.227.227 as permitted sender) smtp.mailfrom=krisman@collabora.com; dmarc=pass (policy=none) header.from=collabora.com X-Rspamd-Server: rspam07 X-HE-Tag: 1646332765-359093 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000001, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Shuah Khan writes: > On 2/28/22 2:37 AM, David Hildenbrand wrote: >> On 24.02.22 22:23, Muhammad Usama Anjum wrote: >>> This introduces three tests: >>> 1) Sanity check soft dirty basic semantics: allocate area, clean, dirty, >>> check if the SD bit flipped. >>> 2) Check VMA reuse: validate the VM_SOFTDIRTY usage >>> 3) Check soft-dirty on huge pages >>> >>> This was motivated by Will Deacon's fix commit 912efa17e512 ("mm: proc: >>> Invalidate TLB after clearing soft-dirty page state"). I was tracking the >>> same issue that he fixed, and this test would have caught it. >>> >> A note that madv_populate.c already contains some SOFTDIRTY tests >> regarding MADV_POPULATE. Eventually we want to factor out >> softdirty/pagemap handling+checks for easier reuse. >> > > Is this patch unnecessary then? It is not unnecessary since the madv test doesn't cover the bug tested here, afaik. But, as mentioned when I originally submitted this patch, it should be merged into selftests/vm/madv_populate.c or, at least, reuse that existing infrastructure. https://lore.kernel.org/lkml/87lf553z5g.fsf@collabora.com/ -- Gabriel Krisman Bertazi