On 04/20/13 19:43, Eric Sandeen wrote:
> On 4/20/13 7:43 AM, Wang Shilong wrote:
>> From: Wang Shilong <wangsl-f...@cn.fujitsu.com>
>>
>> ulist is a generic structure to iterate tree. we export them
>> just in the hope it may be used by other modules. However, until
>> now, it is just used by btrfs. The main point is that ulist_{init,fin}()
>> should be only called by ulist_{alloc,reinit,free}().So we don't need
>> to export them and make them staic will be best choice.
>
> It actually seems like all EXPORT_SYMBOLS should be removed, no?
> I can't imagine that btrfs needs to be exporting any of these
> functions to any other module...
let's see that we move ulists to lib/ for 3.10.
-Arne
>
> Thanks,
> -Eric
>
>> Signed-off-by: Wang Shilong <wangsl-f...@cn.fujitsu.com>
>> ---
>> fs/btrfs/ulist.c | 6 ++----
>> fs/btrfs/ulist.h | 2 --
>> 2 files changed, 2 insertions(+), 6 deletions(-)
>>
>> diff --git a/fs/btrfs/ulist.c b/fs/btrfs/ulist.c
>> index ddc61ca..50fc1c4 100644
>> --- a/fs/btrfs/ulist.c
>> +++ b/fs/btrfs/ulist.c
>> @@ -48,13 +48,12 @@
>> * Note: don't use this function to init an already used ulist, use
>> * ulist_reinit instead.
>> */
>> -void ulist_init(struct ulist *ulist)
>> +static void ulist_init(struct ulist *ulist)
>> {
>> ulist->nnodes = 0;
>> ulist->nodes = ulist->int_nodes;
>> ulist->nodes_alloced = ULIST_SIZE;
>> }
>> -EXPORT_SYMBOL(ulist_init);
>>
>> /**
>> * ulist_fini - free up additionally allocated memory for the ulist
>> @@ -63,7 +62,7 @@ EXPORT_SYMBOL(ulist_init);
>> * This is useful in cases where the base 'struct ulist' has been statically
>> * allocated.
>> */
>> -void ulist_fini(struct ulist *ulist)
>> +static void ulist_fini(struct ulist *ulist)
>> {
>> /*
>> * The first ULIST_SIZE elements are stored inline in struct ulist.
>> @@ -73,7 +72,6 @@ void ulist_fini(struct ulist *ulist)
>> kfree(ulist->nodes);
>> ulist->nodes_alloced = 0; /* in case ulist_fini is called twice */
>> }
>> -EXPORT_SYMBOL(ulist_fini);
>>
>> /**
>> * ulist_reinit - prepare a ulist for reuse
>> diff --git a/fs/btrfs/ulist.h b/fs/btrfs/ulist.h
>> index 21a1963..e05db47 100644
>> --- a/fs/btrfs/ulist.h
>> +++ b/fs/btrfs/ulist.h
>> @@ -60,8 +60,6 @@ struct ulist {
>> struct ulist_node int_nodes[ULIST_SIZE];
>> };
>>
>> -void ulist_init(struct ulist *ulist);
>> -void ulist_fini(struct ulist *ulist);
>> void ulist_reinit(struct ulist *ulist);
>> struct ulist *ulist_alloc(gfp_t gfp_mask);
>> void ulist_free(struct ulist *ulist);
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
