From patchwork Tue Dec 12 19:08:31 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Verma, Vishal L" X-Patchwork-Id: 177518 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:bcd1:0:b0:403:3b70:6f57 with SMTP id r17csp7941910vqy; Tue, 12 Dec 2023 11:09:00 -0800 (PST) X-Google-Smtp-Source: AGHT+IEqnk3iQVeTC5qXsN2aNEwafZPScooUcyMOKqGCfNWjknmTTXVAio/MSfX6B08y1c7iLm5l X-Received: by 2002:a17:903:2585:b0:1d0:6ffd:cea5 with SMTP id jb5-20020a170903258500b001d06ffdcea5mr3453784plb.94.1702408140064; Tue, 12 Dec 2023 11:09:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702408140; cv=none; d=google.com; s=arc-20160816; b=ryuRvNRbukRf8+eRpI7Wi/1fRGCUNrdy6pCloPNkr2admZpkDOBg9WDrbHGFsSI/UF wQtzOKZS179UuPGYPPitALwRNfN0DHzIFyKJlZuQfM/L8uuI2viS/TfXnESludhOG4+v lJRo8WSty5MFOW+tnJLp0EYBWzvBs3k2h/2/bgNqHvBicMQljGeLajSiNIdpFzXVv14u aV6kUnDzB3FEGvnHZxdeJM1LVHffu+aV9eR3kR9kaRxlKLARRJWyAd3yhHBPr6mrFSxX r0uFB4MDc6XCMeuHxCHpsCK1F8vRyFLTFNk5rwvuWmwaImW12Xts5oBylm7/smRAz2kP W3Cw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:in-reply-to:references:message-id :content-transfer-encoding:mime-version:subject:date:from :dkim-signature; bh=zTs4DkKQ9li4pEYgG6Xg9hlT6Otnp8dWYcZG4u3B3X4=; fh=FQ4E/SNuCiiHRf+Px+qtQ7Dk+P7RajOnzWg0q+BEgRM=; b=jnqi87j4KXGgiidDVuNF+xCF3EzdHMx5J3Uu7LVfJVSScrWll80tICtANR6IZPCIfG 9cawDHJ7wHjCXfKTRGjICgu4jPEaLcuL8AGlt3R9srGxgoKl7YRGEu2CqqYK5pSnJibs iDFAIVhC0RUrYcx8qxi9dmKzX1St4JUMOjQFpbAWp/8+FqUTzSpRQzSSpU6PI/Kn5r80 TLqiBsyYrVkV0Wc6yfauMwLxgoN4pNxiF/YPzaRlCeQ4YV6yXeWLqNAREsTVfvmA/U28 dfBHUSX0zwiWiqLODMe4YEhf5sRHw5qi6kmEBGVhkOSrjrX7mopweq2iIQxvLDQ13p47 WGWA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=RdRAwMj8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from snail.vger.email (snail.vger.email. [2620:137:e000::3:7]) by mx.google.com with ESMTPS id i3-20020a170902c94300b001cffce3a2e2si8373564pla.426.2023.12.12.11.08.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Dec 2023 11:09:00 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=RdRAwMj8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 90E818047794; Tue, 12 Dec 2023 11:08:56 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1376751AbjLLTIq (ORCPT + 99 others); Tue, 12 Dec 2023 14:08:46 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41802 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233067AbjLLTIm (ORCPT ); Tue, 12 Dec 2023 14:08:42 -0500 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7FBEEBD; Tue, 12 Dec 2023 11:08:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702408129; x=1733944129; h=from:date:subject:mime-version:content-transfer-encoding: message-id:references:in-reply-to:to:cc; bh=zYL68s17Ug9RUUMduHTMgqPyJT0UZnz1m+I7vUoJDyM=; b=RdRAwMj8PqSJO+wa1peRxSSDRkOYgFVdXPtTk/VWIelNq06Qydd+R6Tz NQKNU6zrFoQ2N28eCSTB29tNS0WPn3XNT9BIeJB6+AlhzGOOiTTaTqLHH 2Cgn8VVd4j+oK6X9oavPpFRm6ZpS2U5kYJ2lWWLBOcCSgrXUyY4wdIro2 /LRDMZv7A8XPj5P+pLO8GMfQBRhftNGX0HyYsLSRAfv23kwxf6RVGvFsm /+ht+YVdrWWVA5C/6zM9NEwtHX51RaXDo8jx9PZiof+cZOf9KztvHWfQ5 hHBpKWIROP0IbUXvxH5swWCg+Q61VpQwvnI29zcpidMIKIRw5VgBn9xE4 w==; X-IronPort-AV: E=McAfee;i="6600,9927,10922"; a="13550594" X-IronPort-AV: E=Sophos;i="6.04,271,1695711600"; d="scan'208";a="13550594" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Dec 2023 11:08:48 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10922"; a="844017861" X-IronPort-AV: E=Sophos;i="6.04,271,1695711600"; d="scan'208";a="844017861" Received: from cmperez2-mobl2.amr.corp.intel.com (HELO [192.168.1.200]) ([10.212.66.25]) by fmsmga004-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Dec 2023 11:08:46 -0800 From: Vishal Verma Date: Tue, 12 Dec 2023 12:08:31 -0700 Subject: [PATCH v4 2/3] dax/bus: Introduce guard(device) for device_{lock,unlock} flows MIME-Version: 1.0 Message-Id: <20231212-vv-dax_abi-v4-2-1351758f0c92@intel.com> References: <20231212-vv-dax_abi-v4-0-1351758f0c92@intel.com> In-Reply-To: <20231212-vv-dax_abi-v4-0-1351758f0c92@intel.com> To: Dan Williams , Vishal Verma , Dave Jiang Cc: linux-kernel@vger.kernel.org, nvdimm@lists.linux.dev, linux-cxl@vger.kernel.org, David Hildenbrand , Dave Hansen , Huang Ying , Joao Martins X-Mailer: b4 0.13-dev-433a8 X-Developer-Signature: v=1; a=openpgp-sha256; l=6772; i=vishal.l.verma@intel.com; h=from:subject:message-id; bh=zYL68s17Ug9RUUMduHTMgqPyJT0UZnz1m+I7vUoJDyM=; b=owGbwMvMwCXGf25diOft7jLG02pJDKkV6/eaxBQWhvSfm7/rjF+CpubEQxa3FHT1g45I+RXON 0i+++VKRykLgxgXg6yYIsvfPR8Zj8ltz+cJTHCEmcPKBDKEgYtTACbyTZyR4c7DK18Pe/HM8gh3 fuIheEm77Cp7lJSx7ouLa+fJTbhRYM3IsPxk3tINju8qjq+63c7Ke00hjNUze9XSsO4VerY8Lne bOQE= X-Developer-Key: i=vishal.l.verma@intel.com; a=openpgp; fpr=F8682BE134C67A12332A2ED07AFA61BEA3B84DFF X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Tue, 12 Dec 2023 11:08:56 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1785104317959434987 X-GMAIL-MSGID: 1785104317959434987 Introduce a guard(device) macro to lock a 'struct device', and unlock it automatically when going out of scope using Scope Based Resource Management semantics. A lot of the sysfs attribute writes in drivers/dax/bus.c benefit from a cleanup using these, so change these where applicable. Cc: Joao Martins Suggested-by: Dan Williams Signed-off-by: Vishal Verma Reviewed-by: Ira Weiny --- include/linux/device.h | 2 + drivers/dax/bus.c | 109 +++++++++++++++++++------------------------------ 2 files changed, 44 insertions(+), 67 deletions(-) diff --git a/include/linux/device.h b/include/linux/device.h index d7a72a8749ea..a83efd9ae949 100644 --- a/include/linux/device.h +++ b/include/linux/device.h @@ -1131,6 +1131,8 @@ void set_secondary_fwnode(struct device *dev, struct fwnode_handle *fwnode); void device_set_of_node_from_dev(struct device *dev, const struct device *dev2); void device_set_node(struct device *dev, struct fwnode_handle *fwnode); +DEFINE_GUARD(device, struct device *, device_lock(_T), device_unlock(_T)) + static inline int dev_num_vf(struct device *dev) { if (dev->bus && dev->bus->num_vf) diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c index 1ff1ab5fa105..ce1356ac6dc2 100644 --- a/drivers/dax/bus.c +++ b/drivers/dax/bus.c @@ -296,9 +296,8 @@ static ssize_t available_size_show(struct device *dev, struct dax_region *dax_region = dev_get_drvdata(dev); unsigned long long size; - device_lock(dev); + guard(device)(dev); size = dax_region_avail_size(dax_region); - device_unlock(dev); return sprintf(buf, "%llu\n", size); } @@ -314,10 +313,9 @@ static ssize_t seed_show(struct device *dev, if (is_static(dax_region)) return -EINVAL; - device_lock(dev); + guard(device)(dev); seed = dax_region->seed; rc = sprintf(buf, "%s\n", seed ? dev_name(seed) : ""); - device_unlock(dev); return rc; } @@ -333,10 +331,9 @@ static ssize_t create_show(struct device *dev, if (is_static(dax_region)) return -EINVAL; - device_lock(dev); + guard(device)(dev); youngest = dax_region->youngest; rc = sprintf(buf, "%s\n", youngest ? dev_name(youngest) : ""); - device_unlock(dev); return rc; } @@ -345,7 +342,14 @@ static ssize_t create_store(struct device *dev, struct device_attribute *attr, const char *buf, size_t len) { struct dax_region *dax_region = dev_get_drvdata(dev); + struct dev_dax_data data = { + .dax_region = dax_region, + .size = 0, + .id = -1, + .memmap_on_memory = false, + }; unsigned long long avail; + struct dev_dax *dev_dax; ssize_t rc; int val; @@ -358,38 +362,26 @@ static ssize_t create_store(struct device *dev, struct device_attribute *attr, if (val != 1) return -EINVAL; - device_lock(dev); + guard(device)(dev); avail = dax_region_avail_size(dax_region); if (avail == 0) - rc = -ENOSPC; - else { - struct dev_dax_data data = { - .dax_region = dax_region, - .size = 0, - .id = -1, - .memmap_on_memory = false, - }; - struct dev_dax *dev_dax = devm_create_dev_dax(&data); + return -ENOSPC; - if (IS_ERR(dev_dax)) - rc = PTR_ERR(dev_dax); - else { - /* - * In support of crafting multiple new devices - * simultaneously multiple seeds can be created, - * but only the first one that has not been - * successfully bound is tracked as the region - * seed. - */ - if (!dax_region->seed) - dax_region->seed = &dev_dax->dev; - dax_region->youngest = &dev_dax->dev; - rc = len; - } - } - device_unlock(dev); + dev_dax = devm_create_dev_dax(&data); + if (IS_ERR(dev_dax)) + return PTR_ERR(dev_dax); - return rc; + /* + * In support of crafting multiple new devices + * simultaneously multiple seeds can be created, + * but only the first one that has not been + * successfully bound is tracked as the region + * seed. + */ + if (!dax_region->seed) + dax_region->seed = &dev_dax->dev; + dax_region->youngest = &dev_dax->dev; + return len; } static DEVICE_ATTR_RW(create); @@ -481,12 +473,9 @@ static int __free_dev_dax_id(struct dev_dax *dev_dax) static int free_dev_dax_id(struct dev_dax *dev_dax) { struct device *dev = &dev_dax->dev; - int rc; - device_lock(dev); - rc = __free_dev_dax_id(dev_dax); - device_unlock(dev); - return rc; + guard(device)(dev); + return __free_dev_dax_id(dev_dax); } static int alloc_dev_dax_id(struct dev_dax *dev_dax) @@ -908,9 +897,8 @@ static ssize_t size_show(struct device *dev, struct dev_dax *dev_dax = to_dev_dax(dev); unsigned long long size; - device_lock(dev); + guard(device)(dev); size = dev_dax_size(dev_dax); - device_unlock(dev); return sprintf(buf, "%llu\n", size); } @@ -1080,15 +1068,12 @@ static ssize_t size_store(struct device *dev, struct device_attribute *attr, return -EINVAL; } - device_lock(dax_region->dev); - if (!dax_region->dev->driver) { - device_unlock(dax_region->dev); + guard(device)(dax_region->dev); + if (!dax_region->dev->driver) return -ENXIO; - } - device_lock(dev); + + guard(device)(dev); rc = dev_dax_resize(dax_region, dev_dax, val); - device_unlock(dev); - device_unlock(dax_region->dev); return rc == 0 ? len : rc; } @@ -1138,18 +1123,14 @@ static ssize_t mapping_store(struct device *dev, struct device_attribute *attr, return rc; rc = -ENXIO; - device_lock(dax_region->dev); - if (!dax_region->dev->driver) { - device_unlock(dax_region->dev); + guard(device)(dax_region->dev); + if (!dax_region->dev->driver) return rc; - } - device_lock(dev); + guard(device)(dev); to_alloc = range_len(&r); if (alloc_is_aligned(dev_dax, to_alloc)) rc = alloc_dev_dax_range(dev_dax, r.start, to_alloc); - device_unlock(dev); - device_unlock(dax_region->dev); return rc == 0 ? len : rc; } @@ -1196,26 +1177,20 @@ static ssize_t align_store(struct device *dev, struct device_attribute *attr, if (!dax_align_valid(val)) return -EINVAL; - device_lock(dax_region->dev); - if (!dax_region->dev->driver) { - device_unlock(dax_region->dev); + guard(device)(dax_region->dev); + if (!dax_region->dev->driver) return -ENXIO; - } - device_lock(dev); - if (dev->driver) { - rc = -EBUSY; - goto out_unlock; - } + guard(device)(dev); + if (dev->driver) + return -EBUSY; align_save = dev_dax->align; dev_dax->align = val; rc = dev_dax_validate_align(dev_dax); if (rc) dev_dax->align = align_save; -out_unlock: - device_unlock(dev); - device_unlock(dax_region->dev); + return rc == 0 ? len : rc; } static DEVICE_ATTR_RW(align);