All of lore.kernel.org
 help / color / mirror / Atom feed
From: Logan Gunthorpe <logang@deltatee.com>
To: linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,
	linux-nvme@lists.infradead.org, linux-rdma@vger.kernel.org,
	linux-nvdimm@lists.01.org, linux-block@vger.kernel.org
Cc: "Jens Axboe" <axboe@kernel.dk>,
	"Benjamin Herrenschmidt" <benh@kernel.crashing.org>,
	"Alex Williamson" <alex.williamson@redhat.com>,
	"Keith Busch" <keith.busch@intel.com>,
	"Jérôme Glisse" <jglisse@redhat.com>,
	"Jason Gunthorpe" <jgg@mellanox.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Max Gurtovoy" <maxg@mellanox.com>,
	"Christoph Hellwig" <hch@lst.de>
Subject: [PATCH v3 02/11] PCI/P2PDMA: Add sysfs group to display p2pmem stats
Date: Mon, 12 Mar 2018 13:35:16 -0600	[thread overview]
Message-ID: <20180312193525.2855-3-logang@deltatee.com> (raw)
In-Reply-To: <20180312193525.2855-1-logang@deltatee.com>

Add a sysfs group to display statistics about P2P memory that is
registered in each PCI device.

Attributes in the group display the total amount of P2P memory, the
amount available and whether it is published or not.

Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
---
 Documentation/ABI/testing/sysfs-bus-pci | 25 +++++++++++++++
 drivers/pci/p2pdma.c                    | 54 +++++++++++++++++++++++++++++++++
 2 files changed, 79 insertions(+)

diff --git a/Documentation/ABI/testing/sysfs-bus-pci b/Documentation/ABI/testing/sysfs-bus-pci
index 44d4b2be92fd..044812c816d0 100644
--- a/Documentation/ABI/testing/sysfs-bus-pci
+++ b/Documentation/ABI/testing/sysfs-bus-pci
@@ -323,3 +323,28 @@ Description:
 
 		This is similar to /sys/bus/pci/drivers_autoprobe, but
 		affects only the VFs associated with a specific PF.
+
+What:		/sys/bus/pci/devices/.../p2pmem/available
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang@deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the amount of memory that has not been
+		allocated (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/size
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang@deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the total amount of memory that the device
+		provides (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/published
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang@deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains a '1' if the memory has been published for
+		use inside the kernel or a '0' if it is only intended
+		for use within the driver that published it.
diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
index 0ee917381dce..fd4789566a56 100644
--- a/drivers/pci/p2pdma.c
+++ b/drivers/pci/p2pdma.c
@@ -24,6 +24,54 @@ struct pci_p2pdma {
 	bool p2pmem_published;
 };
 
+static ssize_t size_show(struct device *dev, struct device_attribute *attr,
+			 char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t size = 0;
+
+	if (pdev->p2pdma->pool)
+		size = gen_pool_size(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", size);
+}
+static DEVICE_ATTR_RO(size);
+
+static ssize_t available_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t avail = 0;
+
+	if (pdev->p2pdma->pool)
+		avail = gen_pool_avail(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", avail);
+}
+static DEVICE_ATTR_RO(available);
+
+static ssize_t published_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+
+	return snprintf(buf, PAGE_SIZE, "%d\n",
+			pdev->p2pdma->p2pmem_published);
+}
+static DEVICE_ATTR_RO(published);
+
+static struct attribute *p2pmem_attrs[] = {
+	&dev_attr_size.attr,
+	&dev_attr_available.attr,
+	&dev_attr_published.attr,
+	NULL,
+};
+
+static const struct attribute_group p2pmem_group = {
+	.attrs = p2pmem_attrs,
+	.name = "p2pmem",
+};
+
 static void pci_p2pdma_percpu_release(struct percpu_ref *ref)
 {
 	struct pci_p2pdma *p2p =
@@ -53,6 +101,7 @@ static void pci_p2pdma_release(void *data)
 	percpu_ref_exit(&pdev->p2pdma->devmap_ref);
 
 	gen_pool_destroy(pdev->p2pdma->pool);
+	sysfs_remove_group(&pdev->dev.kobj, &p2pmem_group);
 	pdev->p2pdma = NULL;
 }
 
@@ -83,9 +132,14 @@ static int pci_p2pdma_setup(struct pci_dev *pdev)
 
 	pdev->p2pdma = p2p;
 
+	error = sysfs_create_group(&pdev->dev.kobj, &p2pmem_group);
+	if (error)
+		goto out_pool_destroy;
+
 	return 0;
 
 out_pool_destroy:
+	pdev->p2pdma = NULL;
 	gen_pool_destroy(p2p->pool);
 out:
 	devm_kfree(&pdev->dev, p2p);
-- 
2.11.0

_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

WARNING: multiple messages have this Message-ID (diff)
From: Logan Gunthorpe <logang@deltatee.com>
To: linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,
	linux-nvme@lists.infradead.org, linux-rdma@vger.kernel.org,
	linux-nvdimm@lists.01.org, linux-block@vger.kernel.org
Cc: "Stephen Bates" <sbates@raithlin.com>,
	"Christoph Hellwig" <hch@lst.de>, "Jens Axboe" <axboe@kernel.dk>,
	"Keith Busch" <keith.busch@intel.com>,
	"Sagi Grimberg" <sagi@grimberg.me>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Jason Gunthorpe" <jgg@mellanox.com>,
	"Max Gurtovoy" <maxg@mellanox.com>,
	"Dan Williams" <dan.j.williams@intel.com>,
	"Jérôme Glisse" <jglisse@redhat.com>,
	"Benjamin Herrenschmidt" <benh@kernel.crashing.org>,
	"Alex Williamson" <alex.williamson@redhat.com>,
	"Logan Gunthorpe" <logang@deltatee.com>
Subject: [PATCH v3 02/11] PCI/P2PDMA: Add sysfs group to display p2pmem stats
Date: Mon, 12 Mar 2018 13:35:16 -0600	[thread overview]
Message-ID: <20180312193525.2855-3-logang@deltatee.com> (raw)
In-Reply-To: <20180312193525.2855-1-logang@deltatee.com>

Add a sysfs group to display statistics about P2P memory that is
registered in each PCI device.

Attributes in the group display the total amount of P2P memory, the
amount available and whether it is published or not.

Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
---
 Documentation/ABI/testing/sysfs-bus-pci | 25 +++++++++++++++
 drivers/pci/p2pdma.c                    | 54 +++++++++++++++++++++++++++++++++
 2 files changed, 79 insertions(+)

diff --git a/Documentation/ABI/testing/sysfs-bus-pci b/Documentation/ABI/testing/sysfs-bus-pci
index 44d4b2be92fd..044812c816d0 100644
--- a/Documentation/ABI/testing/sysfs-bus-pci
+++ b/Documentation/ABI/testing/sysfs-bus-pci
@@ -323,3 +323,28 @@ Description:
 
 		This is similar to /sys/bus/pci/drivers_autoprobe, but
 		affects only the VFs associated with a specific PF.
+
+What:		/sys/bus/pci/devices/.../p2pmem/available
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang@deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the amount of memory that has not been
+		allocated (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/size
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang@deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the total amount of memory that the device
+		provides (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/published
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang@deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains a '1' if the memory has been published for
+		use inside the kernel or a '0' if it is only intended
+		for use within the driver that published it.
diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
index 0ee917381dce..fd4789566a56 100644
--- a/drivers/pci/p2pdma.c
+++ b/drivers/pci/p2pdma.c
@@ -24,6 +24,54 @@ struct pci_p2pdma {
 	bool p2pmem_published;
 };
 
+static ssize_t size_show(struct device *dev, struct device_attribute *attr,
+			 char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t size = 0;
+
+	if (pdev->p2pdma->pool)
+		size = gen_pool_size(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", size);
+}
+static DEVICE_ATTR_RO(size);
+
+static ssize_t available_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t avail = 0;
+
+	if (pdev->p2pdma->pool)
+		avail = gen_pool_avail(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", avail);
+}
+static DEVICE_ATTR_RO(available);
+
+static ssize_t published_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+
+	return snprintf(buf, PAGE_SIZE, "%d\n",
+			pdev->p2pdma->p2pmem_published);
+}
+static DEVICE_ATTR_RO(published);
+
+static struct attribute *p2pmem_attrs[] = {
+	&dev_attr_size.attr,
+	&dev_attr_available.attr,
+	&dev_attr_published.attr,
+	NULL,
+};
+
+static const struct attribute_group p2pmem_group = {
+	.attrs = p2pmem_attrs,
+	.name = "p2pmem",
+};
+
 static void pci_p2pdma_percpu_release(struct percpu_ref *ref)
 {
 	struct pci_p2pdma *p2p =
@@ -53,6 +101,7 @@ static void pci_p2pdma_release(void *data)
 	percpu_ref_exit(&pdev->p2pdma->devmap_ref);
 
 	gen_pool_destroy(pdev->p2pdma->pool);
+	sysfs_remove_group(&pdev->dev.kobj, &p2pmem_group);
 	pdev->p2pdma = NULL;
 }
 
@@ -83,9 +132,14 @@ static int pci_p2pdma_setup(struct pci_dev *pdev)
 
 	pdev->p2pdma = p2p;
 
+	error = sysfs_create_group(&pdev->dev.kobj, &p2pmem_group);
+	if (error)
+		goto out_pool_destroy;
+
 	return 0;
 
 out_pool_destroy:
+	pdev->p2pdma = NULL;
 	gen_pool_destroy(p2p->pool);
 out:
 	devm_kfree(&pdev->dev, p2p);
-- 
2.11.0

WARNING: multiple messages have this Message-ID (diff)
From: Logan Gunthorpe <logang-OTvnGxWRz7hWk0Htik3J/w@public.gmane.org>
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-nvme-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org,
	linux-block-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: "Jens Axboe" <axboe-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>,
	"Benjamin Herrenschmidt"
	<benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>,
	"Alex Williamson"
	<alex.williamson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	"Keith Busch"
	<keith.busch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	"Jérôme Glisse" <jglisse-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	"Jason Gunthorpe" <jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	"Bjorn Helgaas"
	<bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
	"Max Gurtovoy" <maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	"Christoph Hellwig" <hch-jcswGhMUV9g@public.gmane.org>
Subject: [PATCH v3 02/11] PCI/P2PDMA: Add sysfs group to display p2pmem stats
Date: Mon, 12 Mar 2018 13:35:16 -0600	[thread overview]
Message-ID: <20180312193525.2855-3-logang@deltatee.com> (raw)
In-Reply-To: <20180312193525.2855-1-logang-OTvnGxWRz7hWk0Htik3J/w@public.gmane.org>

Add a sysfs group to display statistics about P2P memory that is
registered in each PCI device.

Attributes in the group display the total amount of P2P memory, the
amount available and whether it is published or not.

Signed-off-by: Logan Gunthorpe <logang-OTvnGxWRz7hWk0Htik3J/w@public.gmane.org>
---
 Documentation/ABI/testing/sysfs-bus-pci | 25 +++++++++++++++
 drivers/pci/p2pdma.c                    | 54 +++++++++++++++++++++++++++++++++
 2 files changed, 79 insertions(+)

diff --git a/Documentation/ABI/testing/sysfs-bus-pci b/Documentation/ABI/testing/sysfs-bus-pci
index 44d4b2be92fd..044812c816d0 100644
--- a/Documentation/ABI/testing/sysfs-bus-pci
+++ b/Documentation/ABI/testing/sysfs-bus-pci
@@ -323,3 +323,28 @@ Description:
 
 		This is similar to /sys/bus/pci/drivers_autoprobe, but
 		affects only the VFs associated with a specific PF.
+
+What:		/sys/bus/pci/devices/.../p2pmem/available
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang-OTvnGxWRz7hWk0Htik3J/w@public.gmane.org>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the amount of memory that has not been
+		allocated (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/size
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang-OTvnGxWRz7hWk0Htik3J/w@public.gmane.org>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the total amount of memory that the device
+		provides (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/published
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang-OTvnGxWRz7hWk0Htik3J/w@public.gmane.org>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains a '1' if the memory has been published for
+		use inside the kernel or a '0' if it is only intended
+		for use within the driver that published it.
diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
index 0ee917381dce..fd4789566a56 100644
--- a/drivers/pci/p2pdma.c
+++ b/drivers/pci/p2pdma.c
@@ -24,6 +24,54 @@ struct pci_p2pdma {
 	bool p2pmem_published;
 };
 
+static ssize_t size_show(struct device *dev, struct device_attribute *attr,
+			 char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t size = 0;
+
+	if (pdev->p2pdma->pool)
+		size = gen_pool_size(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", size);
+}
+static DEVICE_ATTR_RO(size);
+
+static ssize_t available_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t avail = 0;
+
+	if (pdev->p2pdma->pool)
+		avail = gen_pool_avail(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", avail);
+}
+static DEVICE_ATTR_RO(available);
+
+static ssize_t published_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+
+	return snprintf(buf, PAGE_SIZE, "%d\n",
+			pdev->p2pdma->p2pmem_published);
+}
+static DEVICE_ATTR_RO(published);
+
+static struct attribute *p2pmem_attrs[] = {
+	&dev_attr_size.attr,
+	&dev_attr_available.attr,
+	&dev_attr_published.attr,
+	NULL,
+};
+
+static const struct attribute_group p2pmem_group = {
+	.attrs = p2pmem_attrs,
+	.name = "p2pmem",
+};
+
 static void pci_p2pdma_percpu_release(struct percpu_ref *ref)
 {
 	struct pci_p2pdma *p2p =
@@ -53,6 +101,7 @@ static void pci_p2pdma_release(void *data)
 	percpu_ref_exit(&pdev->p2pdma->devmap_ref);
 
 	gen_pool_destroy(pdev->p2pdma->pool);
+	sysfs_remove_group(&pdev->dev.kobj, &p2pmem_group);
 	pdev->p2pdma = NULL;
 }
 
@@ -83,9 +132,14 @@ static int pci_p2pdma_setup(struct pci_dev *pdev)
 
 	pdev->p2pdma = p2p;
 
+	error = sysfs_create_group(&pdev->dev.kobj, &p2pmem_group);
+	if (error)
+		goto out_pool_destroy;
+
 	return 0;
 
 out_pool_destroy:
+	pdev->p2pdma = NULL;
 	gen_pool_destroy(p2p->pool);
 out:
 	devm_kfree(&pdev->dev, p2p);
-- 
2.11.0

WARNING: multiple messages have this Message-ID (diff)
From: logang@deltatee.com (Logan Gunthorpe)
Subject: [PATCH v3 02/11] PCI/P2PDMA: Add sysfs group to display p2pmem stats
Date: Mon, 12 Mar 2018 13:35:16 -0600	[thread overview]
Message-ID: <20180312193525.2855-3-logang@deltatee.com> (raw)
In-Reply-To: <20180312193525.2855-1-logang@deltatee.com>

Add a sysfs group to display statistics about P2P memory that is
registered in each PCI device.

Attributes in the group display the total amount of P2P memory, the
amount available and whether it is published or not.

Signed-off-by: Logan Gunthorpe <logang at deltatee.com>
---
 Documentation/ABI/testing/sysfs-bus-pci | 25 +++++++++++++++
 drivers/pci/p2pdma.c                    | 54 +++++++++++++++++++++++++++++++++
 2 files changed, 79 insertions(+)

diff --git a/Documentation/ABI/testing/sysfs-bus-pci b/Documentation/ABI/testing/sysfs-bus-pci
index 44d4b2be92fd..044812c816d0 100644
--- a/Documentation/ABI/testing/sysfs-bus-pci
+++ b/Documentation/ABI/testing/sysfs-bus-pci
@@ -323,3 +323,28 @@ Description:
 
 		This is similar to /sys/bus/pci/drivers_autoprobe, but
 		affects only the VFs associated with a specific PF.
+
+What:		/sys/bus/pci/devices/.../p2pmem/available
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang at deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the amount of memory that has not been
+		allocated (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/size
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang at deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains the total amount of memory that the device
+		provides (in decimal).
+
+What:		/sys/bus/pci/devices/.../p2pmem/published
+Date:		November 2017
+Contact:	Logan Gunthorpe <logang at deltatee.com>
+Description:
+		If the device has any Peer-to-Peer memory registered, this
+	        file contains a '1' if the memory has been published for
+		use inside the kernel or a '0' if it is only intended
+		for use within the driver that published it.
diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
index 0ee917381dce..fd4789566a56 100644
--- a/drivers/pci/p2pdma.c
+++ b/drivers/pci/p2pdma.c
@@ -24,6 +24,54 @@ struct pci_p2pdma {
 	bool p2pmem_published;
 };
 
+static ssize_t size_show(struct device *dev, struct device_attribute *attr,
+			 char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t size = 0;
+
+	if (pdev->p2pdma->pool)
+		size = gen_pool_size(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", size);
+}
+static DEVICE_ATTR_RO(size);
+
+static ssize_t available_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+	size_t avail = 0;
+
+	if (pdev->p2pdma->pool)
+		avail = gen_pool_avail(pdev->p2pdma->pool);
+
+	return snprintf(buf, PAGE_SIZE, "%zd\n", avail);
+}
+static DEVICE_ATTR_RO(available);
+
+static ssize_t published_show(struct device *dev, struct device_attribute *attr,
+			      char *buf)
+{
+	struct pci_dev *pdev = to_pci_dev(dev);
+
+	return snprintf(buf, PAGE_SIZE, "%d\n",
+			pdev->p2pdma->p2pmem_published);
+}
+static DEVICE_ATTR_RO(published);
+
+static struct attribute *p2pmem_attrs[] = {
+	&dev_attr_size.attr,
+	&dev_attr_available.attr,
+	&dev_attr_published.attr,
+	NULL,
+};
+
+static const struct attribute_group p2pmem_group = {
+	.attrs = p2pmem_attrs,
+	.name = "p2pmem",
+};
+
 static void pci_p2pdma_percpu_release(struct percpu_ref *ref)
 {
 	struct pci_p2pdma *p2p =
@@ -53,6 +101,7 @@ static void pci_p2pdma_release(void *data)
 	percpu_ref_exit(&pdev->p2pdma->devmap_ref);
 
 	gen_pool_destroy(pdev->p2pdma->pool);
+	sysfs_remove_group(&pdev->dev.kobj, &p2pmem_group);
 	pdev->p2pdma = NULL;
 }
 
@@ -83,9 +132,14 @@ static int pci_p2pdma_setup(struct pci_dev *pdev)
 
 	pdev->p2pdma = p2p;
 
+	error = sysfs_create_group(&pdev->dev.kobj, &p2pmem_group);
+	if (error)
+		goto out_pool_destroy;
+
 	return 0;
 
 out_pool_destroy:
+	pdev->p2pdma = NULL;
 	gen_pool_destroy(p2p->pool);
 out:
 	devm_kfree(&pdev->dev, p2p);
-- 
2.11.0

  parent reply	other threads:[~2018-03-12 19:29 UTC|newest]

Thread overview: 276+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-12 19:35 [PATCH v3 00/11] Copy Offload in NVMe Fabrics with P2P PCI Memory Logan Gunthorpe
2018-03-12 19:35 ` Logan Gunthorpe
2018-03-12 19:35 ` Logan Gunthorpe
2018-03-12 19:35 ` Logan Gunthorpe
2018-03-12 19:35 ` [PATCH v3 01/11] PCI/P2PDMA: Support peer-to-peer memory Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-13  3:28   ` Sinan Kaya
2018-03-13  3:28     ` Sinan Kaya
2018-03-13  3:28     ` Sinan Kaya
2018-03-13  3:28     ` Sinan Kaya
2018-03-13 16:43     ` Logan Gunthorpe
2018-03-13 16:43       ` Logan Gunthorpe
2018-03-13 16:43       ` Logan Gunthorpe
2018-03-13 16:43       ` Logan Gunthorpe
2018-03-13 17:49       ` Sinan Kaya
2018-03-13 17:49         ` Sinan Kaya
2018-03-13 17:49         ` Sinan Kaya
2018-03-13 17:49         ` Sinan Kaya
2018-03-13 18:44         ` Logan Gunthorpe
2018-03-13 18:44           ` Logan Gunthorpe
2018-03-13 18:44           ` Logan Gunthorpe
2018-03-13 18:44           ` Logan Gunthorpe
2018-03-13 19:10           ` Sinan Kaya
2018-03-13 19:10             ` Sinan Kaya
2018-03-13 19:10             ` Sinan Kaya
2018-03-13 19:10             ` Sinan Kaya
2018-03-13 19:19             ` Logan Gunthorpe
2018-03-13 19:19               ` Logan Gunthorpe
2018-03-13 19:19               ` Logan Gunthorpe
2018-03-13 19:19               ` Logan Gunthorpe
2018-03-13 19:53               ` Sinan Kaya
2018-03-13 19:53                 ` Sinan Kaya
2018-03-13 19:53                 ` Sinan Kaya
2018-03-13 19:53                 ` Sinan Kaya
2018-03-13 20:46                 ` Logan Gunthorpe
2018-03-13 20:46                   ` Logan Gunthorpe
2018-03-13 20:46                   ` Logan Gunthorpe
2018-03-13 20:46                   ` Logan Gunthorpe
2018-03-13 21:22                   ` Sinan Kaya
2018-03-13 21:22                     ` Sinan Kaya
2018-03-13 21:22                     ` Sinan Kaya
2018-03-13 21:22                     ` Sinan Kaya
2018-03-13 22:00                     ` Logan Gunthorpe
2018-03-13 22:00                       ` Logan Gunthorpe
2018-03-13 22:00                       ` Logan Gunthorpe
2018-03-13 22:00                       ` Logan Gunthorpe
2018-03-13 22:29                       ` Sinan Kaya
2018-03-13 22:29                         ` Sinan Kaya
2018-03-13 22:29                         ` Sinan Kaya
2018-03-13 22:29                         ` Sinan Kaya
2018-03-13 22:45                         ` Stephen  Bates
2018-03-13 22:45                           ` Stephen  Bates
2018-03-13 22:45                           ` Stephen  Bates
2018-03-13 22:45                           ` Stephen  Bates
2018-03-13 22:48                         ` Logan Gunthorpe
2018-03-13 22:48                           ` Logan Gunthorpe
2018-03-13 22:48                           ` Logan Gunthorpe
2018-03-13 22:48                           ` Logan Gunthorpe
2018-03-13 23:19                           ` Sinan Kaya
2018-03-13 23:19                             ` Sinan Kaya
2018-03-13 23:19                             ` Sinan Kaya
2018-03-13 23:19                             ` Sinan Kaya
2018-03-13 23:45                             ` Logan Gunthorpe
2018-03-13 23:45                               ` Logan Gunthorpe
2018-03-13 23:45                               ` Logan Gunthorpe
2018-03-13 23:45                               ` Logan Gunthorpe
2018-03-14 12:16                               ` David Laight
2018-03-14 12:16                                 ` David Laight
2018-03-14 12:16                                 ` David Laight
2018-03-14 12:16                                 ` David Laight
2018-03-14 12:16                                 ` David Laight
2018-03-14 16:23                                 ` Logan Gunthorpe
2018-03-14 16:23                                   ` Logan Gunthorpe
2018-03-14 16:23                                   ` Logan Gunthorpe
2018-03-14 16:23                                   ` Logan Gunthorpe
2018-03-13 22:31                       ` Stephen  Bates
2018-03-13 22:31                         ` Stephen  Bates
2018-03-13 22:31                         ` Stephen  Bates
2018-03-13 22:31                         ` Stephen  Bates
2018-03-13 22:31                         ` Stephen  Bates
2018-03-13 23:08                         ` Bjorn Helgaas
2018-03-13 23:08                           ` Bjorn Helgaas
2018-03-13 23:08                           ` Bjorn Helgaas
2018-03-13 23:08                           ` Bjorn Helgaas
2018-03-13 23:21                           ` Logan Gunthorpe
2018-03-13 23:21                             ` Logan Gunthorpe
2018-03-13 23:21                             ` Logan Gunthorpe
2018-03-13 23:21                             ` Logan Gunthorpe
2018-03-14  2:56                             ` Bjorn Helgaas
2018-03-14  2:56                               ` Bjorn Helgaas
2018-03-14  2:56                               ` Bjorn Helgaas
2018-03-14  2:56                               ` Bjorn Helgaas
2018-03-14 14:05                               ` Stephen  Bates
2018-03-14 14:05                                 ` Stephen  Bates
2018-03-14 14:05                                 ` Stephen  Bates
2018-03-14 14:05                                 ` Stephen  Bates
2018-03-14 14:05                                 ` Stephen  Bates
2018-03-14 16:17                               ` Logan Gunthorpe
2018-03-14 16:17                                 ` Logan Gunthorpe
2018-03-14 16:17                                 ` Logan Gunthorpe
2018-03-14 16:17                                 ` Logan Gunthorpe
2018-03-14 18:51                                 ` Bjorn Helgaas
2018-03-14 18:51                                   ` Bjorn Helgaas
2018-03-14 18:51                                   ` Bjorn Helgaas
2018-03-14 18:51                                   ` Bjorn Helgaas
2018-03-14 19:03                                   ` Logan Gunthorpe
2018-03-14 19:03                                     ` Logan Gunthorpe
2018-03-14 19:03                                     ` Logan Gunthorpe
2018-03-14 19:03                                     ` Logan Gunthorpe
2018-03-14 19:28                                     ` Dan Williams
2018-03-14 19:28                                       ` Dan Williams
2018-03-14 19:28                                       ` Dan Williams
2018-03-14 19:28                                       ` Dan Williams
2018-03-14 19:30                                       ` Logan Gunthorpe
2018-03-14 19:30                                         ` Logan Gunthorpe
2018-03-14 19:30                                         ` Logan Gunthorpe
2018-03-14 19:30                                         ` Logan Gunthorpe
2018-03-14 19:34                                       ` Stephen  Bates
2018-03-14 19:34                                         ` Stephen  Bates
2018-03-14 19:34                                         ` Stephen  Bates
2018-03-14 19:34                                         ` Stephen  Bates
2018-03-14 19:34                                         ` Stephen  Bates
2018-03-15  4:00                                         ` Martin K. Petersen
2018-03-15  4:00                                           ` Martin K. Petersen
2018-03-15  4:00                                           ` Martin K. Petersen
2018-03-15  4:00                                           ` Martin K. Petersen
2018-03-15  4:30                                         ` Dan Williams
2018-03-15  4:30                                           ` Dan Williams
2018-03-15  4:30                                           ` Dan Williams
2018-03-15  4:30                                           ` Dan Williams
2018-03-22 22:57                           ` Stephen  Bates
2018-03-22 22:57                             ` Stephen  Bates
2018-03-22 22:57                             ` Stephen  Bates
2018-03-22 22:57                             ` Stephen  Bates
2018-03-22 22:57                             ` Stephen  Bates
2018-03-23 21:50                             ` Bjorn Helgaas
2018-03-23 21:50                               ` Bjorn Helgaas
2018-03-23 21:50                               ` Bjorn Helgaas
2018-03-23 21:50                               ` Bjorn Helgaas
2018-03-23 21:59                               ` Logan Gunthorpe
2018-03-23 21:59                                 ` Logan Gunthorpe
2018-03-23 21:59                                 ` Logan Gunthorpe
2018-03-23 21:59                                 ` Logan Gunthorpe
2018-03-24  3:49                                 ` Bjorn Helgaas
2018-03-24  3:49                                   ` Bjorn Helgaas
2018-03-24  3:49                                   ` Bjorn Helgaas
2018-03-24  3:49                                   ` Bjorn Helgaas
2018-03-24 15:28                                   ` Stephen  Bates
2018-03-24 15:28                                     ` Stephen  Bates
2018-03-24 15:28                                     ` Stephen  Bates
2018-03-24 15:28                                     ` Stephen  Bates
2018-03-24 15:28                                     ` Stephen  Bates
2018-03-26 15:43                                     ` Logan Gunthorpe
2018-03-26 15:43                                       ` Logan Gunthorpe
2018-03-26 15:43                                       ` Logan Gunthorpe
2018-03-26 15:43                                       ` Logan Gunthorpe
2018-03-26 11:11       ` Jonathan Cameron
2018-03-26 11:11         ` Jonathan Cameron
2018-03-26 11:11         ` Jonathan Cameron
2018-03-26 14:01         ` Bjorn Helgaas
2018-03-26 14:01           ` Bjorn Helgaas
2018-03-26 14:01           ` Bjorn Helgaas
2018-03-26 14:01           ` Bjorn Helgaas
2018-03-26 15:46           ` Logan Gunthorpe
2018-03-26 15:46             ` Logan Gunthorpe
2018-03-26 15:46             ` Logan Gunthorpe
2018-03-26 15:46             ` Logan Gunthorpe
2018-03-27  8:47             ` Jonathan Cameron
2018-03-27  8:47               ` Jonathan Cameron
2018-03-27  8:47               ` Jonathan Cameron
2018-03-27  8:47               ` Jonathan Cameron
2018-03-27 15:37               ` Logan Gunthorpe
2018-03-27 15:37                 ` Logan Gunthorpe
2018-03-27 15:37                 ` Logan Gunthorpe
2018-03-27 15:37                 ` Logan Gunthorpe
2018-04-13 21:56               ` Stephen  Bates
2018-04-13 21:56                 ` Stephen  Bates
2018-04-13 21:56                 ` Stephen  Bates
2018-04-13 21:56                 ` Stephen  Bates
2018-04-13 21:56                 ` Stephen  Bates
2018-03-26 16:41         ` Jason Gunthorpe
2018-03-26 16:41           ` Jason Gunthorpe
2018-03-26 16:41           ` Jason Gunthorpe
2018-03-26 16:41           ` Jason Gunthorpe
2018-03-26 17:30           ` Logan Gunthorpe
2018-03-26 17:30             ` Logan Gunthorpe
2018-03-26 17:30             ` Logan Gunthorpe
2018-03-26 17:30             ` Logan Gunthorpe
2018-03-26 19:35             ` Jason Gunthorpe
2018-03-26 19:35               ` Jason Gunthorpe
2018-03-26 19:35               ` Jason Gunthorpe
2018-03-26 19:35               ` Jason Gunthorpe
2018-03-26 20:42               ` Logan Gunthorpe
2018-03-26 20:42                 ` Logan Gunthorpe
2018-03-26 20:42                 ` Logan Gunthorpe
2018-03-26 20:42                 ` Logan Gunthorpe
2018-03-13 18:40     ` Logan Gunthorpe
2018-03-13 18:40       ` Logan Gunthorpe
2018-03-13 18:40       ` Logan Gunthorpe
2018-03-13 18:40       ` Logan Gunthorpe
2018-03-12 19:35 ` Logan Gunthorpe [this message]
2018-03-12 19:35   ` [PATCH v3 02/11] PCI/P2PDMA: Add sysfs group to display p2pmem stats Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35 ` [PATCH v3 03/11] PCI/P2PDMA: Add PCI p2pmem dma mappings to adjust the bus offset Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35 ` [PATCH v3 04/11] PCI/P2PDMA: Clear ACS P2P flags for all devices behind switches Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35 ` [PATCH v3 05/11] PCI/P2PDMA: Add P2P DMA driver writer's documentation Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:41   ` Jonathan Corbet
2018-03-12 19:41     ` Jonathan Corbet
2018-03-12 19:41     ` Jonathan Corbet
2018-03-12 19:41     ` Jonathan Corbet
2018-03-12 21:18     ` Logan Gunthorpe
2018-03-12 21:18       ` Logan Gunthorpe
2018-03-12 21:18       ` Logan Gunthorpe
2018-03-12 21:18       ` Logan Gunthorpe
2018-03-12 19:35 ` [PATCH v3 06/11] block: Introduce PCI P2P flags for request and request queue Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-21  9:27   ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-12 19:35 ` [PATCH v3 07/11] IB/core: Ensure we map P2P memory correctly in rdma_rw_ctx_[init|destroy]() Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-21  9:27   ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-12 19:35 ` [PATCH v3 08/11] nvme-pci: Use PCI p2pmem subsystem to manage the CMB Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-13  1:55   ` Sinan Kaya
2018-03-13  1:55     ` Sinan Kaya
2018-03-13  1:55     ` Sinan Kaya
2018-03-13  1:55     ` Sinan Kaya
2018-03-13  1:58     ` Sinan Kaya
2018-03-13  1:58       ` Sinan Kaya
2018-03-13  1:58       ` Sinan Kaya
2018-03-13  1:58       ` Sinan Kaya
2018-03-12 19:35 ` [PATCH v3 09/11] nvme-pci: Add support for P2P memory in requests Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-21  9:23   ` Christoph Hellwig
2018-03-21  9:23     ` Christoph Hellwig
2018-03-21  9:23     ` Christoph Hellwig
2018-03-21  9:23     ` Christoph Hellwig
2018-03-12 19:35 ` [PATCH v3 10/11] nvme-pci: Add a quirk for a pseudo CMB Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35 ` [PATCH v3 11/11] nvmet: Optionally use PCI P2P memory Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-12 19:35   ` Logan Gunthorpe
2018-03-21  9:27   ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21  9:27     ` Christoph Hellwig
2018-03-21 16:52     ` Logan Gunthorpe
2018-03-21 16:52       ` Logan Gunthorpe
2018-03-21 16:52       ` Logan Gunthorpe
2018-03-21 16:52       ` Logan Gunthorpe

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180312193525.2855-3-logang@deltatee.com \
    --to=logang@deltatee.com \
    --cc=alex.williamson@redhat.com \
    --cc=axboe@kernel.dk \
    --cc=benh@kernel.crashing.org \
    --cc=bhelgaas@google.com \
    --cc=hch@lst.de \
    --cc=jgg@mellanox.com \
    --cc=jglisse@redhat.com \
    --cc=keith.busch@intel.com \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nvdimm@lists.01.org \
    --cc=linux-nvme@lists.infradead.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=maxg@mellanox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.