From patchwork Mon Apr 10 06:05:52 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Murphy Zhou X-Patchwork-Id: 9671987 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 9E72C60244 for ; Mon, 10 Apr 2017 06:06:05 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8DC5D27D0C for ; Mon, 10 Apr 2017 06:06:05 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 8153627F94; Mon, 10 Apr 2017 06:06:05 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 19D8C27D0C for ; Mon, 10 Apr 2017 06:06:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752314AbdDJGGE (ORCPT ); Mon, 10 Apr 2017 02:06:04 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51240 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751702AbdDJGGE (ORCPT ); Mon, 10 Apr 2017 02:06:04 -0400 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 806C280461; Mon, 10 Apr 2017 06:06:03 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 806C280461 Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=xzhou@redhat.com DKIM-Filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 806C280461 Received: from localhost (dhcp-12-130.nay.redhat.com [10.66.12.130]) by smtp.corp.redhat.com (Postfix) with ESMTP id E20B796EA4; Mon, 10 Apr 2017 06:06:02 +0000 (UTC) From: Xiong Zhou To: fstests@vger.kernel.org Cc: ross.zwisler@linux.intel.com, dan.j.williams@intel.com, jmoyer@redhat.com, eguan@redhat.com, Xiong Zhou Subject: [PATCH v2 2/3] DAX-DIO: skip DAX to non-DAX if unsupported Date: Mon, 10 Apr 2017 14:05:52 +0800 Message-Id: <1491804353-1326-2-git-send-email-xzhou@redhat.com> In-Reply-To: <1491804353-1326-1-git-send-email-xzhou@redhat.com> References: <20170407171600.GA29489@linux.intel.com> <1491804353-1326-1-git-send-email-xzhou@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Mon, 10 Apr 2017 06:06:03 +0000 (UTC) Sender: fstests-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP By tweaking SCRATCH_DEV by ndctl, only run DAX mapped area DIO to non-DAX area tests when pmem device is in "memory mode". Signed-off-by: Xiong Zhou --- v2: umount/check scratch fs after test restore SCRATCH_DEV mode in cleanup fix variable names tests/generic/413 | 26 +++++++++++++++++++++++++- tests/xfs/260 | 31 ++++++++++++++++++++++++++++++- 2 files changed, 55 insertions(+), 2 deletions(-) diff --git a/tests/generic/413 b/tests/generic/413 index a1cc514..fbe8f11 100755 --- a/tests/generic/413 +++ b/tests/generic/413 @@ -34,6 +34,9 @@ _cleanup() { cd / rm -f $tmp.* + # restore SCRATCH_DEV to original mode + $NDCTL_PROG create-namespace -f -e $scratch_dev_namespace \ + -m $scratch_dev_mode > /dev/null 2>&1 } # get standard environment, filters and checks @@ -47,9 +50,13 @@ _supported_fs generic _supported_os Linux _require_test _require_scratch_dax +# fsck manually after tests as we will reconfig SCRATCH_DEV +_require_scratch_nocheck _require_test_program "feature" _require_test_program "t_mmap_dio" _require_xfs_io_command "falloc" +_require_ndctl +_require_jq prep_files() { @@ -110,7 +117,7 @@ t_both_nondax() t_mmap_dio_dax() { t_both_dax $1 - t_dax_to_nondax $1 + [ $skip_dax_to_nondax -eq 0 ] && t_dax_to_nondax $1 t_nondax_to_dax $1 t_both_nondax $1 } @@ -126,6 +133,21 @@ do_tests() t_mmap_dio_dax $((64 * 1024 * 1024)) } +skip_dax_to_nondax=0 +# config SCRATCH_DEV to memory mode to support DAX mapped +# area DIO to non-DAX area. + +# save original mode +scratch_dev_mode=$(_ndctl_get_pmem_key_value $SCRATCH_DEV mode) + +# get its namespace +scratch_dev_namespace=$(_ndctl_get_pmem_key_value $SCRATCH_DEV dev) + +# skip dax to non-dax dio if config fails +$NDCTL_PROG create-namespace -f -e $scratch_dev_namespace \ + -m memory > /dev/null 2>&1 \ + || skip_dax_to_nondax=1 + # make fs 2Mb aligned for PMD fault testing mkfs_opts="" if [ "$FSTYP" == "ext4" ]; then @@ -146,6 +168,8 @@ _scratch_mount "-o dax" tsize=$((128 * 1024 * 1024)) do_tests +_scratch_unmount +_check_scratch_fs # success, all done echo "Silence is golden" diff --git a/tests/xfs/260 b/tests/xfs/260 index e613cc0..4e445d3 100755 --- a/tests/xfs/260 +++ b/tests/xfs/260 @@ -34,6 +34,9 @@ _cleanup() { cd / rm -f $tmp.* + # restore SCRATCH_DEV to original mode + $NDCTL_PROG create-namespace -f -e $scratch_dev_namespace \ + -m $scratch_dev_mode > /dev/null 2>&1 } # get standard environment, filters and checks @@ -46,10 +49,14 @@ rm -f $seqres.full _supported_fs xfs _supported_os Linux _require_scratch_dax +# fsck manually after tests as we will reconfig SCRATCH_DEV +_require_scratch_nocheck _require_test_program "feature" _require_test_program "t_mmap_dio" _require_xfs_io_command "chattr" "+/-x" _require_xfs_io_command "falloc" +_require_ndctl +_require_jq prep_files() { @@ -120,7 +127,7 @@ t_both_nondax() t_dax_flag_mmap_dio() { t_both_dax $1 - t_dax_to_nondax $1 + [ $skip_dax_to_nondax -eq 0 ] && t_dax_to_nondax $1 t_nondax_to_dax $1 t_both_nondax $1 } @@ -136,6 +143,21 @@ do_tests() t_dax_flag_mmap_dio $((64 * 1024 * 1024)) } +skip_dax_to_nondax=0 +# config SCRATCH_DEV to memory mode to support DAX mapped +# area DIO to non-DAX area. + +# save original mode +scratch_dev_mode=$(_ndctl_get_pmem_key_value $SCRATCH_DEV mode) + +# get its namespace +scratch_dev_namespace=$(_ndctl_get_pmem_key_value $SCRATCH_DEV dev) + +# skip dax to non-dax dio if config fails +$NDCTL_PROG create-namespace -f -e $scratch_dev_namespace \ + -m memory > /dev/null 2>&1 \ + || skip_dax_to_nondax=1 + # make xfs 2Mb aligned for PMD fault testing _scratch_mkfs "-d su=2m,sw=1" > /dev/null 2>&1 @@ -145,12 +167,19 @@ _scratch_mount "-o dax" tsize=$((128 * 1024 * 1024)) do_tests +# we need to reconfig SCRATCH_DEV to its original mode +# in cleanup, which will corrupt the FS. So fsck here +# before cleanup. _scratch_unmount +_check_scratch_fs # mount again without dax option export MOUNT_OPTIONS="" +export TEST_FS_MOUNT_OPTS="" _scratch_mount do_tests +_scratch_unmount +_check_scratch_fs # success, all done echo "Silence is golden"