lkml.org 
[lkml]   [2022]   [May]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH -next] fuse: return the more nuanced writeback error on close()
From
在 2022/5/18 22:51, Al Viro 写道:
> On Wed, May 18, 2022 at 10:57:29PM +0800, ChenXiaoSong wrote:
>
>> + /* return more nuanced writeback errors */
>> if (err)
>> - return err;
>> + return filemap_check_wb_err(file->f_mapping, 0);
>>
>> err = 0;
>
> As an aside, what the hell is that err = 0 about? Before or after
> that patch, that is - "let's make err zero, in case it had somehow
> magically changed ceased to be so since if (err) bugger_off just above"?
>
> .
>
err = 0 is no longer needed after if (err), I will send v2 patch to
remove the redundant code.

\
 
 \ /
  Last update: 2022-05-23 03:23    [W:0.044 / U:0.720 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site