Christoph Hellwig <hch@xxxxxxxxxxxxx> writes:
> On Sun, Jun 11, 2023 at 11:21:48PM -0700, Christoph Hellwig wrote:
>> Looks good:
>>
>> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Thanks!
>
> Actually, coming back to this:
>
> -to_iomap_page(struct folio *folio)
> +static inline struct iomap_folio_state *iomap_get_ifs
>
> I find the to_* naming much more descriptive for derving the
> private data. get tends to imply grabbing a refcount.
Not always. That would be iomap_ifs_get(ifs)/iomap_ifs_put(ifs).
See this -
static inline void *folio_get_private(struct folio *folio)
{
return folio->private;
}
So, is it ok if we hear from others too on iomap_get_ifs() function
name?
-ritesh
