Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp3688532ybz; Mon, 20 Apr 2020 07:42:08 -0700 (PDT) X-Google-Smtp-Source: APiQypKAqN9pKOHYykULbnwrgf3gKDOYCxJxaOVMD23KX3GVOU8qqU1djuxHhqbVBve7q6A4ZARC X-Received: by 2002:a17:906:288b:: with SMTP id o11mr16905548ejd.237.1587393728525; Mon, 20 Apr 2020 07:42:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1587393728; cv=none; d=google.com; s=arc-20160816; b=BBaL3CZwHxzOfFy7EoLzSSvFvW0Q2/N3mqu6xD8La2kH3qwfEuKAL5rDq7He8iu6Iy YZkzhCdXnF+Dg89BQVelG/RzV+VccTUAJ6ikWHWgn9lud8iB5ry+PRt1zB41iGrHwzXj T6B8O7qi7UjKVCWiM3wTsg3D5+XdCcWApsmed9c7ClnUsQgNxTV+1OE5WqsGX4SFi2LO cgJr7PaNnjmH7p1q0ls6bGM8M8K13VyNcmiGem06We8s88gSVuvCGWRL5gKJuLfOYZZ2 CG2RIZYcyqHCU9abZ+aZ+6zmDDurRWACDHEfWLJ3fe4PbQqI7ZEa5diC+Oa2ady8PY7Y vULg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=kskdakcrRDDo91fKKn7OmT1ABdfxtHTqlHARV8i00xs=; b=en66TrBBa/KANavoZlvviUEql90pZSpzgBvYWu7HDODHkCgBrBC0UfRGX/Tg5r1b16 s/12piZREFJqYg55Y71z+7AE/lziUJZ2wWObAgHmXEtSzvcYdEvdQ6h/KOqlDRpQDsTy /bKVCTlbgC7aL6jJB2gSwGiIjr93URwnfb+0VWkxAb3NczHMt5qzmsE5IA3fZmVuzIca e23HtPtjhi+vhZG9da88mbcwYOPF3gabFLvo1WH8oddHd1IuZwo7PLNDytQgLw6feajQ lWICRhOgB1Y3n7h2xj+KBlXsQIXQKVhkk4JDKTXgqxChFVCKCsQ/zVDYsMBQlOX92Aby IBaQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=aUkNkLAK; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id o12si677651edj.233.2020.04.20.07.41.44; Mon, 20 Apr 2020 07:42:08 -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=@gmail.com header.s=20161025 header.b=aUkNkLAK; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726432AbgDTNUI (ORCPT + 99 others); Mon, 20 Apr 2020 09:20:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38532 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1726049AbgDTNUH (ORCPT ); Mon, 20 Apr 2020 09:20:07 -0400 Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B3CBC061A0C for ; Mon, 20 Apr 2020 06:20:07 -0700 (PDT) Received: by mail-wm1-x344.google.com with SMTP id x25so11003751wmc.0 for ; Mon, 20 Apr 2020 06:20:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=kskdakcrRDDo91fKKn7OmT1ABdfxtHTqlHARV8i00xs=; b=aUkNkLAKtinYXOavLQJR4mxUih+wv7V9IIV5NCBglcrY2wRxfDYBx0oBY6UoEzaY1Z hWMx7UkJYRTqvGRUf9lsw5g/ubgsCVxTBkf1hCbl34ojscJs9LpOkazrxXwHwU7ik/HY GWfF8DuKreVT8/viaIGGkvEvoKu5s2yDarFWFW/vjCXNCzQ6gmBxOreUKkKeGK+8k5fx vgMyY2V9mje8RTJtMxQq/qbZrstI+AQA/hhG3xxfhnyO8dyzCxIGoR70O7FIMSYhRQYX WTPioZNGzmlfpgGRwJX2uMe13cobITxAoNpc34eFSxArzhWpp4YrmcdhnmY/VymONzVa Lu6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=kskdakcrRDDo91fKKn7OmT1ABdfxtHTqlHARV8i00xs=; b=s+pPfiNt4q6/ShXLKF9QM4/wt/I+Z/ZuaBC418qVH1hVsN1hFYQnAudRMyl3cjKjv/ jmq1JyorVMYc5+4AEXKSVgScAHlvHKOhv5NrCHYCw8UqZKzm85K4KmB3c8hEIQ1JcULg KdzPqTzqLP2C8HytdPo2P1tZwlD7ioiszP4n2moqJRRRZWqMb9irgW1k2HoLGQ/LlSty 0Y6BwV3GhSKyvB2S0SvTYnhW8YaUE2LqxIpqYuFDs9i/VPleJO0c5SP0me/HQb1OMn3q 1A1zazDPekqXl/tVQLAIoR4bM6rD/p+5gjntnYXJYOCHwB/c0OeFgIZfJKgDPu8IAsz1 uqPQ== X-Gm-Message-State: AGi0Puaoj3+RrNsJPXDEq0C0xdOCAuxtUrnWqxC6/VVjcjX/sDv9QTaU WcIuI6b9eBNNOzZKzFj+Fdli8DGB X-Received: by 2002:a1c:2457:: with SMTP id k84mr15813838wmk.96.1587388805686; Mon, 20 Apr 2020 06:20:05 -0700 (PDT) Received: from PC192-168-2-103.speedport.ip (p5B05E57B.dip0.t-ipconnect.de. [91.5.229.123]) by smtp.gmail.com with ESMTPSA id z76sm1545815wmc.9.2020.04.20.06.20.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Apr 2020 06:20:05 -0700 (PDT) From: Pankaj Gupta To: linux-kernel@vger.kernel.org, linux-nvdimm@lists.01.org Cc: dan.j.williams@intel.com, vishal.l.verma@intel.com, dave.jiang@intel.com, david@redhat.com, mst@redhat.com, jmoyer@redhat.com, pankaj.gupta@cloud.ionos.com, Pankaj Gupta Subject: [RFC 1/2] pmem: make nvdimm_flush asynchronous Date: Mon, 20 Apr 2020 15:19:46 +0200 Message-Id: <20200420131947.41991-2-pankaj.gupta.linux@gmail.com> X-Mailer: git-send-email 2.24.1 In-Reply-To: <20200420131947.41991-1-pankaj.gupta.linux@gmail.com> References: <20200420131947.41991-1-pankaj.gupta.linux@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch converts nvdimm_flush to return when asynchronous flush is in process. Signed-off-by: Pankaj Gupta --- drivers/nvdimm/pmem.c | 15 ++++++++------- drivers/nvdimm/region_devs.c | 3 +-- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c index 4ffc6f7ca131..747ffaee513b 100644 --- a/drivers/nvdimm/pmem.c +++ b/drivers/nvdimm/pmem.c @@ -192,8 +192,10 @@ static blk_qc_t pmem_make_request(struct request_queue *q, struct bio *bio) struct pmem_device *pmem = q->queuedata; struct nd_region *nd_region = to_region(pmem); - if (bio->bi_opf & REQ_PREFLUSH) - ret = nvdimm_flush(nd_region, bio); + if ((bio->bi_opf & REQ_PREFLUSH) && + nvdimm_flush(nd_region, bio)) { + return BLK_QC_T_NONE; + } do_acct = nd_iostat_start(bio, &start); bio_for_each_segment(bvec, bio, iter) { @@ -207,11 +209,10 @@ static blk_qc_t pmem_make_request(struct request_queue *q, struct bio *bio) if (do_acct) nd_iostat_end(bio, start); - if (bio->bi_opf & REQ_FUA) - ret = nvdimm_flush(nd_region, bio); - - if (ret) - bio->bi_status = errno_to_blk_status(ret); + if (bio->bi_opf & REQ_FUA) { + nvdimm_flush(nd_region, bio); + return BLK_QC_T_NONE; + } bio_endio(bio); return BLK_QC_T_NONE; diff --git a/drivers/nvdimm/region_devs.c b/drivers/nvdimm/region_devs.c index a19e535830d9..1caa13f1523f 100644 --- a/drivers/nvdimm/region_devs.c +++ b/drivers/nvdimm/region_devs.c @@ -1091,8 +1091,7 @@ int nvdimm_flush(struct nd_region *nd_region, struct bio *bio) if (!nd_region->flush) rc = generic_nvdimm_flush(nd_region); else { - if (nd_region->flush(nd_region, bio)) - rc = -EIO; + rc = nd_region->flush(nd_region, bio); } return rc; -- 2.20.1