Skip to content

512K readahead size with thrashing safe readahead

Wu Fengguang linux-embedded linux-fsdevel linux-mm lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest11 patches 10 repliesIn review

Changes since RFC from the cover letter

- move the lenthy intro text to individual patch changelogs
- treat get_capacity()==0 as uninitilized value (Thanks to Vivek Goyal)
- increase readahead size limit for small devices (Thanks to Jens Axboe)
- add fio test results by Vivek Goyal

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/11readahead: limit readahead size for small devicesblock/genhd.c+24 -00 replies
2/11readahead: retain inactive lru pages to be accessed soonmm/readahead.c+44 -00 replies
3/11readahead: bump up the default readahead sizeinclude/linux/mm.h+2 -28 replies
4/11readahead: introduce {MAX|MIN}_READAHEAD_PAGES macros for ease of useblock/blk-core.c fs/fuse/inode.c include/linux/mm.h +1 more+6 -40 replies
5/11readahead: replace ra->mmap_miss with ra->ra_flagsinclude/linux/fs.h mm/filemap.c+31 -62 replies
6/11readahead: thrashing safe context readaheadinclude/linux/fs.h mm/readahead.c+88 -680 replies
7/11readahead: record readahead patternsinclude/linux/fs.h include/linux/mm.h mm/filemap.c +1 more+55 -70 replies
8/11readahead: add tracing eventinclude/trace/events/readahead.h mm/readahead.c+91 -00 replies
9/11readahead: add /debug/readahead/statsmm/Kconfig mm/readahead.c+188 -20 replies
10/11readahead: dont do start-of-file readahead after lseek()fs/read_write.c include/linux/fs.h mm/readahead.c+9 -00 replies
11/11radixtree: speed up next/prev hole searchlib/radix-tree.c+74 -110 replies

Reviews trailers per patch

2 of 11 patches have a Reviewed-by or Acked-by.

PatchJens AxboeVivek GoyalLinus Torvalds
1/11A-b T-b T-b
2/11
3/11T-b
4/11
5/11
6/11
7/11
8/11
9/11
10/11A-b
11/11

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH 00/11] 512K readahead size with thrashing safe readahead

Andrew,

It seems there are no strong objections against the 512KB readahead size. (Or, anyone would prefer an 1MB readahead? Chris Mason picked a 4MB size
for btrfs after all.)

So would you include the patchset for wider tests in -mm? I reordered the patchset a bit: the first 2 patches are good candidates for 2.6.34, while the others may need longer tests.

Changes since RFC:
- move the lenthy intro text to individual patch changelogs
- treat get_capacity()==0 as uninitilized value (Thanks to Vivek Goyal)
- increase readahead size limit for small devices (Thanks to Jens Axboe)
- add fio test results by Vivek Goyal

[PATCH 01/11] readahead: limit readahead size for small devices [PATCH 02/11] readahead: retain inactive lru pages to be accessed soon

[PATCH 03/11] readahead: bump up the default readahead size
[PATCH 04/11] readahead: introduce {MAX|MIN}_READAHEAD_PAGES macros for ease of use [PATCH 05/11] readahead: replace ra->mmap_miss with ra->ra_flags [PATCH 06/11] readahead: thrashing safe context readahead
[PATCH 07/11] readahead: record readahead patterns
[PATCH 08/11] readahead: add tracing event
[PATCH 09/11] readahead: add /debug/readahead/stats
[PATCH 10/11] readahead: dont do start-of-file readahead after lseek() [PATCH 11/11] radixtree: speed up next/prev hole search

Thanks,
Fengguang

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>

Read the cover letter in its thread