Re: [PATCH 87/88] Add configurable prefetch size for layoutget

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2011-06-08 03:15, Peng Tao wrote:
> On 6/8/11, Jim Rees <rees@xxxxxxxxx> wrote:
>> Benny Halevy wrote:
>>
>>   NAK.
>>   This affects all layout types.  In particular it is undesired
>>   for write layouts that extend the file with the objects layout.
>>   The server can extend the layout segments range
>>   over what the client requested so why would the client
>>   ask for artificially large layouts?
>>
>> This has actually been the subject of some debate over Thursday night
>> beers.  The problem we're trying to solve is that the client is spending 98%
>> of its time in layoutget.  This patch gives us something like a 10x
>> speedup.  But many of us think it's not the right fix.  I suggest we discuss
>> next week.
>>

Sure.

>> But note that this patch doesn't change anything unless you set the sysctl.
> there is a default value of 2M. maybe we can set it to page size by
> default so other layout are not affected and block layout can let
> users set it by hand if they care about performance. does this make
> sense?

If doing it at all why use a sysctl rather than a mount option?
Or maybe coding the logic for prefetching the layout iff sequential
access is detected is the right thing to do.

Benny

>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux