On Wed, Jun 21, 2017 at 02:21:20PM -0700, Tahsin Erdogan wrote: > In general, kernel functions indicate success/failure through their return > values. This function returns the status as an output parameter and reserves > the return value for the inode. Make it follow the general convention. > > Signed-off-by: Tahsin Erdogan <tahsin@xxxxxxxxxx> The fact that we have several conventions for error passing, is due to the long history of the ext2/3/4 code base. In the long term, I'd actually like to see us gradually move everyhing to use the ERR_PTR convention. It's a bit more efficient for the common (no error) case, and it allows us to drop an extra parameter from the function signature. Still, it's incrementally better this way, so thanks, added to the ext4 patch queue. - Ted
