lkml.org 
[lkml]   [2021]   [Aug]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] fs: optimise generic_write_check_limits()
From
Date
On 8/6/21 2:28 PM, Matthew Wilcox wrote:
> On Fri, Aug 06, 2021 at 12:22:10PM +0100, Pavel Begunkov wrote:
>> Even though ->s_maxbytes is used by generic_write_check_limits() only in
>> case of O_LARGEFILE, the value is loaded unconditionally, which is heavy
>> and takes 4 indirect loads. Optimise it by not touching ->s_maxbytes,
>> if it's not going to be used.
>
> Is this "optimisation" actually worth anything? Look at how
> force_o_largefile() is used. I would suggest that on the vast majority
> of machines, O_LARGEFILE is always set.

Makes sense to leave it alone then, thanks

--
Pavel Begunkov

\
 
 \ /
  Last update: 2022-09-17 16:11    [W:0.102 / U:0.300 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site