lkml.org 
[lkml]   [2017]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    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@google.com>

    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

    \
     
     \ /
      Last update: 2017-06-22 20:00    [W:3.293 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site