[PATCH 1/1] staging: erofs: Add function comment for erofs/super.c

valdis.kletnieks at vt.edu valdis.kletnieks at vt.edu
Wed Mar 13 02:41:20 EDT 2019


On Wed, 13 Mar 2019 08:48:26 +0800, Gao Xiang said:
(Reversing order of  comments)
> "Release slab Memory" is not a very useful description for this function.

Agreed.

> i_callback is used to free inode structure and other memory related to this inode

However, that's not a bad description at all. :)

(Though I'd consider renaming the function to 'reap_inode_mem()' or something
like that so no description is needed. :)






More information about the Kernelnewbies mailing list