diff options
| author | Christoph Hellwig <hch@lst.de> | 2015-10-03 09:32:59 -0400 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2015-10-14 01:17:57 -0400 |
| commit | 517982229f78b2aebf00a8a337e84e8eeea70b8e (patch) | |
| tree | 6f5f093837a26d5b56874689234dc818951779ac | |
| parent | 45b997737a8025be2825e464e9e9dd5d07160dc3 (diff) | |
configfs: remove old API
Remove the old show_attribute and store_attribute methods and update
the documentation. Also replace the two C samples with a single new
one in the proper samples directory where people expect to find it.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
| -rw-r--r-- | Documentation/filesystems/Makefile | 2 | ||||
| -rw-r--r-- | Documentation/filesystems/configfs/Makefile | 3 | ||||
| -rw-r--r-- | Documentation/filesystems/configfs/configfs.txt | 38 | ||||
| -rw-r--r-- | Documentation/filesystems/configfs/configfs_example_explicit.c | 483 | ||||
| -rw-r--r-- | fs/configfs/file.c | 15 | ||||
| -rw-r--r-- | include/linux/configfs.h | 82 | ||||
| -rw-r--r-- | samples/Kconfig | 6 | ||||
| -rw-r--r-- | samples/Makefile | 3 | ||||
| -rw-r--r-- | samples/configfs/Makefile | 2 | ||||
| -rw-r--r-- | samples/configfs/configfs_sample.c (renamed from Documentation/filesystems/configfs/configfs_example_macros.c) | 140 |
10 files changed, 73 insertions, 701 deletions
diff --git a/Documentation/filesystems/Makefile b/Documentation/filesystems/Makefile index 13483d192ebb..883010ce5e35 100644 --- a/Documentation/filesystems/Makefile +++ b/Documentation/filesystems/Makefile | |||
| @@ -1,5 +1,3 @@ | |||
| 1 | subdir-y := configfs | ||
| 2 | |||
| 3 | # List of programs to build | 1 | # List of programs to build |
| 4 | hostprogs-y := dnotify_test | 2 | hostprogs-y := dnotify_test |
| 5 | 3 | ||
diff --git a/Documentation/filesystems/configfs/Makefile b/Documentation/filesystems/configfs/Makefile deleted file mode 100644 index be7ec5e67dbc..000000000000 --- a/Documentation/filesystems/configfs/Makefile +++ /dev/null | |||
| @@ -1,3 +0,0 @@ | |||
| 1 | ifneq ($(CONFIG_CONFIGFS_FS),) | ||
| 2 | obj-m += configfs_example_explicit.o configfs_example_macros.o | ||
| 3 | endif | ||
diff --git a/Documentation/filesystems/configfs/configfs.txt b/Documentation/filesystems/configfs/configfs.txt index b40fec9d3f53..af68efdbbfad 100644 --- a/Documentation/filesystems/configfs/configfs.txt +++ b/Documentation/filesystems/configfs/configfs.txt | |||
| @@ -160,12 +160,6 @@ among other things. For that, it needs a type. | |||
| 160 | 160 | ||
| 161 | struct configfs_item_operations { | 161 | struct configfs_item_operations { |
| 162 | void (*release)(struct config_item *); | 162 | void (*release)(struct config_item *); |
| 163 | ssize_t (*show_attribute)(struct config_item *, | ||
| 164 | struct configfs_attribute *, | ||
| 165 | char *); | ||
| 166 | ssize_t (*store_attribute)(struct config_item *, | ||
| 167 | struct configfs_attribute *, | ||
| 168 | const char *, size_t); | ||
| 169 | int (*allow_link)(struct config_item *src, | 163 | int (*allow_link)(struct config_item *src, |
| 170 | struct config_item *target); | 164 | struct config_item *target); |
| 171 | int (*drop_link)(struct config_item *src, | 165 | int (*drop_link)(struct config_item *src, |
| @@ -183,9 +177,7 @@ The most basic function of a config_item_type is to define what | |||
| 183 | operations can be performed on a config_item. All items that have been | 177 | operations can be performed on a config_item. All items that have been |
| 184 | allocated dynamically will need to provide the ct_item_ops->release() | 178 | allocated dynamically will need to provide the ct_item_ops->release() |
| 185 | method. This method is called when the config_item's reference count | 179 | method. This method is called when the config_item's reference count |
| 186 | reaches zero. Items that wish to display an attribute need to provide | 180 | reaches zero. |
| 187 | the ct_item_ops->show_attribute() method. Similarly, storing a new | ||
| 188 | attribute value uses the store_attribute() method. | ||
| 189 | 181 | ||
| 190 | [struct configfs_attribute] | 182 | [struct configfs_attribute] |
| 191 | 183 | ||
| @@ -193,6 +185,8 @@ attribute value uses the store_attribute() method. | |||
| 193 | char *ca_name; | 185 | char *ca_name; |
| 194 | struct module *ca_owner; | 186 | struct module *ca_owner; |
| 195 | umode_t ca_mode; | 187 | umode_t ca_mode; |
| 188 | ssize_t (*show)(struct config_item *, char *); | ||
| 189 | ssize_t (*store)(struct config_item *, const char *, size_t); | ||
| 196 | }; | 190 | }; |
| 197 | 191 | ||
| 198 | When a config_item wants an attribute to appear as a file in the item's | 192 | When a config_item wants an attribute to appear as a file in the item's |
| @@ -202,10 +196,10 @@ config_item_type->ct_attrs. When the item appears in configfs, the | |||
| 202 | attribute file will appear with the configfs_attribute->ca_name | 196 | attribute file will appear with the configfs_attribute->ca_name |
| 203 | filename. configfs_attribute->ca_mode specifies the file permissions. | 197 | filename. configfs_attribute->ca_mode specifies the file permissions. |
| 204 | 198 | ||
| 205 | If an attribute is readable and the config_item provides a | 199 | If an attribute is readable and provides a ->show method, that method will |
| 206 | ct_item_ops->show_attribute() method, that method will be called | 200 | be called whenever userspace asks for a read(2) on the attribute. If an |
| 207 | whenever userspace asks for a read(2) on the attribute. The converse | 201 | attribute is writable and provides a ->store method, that method will be |
| 208 | will happen for write(2). | 202 | be called whenever userspace asks for a write(2) on the attribute. |
| 209 | 203 | ||
| 210 | [struct config_group] | 204 | [struct config_group] |
| 211 | 205 | ||
| @@ -311,20 +305,10 @@ the subsystem must be ready for it. | |||
| 311 | [An Example] | 305 | [An Example] |
| 312 | 306 | ||
| 313 | The best example of these basic concepts is the simple_children | 307 | The best example of these basic concepts is the simple_children |
| 314 | subsystem/group and the simple_child item in configfs_example_explicit.c | 308 | subsystem/group and the simple_child item in |
| 315 | and configfs_example_macros.c. It shows a trivial object displaying and | 309 | samples/configfs/configfs_sample.c. It shows a trivial object displaying |
| 316 | storing an attribute, and a simple group creating and destroying these | 310 | and storing an attribute, and a simple group creating and destroying |
| 317 | children. | 311 | these children. |
| 318 | |||
| 319 | The only difference between configfs_example_explicit.c and | ||
| 320 | configfs_example_macros.c is how the attributes of the childless item | ||
| 321 | are defined. The childless item has extended attributes, each with | ||
| 322 | their own show()/store() operation. This follows a convention commonly | ||
| 323 | used in sysfs. configfs_example_explicit.c creates these attributes | ||
| 324 | by explicitly defining the structures involved. Conversely | ||
| 325 | configfs_example_macros.c uses some convenience macros from configfs.h | ||
| 326 | to define the attributes. These macros are similar to their sysfs | ||
| 327 | counterparts. | ||
| 328 | 312 | ||
| 329 | [Hierarchy Navigation and the Subsystem Mutex] | 313 | [Hierarchy Navigation and the Subsystem Mutex] |
| 330 | 314 | ||
diff --git a/Documentation/filesystems/configfs/configfs_example_explicit.c b/Documentation/filesystems/configfs/configfs_example_explicit.c deleted file mode 100644 index 1420233dfa55..000000000000 --- a/Documentation/filesystems/configfs/configfs_example_explicit.c +++ /dev/null | |||
| @@ -1,483 +0,0 @@ | |||
| 1 | /* | ||
| 2 | * vim: noexpandtab ts=8 sts=0 sw=8: | ||
| 3 | * | ||
| 4 | * configfs_example_explicit.c - This file is a demonstration module | ||
| 5 | * containing a number of configfs subsystems. It explicitly defines | ||
| 6 | * each structure without using the helper macros defined in | ||
| 7 | * configfs.h. | ||
| 8 | * | ||
| 9 | * This program is free software; you can redistribute it and/or | ||
| 10 | * modify it under the terms of the GNU General Public | ||
| 11 | * License as published by the Free Software Foundation; either | ||
| 12 | * version 2 of the License, or (at your option) any later version. | ||
| 13 | * | ||
| 14 | * This program is distributed in the hope that it will be useful, | ||
| 15 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
| 16 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | ||
| 17 | * General Public License for more details. | ||
| 18 | * | ||
| 19 | * You should have received a copy of the GNU General Public | ||
| 20 | * License along with this program; if not, write to the | ||
| 21 | * Free Software Foundation, Inc., 59 Temple Place - Suite 330, | ||
| 22 | * Boston, MA 021110-1307, USA. | ||
| 23 | * | ||
| 24 | * Based on sysfs: | ||
| 25 | * sysfs is Copyright (C) 2001, 2002, 2003 Patrick Mochel | ||
| 26 | * | ||
| 27 | * configfs Copyright (C) 2005 Oracle. All rights reserved. | ||
| 28 | */ | ||
| 29 | |||
| 30 | #include <linux/init.h> | ||
| 31 | #include <linux/module.h> | ||
| 32 | #include <linux/slab.h> | ||
| 33 | |||
| 34 | #include <linux/configfs.h> | ||
| 35 | |||
| 36 | |||
| 37 | |||
| 38 | /* | ||
| 39 | * 01-childless | ||
| 40 | * | ||
| 41 | * This first example is a childless subsystem. It cannot create | ||
| 42 | * any config_items. It just has attributes. | ||
| 43 | * | ||
| 44 | * Note that we are enclosing the configfs_subsystem inside a container. | ||
| 45 | * This is not necessary if a subsystem has no attributes directly | ||
| 46 | * on the subsystem. See the next example, 02-simple-children, for | ||
| 47 | * such a subsystem. | ||
| 48 | */ | ||
| 49 | |||
| 50 | struct childless { | ||
| 51 | struct configfs_subsystem subsys; | ||
| 52 | int showme; | ||
| 53 | int storeme; | ||
| 54 | }; | ||
| 55 | |||
| 56 | struct childless_attribute { | ||
| 57 | struct configfs_attribute attr; | ||
| 58 | ssize_t (*show)(struct childless *, char *); | ||
| 59 | ssize_t (*store)(struct childless *, const char *, size_t); | ||
| 60 | }; | ||
| 61 | |||
| 62 | static inline struct childless *to_childless(struct config_item *item) | ||
| 63 | { | ||
| 64 | return item ? container_of(to_configfs_subsystem(to_config_group(item)), struct childless, subsys) : NULL; | ||
| 65 | } | ||
| 66 | |||
| 67 | static ssize_t childless_showme_read(struct childless *childless, | ||
| 68 | char *page) | ||
| 69 | { | ||
| 70 | ssize_t pos; | ||
| 71 | |||
| 72 | pos = sprintf(page, "%d\n", childless->showme); | ||
| 73 | childless->showme++; | ||
| 74 | |||
| 75 | return pos; | ||
| 76 | } | ||
| 77 | |||
| 78 | static ssize_t childless_storeme_read(struct childless *childless, | ||
| 79 | char *page) | ||
| 80 | { | ||
| 81 | return sprintf(page, "%d\n", childless->storeme); | ||
| 82 | } | ||
| 83 | |||
| 84 | static ssize_t childless_storeme_write(struct childless *childless, | ||
| 85 | const char *page, | ||
| 86 | size_t count) | ||
| 87 | { | ||
| 88 | unsigned long tmp; | ||
| 89 | char *p = (char *) page; | ||
| 90 | |||
| 91 | tmp = simple_strtoul(p, &p, 10); | ||
| 92 | if ((*p != '\0') && (*p != '\n')) | ||
| 93 | return -EINVAL; | ||
| 94 | |||
| 95 | if (tmp > INT_MAX) | ||
| 96 | return -ERANGE; | ||
| 97 | |||
| 98 | childless->storeme = tmp; | ||
| 99 | |||
| 100 | return count; | ||
| 101 | } | ||
| 102 | |||
| 103 | static ssize_t childless_description_read(struct childless *childless, | ||
| 104 | char *page) | ||
| 105 | { | ||
| 106 | return sprintf(page, | ||
| 107 | "[01-childless]\n" | ||
| 108 | "\n" | ||
| 109 | "The childless subsystem is the simplest possible subsystem in\n" | ||
| 110 | "configfs. It does not support the creation of child config_items.\n" | ||
| 111 | "It only has a few attributes. In fact, it isn't much different\n" | ||
| 112 | "than a directory in /proc.\n"); | ||
| 113 | } | ||
| 114 | |||
| 115 | static struct childless_attribute childless_attr_showme = { | ||
| 116 | .attr = { .ca_owner = THIS_MODULE, .ca_name = "showme", .ca_mode = S_IRUGO }, | ||
| 117 | .show = childless_showme_read, | ||
| 118 | }; | ||
| 119 | static struct childless_attribute childless_attr_storeme = { | ||
| 120 | .attr = { .ca_owner = THIS_MODULE, .ca_name = "storeme", .ca_mode = S_IRUGO | S_IWUSR }, | ||
| 121 | .show = childless_storeme_read, | ||
| 122 | .store = childless_storeme_write, | ||
| 123 | }; | ||
| 124 | static struct childless_attribute childless_attr_description = { | ||
| 125 | .attr = { .ca_owner = THIS_MODULE, .ca_name = "description", .ca_mode = S_IRUGO }, | ||
| 126 | .show = childless_description_read, | ||
| 127 | }; | ||
| 128 | |||
| 129 | static struct configfs_attribute *childless_attrs[] = { | ||
| 130 | &childless_attr_showme.attr, | ||
| 131 | &childless_attr_storeme.attr, | ||
| 132 | &childless_attr_description.attr, | ||
| 133 | NULL, | ||
| 134 | }; | ||
| 135 | |||
| 136 | static ssize_t childless_attr_show(struct config_item *item, | ||
| 137 | struct configfs_attribute *attr, | ||
| 138 | char *page) | ||
| 139 | { | ||
| 140 | struct childless *childless = to_childless(item); | ||
| 141 | struct childless_attribute *childless_attr = | ||
| 142 | container_of(attr, struct childless_attribute, attr); | ||
| 143 | ssize_t ret = 0; | ||
| 144 | |||
| 145 | if (childless_attr->show) | ||
| 146 | ret = childless_attr->show(childless, page); | ||
| 147 | return ret; | ||
| 148 | } | ||
| 149 | |||
| 150 | static ssize_t childless_attr_store(struct config_item *item, | ||
| 151 | struct configfs_attribute *attr, | ||
| 152 | const char *page, size_t count) | ||
| 153 | { | ||
| 154 | struct childless *childless = to_childless(item); | ||
| 155 | struct childless_attribute *childless_attr = | ||
| 156 | container_of(attr, struct childless_attribute, attr); | ||
| 157 | ssize_t ret = -EINVAL; | ||
| 158 | |||
| 159 | if (childless_attr->store) | ||
| 160 | ret = childless_attr->store(childless, page, count); | ||
| 161 | return ret; | ||
| 162 | } | ||
| 163 | |||
| 164 | static struct configfs_item_operations childless_item_ops = { | ||
| 165 | .show_attribute = childless_attr_show, | ||
| 166 | .store_attribute = childless_attr_store, | ||
| 167 | }; | ||
| 168 | |||
| 169 | static struct config_item_type childless_type = { | ||
| 170 | .ct_item_ops = &childless_item_ops, | ||
| 171 | .ct_attrs = childless_attrs, | ||
| 172 | .ct_owner = THIS_MODULE, | ||
| 173 | }; | ||
| 174 | |||
| 175 | static struct childless childless_subsys = { | ||
| 176 | .subsys = { | ||
| 177 | .su_group = { | ||
| 178 | .cg_item = { | ||
| 179 | .ci_namebuf = "01-childless", | ||
| 180 | .ci_type = &childless_type, | ||
| 181 | }, | ||
| 182 | }, | ||
| 183 | }, | ||
| 184 | }; | ||
| 185 | |||
| 186 | |||
| 187 | /* ----------------------------------------------------------------- */ | ||
| 188 | |||
| 189 | /* | ||
| 190 | * 02-simple-children | ||
| 191 | * | ||
| 192 | * This example merely has a simple one-attribute child. Note that | ||
| 193 | * there is no extra attribute structure, as the child's attribute is | ||
| 194 | * known from the get-go. Also, there is no container for the | ||
| 195 | * subsystem, as it has no attributes of its own. | ||
| 196 | */ | ||
| 197 | |||
| 198 | struct simple_child { | ||
| 199 | struct config_item item; | ||
| 200 | int storeme; | ||
| 201 | }; | ||
| 202 | |||
| 203 | static inline struct simple_child *to_simple_child(struct config_item *item) | ||
| 204 | { | ||
| 205 | return item ? container_of(item, struct simple_child, item) : NULL; | ||
| 206 | } | ||
| 207 | |||
| 208 | static struct configfs_attribute simple_child_attr_storeme = { | ||
| 209 | .ca_owner = THIS_MODULE, | ||
| 210 | .ca_name = "storeme", | ||
| 211 | .ca_mode = S_IRUGO | S_IWUSR, | ||
| 212 | }; | ||
| 213 | |||
| 214 | static struct configfs_attribute *simple_child_attrs[] = { | ||
| 215 | &simple_child_attr_storeme, | ||
| 216 | NULL, | ||
| 217 | }; | ||
| 218 | |||
| 219 | static ssize_t simple_child_attr_show(struct config_item *item, | ||
| 220 | struct configfs_attribute *attr, | ||
| 221 | char *page) | ||
| 222 | { | ||
| 223 | ssize_t count; | ||
| 224 | struct simple_child *simple_child = to_simple_child(item); | ||
| 225 | |||
| 226 | count = sprintf(page, "%d\n", simple_child->storeme); | ||
| 227 | |||
| 228 | return count; | ||
| 229 | } | ||
| 230 | |||
| 231 | static ssize_t simple_child_attr_store(struct config_item *item, | ||
| 232 | struct configfs_attribute *attr, | ||
| 233 | const char *page, size_t count) | ||
| 234 | { | ||
| 235 | struct simple_child *simple_child = to_simple_child(item); | ||
| 236 | unsigned long tmp; | ||
| 237 | char *p = (char *) page; | ||
| 238 | |||
| 239 | tmp = simple_strtoul(p, &p, 10); | ||
| 240 | if (!p || (*p && (*p != '\n'))) | ||
| 241 | return -EINVAL; | ||
| 242 | |||
| 243 | if (tmp > INT_MAX) | ||
| 244 | return -ERANGE; | ||
| 245 | |||
| 246 | simple_child->storeme = tmp; | ||
| 247 | |||
| 248 | return count; | ||
| 249 | } | ||
| 250 | |||
| 251 | static void simple_child_release(struct config_item *item) | ||
| 252 | { | ||
| 253 | kfree(to_simple_child(item)); | ||
| 254 | } | ||
| 255 | |||
| 256 | static struct configfs_item_operations simple_child_item_ops = { | ||
| 257 | .release = simple_child_release, | ||
| 258 | .show_attribute = simple_child_attr_show, | ||
| 259 | .store_attribute = simple_child_attr_store, | ||
| 260 | }; | ||
| 261 | |||
| 262 | static struct config_item_type simple_child_type = { | ||
| 263 | .ct_item_ops = &simple_child_item_ops, | ||
| 264 | .ct_attrs = simple_child_attrs, | ||
| 265 | .ct_owner = THIS_MODULE, | ||
| 266 | }; | ||
| 267 | |||
| 268 | |||
| 269 | struct simple_children { | ||
| 270 | struct config_group group; | ||
| 271 | }; | ||
| 272 | |||
| 273 | static inline struct simple_children *to_simple_children(struct config_item *item) | ||
| 274 | { | ||
| 275 | return item ? container_of(to_config_group(item), struct simple_children, group) : NULL; | ||
| 276 | } | ||
| 277 | |||
| 278 | static struct config_item *simple_children_make_item(struct config_group *group, const char *name) | ||
| 279 | { | ||
| 280 | struct simple_child *simple_child; | ||
| 281 | |||
| 282 | simple_child = kzalloc(sizeof(struct simple_child), GFP_KERNEL); | ||
| 283 | if (!simple_child) | ||
| 284 | return ERR_PTR(-ENOMEM); | ||
| 285 | |||
| 286 | config_item_init_type_name(&simple_child->item, name, | ||
| 287 | &simple_child_type); | ||
| 288 | |||
| 289 | simple_child->storeme = 0; | ||
| 290 | |||
| 291 | return &simple_child->item; | ||
| 292 | } | ||
| 293 | |||
| 294 | static struct configfs_attribute simple_children_attr_description = { | ||
| 295 | .ca_owner = THIS_MODULE, | ||
| 296 | .ca_name = "description", | ||
| 297 | .ca_mode = S_IRUGO, | ||
| 298 | }; | ||
| 299 | |||
| 300 | static struct configfs_attribute *simple_children_attrs[] = { | ||
| 301 | &simple_children_attr_description, | ||
| 302 | NULL, | ||
| 303 | }; | ||
| 304 | |||
| 305 | static ssize_t simple_children_attr_show(struct config_item *item, | ||
| 306 | struct configfs_attribute *attr, | ||
| 307 | char *page) | ||
| 308 | { | ||
| 309 | return sprintf(page, | ||
| 310 | "[02-simple-children]\n" | ||
| 311 | "\n" | ||
| 312 | "This subsystem allows the creation of child config_items. These\n" | ||
| 313 | "items have only one attribute that is readable and writeable.\n"); | ||
| 314 | } | ||
| 315 | |||
| 316 | static void simple_children_release(struct config_item *item) | ||
| 317 | { | ||
| 318 | kfree(to_simple_children(item)); | ||
| 319 | } | ||
| 320 | |||
| 321 | static struct configfs_item_operations simple_children_item_ops = { | ||
| 322 | .release = simple_children_release, | ||
| 323 | .show_attribute = simple_children_attr_show, | ||
| 324 | }; | ||
| 325 | |||
| 326 | /* | ||
| 327 | * Note that, since no extra work is required on ->drop_item(), | ||
| 328 | * no ->drop_item() is provided. | ||
| 329 | */ | ||
| 330 | static struct configfs_group_operations simple_children_group_ops = { | ||
| 331 | .make_item = simple_children_make_item, | ||
| 332 | }; | ||
| 333 | |||
| 334 | static struct config_item_type simple_children_type = { | ||
| 335 | .ct_item_ops = &simple_children_item_ops, | ||
| 336 | .ct_group_ops = &simple_children_group_ops, | ||
| 337 | .ct_attrs = simple_children_attrs, | ||
| 338 | .ct_owner = THIS_MODULE, | ||
| 339 | }; | ||
| 340 | |||
| 341 | static struct configfs_subsystem simple_children_subsys = { | ||
| 342 | .su_group = { | ||
| 343 | .cg_item = { | ||
| 344 | .ci_namebuf = "02-simple-children", | ||
| 345 | .ci_type = &simple_children_type, | ||
| 346 | }, | ||
| 347 | }, | ||
| 348 | }; | ||
| 349 | |||
| 350 | |||
| 351 | /* ----------------------------------------------------------------- */ | ||
| 352 | |||
| 353 | /* | ||
| 354 | * 03-group-children | ||
| 355 | * | ||
| 356 | * This example reuses the simple_children group from above. However, | ||
| 357 | * the simple_children group is not the subsystem itself, it is a | ||
| 358 | * child of the subsystem. Creation of a group in the subsystem creates | ||
| 359 | * a new simple_children group. That group can then have simple_child | ||
| 360 | * children of its own. | ||
| 361 | */ | ||
| 362 | |||
| 363 | static struct config_group *group_children_make_group(struct config_group *group, const char *name) | ||
| 364 | { | ||
| 365 | struct simple_children *simple_children; | ||
| 366 | |||
| 367 | simple_children = kzalloc(sizeof(struct simple_children), | ||
| 368 | GFP_KERNEL); | ||
| 369 | if (!simple_children) | ||
| 370 | return ERR_PTR(-ENOMEM); | ||
| 371 | |||
| 372 | config_group_init_type_name(&simple_children->group, name, | ||
| 373 | &simple_children_type); | ||
| 374 | |||
| 375 | return &simple_children->group; | ||
| 376 | } | ||
| 377 | |||
| 378 | static struct configfs_attribute group_children_attr_description = { | ||
| 379 | .ca_owner = THIS_MODULE, | ||
| 380 | .ca_name = "description", | ||
| 381 | .ca_mode = S_IRUGO, | ||
| 382 | }; | ||
| 383 | |||
| 384 | static struct configfs_attribute *group_children_attrs[] = { | ||
| 385 | &group_children_attr_description, | ||
| 386 | NULL, | ||
| 387 | }; | ||
| 388 | |||
| 389 | static ssize_t group_children_attr_show(struct config_item *item, | ||
| 390 | struct configfs_attribute *attr, | ||
| 391 | char *page) | ||
| 392 | { | ||
| 393 | return sprintf(page, | ||
| 394 | "[03-group-children]\n" | ||
| 395 | "\n" | ||
| 396 | "This subsystem allows the creation of child config_groups. These\n" | ||
| 397 | "groups are like the subsystem simple-children.\n"); | ||
| 398 | } | ||
| 399 | |||
| 400 | static struct configfs_item_operations group_children_item_ops = { | ||
| 401 | .show_attribute = group_children_attr_show, | ||
| 402 | }; | ||
| 403 | |||
| 404 | /* | ||
| 405 | * Note that, since no extra work is required on ->drop_item(), | ||
| 406 | * no ->drop_item() is provided. | ||
| 407 | */ | ||
| 408 | static struct configfs_group_operations group_children_group_ops = { | ||
| 409 | .make_group = group_children_make_group, | ||
| 410 | }; | ||
| 411 | |||
| 412 | static struct config_item_type group_children_type = { | ||
| 413 | .ct_item_ops = &group_children_item_ops, | ||
| 414 | .ct_group_ops = &group_children_group_ops, | ||
| 415 | .ct_attrs = group_children_attrs, | ||
| 416 | .ct_owner = THIS_MODULE, | ||
| 417 | }; | ||
| 418 | |||
| 419 | static struct configfs_subsystem group_children_subsys = { | ||
| 420 | .su_group = { | ||
| 421 | .cg_item = { | ||
| 422 | .ci_namebuf = "03-group-children", | ||
| 423 | .ci_type = &group_children_type, | ||
| 424 | }, | ||
| 425 | }, | ||
| 426 | }; | ||
| 427 | |||
| 428 | /* ----------------------------------------------------------------- */ | ||
| 429 | |||
| 430 | /* | ||
| 431 | * We're now done with our subsystem definitions. | ||
| 432 | * For convenience in this module, here's a list of them all. It | ||
| 433 | * allows the init function to easily register them. Most modules | ||
| 434 | * will only have one subsystem, and will only call register_subsystem | ||
| 435 | * on it directly. | ||
| 436 | */ | ||
| 437 | static struct configfs_subsystem *example_subsys[] = { | ||
| 438 | &childless_subsys.subsys, | ||
| 439 | &simple_children_subsys, | ||
| 440 | &group_children_subsys, | ||
| 441 | NULL, | ||
| 442 | }; | ||
| 443 | |||
| 444 | static int __init configfs_example_init(void) | ||
| 445 | { | ||
| 446 | int ret; | ||
| 447 | int i; | ||
| 448 | struct configfs_subsystem *subsys; | ||
| 449 | |||
| 450 | for (i = 0; example_subsys[i]; i++) { | ||
| 451 | subsys = example_subsys[i]; | ||
| 452 | |||
| 453 | config_group_init(&subsys->su_group); | ||
| 454 | mutex_init(&subsys->su_mutex); | ||
| 455 | ret = configfs_register_subsystem(subsys); | ||
| 456 | if (ret) { | ||
| 457 | printk(KERN_ERR "Error %d while registering subsystem %s\n", | ||
| 458 | ret, | ||
| 459 | subsys->su_group.cg_item.ci_namebuf); | ||
| 460 | goto out_unregister; | ||
| 461 | } | ||
| 462 | } | ||
| 463 | |||
| 464 | return 0; | ||
| 465 | |||
| 466 | out_unregister: | ||
| 467 | for (i--; i >= 0; i--) | ||
| 468 | configfs_unregister_subsystem(example_subsys[i]); | ||
| 469 | |||
| 470 | return ret; | ||
| 471 | } | ||
| 472 | |||
| 473 | static void __exit configfs_example_exit(void) | ||
| 474 | { | ||
| 475 | int i; | ||
| 476 | |||
| 477 | for (i = 0; example_subsys[i]; i++) | ||
| 478 | configfs_unregister_subsystem(example_subsys[i]); | ||
| 479 | } | ||
| 480 | |||
| 481 | module_init(configfs_example_init); | ||
| 482 | module_exit(configfs_example_exit); | ||
| 483 | MODULE_LICENSE("GPL"); | ||
diff --git a/fs/configfs/file.c b/fs/configfs/file.c index 106ca589e90a..d39099ea7df7 100644 --- a/fs/configfs/file.c +++ b/fs/configfs/file.c | |||
| @@ -65,7 +65,6 @@ static int fill_read_buffer(struct dentry * dentry, struct configfs_buffer * buf | |||
| 65 | { | 65 | { |
| 66 | struct configfs_attribute * attr = to_attr(dentry); | 66 | struct configfs_attribute * attr = to_attr(dentry); |
| 67 | struct config_item * item = to_item(dentry->d_parent); | 67 | struct config_item * item = to_item(dentry->d_parent); |
| 68 | struct configfs_item_operations * ops = buffer->ops; | ||
| 69 | int ret = 0; | 68 | int ret = 0; |
| 70 | ssize_t count; | 69 | ssize_t count; |
| 71 | 70 | ||
| @@ -74,10 +73,7 @@ static int fill_read_buffer(struct dentry * dentry, struct configfs_buffer * buf | |||
| 74 | if (!buffer->page) | 73 | if (!buffer->page) |
| 75 | return -ENOMEM; | 74 | return -ENOMEM; |
| 76 | 75 | ||
| 77 | if (ops->show_attribute) | 76 | count = attr->show(item, buffer->page); |
| 78 | count = ops->show_attribute(item, attr, buffer->page); | ||
| 79 | else | ||
| 80 | count = attr->show(item, buffer->page); | ||
| 81 | 77 | ||
| 82 | buffer->needs_read_fill = 0; | 78 | buffer->needs_read_fill = 0; |
| 83 | BUG_ON(count > (ssize_t)SIMPLE_ATTR_SIZE); | 79 | BUG_ON(count > (ssize_t)SIMPLE_ATTR_SIZE); |
| @@ -175,10 +171,7 @@ flush_write_buffer(struct dentry * dentry, struct configfs_buffer * buffer, size | |||
| 175 | { | 171 | { |
| 176 | struct configfs_attribute * attr = to_attr(dentry); | 172 | struct configfs_attribute * attr = to_attr(dentry); |
| 177 | struct config_item * item = to_item(dentry->d_parent); | 173 | struct config_item * item = to_item(dentry->d_parent); |
| 178 | struct configfs_item_operations * ops = buffer->ops; | ||
| 179 | 174 | ||
| 180 | if (ops->store_attribute) | ||
| 181 | return ops->store_attribute(item, attr, buffer->page, count); | ||
| 182 | return attr->store(item, buffer->page, count); | 175 | return attr->store(item, buffer->page, count); |
| 183 | } | 176 | } |
| 184 | 177 | ||
| @@ -243,8 +236,7 @@ static int check_perm(struct inode * inode, struct file * file) | |||
| 243 | * and we must have a store method. | 236 | * and we must have a store method. |
| 244 | */ | 237 | */ |
| 245 | if (file->f_mode & FMODE_WRITE) { | 238 | if (file->f_mode & FMODE_WRITE) { |
| 246 | if (!(inode->i_mode & S_IWUGO) || | 239 | if (!(inode->i_mode & S_IWUGO) || !attr->store) |
| 247 | (!ops->store_attribute && !attr->store)) | ||
| 248 | goto Eaccess; | 240 | goto Eaccess; |
| 249 | 241 | ||
| 250 | } | 242 | } |
| @@ -254,8 +246,7 @@ static int check_perm(struct inode * inode, struct file * file) | |||
| 254 | * must be a show method for it. | 246 | * must be a show method for it. |
| 255 | */ | 247 | */ |
| 256 | if (file->f_mode & FMODE_READ) { | 248 | if (file->f_mode & FMODE_READ) { |
| 257 | if (!(inode->i_mode & S_IRUGO) || | 249 | if (!(inode->i_mode & S_IRUGO) || !attr->show) |
| 258 | (!ops->show_attribute && !attr->show)) | ||
| 259 | goto Eaccess; | 250 | goto Eaccess; |
| 260 | } | 251 | } |
| 261 | 252 | ||
diff --git a/include/linux/configfs.h b/include/linux/configfs.h index 85e9956a86de..a8a335b7fce0 100644 --- a/include/linux/configfs.h +++ b/include/linux/configfs.h | |||
| @@ -155,86 +155,6 @@ static struct configfs_attribute _pfx##attr_##_name = { \ | |||
| 155 | } | 155 | } |
| 156 | 156 | ||
| 157 | /* | 157 | /* |
| 158 | * Users often need to create attribute structures for their configurable | ||
| 159 | * attributes, containing a configfs_attribute member and function pointers | ||
| 160 | * for the show() and store() operations on that attribute. If they don't | ||
| 161 | * need anything else on the extended attribute structure, they can use | ||
| 162 | * this macro to define it The argument _item is the name of the | ||
| 163 | * config_item structure. | ||
| 164 | */ | ||
| 165 | #define CONFIGFS_ATTR_STRUCT(_item) \ | ||
| 166 | struct _item##_attribute { \ | ||
| 167 | struct configfs_attribute attr; \ | ||
| 168 | ssize_t (*show)(struct _item *, char *); \ | ||
| 169 | ssize_t (*store)(struct _item *, const char *, size_t); \ | ||
| 170 | } | ||
| 171 | |||
| 172 | /* | ||
| 173 | * With the extended attribute structure, users can use this macro | ||
| 174 | * (similar to sysfs' __ATTR) to make defining attributes easier. | ||
| 175 | * An example: | ||
| 176 | * #define MYITEM_ATTR(_name, _mode, _show, _store) \ | ||
| 177 | * struct myitem_attribute childless_attr_##_name = \ | ||
| 178 | * __CONFIGFS_ATTR(_name, _mode, _show, _store) | ||
| 179 | */ | ||
| 180 | #define __CONFIGFS_ATTR(_name, _mode, _show, _store) \ | ||
| 181 | { \ | ||
| 182 | .attr = { \ | ||
| 183 | .ca_name = __stringify(_name), \ | ||
| 184 | .ca_mode = _mode, \ | ||
| 185 | .ca_owner = THIS_MODULE, \ | ||
| 186 | }, \ | ||
| 187 | .show = _show, \ | ||
| 188 | .store = _store, \ | ||
| 189 | } | ||
| 190 | /* Here is a readonly version, only requiring a show() operation */ | ||
| 191 | #define __CONFIGFS_ATTR_RO(_name, _show) \ | ||
| 192 | { \ | ||
| 193 | .attr = { \ | ||
| 194 | .ca_name = __stringify(_name), \ | ||
| 195 | .ca_mode = 0444, \ | ||
| 196 | .ca_owner = THIS_MODULE, \ | ||
| 197 | }, \ | ||
| 198 | .show = _show, \ | ||
| 199 | } | ||
| 200 | |||
| 201 | /* | ||
| 202 | * With these extended attributes, the simple show_attribute() and | ||
| 203 | * store_attribute() operations need to call the show() and store() of the | ||
| 204 | * attributes. This is a common pattern, so we provide a macro to define | ||
| 205 | * them. The argument _item is the name of the config_item structure. | ||
| 206 | * This macro expects the attributes to be named "struct <name>_attribute" | ||
| 207 | * and the function to_<name>() to exist; | ||
| 208 | */ | ||
| 209 | #define CONFIGFS_ATTR_OPS(_item) \ | ||
| 210 | static ssize_t _item##_attr_show(struct config_item *item, \ | ||
| 211 | struct configfs_attribute *attr, \ | ||
| 212 | char *page) \ | ||
| 213 | { \ | ||
| 214 | struct _item *_item = to_##_item(item); \ | ||
| 215 | struct _item##_attribute *_item##_attr = \ | ||
| 216 | container_of(attr, struct _item##_attribute, attr); \ | ||
| 217 | ssize_t ret = 0; \ | ||
| 218 | \ | ||
| 219 | if (_item##_attr->show) \ | ||
| 220 | ret = _item##_attr->show(_item, page); \ | ||
| 221 | return ret; \ | ||
| 222 | } \ | ||
| 223 | static ssize_t _item##_attr_store(struct config_item *item, \ | ||
| 224 | struct configfs_attribute *attr, \ | ||
| 225 | const char *page, size_t count) \ | ||
| 226 | { \ | ||
| 227 | struct _item *_item = to_##_item(item); \ | ||
| 228 | struct _item##_attribute *_item##_attr = \ | ||
| 229 | container_of(attr, struct _item##_attribute, attr); \ | ||
| 230 | ssize_t ret = -EINVAL; \ | ||
| 231 | \ | ||
| 232 | if (_item##_attr->store) \ | ||
| 233 | ret = _item##_attr->store(_item, page, count); \ | ||
| 234 | return ret; \ | ||
| 235 | } | ||
| 236 | |||
| 237 | /* | ||
| 238 | * If allow_link() exists, the item can symlink(2) out to other | 158 | * If allow_link() exists, the item can symlink(2) out to other |
| 239 | * items. If the item is a group, it may support mkdir(2). | 159 | * items. If the item is a group, it may support mkdir(2). |
| 240 | * Groups supply one of make_group() and make_item(). If the | 160 | * Groups supply one of make_group() and make_item(). If the |
| @@ -250,8 +170,6 @@ static ssize_t _item##_attr_store(struct config_item *item, \ | |||
| 250 | */ | 170 | */ |
| 251 | struct configfs_item_operations { | 171 | struct configfs_item_operations { |
| 252 | void (*release)(struct config_item *); | 172 | void (*release)(struct config_item *); |
| 253 | ssize_t (*show_attribute)(struct config_item *, struct configfs_attribute *,char *); | ||
| 254 | ssize_t (*store_attribute)(struct config_item *,struct configfs_attribute *,const char *, size_t); | ||
| 255 | int (*allow_link)(struct config_item *src, struct config_item *target); | 173 | int (*allow_link)(struct config_item *src, struct config_item *target); |
| 256 | int (*drop_link)(struct config_item *src, struct config_item *target); | 174 | int (*drop_link)(struct config_item *src, struct config_item *target); |
| 257 | }; | 175 | }; |
diff --git a/samples/Kconfig b/samples/Kconfig index 224ebb46bed5..d54f28c6dc5e 100644 --- a/samples/Kconfig +++ b/samples/Kconfig | |||
| @@ -70,4 +70,10 @@ config SAMPLE_LIVEPATCH | |||
| 70 | Builds a sample live patch that replaces the procfs handler | 70 | Builds a sample live patch that replaces the procfs handler |
| 71 | for /proc/cmdline to print "this has been live patched". | 71 | for /proc/cmdline to print "this has been live patched". |
| 72 | 72 | ||
| 73 | config SAMPLE_CONFIGFS | ||
| 74 | tristate "Build configfs patching sample -- loadable modules only" | ||
| 75 | depends on CONFIGFS_FS && m | ||
| 76 | help | ||
| 77 | Builds a sample configfs interface. | ||
| 78 | |||
| 73 | endif # SAMPLES | 79 | endif # SAMPLES |
diff --git a/samples/Makefile b/samples/Makefile index f00257bcc5a7..48001d7e23f0 100644 --- a/samples/Makefile +++ b/samples/Makefile | |||
| @@ -1,4 +1,5 @@ | |||
| 1 | # Makefile for Linux samples code | 1 | # Makefile for Linux samples code |
| 2 | 2 | ||
| 3 | obj-$(CONFIG_SAMPLES) += kobject/ kprobes/ trace_events/ livepatch/ \ | 3 | obj-$(CONFIG_SAMPLES) += kobject/ kprobes/ trace_events/ livepatch/ \ |
| 4 | hw_breakpoint/ kfifo/ kdb/ hidraw/ rpmsg/ seccomp/ | 4 | hw_breakpoint/ kfifo/ kdb/ hidraw/ rpmsg/ seccomp/ \ |
| 5 | configfs/ | ||
diff --git a/samples/configfs/Makefile b/samples/configfs/Makefile new file mode 100644 index 000000000000..a9afd99630fc --- /dev/null +++ b/samples/configfs/Makefile | |||
| @@ -0,0 +1,2 @@ | |||
| 1 | |||
| 2 | obj-$(CONFIG_SAMPLE_CONFIGFS) += configfs_sample.o | ||
diff --git a/Documentation/filesystems/configfs/configfs_example_macros.c b/samples/configfs/configfs_sample.c index 327dfbc640a9..1ea33119e532 100644 --- a/Documentation/filesystems/configfs/configfs_example_macros.c +++ b/samples/configfs/configfs_sample.c | |||
| @@ -54,18 +54,13 @@ struct childless { | |||
| 54 | 54 | ||
| 55 | static inline struct childless *to_childless(struct config_item *item) | 55 | static inline struct childless *to_childless(struct config_item *item) |
| 56 | { | 56 | { |
| 57 | return item ? container_of(to_configfs_subsystem(to_config_group(item)), struct childless, subsys) : NULL; | 57 | return item ? container_of(to_configfs_subsystem(to_config_group(item)), |
| 58 | struct childless, subsys) : NULL; | ||
| 58 | } | 59 | } |
| 59 | 60 | ||
| 60 | CONFIGFS_ATTR_STRUCT(childless); | 61 | static ssize_t childless_showme_show(struct config_item *item, char *page) |
| 61 | #define CHILDLESS_ATTR(_name, _mode, _show, _store) \ | ||
| 62 | struct childless_attribute childless_attr_##_name = __CONFIGFS_ATTR(_name, _mode, _show, _store) | ||
| 63 | #define CHILDLESS_ATTR_RO(_name, _show) \ | ||
| 64 | struct childless_attribute childless_attr_##_name = __CONFIGFS_ATTR_RO(_name, _show); | ||
| 65 | |||
| 66 | static ssize_t childless_showme_read(struct childless *childless, | ||
| 67 | char *page) | ||
| 68 | { | 62 | { |
| 63 | struct childless *childless = to_childless(item); | ||
| 69 | ssize_t pos; | 64 | ssize_t pos; |
| 70 | 65 | ||
| 71 | pos = sprintf(page, "%d\n", childless->showme); | 66 | pos = sprintf(page, "%d\n", childless->showme); |
| @@ -74,16 +69,15 @@ static ssize_t childless_showme_read(struct childless *childless, | |||
| 74 | return pos; | 69 | return pos; |
| 75 | } | 70 | } |
| 76 | 71 | ||
| 77 | static ssize_t childless_storeme_read(struct childless *childless, | 72 | static ssize_t childless_storeme_show(struct config_item *item, char *page) |
| 78 | char *page) | ||
| 79 | { | 73 | { |
| 80 | return sprintf(page, "%d\n", childless->storeme); | 74 | return sprintf(page, "%d\n", to_childless(item)->storeme); |
| 81 | } | 75 | } |
| 82 | 76 | ||
| 83 | static ssize_t childless_storeme_write(struct childless *childless, | 77 | static ssize_t childless_storeme_store(struct config_item *item, |
| 84 | const char *page, | 78 | const char *page, size_t count) |
| 85 | size_t count) | ||
| 86 | { | 79 | { |
| 80 | struct childless *childless = to_childless(item); | ||
| 87 | unsigned long tmp; | 81 | unsigned long tmp; |
| 88 | char *p = (char *) page; | 82 | char *p = (char *) page; |
| 89 | 83 | ||
| @@ -99,8 +93,7 @@ static ssize_t childless_storeme_write(struct childless *childless, | |||
| 99 | return count; | 93 | return count; |
| 100 | } | 94 | } |
| 101 | 95 | ||
| 102 | static ssize_t childless_description_read(struct childless *childless, | 96 | static ssize_t childless_description_show(struct config_item *item, char *page) |
| 103 | char *page) | ||
| 104 | { | 97 | { |
| 105 | return sprintf(page, | 98 | return sprintf(page, |
| 106 | "[01-childless]\n" | 99 | "[01-childless]\n" |
| @@ -111,26 +104,18 @@ static ssize_t childless_description_read(struct childless *childless, | |||
| 111 | "than a directory in /proc.\n"); | 104 | "than a directory in /proc.\n"); |
| 112 | } | 105 | } |
| 113 | 106 | ||
| 114 | CHILDLESS_ATTR_RO(showme, childless_showme_read); | 107 | CONFIGFS_ATTR_RO(childless_, showme); |
| 115 | CHILDLESS_ATTR(storeme, S_IRUGO | S_IWUSR, childless_storeme_read, | 108 | CONFIGFS_ATTR(childless_, storeme); |
| 116 | childless_storeme_write); | 109 | CONFIGFS_ATTR_RO(childless_, description); |
| 117 | CHILDLESS_ATTR_RO(description, childless_description_read); | ||
| 118 | 110 | ||
| 119 | static struct configfs_attribute *childless_attrs[] = { | 111 | static struct configfs_attribute *childless_attrs[] = { |
| 120 | &childless_attr_showme.attr, | 112 | &childless_attr_showme, |
| 121 | &childless_attr_storeme.attr, | 113 | &childless_attr_storeme, |
| 122 | &childless_attr_description.attr, | 114 | &childless_attr_description, |
| 123 | NULL, | 115 | NULL, |
| 124 | }; | 116 | }; |
| 125 | 117 | ||
| 126 | CONFIGFS_ATTR_OPS(childless); | ||
| 127 | static struct configfs_item_operations childless_item_ops = { | ||
| 128 | .show_attribute = childless_attr_show, | ||
| 129 | .store_attribute = childless_attr_store, | ||
| 130 | }; | ||
| 131 | |||
| 132 | static struct config_item_type childless_type = { | 118 | static struct config_item_type childless_type = { |
| 133 | .ct_item_ops = &childless_item_ops, | ||
| 134 | .ct_attrs = childless_attrs, | 119 | .ct_attrs = childless_attrs, |
| 135 | .ct_owner = THIS_MODULE, | 120 | .ct_owner = THIS_MODULE, |
| 136 | }; | 121 | }; |
| @@ -168,32 +153,13 @@ static inline struct simple_child *to_simple_child(struct config_item *item) | |||
| 168 | return item ? container_of(item, struct simple_child, item) : NULL; | 153 | return item ? container_of(item, struct simple_child, item) : NULL; |
| 169 | } | 154 | } |
| 170 | 155 | ||
| 171 | static struct configfs_attribute simple_child_attr_storeme = { | 156 | static ssize_t simple_child_storeme_show(struct config_item *item, char *page) |
| 172 | .ca_owner = THIS_MODULE, | ||
| 173 | .ca_name = "storeme", | ||
| 174 | .ca_mode = S_IRUGO | S_IWUSR, | ||
| 175 | }; | ||
| 176 | |||
| 177 | static struct configfs_attribute *simple_child_attrs[] = { | ||
| 178 | &simple_child_attr_storeme, | ||
| 179 | NULL, | ||
| 180 | }; | ||
| 181 | |||
| 182 | static ssize_t simple_child_attr_show(struct config_item *item, | ||
| 183 | struct configfs_attribute *attr, | ||
| 184 | char *page) | ||
| 185 | { | 157 | { |
| 186 | ssize_t count; | 158 | return sprintf(page, "%d\n", to_simple_child(item)->storeme); |
| 187 | struct simple_child *simple_child = to_simple_child(item); | ||
| 188 | |||
| 189 | count = sprintf(page, "%d\n", simple_child->storeme); | ||
| 190 | |||
| 191 | return count; | ||
| 192 | } | 159 | } |
| 193 | 160 | ||
| 194 | static ssize_t simple_child_attr_store(struct config_item *item, | 161 | static ssize_t simple_child_storeme_store(struct config_item *item, |
| 195 | struct configfs_attribute *attr, | 162 | const char *page, size_t count) |
| 196 | const char *page, size_t count) | ||
| 197 | { | 163 | { |
| 198 | struct simple_child *simple_child = to_simple_child(item); | 164 | struct simple_child *simple_child = to_simple_child(item); |
| 199 | unsigned long tmp; | 165 | unsigned long tmp; |
| @@ -211,6 +177,13 @@ static ssize_t simple_child_attr_store(struct config_item *item, | |||
| 211 | return count; | 177 | return count; |
| 212 | } | 178 | } |
| 213 | 179 | ||
| 180 | CONFIGFS_ATTR(simple_child_, storeme); | ||
| 181 | |||
| 182 | static struct configfs_attribute *simple_child_attrs[] = { | ||
| 183 | &simple_child_attr_storeme, | ||
| 184 | NULL, | ||
| 185 | }; | ||
| 186 | |||
| 214 | static void simple_child_release(struct config_item *item) | 187 | static void simple_child_release(struct config_item *item) |
| 215 | { | 188 | { |
| 216 | kfree(to_simple_child(item)); | 189 | kfree(to_simple_child(item)); |
| @@ -218,8 +191,6 @@ static void simple_child_release(struct config_item *item) | |||
| 218 | 191 | ||
| 219 | static struct configfs_item_operations simple_child_item_ops = { | 192 | static struct configfs_item_operations simple_child_item_ops = { |
| 220 | .release = simple_child_release, | 193 | .release = simple_child_release, |
| 221 | .show_attribute = simple_child_attr_show, | ||
| 222 | .store_attribute = simple_child_attr_store, | ||
| 223 | }; | 194 | }; |
| 224 | 195 | ||
| 225 | static struct config_item_type simple_child_type = { | 196 | static struct config_item_type simple_child_type = { |
| @@ -235,10 +206,12 @@ struct simple_children { | |||
| 235 | 206 | ||
| 236 | static inline struct simple_children *to_simple_children(struct config_item *item) | 207 | static inline struct simple_children *to_simple_children(struct config_item *item) |
| 237 | { | 208 | { |
| 238 | return item ? container_of(to_config_group(item), struct simple_children, group) : NULL; | 209 | return item ? container_of(to_config_group(item), |
| 210 | struct simple_children, group) : NULL; | ||
| 239 | } | 211 | } |
| 240 | 212 | ||
| 241 | static struct config_item *simple_children_make_item(struct config_group *group, const char *name) | 213 | static struct config_item *simple_children_make_item(struct config_group *group, |
| 214 | const char *name) | ||
| 242 | { | 215 | { |
| 243 | struct simple_child *simple_child; | 216 | struct simple_child *simple_child; |
| 244 | 217 | ||
| @@ -254,20 +227,8 @@ static struct config_item *simple_children_make_item(struct config_group *group, | |||
| 254 | return &simple_child->item; | 227 | return &simple_child->item; |
| 255 | } | 228 | } |
| 256 | 229 | ||
| 257 | static struct configfs_attribute simple_children_attr_description = { | 230 | static ssize_t simple_children_description_show(struct config_item *item, |
| 258 | .ca_owner = THIS_MODULE, | 231 | char *page) |
| 259 | .ca_name = "description", | ||
| 260 | .ca_mode = S_IRUGO, | ||
| 261 | }; | ||
| 262 | |||
| 263 | static struct configfs_attribute *simple_children_attrs[] = { | ||
| 264 | &simple_children_attr_description, | ||
| 265 | NULL, | ||
| 266 | }; | ||
| 267 | |||
| 268 | static ssize_t simple_children_attr_show(struct config_item *item, | ||
| 269 | struct configfs_attribute *attr, | ||
| 270 | char *page) | ||
| 271 | { | 232 | { |
| 272 | return sprintf(page, | 233 | return sprintf(page, |
| 273 | "[02-simple-children]\n" | 234 | "[02-simple-children]\n" |
| @@ -276,6 +237,13 @@ static ssize_t simple_children_attr_show(struct config_item *item, | |||
| 276 | "items have only one attribute that is readable and writeable.\n"); | 237 | "items have only one attribute that is readable and writeable.\n"); |
| 277 | } | 238 | } |
| 278 | 239 | ||
| 240 | CONFIGFS_ATTR_RO(simple_children_, description); | ||
| 241 | |||
| 242 | static struct configfs_attribute *simple_children_attrs[] = { | ||
| 243 | &simple_children_attr_description, | ||
| 244 | NULL, | ||
| 245 | }; | ||
| 246 | |||
| 279 | static void simple_children_release(struct config_item *item) | 247 | static void simple_children_release(struct config_item *item) |
| 280 | { | 248 | { |
| 281 | kfree(to_simple_children(item)); | 249 | kfree(to_simple_children(item)); |
| @@ -283,7 +251,6 @@ static void simple_children_release(struct config_item *item) | |||
| 283 | 251 | ||
| 284 | static struct configfs_item_operations simple_children_item_ops = { | 252 | static struct configfs_item_operations simple_children_item_ops = { |
| 285 | .release = simple_children_release, | 253 | .release = simple_children_release, |
| 286 | .show_attribute = simple_children_attr_show, | ||
| 287 | }; | 254 | }; |
| 288 | 255 | ||
| 289 | /* | 256 | /* |
| @@ -323,7 +290,8 @@ static struct configfs_subsystem simple_children_subsys = { | |||
| 323 | * children of its own. | 290 | * children of its own. |
| 324 | */ | 291 | */ |
| 325 | 292 | ||
| 326 | static struct config_group *group_children_make_group(struct config_group *group, const char *name) | 293 | static struct config_group *group_children_make_group( |
| 294 | struct config_group *group, const char *name) | ||
| 327 | { | 295 | { |
| 328 | struct simple_children *simple_children; | 296 | struct simple_children *simple_children; |
| 329 | 297 | ||
| @@ -338,20 +306,8 @@ static struct config_group *group_children_make_group(struct config_group *group | |||
| 338 | return &simple_children->group; | 306 | return &simple_children->group; |
| 339 | } | 307 | } |
| 340 | 308 | ||
| 341 | static struct configfs_attribute group_children_attr_description = { | 309 | static ssize_t group_children_description_show(struct config_item *item, |
| 342 | .ca_owner = THIS_MODULE, | 310 | char *page) |
| 343 | .ca_name = "description", | ||
| 344 | .ca_mode = S_IRUGO, | ||
| 345 | }; | ||
| 346 | |||
| 347 | static struct configfs_attribute *group_children_attrs[] = { | ||
| 348 | &group_children_attr_description, | ||
| 349 | NULL, | ||
| 350 | }; | ||
| 351 | |||
| 352 | static ssize_t group_children_attr_show(struct config_item *item, | ||
| 353 | struct configfs_attribute *attr, | ||
| 354 | char *page) | ||
| 355 | { | 311 | { |
| 356 | return sprintf(page, | 312 | return sprintf(page, |
| 357 | "[03-group-children]\n" | 313 | "[03-group-children]\n" |
| @@ -360,8 +316,11 @@ static ssize_t group_children_attr_show(struct config_item *item, | |||
| 360 | "groups are like the subsystem simple-children.\n"); | 316 | "groups are like the subsystem simple-children.\n"); |
| 361 | } | 317 | } |
| 362 | 318 | ||
| 363 | static struct configfs_item_operations group_children_item_ops = { | 319 | CONFIGFS_ATTR_RO(group_children_, description); |
| 364 | .show_attribute = group_children_attr_show, | 320 | |
| 321 | static struct configfs_attribute *group_children_attrs[] = { | ||
| 322 | &group_children_attr_description, | ||
| 323 | NULL, | ||
| 365 | }; | 324 | }; |
| 366 | 325 | ||
| 367 | /* | 326 | /* |
| @@ -373,7 +332,6 @@ static struct configfs_group_operations group_children_group_ops = { | |||
| 373 | }; | 332 | }; |
| 374 | 333 | ||
| 375 | static struct config_item_type group_children_type = { | 334 | static struct config_item_type group_children_type = { |
| 376 | .ct_item_ops = &group_children_item_ops, | ||
| 377 | .ct_group_ops = &group_children_group_ops, | 335 | .ct_group_ops = &group_children_group_ops, |
| 378 | .ct_attrs = group_children_attrs, | 336 | .ct_attrs = group_children_attrs, |
| 379 | .ct_owner = THIS_MODULE, | 337 | .ct_owner = THIS_MODULE, |
