From patchwork Thu Jul 5 06:49:56 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dan Williams X-Patchwork-Id: 10508267 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 EB88C60116 for ; Thu, 5 Jul 2018 06:59:57 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D893028DF2 for ; Thu, 5 Jul 2018 06:59:57 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id CCAF228E49; Thu, 5 Jul 2018 06:59:57 +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=-2.9 required=2.0 tests=BAYES_00, MAILING_LIST_MULTI, RCVD_IN_DNSWL_NONE autolearn=unavailable version=3.3.1 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4F86428DF2 for ; Thu, 5 Jul 2018 06:59:57 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 763A16B0277; Thu, 5 Jul 2018 02:59:55 -0400 (EDT) Delivered-To: linux-mm-outgoing@kvack.org Received: by kanga.kvack.org (Postfix, from userid 40) id 711F56B0278; Thu, 5 Jul 2018 02:59:55 -0400 (EDT) X-Original-To: int-list-linux-mm@kvack.org X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 627126B0279; Thu, 5 Jul 2018 02:59:55 -0400 (EDT) X-Original-To: linux-mm@kvack.org X-Delivered-To: linux-mm@kvack.org Received: from mail-pl0-f70.google.com (mail-pl0-f70.google.com [209.85.160.70]) by kanga.kvack.org (Postfix) with ESMTP id 241546B0277 for ; Thu, 5 Jul 2018 02:59:55 -0400 (EDT) Received: by mail-pl0-f70.google.com with SMTP id t19-v6so1422410plo.9 for ; Wed, 04 Jul 2018 23:59:55 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-original-authentication-results:x-gm-message-state:subject:from :to:cc:date:message-id:in-reply-to:references:user-agent :mime-version:content-transfer-encoding; bh=0xBlKbIiAZAjDHWJse6OFnnNgA8yroLCRdFGcVISi3Q=; b=W1TDLSyqSay5iLxtRUaR1VCo2SqbK7HFPBgDfkFyEJ15InnBMvQJMBSbWmE7ODycUb ltD+OHqEdOl7sQbR0D0qxwVGo0a3DnM6jZ4/bvjW/+Ez/Y+BtX7JEUVfMIq1HMK80xaR pz4QXrxxTbyoO0coTbG/puqGA3rFcq+0gPIzQF5WKkv7vbK4N7yjSb8gh9l45QEH24B4 bY+jW12avcZ1oEcv1z3RKBZGjhj/MR/W28VwbnWynNso93iIbOOgqSbiioiBbHAe7ykp 3Ncbe8PyGSY0rg3suvO0tjQBW35vKuH8p6234hoWKgeD9cpZG6ZvqxKlZ7PU7axX09zF NiPw== X-Original-Authentication-Results: mx.google.com; spf=pass (google.com: domain of dan.j.williams@intel.com designates 192.55.52.136 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com X-Gm-Message-State: APt69E17GXJJfwCf0pO3yEuukyJCgdSW9Y7KQlD7q6qDaFNNd3ck0J2i dZyLzWzctvKpJdlkXVGdnY1jOigjuAUkl3PlQH2oks1ntL2AJTM4UGBbiHfORbpYIC5/Nv9FnV0 6Ix9k2mKBTJjus+qFnX2qMPpdwqMz0VQlxrBdnrB65yY2SAZUOpkndnDjrsbMEnhMfQ== X-Received: by 2002:a63:195e:: with SMTP id 30-v6mr2328802pgz.192.1530773994818; Wed, 04 Jul 2018 23:59:54 -0700 (PDT) X-Google-Smtp-Source: AAOMgpftrVJESLvLJXr7huGFDHduy6w1GcK4+xNmfH4vF7NPqFKgU9fP26HppPcEHIB3UhoRrQN4 X-Received: by 2002:a63:195e:: with SMTP id 30-v6mr2328767pgz.192.1530773994103; Wed, 04 Jul 2018 23:59:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530773994; cv=none; d=google.com; s=arc-20160816; b=rpLdGLtwWwLEs8oCXjVKjpBWlL3OoyulupENcjaRoSJnrPgQ3nZrzMDQDYy0eB+j0t w8x5MLENOhbtVXMZO122l34yUfOtvTUIVnaIbV2fw7P99CZ9MGdzWRWjOLoK1P4UFE3T ch0P8ZNsptypmoIxl+Yl+MapZ3hgE7/gK3xMSLReBZ9NDRq9GXXH94vKzJUgEmJTsEab leCM5Qc0ZKoGLoznfcnh97qNqWu5AtByDrbgbniH6k/0Czlz5cS9I4n+drn3q3FK0Nqq KN9g4EAZXp6kQy/AI4o7KoURQosqSSKSlKKkwgCCckaQWLXwViz2NY6B+DfPkAGJXDW+ bRoQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:user-agent:references :in-reply-to:message-id:date:cc:to:from:subject :arc-authentication-results; bh=0xBlKbIiAZAjDHWJse6OFnnNgA8yroLCRdFGcVISi3Q=; b=LzoVPoQDMJtUVVmG1BpsDQeW1Anbqx+3PBkQk9kb0gHXEsHUEBru+AMLHrZ/vSxxQ4 un0tLkE40N9tQdSMuDa517/Ej4V2R/5l3owMOgKgbsqNAnZd+C+Bqzapyq4b18KwUhRG 4ql65OvRhzwAn0cNIuOYf7h87mFTXs+99939OU0r8++BLoJga8guwAMgA36/HsEmMu+C L2jhviWrxgFZQj9SUXZJMAX3JZ9pAVSwTLhZKZF4+poId2JRq5kPdaI+CtjmTIvAGMZP SdoAWtAR3psHwEA8O+SRokULhhOsSdktolQV5D88w83sfA5xdgXnVOEF2qsQNboJiSB0 9zfw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dan.j.williams@intel.com designates 192.55.52.136 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from mga12.intel.com (mga12.intel.com. [192.55.52.136]) by mx.google.com with ESMTPS id u76-v6si6173142pfj.58.2018.07.04.23.59.53 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 04 Jul 2018 23:59:54 -0700 (PDT) Received-SPF: pass (google.com: domain of dan.j.williams@intel.com designates 192.55.52.136 as permitted sender) client-ip=192.55.52.136; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dan.j.williams@intel.com designates 192.55.52.136 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 04 Jul 2018 23:59:53 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,311,1526367600"; d="scan'208";a="237968468" Received: from dwillia2-desk3.jf.intel.com (HELO dwillia2-desk3.amr.corp.intel.com) ([10.54.39.16]) by orsmga005.jf.intel.com with ESMTP; 04 Jul 2018 23:59:53 -0700 Subject: [PATCH 10/13] filesystem-dax: Make mount time pfn validation a debug check From: Dan Williams To: akpm@linux-foundation.org Cc: Jan Kara , Christoph Hellwig , Ross Zwisler , vishal.l.verma@intel.com, linux-nvdimm@lists.01.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Date: Wed, 04 Jul 2018 23:49:56 -0700 Message-ID: <153077339595.40830.16578300356324475234.stgit@dwillia2-desk3.amr.corp.intel.com> In-Reply-To: <153077334130.40830.2714147692560185329.stgit@dwillia2-desk3.amr.corp.intel.com> References: <153077334130.40830.2714147692560185329.stgit@dwillia2-desk3.amr.corp.intel.com> User-Agent: StGit/0.18-2-gc94f MIME-Version: 1.0 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: X-Virus-Scanned: ClamAV using ClamSMTP Do not ask for dax_direct_access() to retrieve a pfn in the DAX_DRIVER_DEBUG=n case. This avoids an early call to memmap_sync() in the driver. Now that QUEUE_FLAG_DAX usage has been fixed the validation of the pfn is only useful for dax driver developers. It is safe to assume that pmem, dcssblk, and device-mapper-dax are correct with respect to dax operation, so only retrieve the pfn for debug builds when qualifying a new dax driver, if one ever arrives. The moves the first consumption of a pfn from ->direct_access() to the first dax mapping fault, rather than initial filesystem mount. I.e. more time for memmap init to run in the background. Cc: Jan Kara Cc: Christoph Hellwig Cc: Ross Zwisler Signed-off-by: Dan Williams --- drivers/dax/Kconfig | 10 ++++++++ drivers/dax/super.c | 64 ++++++++++++++++++++++++++++++++------------------- 2 files changed, 50 insertions(+), 24 deletions(-) diff --git a/drivers/dax/Kconfig b/drivers/dax/Kconfig index e0700bf4893a..b32f8827b983 100644 --- a/drivers/dax/Kconfig +++ b/drivers/dax/Kconfig @@ -9,6 +9,16 @@ menuconfig DAX if DAX +config DAX_DRIVER_DEBUG + bool "DAX: driver debug" + help + Enable validation of the page frame objects returned from a + driver's 'direct_access' operation. This validation is + performed relative to the requirements of the FS_DAX and + FS_DAX_LIMITED configuration options. If you are validating + the implementation of a dax device driver say Y otherwise + say N. + config DEV_DAX tristate "Device DAX: direct access mapping device" depends on TRANSPARENT_HUGEPAGE diff --git a/drivers/dax/super.c b/drivers/dax/super.c index 903d9c473749..87b1c55b7c7a 100644 --- a/drivers/dax/super.c +++ b/drivers/dax/super.c @@ -72,6 +72,41 @@ struct dax_device *fs_dax_get_by_bdev(struct block_device *bdev) EXPORT_SYMBOL_GPL(fs_dax_get_by_bdev); #endif +static bool validate_dax_pfn(pfn_t *pfn) +{ + bool dax_enabled = false; + + /* + * Unless debugging a new dax driver, or new dax architecture + * support there is no need to check the pfn. Delay the kernel's + * first need for a dax pfn until first userspace dax fault. + */ + if (!pfn) + return true; + + if (IS_ENABLED(CONFIG_FS_DAX_LIMITED) && pfn_t_special(*pfn)) { + /* + * An arch that has enabled the pmem api should also + * have its drivers support pfn_t_devmap() + * + * This is a developer warning and should not trigger in + * production. dax_flush() will crash since it depends + * on being able to do (page_address(pfn_to_page())). + */ + WARN_ON(IS_ENABLED(CONFIG_ARCH_HAS_PMEM_API)); + dax_enabled = true; + } else if (pfn_t_devmap(*pfn)) { + struct dev_pagemap *pgmap; + + pgmap = get_dev_pagemap(pfn_t_to_pfn(*pfn), NULL); + if (pgmap && pgmap->type == MEMORY_DEVICE_FS_DAX) + dax_enabled = true; + put_dev_pagemap(pgmap); + } + + return dax_enabled; +} + /** * __bdev_dax_supported() - Check if the device supports dax for filesystem * @bdev: block device to check @@ -85,11 +120,10 @@ EXPORT_SYMBOL_GPL(fs_dax_get_by_bdev); bool __bdev_dax_supported(struct block_device *bdev, int blocksize) { struct dax_device *dax_dev; - bool dax_enabled = false; + pfn_t _pfn, *pfn; pgoff_t pgoff; int err, id; void *kaddr; - pfn_t pfn; long len; char buf[BDEVNAME_SIZE]; @@ -113,8 +147,10 @@ bool __bdev_dax_supported(struct block_device *bdev, int blocksize) return false; } + pfn = IS_ENABLED(DAX_DRIVER_DEBUG) ? &_pfn : NULL; + id = dax_read_lock(); - len = dax_direct_access(dax_dev, pgoff, 1, &kaddr, &pfn); + len = dax_direct_access(dax_dev, pgoff, 1, &kaddr, pfn); dax_read_unlock(id); put_dax(dax_dev); @@ -125,27 +161,7 @@ bool __bdev_dax_supported(struct block_device *bdev, int blocksize) return false; } - if (IS_ENABLED(CONFIG_FS_DAX_LIMITED) && pfn_t_special(pfn)) { - /* - * An arch that has enabled the pmem api should also - * have its drivers support pfn_t_devmap() - * - * This is a developer warning and should not trigger in - * production. dax_flush() will crash since it depends - * on being able to do (page_address(pfn_to_page())). - */ - WARN_ON(IS_ENABLED(CONFIG_ARCH_HAS_PMEM_API)); - dax_enabled = true; - } else if (pfn_t_devmap(pfn)) { - struct dev_pagemap *pgmap; - - pgmap = get_dev_pagemap(pfn_t_to_pfn(pfn), NULL); - if (pgmap && pgmap->type == MEMORY_DEVICE_FS_DAX) - dax_enabled = true; - put_dev_pagemap(pgmap); - } - - if (!dax_enabled) { + if (!validate_dax_pfn(pfn)) { pr_debug("%s: error: dax support not enabled\n", bdevname(bdev, buf)); return false;