On Thu, Dec 08, 2022 at 10:32:32AM +0800, Ye Bin wrote: > From: Ye Bin <yebin10@xxxxxxxxxx> > > Prepend ext4 prefix to function names of xattr_find_entry() and > __xattr_check_inode() for consistency with other functions in ext4 > xattr module. The reason why these function names don't have the ext4 prefix is because functions are static, so there is need to worry about namespace pollution. For static functions, there is no real need to add the ext4 prefix, so why not keep the functions shorter? Yes, there are places where static functions have the ext4 prefix. The main reason why we have changed it is that it's not worth the code churn. Similarly, adding the ext4 prefix is *definitely* not worth the code churn. Cheers, - Ted
