git: d7d833e20ba3 - main - ibcore: Fix unable to change lifespan entry for hw_counters.

Hans Petter Selasky hselasky at FreeBSD.org
Mon Jul 12 13:09:32 UTC 2021


The branch main has been updated by hselasky:

URL: https://cgit.FreeBSD.org/src/commit/?id=d7d833e20ba33f5b9f3052a534af7ecdd602f152

commit d7d833e20ba33f5b9f3052a534af7ecdd602f152
Author:     Hans Petter Selasky <hselasky at FreeBSD.org>
AuthorDate: 2021-06-16 13:01:39 +0000
Commit:     Hans Petter Selasky <hselasky at FreeBSD.org>
CommitDate: 2021-07-12 12:22:30 +0000

    ibcore: Fix unable to change lifespan entry for hw_counters.
    
    This patch fixes the case where 'lifespan' entry of the hw_counters
    is not writable. Currently write callback is not exposed for for
    the hw_counters sysfs operation. Due to this, modifying lifespan
    value results into permission denied error in below example.
    
    echo 10 > /sys/class/infiniband/mlx5_0/ports/1/hw_counters/lifespan
    -bash: /sys/class/infiniband/mlx5_0/ports/1/hw_counters/lifespan:
    Permission denied
    
    This patch adds the hook to modify any attribute which implements
    store() operation.
    
    Linux commit:
    79c4d80b43b8e43684894574a508a871f0c196bf
    
    MFC after:      1 week
    Reviewed by:    kib
    Sponsored by:   Mellanox Technologies // NVIDIA Networking
---
 sys/ofed/drivers/infiniband/core/ib_sysfs.c | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/sys/ofed/drivers/infiniband/core/ib_sysfs.c b/sys/ofed/drivers/infiniband/core/ib_sysfs.c
index d12b5803e722..55e7b5395b8e 100644
--- a/sys/ofed/drivers/infiniband/core/ib_sysfs.c
+++ b/sys/ofed/drivers/infiniband/core/ib_sysfs.c
@@ -114,8 +114,22 @@ static ssize_t port_attr_show(struct kobject *kobj,
 	return port_attr->show(p, port_attr, buf);
 }
 
+static ssize_t port_attr_store(struct kobject *kobj,
+			       struct attribute *attr,
+			       const char *buf, size_t count)
+{
+	struct port_attribute *port_attr =
+		container_of(attr, struct port_attribute, attr);
+	struct ib_port *p = container_of(kobj, struct ib_port, kobj);
+
+	if (!port_attr->store)
+		return -EIO;
+	return port_attr->store(p, port_attr, buf, count);
+}
+
 static const struct sysfs_ops port_sysfs_ops = {
-	.show = port_attr_show
+	.show	= port_attr_show,
+	.store	= port_attr_store
 };
 
 static ssize_t gid_attr_show(struct kobject *kobj,


More information about the dev-commits-src-all mailing list