[PATCH 01/23] configfs: add show and store methods to struct configfs_attribute

2015-10-03 Thread Christoph Hellwig 
Add methods to struct configfs_attribute to directly show and store
attributes without adding boilerplate code to every user.  In addition
to the methods this also adds 3 helper macros to define read/write,
read-only and write-only attributes with a single line of code.

Signed-off-by: Christoph Hellwig 
Reviewed-by: Nicholas Bellinger 
Acked-by: Greg Kroah-Hartman 
---
 fs/configfs/file.c   | 17 -
 include/linux/configfs.h | 27 +++
 2 files changed, 39 insertions(+), 5 deletions(-)

diff --git a/fs/configfs/file.c b/fs/configfs/file.c
index 403269f..106ca58 100644
--- a/fs/configfs/file.c
+++ b/fs/configfs/file.c
@@ -74,7 +74,11 @@ static int fill_read_buffer(struct dentry * dentry, struct 
configfs_buffer * buf
if (!buffer->page)
return -ENOMEM;
 
-   count = ops->show_attribute(item,attr,buffer->page);
+   if (ops->show_attribute)
+   count = ops->show_attribute(item, attr, buffer->page);
+   else
+   count = attr->show(item, buffer->page);
+
buffer->needs_read_fill = 0;
BUG_ON(count > (ssize_t)SIMPLE_ATTR_SIZE);
if (count >= 0)
@@ -173,7 +177,9 @@ flush_write_buffer(struct dentry * dentry, struct 
configfs_buffer * buffer, size
struct config_item * item = to_item(dentry->d_parent);
struct configfs_item_operations * ops = buffer->ops;
 
-   return ops->store_attribute(item,attr,buffer->page,count);
+   if (ops->store_attribute)
+   return ops->store_attribute(item, attr, buffer->page, count);
+   return attr->store(item, buffer->page, count);
 }
 
 
@@ -237,8 +243,8 @@ static int check_perm(struct inode * inode, struct file * 
file)
 * and we must have a store method.
 */
if (file->f_mode & FMODE_WRITE) {
-
-   if (!(inode->i_mode & S_IWUGO) || !ops->store_attribute)
+   if (!(inode->i_mode & S_IWUGO) ||
+   (!ops->store_attribute && !attr->store))
goto Eaccess;
 
}
@@ -248,7 +254,8 @@ static int check_perm(struct inode * inode, struct file * 
file)
 * must be a show method for it.
 */
if (file->f_mode & FMODE_READ) {
-   if (!(inode->i_mode & S_IRUGO) || !ops->show_attribute)
+   if (!(inode->i_mode & S_IRUGO) ||
+   (!ops->show_attribute && !attr->show))
goto Eaccess;
}
 
diff --git a/include/linux/configfs.h b/include/linux/configfs.h
index 63a36e8..85e9956 100644
--- a/include/linux/configfs.h
+++ b/include/linux/configfs.h
@@ -125,8 +125,35 @@ struct configfs_attribute {
const char  *ca_name;
struct module   *ca_owner;
umode_t ca_mode;
+   ssize_t (*show)(struct config_item *, char *);
+   ssize_t (*store)(struct config_item *, const char *, size_t);
 };
 
+#define CONFIGFS_ATTR(_pfx, _name) \
+static struct configfs_attribute _pfx##attr_##_name = {\
+   .ca_name= __stringify(_name),   \
+   .ca_mode= S_IRUGO | S_IWUSR,\
+   .ca_owner   = THIS_MODULE,  \
+   .show   = _pfx##_name##_show,   \
+   .store  = _pfx##_name##_store,  \
+}
+
+#define CONFIGFS_ATTR_RO(_pfx, _name)  \
+static struct configfs_attribute _pfx##attr_##_name = {\
+   .ca_name= __stringify(_name),   \
+   .ca_mode= S_IRUGO,  \
+   .ca_owner   = THIS_MODULE,  \
+   .show   = _pfx##_name##_show,   \
+}
+
+#define CONFIGFS_ATTR_WO(_pfx, _name)  \
+static struct configfs_attribute _pfx##attr_##_name = {\
+   .ca_name= __stringify(_name),   \
+   .ca_mode= S_IWUSR,  \
+   .ca_owner   = THIS_MODULE,  \
+   .store  = _pfx##_name##_store,  \
+}
+
 /*
  * Users often need to create attribute structures for their configurable
  * attributes, containing a configfs_attribute member and function pointers
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 01/23] configfs: add show and store methods to struct configfs_attribute

2015-09-26 Thread Nicholas A. Bellinger 
On Fri, 2015-09-25 at 06:49 -0700, Christoph Hellwig wrote:
> Add methods to struct configfs_attribute to directly show and store
> attributes without adding boilerplate code to every user.  In addition
> to the methods this also adds 3 helper macros to define read/write,
> read-only and write-only attributes with a single line of code.
> 
> Signed-off-by: Christoph Hellwig 
> ---
>  fs/configfs/file.c   | 17 -
>  include/linux/configfs.h | 27 +++
>  2 files changed, 39 insertions(+), 5 deletions(-)
> 

Reviewed-by: Nicholas Bellinger 

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 01/23] configfs: add show and store methods to struct configfs_attribute

2015-09-25 Thread Greg KH 
On Fri, Sep 25, 2015 at 10:28:57AM -0400, Tejun Heo wrote:
> Hello, Christoph.
> 
> On Fri, Sep 25, 2015 at 06:49:38AM -0700, Christoph Hellwig wrote:
> > Add methods to struct configfs_attribute to directly show and store
> > attributes without adding boilerplate code to every user.  In addition
> > to the methods this also adds 3 helper macros to define read/write,
> > read-only and write-only attributes with a single line of code.
> 
> This looks great to me.  It doesn't look like the whole attribute
> thing played out as hoped.  It'd be great if we can move away from it.
> 
> Thanks a lot for the work!

Looks good to me as well, no objection from my side for the USB or core
configfs stuff:

Acked-by: Greg Kroah-Hartman 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 01/23] configfs: add show and store methods to struct configfs_attribute

2015-09-25 Thread Christoph Hellwig 
Add methods to struct configfs_attribute to directly show and store
attributes without adding boilerplate code to every user.  In addition
to the methods this also adds 3 helper macros to define read/write,
read-only and write-only attributes with a single line of code.

Signed-off-by: Christoph Hellwig 
---
 fs/configfs/file.c   | 17 -
 include/linux/configfs.h | 27 +++
 2 files changed, 39 insertions(+), 5 deletions(-)

diff --git a/fs/configfs/file.c b/fs/configfs/file.c
index 403269f..106ca58 100644
--- a/fs/configfs/file.c
+++ b/fs/configfs/file.c
@@ -74,7 +74,11 @@ static int fill_read_buffer(struct dentry * dentry, struct 
configfs_buffer * buf
if (!buffer->page)
return -ENOMEM;
 
-   count = ops->show_attribute(item,attr,buffer->page);
+   if (ops->show_attribute)
+   count = ops->show_attribute(item, attr, buffer->page);
+   else
+   count = attr->show(item, buffer->page);
+
buffer->needs_read_fill = 0;
BUG_ON(count > (ssize_t)SIMPLE_ATTR_SIZE);
if (count >= 0)
@@ -173,7 +177,9 @@ flush_write_buffer(struct dentry * dentry, struct 
configfs_buffer * buffer, size
struct config_item * item = to_item(dentry->d_parent);
struct configfs_item_operations * ops = buffer->ops;
 
-   return ops->store_attribute(item,attr,buffer->page,count);
+   if (ops->store_attribute)
+   return ops->store_attribute(item, attr, buffer->page, count);
+   return attr->store(item, buffer->page, count);
 }
 
 
@@ -237,8 +243,8 @@ static int check_perm(struct inode * inode, struct file * 
file)
 * and we must have a store method.
 */
if (file->f_mode & FMODE_WRITE) {
-
-   if (!(inode->i_mode & S_IWUGO) || !ops->store_attribute)
+   if (!(inode->i_mode & S_IWUGO) ||
+   (!ops->store_attribute && !attr->store))
goto Eaccess;
 
}
@@ -248,7 +254,8 @@ static int check_perm(struct inode * inode, struct file * 
file)
 * must be a show method for it.
 */
if (file->f_mode & FMODE_READ) {
-   if (!(inode->i_mode & S_IRUGO) || !ops->show_attribute)
+   if (!(inode->i_mode & S_IRUGO) ||
+   (!ops->show_attribute && !attr->show))
goto Eaccess;
}
 
diff --git a/include/linux/configfs.h b/include/linux/configfs.h
index 63a36e8..85e9956 100644
--- a/include/linux/configfs.h
+++ b/include/linux/configfs.h
@@ -125,8 +125,35 @@ struct configfs_attribute {
const char  *ca_name;
struct module   *ca_owner;
umode_t ca_mode;
+   ssize_t (*show)(struct config_item *, char *);
+   ssize_t (*store)(struct config_item *, const char *, size_t);
 };
 
+#define CONFIGFS_ATTR(_pfx, _name) \
+static struct configfs_attribute _pfx##attr_##_name = {\
+   .ca_name= __stringify(_name),   \
+   .ca_mode= S_IRUGO | S_IWUSR,\
+   .ca_owner   = THIS_MODULE,  \
+   .show   = _pfx##_name##_show,   \
+   .store  = _pfx##_name##_store,  \
+}
+
+#define CONFIGFS_ATTR_RO(_pfx, _name)  \
+static struct configfs_attribute _pfx##attr_##_name = {\
+   .ca_name= __stringify(_name),   \
+   .ca_mode= S_IRUGO,  \
+   .ca_owner   = THIS_MODULE,  \
+   .show   = _pfx##_name##_show,   \
+}
+
+#define CONFIGFS_ATTR_WO(_pfx, _name)  \
+static struct configfs_attribute _pfx##attr_##_name = {\
+   .ca_name= __stringify(_name),   \
+   .ca_mode= S_IWUSR,  \
+   .ca_owner   = THIS_MODULE,  \
+   .store  = _pfx##_name##_store,  \
+}
+
 /*
  * Users often need to create attribute structures for their configurable
  * attributes, containing a configfs_attribute member and function pointers
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 01/23] configfs: add show and store methods to struct configfs_attribute

2015-09-25 Thread Tejun Heo 
Hello, Christoph.

On Fri, Sep 25, 2015 at 06:49:38AM -0700, Christoph Hellwig wrote:
> Add methods to struct configfs_attribute to directly show and store
> attributes without adding boilerplate code to every user.  In addition
> to the methods this also adds 3 helper macros to define read/write,
> read-only and write-only attributes with a single line of code.

This looks great to me.  It doesn't look like the whole attribute
thing played out as hoped.  It'd be great if we can move away from it.

Thanks a lot for the work!

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html