Re: [PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-19 Thread Shaohua Li
On Thu, 2011-01-20 at 10:46 +0800, Andrew Morton wrote: > On Thu, 20 Jan 2011 10:34:18 +0800 Shaohua Li wrote: > > > > > Under a harddisk based netbook with Meego, the metadata readahead > > > > reduced about 3.5s boot time in average from total 16s. > > > > > > That's a respectable speedup.

Re: [PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-19 Thread Andrew Morton
On Thu, 20 Jan 2011 10:34:18 +0800 Shaohua Li wrote: > > > Under a harddisk based netbook with Meego, the metadata readahead > > > reduced about 3.5s boot time in average from total 16s. > > > > That's a respectable speedup. And it *needs* to be a good speedup, > > given how hacky all of this

Re: [PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-19 Thread Shaohua Li
On Thu, 2011-01-20 at 04:34 +0800, Andrew Morton wrote: > On Wed, 19 Jan 2011 09:15:15 +0800 > Shaohua Li wrote: > > > We have file readahead to do asyn file read, but has no metadata > > readahead. For a list of files, their metadata is stored in fragmented > > disk space and metadata read is

Re: [PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-19 Thread Shaohua Li
On Thu, 2011-01-20 at 05:33 +0800, David Nicol wrote: > On Wed, Jan 19, 2011 at 2:34 PM, Andrew Morton > wrote: > > > It would be much more valuable if those 3.5 seconds were available to > > devices which really really care about bootup times, but very few of > > those devices use rotating disks

Re: [PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-19 Thread David Nicol
On Wed, Jan 19, 2011 at 2:34 PM, Andrew Morton wrote: > It would be much more valuable if those 3.5 seconds were available to > devices which really really care about bootup times, but very few of > those devices use rotating disks nowadays, I expect? And don't rotating disk modules read and buf

Re: [PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-19 Thread Andrew Morton
On Wed, 19 Jan 2011 09:15:15 +0800 Shaohua Li wrote: > We have file readahead to do asyn file read, but has no metadata > readahead. For a list of files, their metadata is stored in fragmented > disk space and metadata read is a sync operation, which impacts the > efficiency of readahead much.

[PATCH v3 0/5]add new ioctls to do metadata readahead in btrfs

2011-01-18 Thread Shaohua Li
Hi, We have file readahead to do asyn file read, but has no metadata readahead. For a list of files, their metadata is stored in fragmented disk space and metadata read is a sync operation, which impacts the efficiency of readahead much. The patches try to add meatadata readahead for btrfs. It ha