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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1AF19E92718 for ; Thu, 5 Oct 2023 15:48:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234418AbjJEPsw (ORCPT ); Thu, 5 Oct 2023 11:48:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52656 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233088AbjJEPsI (ORCPT ); Thu, 5 Oct 2023 11:48:08 -0400 Received: from esa9.hc1455-7.c3s2.iphmx.com (esa9.hc1455-7.c3s2.iphmx.com [139.138.36.223]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8BE65618D for ; Thu, 5 Oct 2023 07:30:26 -0700 (PDT) X-IronPort-AV: E=McAfee;i="6600,9927,10853"; a="123231749" X-IronPort-AV: E=Sophos;i="6.03,202,1694703600"; d="scan'208";a="123231749" Received: from unknown (HELO yto-r4.gw.nic.fujitsu.com) ([218.44.52.220]) by esa9.hc1455-7.c3s2.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Oct 2023 17:53:16 +0900 Received: from yto-m1.gw.nic.fujitsu.com (yto-nat-yto-m1.gw.nic.fujitsu.com [192.168.83.64]) by yto-r4.gw.nic.fujitsu.com (Postfix) with ESMTP id CD08FCD6DC for ; Thu, 5 Oct 2023 17:53:14 +0900 (JST) Received: from kws-ab3.gw.nic.fujitsu.com (kws-ab3.gw.nic.fujitsu.com [192.51.206.21]) by yto-m1.gw.nic.fujitsu.com (Postfix) with ESMTP id 1CEFFCFF9A for ; Thu, 5 Oct 2023 17:53:14 +0900 (JST) Received: from edo.cn.fujitsu.com (edo.cn.fujitsu.com [10.167.33.5]) by kws-ab3.gw.nic.fujitsu.com (Postfix) with ESMTP id 95BC52007781C for ; Thu, 5 Oct 2023 17:53:13 +0900 (JST) Received: from [10.193.128.127] (unknown [10.193.128.127]) by edo.cn.fujitsu.com (Postfix) with ESMTP id B4E4C1A0070; Thu, 5 Oct 2023 16:53:12 +0800 (CST) Message-ID: Date: Thu, 5 Oct 2023 16:53:12 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] xfs: drop experimental warning for FSDAX To: "Darrick J. Wong" Cc: Chandan Babu R , Dave Chinner , Dan Williams , Andrew Morton , linux-xfs@vger.kernel.org, nvdimm@lists.linux.dev References: <87y1gs83yq.fsf@debian-BULLSEYE-live-builder-AMD64> <20230927083034.90bd6336229dd00af601e0ef@linux-foundation.org> <9c3cbc0c-7135-4006-ad4a-2abce0a556b0@fujitsu.com> <20230928092052.9775e59262c102dc382513ef@linux-foundation.org> <20230928171339.GJ11439@frogsfrogsfrogs> <99279735-2d17-405f-bade-9501a296d817@fujitsu.com> <651718a6a6e2c_c558e2943e@dwillia2-xfh.jf.intel.com.notmuch> <87y1gltcvg.fsf@debian-BULLSEYE-live-builder-AMD64> <20231005000809.GN21298@frogsfrogsfrogs> From: Shiyang Ruan In-Reply-To: <20231005000809.GN21298@frogsfrogsfrogs> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-TM-AS-GCONF: 00 X-TM-AS-Product-Ver: IMSS-9.1.0.1417-9.0.0.1002-27916.006 X-TM-AS-User-Approved-Sender: Yes X-TMASE-Version: IMSS-9.1.0.1417-9.0.1002-27916.006 X-TMASE-Result: 10--16.958400-10.000000 X-TMASE-MatchedRID: +Gnn4vg1rdyPvrMjLFD6eHchRkqzj/bEC/ExpXrHizwUta+LQgEZMPfl GT6FkvTPZxpAP+mD6wCDs24hteATn9/K1ikJIsLOKsurITpSv+PVBDonH99+VuZYcdJgScjxgbP PjZPQM/k9QxA+EGg/dOvPanPbI41jNtywwIf5ksW/x8sBfQ/nQPzrlCVXDGYb0y9hkRwHPXGGPn eCOFgqRJT6yVaXFFqMyFxL8McBbl5x1XV+/0DNMFK9xlSOuRoWYjDXdM/x2VPWeQtrcncLfYtxj ghoBojmAd0cn1fMeA4C/xl8pUGMgZlbG0HPEaFBA9lly13c/gEBmf/gD11vZLcIt210bWgILhd6 ma7WE8vQ2jWaSvYDWNZZVqi+HeLhBOwoKTOp+iB85pjA/x1xftDEMPvvoocvP4H+2nyK0FOhhi9 o+RXHs6GvCFqoKSwTX7bicKxRIU352SN33UuAALnUfzScNiR3DWKb6PcVvpiOhzOa6g8KrZzIAI ytA760+kpp1tsNgPNe20TL477TKxIwZxusN18Nz8JWjFN4feM= X-TMASE-SNAP-Result: 1.821001.0001-0-1-22:0,33:0,34:0-0 Precedence: bulk List-ID: X-Mailing-List: linux-xfs@vger.kernel.org 在 2023/10/5 8:08, Darrick J. Wong 写道: >>> >>> Sorry, I sent the list below to Chandan, didn't cc the maillist >>> because it's just a rough list rather than a PR: >>> >>> >>> 1. subject: [v3] xfs: correct calculation for agend and blockcount >>> url: >>> https://lore.kernel.org/linux-xfs/20230913102942.601271-1-ruansy.fnst@fujitsu.com/ >>> note: This one is a fix patch for commit: 5cf32f63b0f4 ("xfs: >>> fix the calculation for "end" and "length""). >>> It can solve the fail of xfs/55[0-2]: the programs >>> accessing the DAX file may not be notified as expected, >>> because the length always 1 block less than actual. Then >>> this patch fixes this. >>> >>> >>> 2. subject: [v15] mm, pmem, xfs: Introduce MF_MEM_PRE_REMOVE for unbind >>> url: >>> https://lore.kernel.org/linux-xfs/20230928103227.250550-1-ruansy.fnst@fujitsu.com/T/#u >>> note: This is a feature patch. It handles the pre-remove event >>> of DAX device, by notifying kernel/user space before actually >>> removing. >>> It has been picked by Andrew in his >>> mm-hotfixes-unstable. I am not sure whether you or he will >>> merge this one. >>> >>> >>> 3. subject: [v1] xfs: drop experimental warning for FSDAX >>> url: >>> https://lore.kernel.org/linux-xfs/20230915063854.1784918-1-ruansy.fnst@fujitsu.com/ >>> note: With the patches mentioned above, I did a lot of tests, >>> including xfstests and blackbox tests, the FSDAX function looks >>> good now. So I think the experimental warning could be dropped. >> >> Darrick/Dave, Could you please review the above patch and let us know if you >> have any objections? > > The first two patches are ok. The third one ... well I was about to say > ok but then this happened with generic/269 on a 6.6-rc4 kernel and those > two patches applied: Hi Darrick, Thanks for testing. I just tested this case (generic/269) on v6.6-rc4 with my 3 patches again, but it didn't fail. Such WARNING message didn't show in dmesg too. My local.config is shown as below: [nodax_reflink] export FSTYP=xfs export TEST_DEV=/dev/pmem0 export TEST_DIR=/mnt/test export SCRATCH_DEV=/dev/pmem1 export SCRATCH_MNT=/mnt/scratch export MKFS_OPTIONS="-m reflink=1,rmapbt=1" [dax_reflink] export FSTYP=xfs export TEST_DEV=/dev/pmem0 export TEST_DIR=/mnt/test export SCRATCH_DEV=/dev/pmem1 export SCRATCH_MNT=/mnt/scratch export MKFS_OPTIONS="-m reflink=1,rmapbt=1" export MOUNT_OPTIONS="-o dax" export TEST_FS_MOUNT_OPTS="-o dax" And tools version are: - xfstests (v2023.09.03) - xfsprogs (v6.4.0) Could you show me more info (such as kernel config, local.config) ? So that I can find out what exactly is going wrong. -- Thanks, Ruan.