Thread (7 messages) 7 messages, 6 authors, 2016-03-11

Re: [PATCH] configfs: switch ->default groups to a linked list

From: Nicholas A. Bellinger <hidden>
Date: 2016-02-27 23:22:26
Also in: lkml, ocfs2-devel

On Fri, 2016-02-26 at 13:33 +0100, Christoph Hellwig wrote:
Replace the current NULL-terminated array of default groups with a linked
list.  This gets rid of lots of nasty code to size and/or dynamically
allocate the array.

While we're at it also provide a conveniant helper to remove the default
groups.

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 Documentation/filesystems/configfs/configfs.txt |  11 +-
 drivers/infiniband/core/cma_configfs.c          |  31 ++--
 drivers/target/iscsi/iscsi_target_configfs.c    |  75 +++------
 drivers/target/target_core_configfs.c           | 203 +++++-------------------
 drivers/target/target_core_fabric_configfs.c    | 194 ++++++----------------
 drivers/target/target_core_internal.h           |   1 -
 drivers/target/target_core_stat.c               |  41 ++---
 drivers/usb/gadget/configfs.c                   |  36 ++---
 drivers/usb/gadget/function/f_mass_storage.c    |   6 +-
 drivers/usb/gadget/function/f_rndis.c           |   1 -
 drivers/usb/gadget/function/uvc_configfs.c      | 198 +++++++++--------------
 fs/configfs/dir.c                               |  44 +++--
 fs/configfs/item.c                              |   1 +
 fs/dlm/config.c                                 |  38 +----
 fs/ocfs2/cluster/nodemanager.c                  |  22 +--
 include/linux/configfs.h                        |  11 +-
 include/target/target_core_base.h               |   3 -
 17 files changed, 284 insertions(+), 632 deletions(-)
This is a very nice simplification to drop the long-standing fixed size
->default_groups[] usage tree-wide for configfs consumers.

For the target parts:

Acked-by: Nicholas Bellinger <redacted>

Also, this applies cleanly to target-pending/for-next, so I don't see a
problem merging it up via your new configfs tree.

Otherwise let me know, and I'll be happy to merge via target-pending.

Thanks HCH.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help