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@xxxxxx> > --- > 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 <nab@xxxxxxxxxxxxxxx> 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. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html