Re: [PATCH] maple_tree: export symbol mas_preallocate()

From: Christoph Hellwig
Date: Wed Mar 08 2023 - 09:46:24 EST


On Wed, Mar 08, 2023 at 03:34:29PM +0100, Danilo Krummrich wrote:
> On 3/8/23 11:59, Vlastimil Babka wrote:
>> On 3/2/23 02:52, Liam R. Howlett wrote:
>>> Thanks for the patch. This should indeed be exported.
>>>
>>> Reviewed-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx>
>>
>> What uses it? Don't we export only things used by in-kernel modules?
>
> mas_preallocate() is part of the maple tree's 'Advanced API'. All other
> functions of this API are exported already.
>
> More specifically, e.g. mas_store_prealloc() is exported which doesn't make
> a lot of sense without mas_preallocate() being available too.
>
> I don't think it is used outside of mm yet, but as a generic tree
> implementation maple tree is pretty likely to be used in code built as
> module, e.g. drivers.

If anyone wants to use they can add the export as needed.