From patchwork Sat Apr 18 01:35:46 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dan Williams X-Patchwork-Id: 6236181 Return-Path: X-Original-To: patchwork-linux-nvdimm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id A395C9F313 for ; Sat, 18 Apr 2015 01:38:30 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 860BC2038E for ; Sat, 18 Apr 2015 01:38:29 +0000 (UTC) Received: from ml01.01.org (ml01.01.org [198.145.21.10]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 543BC2038D for ; Sat, 18 Apr 2015 01:38:28 +0000 (UTC) Received: from ml01.vlan14.01.org (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 49742813BA; Fri, 17 Apr 2015 18:38:28 -0700 (PDT) X-Original-To: linux-nvdimm@lists.01.org Delivered-To: linux-nvdimm@lists.01.org Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by ml01.01.org (Postfix) with ESMTP id 120C281388 for ; Fri, 17 Apr 2015 18:38:27 -0700 (PDT) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga103.jf.intel.com with ESMTP; 17 Apr 2015 18:38:26 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,598,1422950400"; d="scan'208";a="482460296" Received: from dwillia2-desk3.jf.intel.com (HELO dwillia2-desk3.amr.corp.intel.com) ([10.23.232.36]) by FMSMGA003.fm.intel.com with ESMTP; 17 Apr 2015 18:38:26 -0700 From: Dan Williams To: linux-nvdimm@lists.01.org Date: Fri, 17 Apr 2015 21:35:46 -0400 Message-ID: <20150418013546.25237.86836.stgit@dwillia2-desk3.amr.corp.intel.com> In-Reply-To: <20150418013256.25237.96403.stgit@dwillia2-desk3.amr.corp.intel.com> References: <20150418013256.25237.96403.stgit@dwillia2-desk3.amr.corp.intel.com> User-Agent: StGit/0.17.1-8-g92dd MIME-Version: 1.0 Cc: Neil Brown , Greg KH , linux-kernel@vger.kernel.org Subject: [Linux-nvdimm] [PATCH 06/21] nd: ndctl class device, and nd bus attributes X-BeenThere: linux-nvdimm@lists.01.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Linux-nvdimm developer list." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This is the position (device topology) independent method to find all the NFIT-defined buses in the system. The expectation is that there will only ever be one "nd" bus discovered via /sys/class/nd/ndctl0. However, we allow for the possibility of multiple buses and they will listed in discovery order as ndctl0...ndctlN. This character device hosts the ioctl for passing control messages (as defined by the NFIT spec). The "format" and "revision" attributes of this device identify the format of the messages. In the event an NFIT is registered with an unknown/unsupported control message format then the "format" attribute will not be visible. Cc: Greg KH Cc: Neil Brown Signed-off-by: Dan Williams --- drivers/block/nd/Makefile | 1 drivers/block/nd/bus.c | 84 +++++++++++++++++++++++++++++++++++++++++ drivers/block/nd/core.c | 71 ++++++++++++++++++++++++++++++++++- drivers/block/nd/nd-private.h | 5 ++ 4 files changed, 160 insertions(+), 1 deletion(-) create mode 100644 drivers/block/nd/bus.c diff --git a/drivers/block/nd/Makefile b/drivers/block/nd/Makefile index c6bec0c185c5..7772fb599809 100644 --- a/drivers/block/nd/Makefile +++ b/drivers/block/nd/Makefile @@ -20,3 +20,4 @@ obj-$(CONFIG_NFIT_ACPI) += nd_acpi.o nd_acpi-y := acpi.o nd-y := core.o +nd-y += bus.o diff --git a/drivers/block/nd/bus.c b/drivers/block/nd/bus.c new file mode 100644 index 000000000000..c27db50511f2 --- /dev/null +++ b/drivers/block/nd/bus.c @@ -0,0 +1,84 @@ +/* + * Copyright(c) 2013-2015 Intel Corporation. All rights reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of version 2 of the GNU General Public License as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + */ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt +#include +#include +#include +#include +#include +#include "nd-private.h" +#include "nfit.h" + +static int nd_major; +static struct class *nd_class; + +int nd_bus_create_ndctl(struct nd_bus *nd_bus) +{ + dev_t devt = MKDEV(nd_major, nd_bus->id); + struct device *dev; + + dev = device_create(nd_class, &nd_bus->dev, devt, nd_bus, "ndctl%d", + nd_bus->id); + + if (IS_ERR(dev)) { + dev_dbg(&nd_bus->dev, "failed to register ndctl%d: %ld\n", + nd_bus->id, PTR_ERR(dev)); + return PTR_ERR(dev); + } + return 0; +} + +void nd_bus_destroy_ndctl(struct nd_bus *nd_bus) +{ + device_destroy(nd_class, MKDEV(nd_major, nd_bus->id)); +} + +static long nd_ioctl(struct file *file, unsigned int cmd, unsigned long arg) +{ + return -ENXIO; +} + +static const struct file_operations nd_bus_fops = { + .owner = THIS_MODULE, + .open = nonseekable_open, + .unlocked_ioctl = nd_ioctl, + .compat_ioctl = nd_ioctl, + .llseek = noop_llseek, +}; + +int __init nd_bus_init(void) +{ + int rc; + + rc = register_chrdev(0, "ndctl", &nd_bus_fops); + if (rc < 0) + return rc; + nd_major = rc; + + nd_class = class_create(THIS_MODULE, "nd"); + if (IS_ERR(nd_class)) + goto err_class; + + return 0; + + err_class: + unregister_chrdev(nd_major, "ndctl"); + + return rc; +} + +void __exit nd_bus_exit(void) +{ + class_destroy(nd_class); + unregister_chrdev(nd_major, "ndctl"); +} diff --git a/drivers/block/nd/core.c b/drivers/block/nd/core.c index d126799e7ff7..d6a666b9228b 100644 --- a/drivers/block/nd/core.c +++ b/drivers/block/nd/core.c @@ -14,12 +14,15 @@ #include #include #include +#include #include #include #include #include "nd-private.h" #include "nfit.h" +LIST_HEAD(nd_bus_list); +DEFINE_MUTEX(nd_bus_list_mutex); static DEFINE_IDA(nd_ida); static bool warn_checksum; @@ -68,6 +71,53 @@ struct nd_bus *to_nd_bus(struct device *dev) return nd_bus; } +static const char *nd_bus_provider(struct nd_bus *nd_bus) +{ + struct nfit_bus_descriptor *nfit_desc = nd_bus->nfit_desc; + struct device *parent = nd_bus->dev.parent; + + if (nfit_desc->provider_name) + return nfit_desc->provider_name; + else if (parent) + return dev_name(parent); + else + return "unknown"; +} + +static ssize_t provider_show(struct device *dev, + struct device_attribute *attr, char *buf) +{ + struct nd_bus *nd_bus = to_nd_bus(dev); + + return sprintf(buf, "%s\n", nd_bus_provider(nd_bus)); +} +static DEVICE_ATTR_RO(provider); + +static ssize_t revision_show(struct device *dev, + struct device_attribute *attr, char *buf) +{ + struct nd_bus *nd_bus = to_nd_bus(dev); + struct nfit __iomem *nfit = nd_bus->nfit_desc->nfit_base; + + return sprintf(buf, "%d\n", readb(&nfit->revision)); +} +static DEVICE_ATTR_RO(revision); + +static struct attribute *nd_bus_attributes[] = { + &dev_attr_provider.attr, + &dev_attr_revision.attr, + NULL, +}; + +static struct attribute_group nd_bus_attribute_group = { + .attrs = nd_bus_attributes, +}; + +static const struct attribute_group *nd_bus_attribute_groups[] = { + &nd_bus_attribute_group, + NULL, +}; + static void *nd_bus_new(struct device *parent, struct nfit_bus_descriptor *nfit_desc) { @@ -81,6 +131,7 @@ static void *nd_bus_new(struct device *parent, INIT_LIST_HEAD(&nd_bus->bdws); INIT_LIST_HEAD(&nd_bus->memdevs); INIT_LIST_HEAD(&nd_bus->dimms); + INIT_LIST_HEAD(&nd_bus->list); nd_bus->id = ida_simple_get(&nd_ida, 0, 0, GFP_KERNEL); if (nd_bus->id < 0) { kfree(nd_bus); @@ -89,6 +140,7 @@ static void *nd_bus_new(struct device *parent, nd_bus->nfit_desc = nfit_desc; nd_bus->dev.parent = parent; nd_bus->dev.release = nd_bus_release; + nd_bus->dev.groups = nd_bus_attribute_groups; dev_set_name(&nd_bus->dev, "ndbus%d", nd_bus->id); rc = device_register(&nd_bus->dev); if (rc) { @@ -428,6 +480,14 @@ static struct nd_bus *nd_bus_probe(struct nd_bus *nd_bus) if (rc) goto err; + rc = nd_bus_create_ndctl(nd_bus); + if (rc) + goto err; + + mutex_lock(&nd_bus_list_mutex); + list_add_tail(&nd_bus->list, &nd_bus_list); + mutex_unlock(&nd_bus_list_mutex); + return nd_bus; err: put_device(&nd_bus->dev); @@ -458,6 +518,13 @@ void nfit_bus_unregister(struct nd_bus *nd_bus) { if (!nd_bus) return; + + mutex_lock(&nd_bus_list_mutex); + list_del_init(&nd_bus->list); + mutex_unlock(&nd_bus_list_mutex); + + nd_bus_destroy_ndctl(nd_bus); + device_unregister(&nd_bus->dev); } EXPORT_SYMBOL(nfit_bus_unregister); @@ -472,11 +539,13 @@ static __init int nd_core_init(void) BUILD_BUG_ON(sizeof(struct nfit_dcr) != 80); BUILD_BUG_ON(sizeof(struct nfit_bdw) != 40); - return 0; + return nd_bus_init(); } static __exit void nd_core_exit(void) { + WARN_ON(!list_empty(&nd_bus_list)); + nd_bus_exit(); } MODULE_LICENSE("GPL v2"); MODULE_AUTHOR("Intel Corporation"); diff --git a/drivers/block/nd/nd-private.h b/drivers/block/nd/nd-private.h index 0ede8818f320..4bcc9c96cb4d 100644 --- a/drivers/block/nd/nd-private.h +++ b/drivers/block/nd/nd-private.h @@ -57,5 +57,10 @@ struct nd_mem { struct nfit_spa __iomem *nfit_spa_bdw; struct list_head list; }; + struct nd_bus *to_nd_bus(struct device *dev); +int __init nd_bus_init(void); +void __exit nd_bus_exit(void); +int nd_bus_create_ndctl(struct nd_bus *nd_bus); +void nd_bus_destroy_ndctl(struct nd_bus *nd_bus); #endif /* __ND_PRIVATE_H__ */