Adaptive read-ahead V5
Subsystems (11)
cover Adaptive read-ahead V5
This version of adaptive read-ahead patch features various clean ups. Changes include:
- rewrote context based method to make it clean and robust
- improved accuracy of stateful thrashing threshold estimation
- nr_page_aging made right
- sorted out the thrashing protection logic
- enhanced debug/accounting facilities
Regards,
Wu Fengguang
PATCH 1/13 mm: delayed page activation
When a page is referenced the second time in inactive_list, mark it with PG_activate instead of moving it into active_list immediately. The actual moving work is delayed to vmscan time.
This means two essential changes:
- keeps the adjecency of pages in lru;
- lifts the page reference counter max from 1 to 3.
And leads to the following improvements:
- read-ahead for a leading reader will not be disturbed by a following reader;
- enables the thrashing protection logic to save pages for following readers;
- keeping relavant pages together helps improve I/O efficiency;
- and also helps decrease vm fragmantation;
- increased refcnt space might help page replacement algorithms.
---
include/linux/page-flags.h | 31 +++++++++++++++++++++++++++++++ mm/page_alloc.c | 1 + mm/swap.c | 9 ++++----- mm/vmscan.c | 6 ++++++ 4 files changed, 42 insertions(+), 5 deletions(-)
include/linux/page-flags.h +31 -0
@@ -76,6 +76,7 @@#define PG_reclaim 17/* To be reclaimed asap */ #define PG_nosave_free 18/* Free, should not be written */ #define PG_uncached 19/* Page has been mapped as uncached */ +#define PG_activate 20/* delayed activate */ /* * Global page accounting. One instance per CPU. Only unsigned longs are
@@ -308,6 +309,12 @@ extern void __mod_page_state(unsigned lo#define SetPageUncached (page) set_bit(PG_uncached, &(page)->flags)#define ClearPageUncached (page) clear_bit(PG_uncached, &(page)->flags) +#define PageActivate (page) test_bit(PG_activate, &(page)->flags) +#define SetPageActivate (page) set_bit(PG_activate, &(page)->flags) +#define ClearPageActivate (page) clear_bit(PG_activate, &(page)->flags) +#define TestClearPageActivate (page) test_and_clear_bit(PG_activate, &(page)->flags) +#define TestSetPageActivate (page) test_and_set_bit(PG_activate, &(page)->flags) +struct page ;/* forward declaration */ int test_clear_page_dirty (struct page *page);
@@ -333,4 +340,28 @@ static inline void set_page_writeback(st#define ClearPageFsMisc (page) clear_bit(PG_fs_misc, &(page)->flags)#define TestClearPageFsMisc (page) test_and_clear_bit(PG_fs_misc, &(page)->flags) +#if PG_activate < PG_referenced +#error unexpected page flags order +#endif + +#define PAGE_REFCNT_0 0 +#define PAGE_REFCNT_1 (1 << PG_referenced) +#define PAGE_REFCNT_2 (1 << PG_activate) +#define PAGE_REFCNT_3 ((1 << PG_activate) | (1 << PG_referenced)) +#define PAGE_REFCNT_MASK PAGE_REFCNT_3 + +/* +* STATUS REFERENCE COUNT +* __ 0 +* _R PAGE_REFCNT_1 +* A_ PAGE_REFCNT_2 +* AR PAGE_REFCNT_3 +* +* A/R: Active / Referenced +*/ +static inline unsigned long page_refcnt (struct page *page) +{ +return page->flags &PAGE_REFCNT_MASK ; +} +#endif /* PAGE_FLAGS_H */
mm/page_alloc.c +1 -0
@@ -488,6 +488,7 @@ static void prep_new_page(struct page *p page->flags &= ~(1 << PG_uptodate |1 << PG_error |1 << PG_referenced |1 << PG_arch_1 | +1 << PG_activate |1 << PG_checked |1 << PG_mappedtodisk);set_page_private (page,0 );set_page_refs (page, order);
mm/swap.c +4 -5
@@ -29,7 +29,6 @@#include <linux/percpu.h> #include <linux/cpu.h> #include <linux/notifier.h> -#include <linux/init.h> /* How many pages do we try to swap or page in/out together? */ int page_cluster;
@@ -117,13 +116,13 @@ void fastcall activate_page(struct page* Mark a page as having seen activity. * * inactive,unreferenced -> inactive,referenced -* inactive,referenced -> active,unreferenced -* active,unreferenced -> active,referenced +* inactive,referenced -> activate,unreferenced +* activate,unreferenced -> activate,referenced */ void fastcall mark_page_accessed(struct page *page) { -if (!PageActive (page) &&PageReferenced (page) &&PageLRU (page)) { -activate_page (page); +if (!PageActivate (page) &&PageReferenced (page) &&PageLRU (page)) { +SetPageActivate (page);ClearPageReferenced (page); }else if (!PageReferenced (page)) {SetPageReferenced (page);
mm/vmscan.c +6 -0
@@ -407,6 +407,12 @@ static int shrink_list(struct list_headif (PageWriteback (page)) goto keep_locked; +if (PageActivate (page)) { +ClearPageActivate (page); +ClearPageReferenced (page); + goto activate_locked; + } + referenced =page_referenced (page,1 , sc->priority <=0 );/* In active use or really unfreeable? Activate it. */ if (referenced &&page_mapping_inuse (page))
--
PATCH 2/13 mm: balance page aging between zones
In reply to Wu FengguangThe page aging rates are currently imbalanced, the gap can be as large as 3 times, which can severely damage read-ahead requests and shorten their effective life time.
This patch adds three variables in struct zone to keep track of page aging rate, and keeps them in sync on vmscan/pagealloc time. The nr_page_aging is a per-zone counter-part to the per-cpu pgscan_{kswapd,direct}_{zone name}.
The direct page reclaim path is not touched, for it needs non-trival changes. It is ok as long as (pgscan_kswapd_* > pgscan_direct_*).
__alloc_pages() is changed a lot, which needs comfirmation from NUMA gurus. The basic idea is to do reclaim in batches, and keep the zones inside one batch balanced. There can be different policies in the partitioning. One obvious choice for the first batch would be the zones on current node and nearby memory nodes without cpu.
---
include/linux/mmzone.h | 51 +++++++++++++++++++++++++ mm/page_alloc.c | 97 ++++++++++++++++++++++++++++++++++++++++--------- mm/vmscan.c | 32 +++++++++++++--- 3 files changed, 158 insertions(+), 22 deletions(-)
include/linux/mmzone.h +51 -0
@@ -160,6 +160,20 @@ struct zone {unsigned long pages_scanned;/* since last reclaim */ int all_unreclaimable;/* All pages pinned */ +/* Fields for balanced page aging: +* nr_page_aging - The accumulated number of activities that may +* cause page aging, that is, make some pages closer +* to the tail of inactive_list. +* aging_milestone - A snapshot of nr_page_aging every time a full +* inactive_list of pages become aged. +* page_age - A normalized value showing the percent of pages +* have been aged. It is compared between zones to +* balance the rate of page aging. +*/ +unsigned long nr_page_aging; +unsigned long aging_milestone; +unsigned long page_age; +/* * Does the allocator try to reclaim pages from the zone as soon * as it fails a watermark_ok() in __alloc_pages?
@@ -343,6 +357,43 @@ static inline void memory_present(int niunsigned long __init node_memmap_size_bytes (int ,unsigned long ,unsigned long );#endif +#ifdef CONFIG_HIGHMEM64G +#define PAGE_AGE_SHIFT 8 +#elif BITS_PER_LONG ==32 +#define PAGE_AGE_SHIFT 12 +#elif BITS_PER_LONG ==64 +#define PAGE_AGE_SHIFT 20 +#else +#error unknown BITS_PER_LONG +#endif +#define PAGE_AGE_MASK ((1 << PAGE_AGE_SHIFT) - 1) + +/* +* The percent of pages in inactive_list that have been scanned / aged. +* It's not really ##%, but a high resolution normalized value. +*/ +static inline void update_page_age (struct zone *z) +{ +if (z->nr_page_aging - z->aging_milestone > z->nr_inactive) + z->aging_milestone += z->nr_inactive; + + z->page_age = ((z->nr_page_aging - z->aging_milestone) + <<PAGE_AGE_SHIFT ) / (1 + z->nr_inactive); +} + +/* +* The simplified code is: +* return (a->page_age > b->page_age); +* The complexity deals with the wrap-around problem. +* Two page ages not close enough should also be ignored: +* they are out of sync and the comparison may be nonsense. +*/ +static inline int pages_more_aged (struct zone *a,struct zone *b) +{ +return ((b->page_age - a->page_age) &PAGE_AGE_MASK ) > +PAGE_AGE_MASK - (1 << (PAGE_AGE_SHIFT -2 )); +} +/* * zone_idx() returns 0 for the ZONE_DMA zone, 1 for the ZONE_NORMAL zone, etc. */
mm/page_alloc.c +81 -16
171 lines not on this page · raw message
mm/vmscan.c +26 -6
@@ -653,6 +653,8 @@ static void shrink_cache(struct zone *zo goto done; max_scan -= nr_scan; + zone->nr_page_aging += nr_scan; +update_page_age (zone);if (current_is_kswapd ())mod_page_state_zone (zone, pgscan_kswapd, nr_scan);else
@@ -1068,6 +1070,7 @@ loop_again:for (priority =DEF_PRIORITY ; priority >=0 ; priority--) {int end_zone =0 ;/* Inclusive. 0 = ZONE_DMA */ +int begin_zone =-1 ;unsigned long lru_pages =0 ; all_zones_ok =1 ;
@@ -1089,16 +1092,33 @@ loop_again:if (!zone_watermark_ok (zone, order, zone->pages_high,0 ,0 ,0 )) { - end_zone = i; - goto scan; +if (!end_zone) + begin_zone = end_zone = i; +else /* if (begin_zone == i + 1) */ + begin_zone = i; } } - goto out; +if (begin_zone <0 ) + goto out; }else { + begin_zone =0 ; end_zone = pgdat->nr_zones -1 ; } -scan: -for (i =0 ; i <= end_zone; i++) { + +/* +* Prepare enough free pages for zones with small page_age, +* they are going to be reclaimed in the page allocation. +*/ +while (end_zone < pgdat->nr_zones -1 && +pages_more_aged (pgdat->node_zones + end_zone, + pgdat->node_zones + end_zone +1 )) + end_zone++; +while (begin_zone && +pages_more_aged (pgdat->node_zones + begin_zone, + pgdat->node_zones + begin_zone -1 )) + begin_zone--; + +for (i = begin_zone; i <= end_zone; i++) {struct zone *zone = pgdat->node_zones + i; lru_pages += zone->nr_active + zone->nr_inactive;
@@ -1113,7 +1133,7 @@ scan:* pages behind kswapd's direction of progress, which would * cause too much scanning of the lower zones. */ -for (i =0 ; i <= end_zone; i++) { +for (i = begin_zone; i <= end_zone; i++) {struct zone *zone = pgdat->node_zones + i;int nr_slab;
--
PATCH 3/13 radixtree: sync with mainline
In reply to Wu FengguangThe patch from Christoph Lameter:
[PATCH] radix-tree: Remove unnecessary indirections and clean up code
is only partially merged into -mm tree. This patch completes it.
---
lib/radix-tree.c | 12 +++++------- 1 files changed, 5 insertions(+), 7 deletions(-)
lib/radix-tree.c +5 -7
@@ -286,27 +286,25 @@ static inline void **__lookup_slot(struc unsigned long index) {unsigned int height, shift; -struct radix_tree_node **slot; +struct radix_tree_node *slot; height = root->height;if (index >radix_tree_maxindex (height))return NULL ; shift = (height-1 ) *RADIX_TREE_MAP_SHIFT ; - slot = &root->rnode; + slot = root->rnode;while (height >0 ) { -if (*slot ==NULL ) +if (slot ==NULL )return NULL ; - slot = (struct radix_tree_node **) - ((*slot)->slots + - ((index >> shift) &RADIX_TREE_MAP_MASK )); + slot = slot->slots[(index >> shift) &RADIX_TREE_MAP_MASK ]; shift -=RADIX_TREE_MAP_SHIFT ; height--; } -return (void **)slot; +return slot; }/**
--
PATCH 4/13 radixtree: look-aside cache
In reply to Wu FengguangThis introduces a set of lookup functions to radix tree for the read-ahead logic. Other access patterns with high locality may also benefit from them.
Most of them are best inlined, so some macros/structs in .c are moved into .h.
- radix_tree_lookup_node(root, index, level)
Perform partial lookup, return the @level'th parent of the slot at @index.
- radix_tree_cache_lookup(root, cache, index) Perform lookup with the aid of a look-aside cache.
- radix_tree_cache_xxx()
Query various information about the cache.
- radix_tree_lookup_head(root, index, max_scan)
- radix_tree_lookup_tail(root, index, max_scan)
[head, tail) is a segment with continuous pages. The functions search for the head and tail index of the segment at @index.
---
include/linux/radix-tree.h | 149 ++++++++++++++++++++++++++++++++++++++++++++- lib/radix-tree.c | 142 ++++++++++++++++++++++++++++++++---------- 2 files changed, 254 insertions(+), 37 deletions(-)
include/linux/radix-tree.h +147 -2
@@ -22,12 +22,39 @@#include <linux/preempt.h> #include <linux/types.h> +#ifdef __KERNEL__ +#define RADIX_TREE_MAP_SHIFT 6 +#else +#define RADIX_TREE_MAP_SHIFT 3/* For more stressful testing */ +#endif +#define RADIX_TREE_TAGS 2 + +#define RADIX_TREE_MAP_SIZE (1UL << RADIX_TREE_MAP_SHIFT) +#define RADIX_TREE_MAP_MASK (RADIX_TREE_MAP_SIZE-1) + +#define RADIX_TREE_TAG_LONGS \ + ((RADIX_TREE_MAP_SIZE + BITS_PER_LONG - 1) / BITS_PER_LONG) + +struct radix_tree_node { +unsigned int count; +void *slots[RADIX_TREE_MAP_SIZE ]; +unsigned long tags[RADIX_TREE_TAGS ][RADIX_TREE_TAG_LONGS ]; +}; +struct radix_tree_root {unsigned int height;unsigned int gfp_mask;struct radix_tree_node *rnode; }; +/* +* Support access patterns with strong locality. +*/ +struct radix_tree_cache { +unsigned long first_index; +struct radix_tree_node *tree_node; +}; +#define RADIX_TREE_INIT (mask) { \ .height = 0, \ .gfp_mask = (mask), \
@@ -45,9 +72,13 @@ do { \ }while (0 ) int radix_tree_insert(struct radix_tree_root *, unsigned long, void *); -void *radix_tree_lookup (struct radix_tree_root *,unsigned long ); -void **radix_tree_lookup_slot (struct radix_tree_root *,unsigned long ); +void *radix_tree_lookup_node (struct radix_tree_root *,unsigned long , +unsigned int );void *radix_tree_delete (struct radix_tree_root *,unsigned long ); +unsigned long radix_tree_lookup_head (struct radix_tree_root *root, +unsigned long index,unsigned int max_scan); +unsigned long radix_tree_lookup_tail (struct radix_tree_root *root, +unsigned long index,unsigned int max_scan);unsigned int radix_tree_gang_lookup (struct radix_tree_root *root,void **results,unsigned long first_index,unsigned int max_items);
@@ -69,4 +100,118 @@ static inline void radix_tree_preload_en preempt_enable(); } +/** +* radix_tree_lookup - perform lookup operation on a radix tree +* @root: radix tree root +* @index: index key +* +* Lookup the item at the position @index in the radix tree @root. +*/ +static inline void *radix_tree_lookup (struct radix_tree_root *root, +unsigned long index) +{ +return radix_tree_lookup_node (root, index,0 ); +} + +/** +* radix_tree_lookup_slot - lookup a slot in a radix tree +* @root: radix tree root +* @index: index key +* +* Lookup the slot corresponding to the position @index in the radix tree +* @root. This is useful for update-if-exists operations. +*/ +static inline void **radix_tree_lookup_slot (struct radix_tree_root *root, +unsigned long index) +{ +struct radix_tree_node *node; + + node =radix_tree_lookup_node (root, index,1 ); +return node->slots + (index &RADIX_TREE_MAP_MASK ); +} + +/** +* radix_tree_cache_lookup_node - cached lookup node +* @root: radix tree root +* @cache: look-aside cache +* @index: index key +* +* Lookup the item at the position @index in the radix tree @root, +* and return the node @level levels from the bottom in the search path. +* @cache stores the last accessed upper level tree node by this +* function, and is always checked first before searching in the tree. +* It can improve speed for access patterns with strong locality. +* NOTE: +* - The cache becomes invalid on leaving the lock; +* - Do not intermix calls with different @level. +*/ +static inline void *radix_tree_cache_lookup_node (struct radix_tree_root *root, +struct radix_tree_cache *cache, +unsigned long index,unsigned int level) +{ +struct radix_tree_node *node; +unsigned long i; +unsigned long mask; + +if (level && level >= root->height) +return root->rnode; + + i = ((index >> (level *RADIX_TREE_MAP_SHIFT )) &RADIX_TREE_MAP_MASK ); + mask = ~((RADIX_TREE_MAP_SIZE << (level *RADIX_TREE_MAP_SHIFT )) -1 ); + +if ((index & mask) == cache->first_index) +return cache->tree_node->slots[i]; + + node =radix_tree_lookup_node (root, index, level +1 ); +if (!node) +return 0 ; + + cache->tree_node = node; + cache->first_index = (index & mask); +return node->slots[i]; +} + +/** +* radix_tree_cache_lookup - cached lookup page +* @root: radix tree root +* @cache: look-aside cache +* @index: index key +* +* Lookup the item at the position @index in the radix tree @root. +*/ +static inline void *radix_tree_cache_lookup (struct radix_tree_root *root, +struct radix_tree_cache *cache, +unsigned long index) +{ +return radix_tree_cache_lookup_node (root, cache, index,0 ); +} + +static inline void radix_tree_cache_init (struct radix_tree_cache *cache) +{ + cache->first_index =0x77 ; +} + +static inline int radix_tree_cache_size (struct radix_tree_cache *cache) +{ +return RADIX_TREE_MAP_SIZE ; +} + +static inline int radix_tree_cache_count (struct radix_tree_cache *cache) +{ +if (cache->first_index !=0x77 ) +return cache->tree_node->count; +else +return 0 ; +} + +static inline int radix_tree_cache_full (struct radix_tree_cache *cache) +{ +return radix_tree_cache_count (cache) ==radix_tree_cache_size (cache); +} + +static inline int radix_tree_cache_first_index (struct radix_tree_cache *cache) +{ +return cache->first_index; +} +#endif /* _LINUX_RADIX_TREE_H */
lib/radix-tree.c +107 -35
183 lines not on this page · raw message
--
PATCH 5/13 readahead: some preparation
In reply to Wu FengguangSome random changes that do not fit in elsewhere.
---
mm/readahead.c | 165 ++++++++++++++++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 157 insertions(+), 8 deletions(-)
mm/readahead.c +157 -8
@@ -15,13 +15,36 @@#include <linux/backing-dev.h> #include <linux/pagevec.h> +/* +* Debug facilities. +*/ +#define DEBUG_READAHEAD +#ifdef DEBUG_READAHEAD + +#define dprintk(args...) \ + if (readahead_ratio & 1) printk(KERN_DEBUG args) +#define ddprintk(args...) \ + if ((readahead_ratio & 3) == 3) printk(KERN_DEBUG args) + +#else /* DEBUG_READAHEAD */ + +#define dprintk(args...) do {} while(0) +#define ddprintk(args...) do {} while(0) + +#endif /* DEBUG_READAHEAD */ + + +/* The default max/min read-ahead pages. */ +#define MAX_RA_PAGES (VM_MAX_READAHEAD >> (PAGE_CACHE_SHIFT - 10)) +#define MIN_RA_PAGES (VM_MIN_READAHEAD >> (PAGE_CACHE_SHIFT - 10)) +void default_unplug_io_fn (struct backing_dev_info *bdi,struct page *page) { }EXPORT_SYMBOL (default_unplug_io_fn);struct backing_dev_info default_backing_dev_info = { - .ra_pages = (VM_MAX_READAHEAD *1024 ) /PAGE_CACHE_SIZE , + .ra_pages =MAX_RA_PAGES , .state =0 , .capabilities =BDI_CAP_MAP_COPY , .unplug_io_fn = default_unplug_io_fn,
@@ -50,7 +73,7 @@ static inline unsigned long get_max_readstatic inline unsigned long get_min_readahead (struct file_ra_state *ra) { -return (VM_MIN_READAHEAD *1024 ) /PAGE_CACHE_SIZE ; +return MIN_RA_PAGES ; }static inline void ra_off (struct file_ra_state *ra)
@@ -255,10 +278,11 @@ out:*/ static int __do_page_cache_readahead (struct address_space *mapping,struct file *filp, -unsigned long offset,unsigned long nr_to_read) +unsigned long offset,unsigned long nr_to_read, +unsigned long lookahead_size) {struct inode *inode = mapping->host; -struct page *page; +struct page *page =NULL ;unsigned long end_index;/* The last page we want to read */ LIST_HEAD (page_pool);int page_idx;
@@ -268,7 +292,7 @@ __do_page_cache_readahead(struct addressif (isize ==0 ) goto out; - end_index = ((isize -1 ) >>PAGE_CACHE_SHIFT ); + end_index = ((isize -1 ) >>PAGE_CACHE_SHIFT );/* * Preallocate as many pages as we will need.
@@ -291,6 +315,9 @@ __do_page_cache_readahead(struct addressbreak ; page->index = page_offset;list_add (&page->lru, &page_pool); +if (readahead_ratio >9 && + page_idx == nr_to_read - lookahead_size) +SetPageReadahead (page); ret++; }read_unlock_irq (&mapping->tree_lock);
@@ -327,7 +354,7 @@ int force_page_cache_readahead(struct adif (this_chunk > nr_to_read) this_chunk = nr_to_read; err =__do_page_cache_readahead (mapping, filp, - offset, this_chunk); + offset, this_chunk,0 );if (err <0 ) { ret = err;break ;
@@ -374,7 +401,7 @@ int do_page_cache_readahead(struct addreif (bdi_read_congested (mapping->backing_dev_info))return -1 ; -return __do_page_cache_readahead (mapping, filp, offset, nr_to_read); +return __do_page_cache_readahead (mapping, filp, offset, nr_to_read,0 ); }/*
@@ -394,7 +421,10 @@ blockable_page_cache_readahead(struct adif (!block &&bdi_read_congested (mapping->backing_dev_info))return 0 ; - actual =__do_page_cache_readahead (mapping, filp, offset, nr_to_read); + actual =__do_page_cache_readahead (mapping, filp, offset, nr_to_read,0 ); + +dprintk ("blockable-readahead(ino=%lu, ra=%lu+%lu) = %d\n" , + mapping->host->i_ino, offset, nr_to_read, actual);return check_ra_success (ra, nr_to_read, actual); }
@@ -559,3 +589,122 @@ unsigned long max_sane_readahead(unsigne__get_zone_counts (&active, &inactive, &free,NODE_DATA (numa_node_id ()));return min (nr, (inactive + free) /2 ); } + +/* +* Adaptive read-ahead. +* +* Good read patterns are compact both in space and time. The read-ahead logic +* tries to grant larger read-ahead size to better readers under the constraint +* of system memory and load pressures. +* +* It employs two methods to estimate the max thrashing safe read-ahead size: +* 1. state based - the default one +* 2. context based - the fail safe one +* The integration of the dual methods has the merit of being agile and robust. +* It makes the overall design clean: special cases are handled in general by +* the stateless method, leaving the stateful one simple and fast. +* +* To improve throughput and decrease read delay, the logic 'looks ahead'. +* In every read-ahead chunk, it selects one page and tag it with PG_readahead. +* Later when the page with PG_readahead is to be read, the logic knows that +* it's time to carry out the next read-ahead chunk in advance. +* +* a read-ahead chunk +* +-----------------------------------------+ +* | # PG_readahead | +* +-----------------------------------------+ +* ^ When this page is read, we submit I/O for the next read-ahead. +* +* +* Here are some variable names used frequently: +* +* |<------- la_size ------>| +* +-----------------------------------------+ +* | # | +* +-----------------------------------------+ +* ra_index -->|<---------------- ra_size -------------->| +* +*/ + +#define next_page (pg) (list_entry((pg)->lru.prev, struct page, lru)) +#define prev_page (pg) (list_entry((pg)->lru.next, struct page, lru)) + +/* +* The nature of read-ahead allows most tests to fail or even be wrong. +* Here we just do not bother to call get_page(), it's meaningless anyway. +*/ +static inline struct page *__find_page (struct address_space *mapping, +unsigned long offset) +{ +return radix_tree_lookup (&mapping->page_tree, offset); +} + +struct page *find_page (struct address_space *mapping,unsigned long offset) +{ +struct page *page; + +read_lock_irq (&mapping->tree_lock); + page =__find_page (mapping, offset); +read_unlock_irq (&mapping->tree_lock); +#ifdef DEBUG_READAHEAD_RADIXTREE +if (page) +BUG_ON (page->index != offset); +#endif +return page; +} + +/* +* Move pages in danger (of thrashing) to the head of inactive_list. +* Not expected to happen frequently. +*/ +static int rescue_pages (struct page *page,unsigned long nr_pages) +{ +unsigned long pgrescue; +unsigned long index; +struct address_space *mapping; +struct zone *zone; + +BUG_ON (!nr_pages || !page); + pgrescue =0 ; + index =page_index (page); + mapping =page_mapping (page); + +dprintk ("rescue_pages(ino=%lu, index=%lu nr=%lu)\n" , + mapping->host->i_ino, index, nr_pages); + +for (;;) { + zone =page_zone (page); +spin_lock_irq (&zone->lru_lock); + +if (!PageLRU (page)) + goto out_unlock; + +while (page_mapping (page) == mapping && +page_index (page) == index) { +struct page *the_page = page; + page =next_page (page); +if (!PageActive (the_page) && + !PageActivate (the_page) && + !PageLocked (the_page) && +page_count (the_page) ==1 ) { +list_move (&the_page->lru, &zone->inactive_list); + pgrescue++; + } + index++; +if (!--nr_pages) + goto out_unlock; + } + +spin_unlock_irq (&zone->lru_lock); + + page =find_page (mapping, index); +if (!page) + goto out; + } +out_unlock: +spin_unlock_irq (&zone->lru_lock); +out: +ra_account (0 ,RA_EVENT_READAHEAD_RESCUE , pgrescue); + +return nr_pages ? index :0 ; +}
--
PATCH 6/13 readahead: call scheme
In reply to Wu FengguangAn new page flag PG_readahead is introduced as a look-ahead mark. The look-ahead mark corresponds to `ahead_start' of the current logic.
The read-ahead logic is called when
- read reaches a look-ahead mark;
- read on a non-present page.
And ra_access() is called on every page reference to maintain the cache_hit counter.
This scheme has the following benefits:
- makes all stateful/stateless methods happy;
- eliminates the cache hit problem naturally;
- lives in harmony with application managed read-aheads via fadvise/madvise.
---
include/linux/mm.h | 7 + include/linux/page-flags.h | 5 + mm/filemap.c | 66 +++++++++++++++--- mm/readahead.c | 162 +++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 229 insertions(+), 11 deletions(-)
include/linux/page-flags.h +5 -0
@@ -77,6 +77,7 @@#define PG_nosave_free 18/* Free, should not be written */ #define PG_uncached 19/* Page has been mapped as uncached */ #define PG_activate 20/* delayed activate */ +#define PG_readahead 21/* check readahead when reading this page */ /* * Global page accounting. One instance per CPU. Only unsigned longs are
@@ -315,6 +316,10 @@ extern void __mod_page_state(unsigned lo#define TestClearPageActivate (page) test_and_clear_bit(PG_activate, &(page)->flags)#define TestSetPageActivate (page) test_and_set_bit(PG_activate, &(page)->flags) +#define PageReadahead (page) test_bit(PG_readahead, &(page)->flags) +#define SetPageReadahead (page) set_bit(PG_readahead, &(page)->flags) +#define TestClearPageReadahead (page) test_and_clear_bit(PG_readahead, &(page)->flags) +struct page ;/* forward declaration */ int test_clear_page_dirty (struct page *page);
include/linux/mm.h +7 -0
@@ -944,6 +944,13 @@ unsigned long page_cache_readahead(struvoid handle_ra_miss (struct address_space *mapping,struct file_ra_state *ra,pgoff_t offset);unsigned long max_sane_readahead (unsigned long nr); +unsigned long +page_cache_readahead_adaptive (struct address_space *mapping, +struct file_ra_state *ra,struct file *filp, +struct page *prev_page,struct page *page, +unsigned long first_index, +unsigned long index,unsigned long last_index); +void fastcall ra_access(struct file_ra_state *ra, struct page *page);/* Do stack extension */ extern int expand_stack (struct vm_area_struct *vma,unsigned long address);
mm/filemap.c +55 -11
183 lines not on this page · raw message
mm/readahead.c +162 -0
171 lines not on this page · raw message
--
PATCH 7/13 readahead: tunable parameters
In reply to Wu Fengguang- new entry in /proc/sys/vm/readahead_ratio;
- limit loopdev file read-ahead size to 32kb;
- limit mmap read-around size to 256kb;
- dynamic read-ahead min/max bound.
---
Documentation/sysctl/vm.txt | 14 ++++++++++++++ drivers/block/loop.c | 5 +++++ include/linux/mm.h | 5 ++++- include/linux/sysctl.h | 1 + kernel/sysctl.c | 11 +++++++++++ mm/filemap.c | 7 +++++++ mm/readahead.c | 31 +++++++++++++++++++++++++++++++ 7 files changed, 73 insertions(+), 1 deletion(-)
Documentation/sysctl/vm.txt +14 -0
@@ -27,6 +27,7 @@ Currently, these files are in /proc/sys/ - laptop_mode - block_dump - swap_prefetch +- readahead_ratio ==============================================================
@@ -114,3 +115,16 @@ except when laptop_mode is enabled and t Setting it to 0 disables prefetching entirely. The default value is dependant on ramsize. + +============================================================== + +readahead_ratio + +This limits read-ahead size to percent of the thrashing-threshold. +The thrashing-threshold is dynamicly estimated according to the +_history_ read speed and system load, and used to limit the +_future_ read-ahead request size. So you should set it to a low +value if you have not enough memory to counteract the I/O load +fluctuation. + +The default value is 50.
include/linux/mm.h +4 -1
@@ -927,11 +927,14 @@ extern int filemap_populate(struct vm_arint write_one_page (struct page *page,int wait);/* readahead.c */ -#define VM_MAX_READAHEAD 128/* kbytes */ +#define VM_MAX_READAHEAD 1024/* kbytes */ #define VM_MIN_READAHEAD 16/* kbytes (includes current page) */ #define VM_MAX_CACHE_HIT 256/* max pages in a row in cache before * turning readahead off */ +/* turn on read-ahead thrashing protection if (readahead_ratio >= ##) */ +#define VM_READAHEAD_PROTECT_RATIO 80 +int do_page_cache_readahead (struct address_space *mapping,struct file *filp,unsigned long offset,unsigned long nr_to_read); int force_page_cache_readahead(struct address_space *mapping, struct file *filp,
include/linux/sysctl.h +1 -0
@@ -182,6 +182,7 @@ enumVM_LEGACY_VA_LAYOUT =27 ,/* legacy/compatibility virtual address space layout */ VM_SWAP_TOKEN_TIMEOUT =28 ,/* default time for token time out */ VM_SWAP_PREFETCH =29 ,/* int: amount to swap prefetch */ + VM_READAHEAD_RATIO=30, /* percent of read-ahead size to thrashing-threshold */ };
kernel/sysctl.c +11 -0
@@ -67,6 +67,7 @@ extern int min_free_kbytes;extern int printk_ratelimit_jiffies;extern int printk_ratelimit_burst;extern int pid_max_min, pid_max_max; +extern int readahead_ratio;#if defined(CONFIG_X86_LOCAL_APIC ) && defined(CONFIG_X86 )int unknown_nmi_panic;
@@ -869,6 +870,16 @@ static ctl_table vm_table[] = { },#endif #endif + { + .ctl_name = VM_READAHEAD_RATIO, + .procname = "readahead_ratio", + .data = &readahead_ratio, + .maxlen = sizeof(readahead_ratio), + .mode = 0644, + .proc_handler = &proc_dointvec, + .strategy = &sysctl_intvec, + .extra1 = &zero, + }, { .ctl_name = 0 } };
drivers/block/loop.c +5 -0
@@ -768,6 +768,11 @@ static int loop_set_fd(struct loop_devic mapping = file->f_mapping; inode = mapping->host; +/* +* The upper layer should already do proper read-ahead, +* unlimited read-ahead here only ruins the cache hit rate. +*/ + file->f_ra.ra_pages =32 >> (PAGE_CACHE_SHIFT -10 );if (!(file->f_mode &FMODE_WRITE )) lo_flags |=LO_FLAGS_READ_ONLY ;
mm/filemap.c +7 -0
@@ -1312,6 +1312,13 @@ retry_find:if (ra_pages) {pgoff_t start =0 ; +/* +* Max read-around should be much smaller than +* max read-ahead. +* How about adding a tunable parameter for this? +*/ +if (ra_pages >64 ) + ra_pages =64 ;if (pgoff > ra_pages /2 ) start = pgoff - ra_pages /2 ;do_page_cache_readahead (mapping, file, start, ra_pages);
mm/readahead.c +31 -0
@@ -15,6 +15,13 @@#include <linux/backing-dev.h> #include <linux/pagevec.h> +/* Set look-ahead size to 1/8 of the thrashing-threshold. */ +#define LOOKAHEAD_RATIO 8 + +/* Set read-ahead size to ##% of the thrashing-threshold. */ +int readahead_ratio =0 ; +EXPORT_SYMBOL (readahead_ratio); +/* Detailed classification of read-ahead behaviors. */ #define RA_CLASS_SHIFT 3#define RA_CLASS_MASK ((1 << RA_CLASS_SHIFT) - 1)
@@ -742,6 +749,30 @@ out: }/* +* ra_size is mainly determined by: +* 1. sequential-start: min(KB(16 + mem_mb/16), KB(64)) +* 2. sequential-max: min(KB(64 + mem_mb*64), KB(2048)) +* 3. sequential: (thrashing-threshold) * readahead_ratio / 100 +* +* Table of concrete numbers for 4KB page size: +* (inactive + free) (in MB): 4 8 16 32 64 128 256 512 1024 +* initial ra_size (in KB): 16 16 16 16 20 24 32 48 64 +* max ra_size (in KB): 320 576 1088 2048 2048 2048 2048 2048 2048 +*/ +static inline void get_readahead_bounds (struct file_ra_state *ra, +unsigned long *ra_min, +unsigned long *ra_max) +{ +unsigned long mem_mb; + +#define KB (size) (((size) * 1024) / PAGE_CACHE_SIZE) + mem_mb =nr_free_inactive () *PAGE_CACHE_SIZE /1024 /1024 ; + *ra_max =min (min (KB (64 + mem_mb*64 ),KB (2048 )), ra->ra_pages); + *ra_min =min (min (KB (VM_MIN_READAHEAD + mem_mb/16 ),KB (128 )), *ra_max/2 ); +#undef KB +} + +/* * This is the entry point of the adaptive read-ahead logic. * * It is only called on two conditions:
--
PATCH 8/13 readahead: state based method
In reply to Wu FengguangThis is the fast code path.
Major steps:
- estimate a thrashing safe ra_size;
- assemble the next read-ahead request in file->f_ra;
- submit it.
---
include/linux/fs.h | 8 + mm/readahead.c | 333 +++++++++++++++++++++++++++++++++++++++++++++++++++++ mm/swap.c | 3 mm/vmscan.c | 5 4 files changed, 348 insertions(+), 1 deletion(-)
include/linux/fs.h +7 -1
@@ -564,13 +564,19 @@ struct file_ra_state {unsigned long start;/* Current window */ unsigned long size;unsigned long flags;/* ra flags RA_FLAG_xxx*/ -unsigned long cache_hit;/* cache hit count*/ +uint64_t cache_hit;/* cache hit count*/ unsigned long prev_page;/* Cache last read() position */ unsigned long ahead_start;/* Ahead window */ unsigned long ahead_size;unsigned long ra_pages;/* Maximum readahead window */ unsigned long mmap_hit;/* Cache hit stat for mmap accesses */ unsigned long mmap_miss;/* Cache miss stat for mmap accesses */ + +unsigned long age; +unsigned long la_index; +unsigned long ra_index; +unsigned long lookahead_index; +unsigned long readahead_index; };#define RA_FLAG_MISS 0x01/* a cache miss occured against this file */ #define RA_FLAG_INCACHE 0x02/* file is already in cache */
mm/vmscan.c +5 -0
@@ -370,6 +370,8 @@ static pageout_t pageout(struct page *pareturn PAGE_CLEAN ; } +DECLARE_PER_CPU(unsigned long, smooth_aging); +/* * shrink_list adds the number of reclaimed pages to sc->nr_reclaimed */
@@ -417,6 +419,8 @@ static int shrink_list(struct list_head/* In active use or really unfreeable? Activate it. */ if (referenced &&page_mapping_inuse (page)) goto activate_locked; +if (!referenced) +__get_cpu_var (smooth_aging)++;#ifdef CONFIG_SWAP /*
@@ -774,6 +778,7 @@ refill_inactive_zone(struct zone *zone,list_add (&page->lru, &l_active);continue ; } +__get_cpu_var (smooth_aging)++; }list_add (&page->lru, &l_inactive); }
mm/swap.c +3 -0
@@ -112,6 +112,8 @@ void fastcall activate_page(struct pagespin_unlock_irq (&zone->lru_lock); } +DECLARE_PER_CPU(unsigned long, smooth_aging); +/* * Mark a page as having seen activity. *
@@ -126,6 +128,7 @@ void fastcall mark_page_accessed(structClearPageReferenced (page); }else if (!PageReferenced (page)) {SetPageReferenced (page); +__get_cpu_var (smooth_aging)++; } }
mm/readahead.c +333 -0
345 lines not on this page · raw message
--
PATCH 9/13 readahead: context based method
In reply to Wu FengguangThis is the slow code path.
No valid state info is available, so the page cache is queried to abtain the required position/timing infomation.
Major steps:
- look back/forward to find the ra_index;
- look back to estimate a thrashing safe ra_size;
- assemble the next read-ahead request in file->f_ra;
- submit it.
---
mm/readahead.c | 336 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 335 insertions(+), 1 deletion(-)
mm/readahead.c +335 -1
@@ -1080,6 +1080,340 @@ state_based_readahead(struct address_spareturn ra_dispatch (ra, mapping, filp); } +/* +* Page cache context based estimation of read-ahead/look-ahead size/index. +* +* The logic first looks backward in the inactive_list to get an estimation of +* the thrashing-threshold, and then, if necessary, looks forward to determine +* the start point of next read-ahead. +* +* The estimation theory can be illustrated with figure: +* +* chunk A chunk B chunk C head +* +* l01 l11 l12 l21 l22 +*| |-->|-->| |------>|-->| |------>| +*| +-------+ +-----------+ +-------------+ | +*| | # | | # | | # | | +*| +-------+ +-----------+ +-------------+ | +*| |<==============|<===========================|<============================| +* L0 L1 L2 +* +* Let f(l) = L be a map from +* l: the number of pages read by the stream +* to +* L: the number of pages pushed into inactive_list in the mean time +* then +* f(l01) <= L0 +* f(l11 + l12) = L1 +* f(l21 + l22) = L2 +* ... +* f(l01 + l11 + ...) <= Sum(L0 + L1 + ...) +* <= Length(inactive_list) = f(thrashing-threshold) +* +* So the count of countinuous history pages left in the inactive_list is always +* a lower estimation of the true thrashing-threshold. +*/ + +/* +* STATUS REFERENCE COUNT TYPE +* A__ 0 not in inactive list +* ___ 0 fresh +* __R PAGE_REFCNT_1 stale +* _a_ PAGE_REFCNT_2 disturbed once +* _aR PAGE_REFCNT_3 disturbed twice +* +* A/a/R: Active / aCTIVATE / Referenced +*/ +static inline unsigned long cold_page_refcnt (struct page *page) +{ +if (!page ||PageActive (page)) +return 0 ; + +return page_refcnt (page); +} + +static inline char page_refcnt_symbol (struct page *page) +{ +if (!page) +return 'X' ; +if (PageActive (page)) +return 'A' ; +switch (page_refcnt (page)) { +case 0 : +return '_' ; +case PAGE_REFCNT_1 : +return '-' ; +case PAGE_REFCNT_2 : +return '=' ; +case PAGE_REFCNT_3 : +return '#' ; + } +return '?' ; +} + +/* +* Count/estimate cache hits in range [first_index, last_index]. +* The estimation is simple and a bit optimistic. +*/ +static int count_cache_hit (struct address_space *mapping, +unsigned long first_index,unsigned long last_index) +{ +static int steps[8 ] = {0 ,4 ,2 ,6 ,1 ,3 ,5 ,7 }; +struct page *page; +int size = last_index - first_index +1 ; +int count =0 ; +int i; + +read_lock_irq (&mapping->tree_lock); + +for (i =0 ; i <8 ;) { + page =__find_page (mapping, + first_index + size * steps[i++] /8 ); +if (cold_page_refcnt (page) >=PAGE_REFCNT_1 && ++count >=2 ) +break ; + } + +read_unlock_irq (&mapping->tree_lock); + +return size * count / i; +} + +/* +* Look back and check history pages to estimate thrashing-threshold. +*/ +static int query_page_cache (struct address_space *mapping, +unsigned long *remain,unsigned long offset, +unsigned long ra_min,unsigned long ra_max) +{ +int step; +int count; +unsigned long index; +unsigned long nr_lookback; +struct radix_tree_cache cache; + +/* +* Scan backward and check the near @ra_max pages. +* The count here determines ra_size. +*/ +read_lock_irq (&mapping->tree_lock); + index =radix_tree_lookup_head (&mapping->page_tree, offset, ra_max); +read_unlock_irq (&mapping->tree_lock); +#ifdef DEBUG_READAHEAD_RADIXTREE +if (index <= offset) { +WARN_ON (!find_page (mapping, index)); +if (index + ra_max > offset) +WARN_ON (find_page (mapping, index -1 )); + }else { +BUG_ON (index > offset +1 ); +WARN_ON (find_page (mapping, offset)); + } +#endif + + *remain = offset - index +1 ; + +if (unlikely (*remain <= ra_min)) { + count = ra_min; + goto out; + } + + count =count_cache_hit (mapping, index, offset); +if (count < ra_min) + count = ra_min; +if (unlikely (count *2 < offset - index)) + goto out; + +if (*remain < ra_max) + goto out; + +/* +* Check the far pages coarsely. +* The big count here helps increase la_size. +*/ + nr_lookback = ra_max * (LOOKAHEAD_RATIO +1 ) * +100 / (readahead_ratio +1 ); +if (nr_lookback > offset) + nr_lookback = offset; + +radix_tree_cache_init (&cache); +read_lock_irq (&mapping->tree_lock); +for (step =2 * ra_max; step < nr_lookback; step += ra_max) { +struct radix_tree_node *node; + node =radix_tree_cache_lookup_node (&mapping->page_tree, + &cache, offset - step,1 ); +if (!node) +break ; +#ifdef DEBUG_READAHEAD_RADIXTREE +if (node !=radix_tree_lookup_node (&mapping->page_tree, + offset - step,1 )) { +read_unlock_irq (&mapping->tree_lock); +printk (KERN_ERR "check radix_tree_cache_lookup_node!\n" ); +return 1 ; + } +#endif + } +read_unlock_irq (&mapping->tree_lock); + +/* +* For sequential read that extends from index 0, the counted value +* may well be far under the true threshold, so return it unmodified +* for further process in adjust_rala_accelerated(). +*/ +if (step < offset) + count = step * readahead_ratio /100 ; +else + count = offset; + +out: +return count; +} + +/* +* Scan backward in the file for the first non-present page. +*/ +static inline unsigned long first_absent_page_bw (struct address_space *mapping, +unsigned long index,unsigned long max_scan) +{ +struct radix_tree_cache cache; +struct page *page; +unsigned long origin; + + origin = index; +if (max_scan > index) + max_scan = index; +radix_tree_cache_init (&cache); +read_lock_irq (&mapping->tree_lock); +for (;;) { + page =radix_tree_cache_lookup (&mapping->page_tree, + &cache, --index); +if (page) { + index++; +break ; + } +if (origin - index > max_scan) +break ; + } +read_unlock_irq (&mapping->tree_lock); + +return index; +} + +/* +* Scan forward in the file for the first non-present page. +*/ +static inline unsigned long first_absent_page (struct address_space *mapping, +unsigned long index,unsigned long max_scan) +{ +unsigned long ra_index; + +read_lock_irq (&mapping->tree_lock); + ra_index =radix_tree_lookup_tail (&mapping->page_tree, + index +1 , max_scan); +read_unlock_irq (&mapping->tree_lock); + +#ifdef DEBUG_READAHEAD_RADIXTREE +BUG_ON (ra_index <= index); +if (index + max_scan > index) { +if (ra_index <= index + max_scan) +WARN_ON (find_page (mapping, ra_index)); +WARN_ON (!find_page (mapping, ra_index -1 )); + } +#endif + +if (ra_index <= index + max_scan) +return ra_index; +else +return 0 ; +} + +/* +* Determine the request parameters for context based read-ahead that extends +* from start of file. +* +* The major weakness of stateless method is perhaps the slow grow up speed of +* ra_size. The logic tries to make up for this in the important case of +* sequential reads that extend from start of file. In this case, the ra_size +* is not choosed to make the whole next chunk safe(as in normal ones). Only +* half of which is safe. The added 'unsafe' half is the look-ahead part. It +* is expected to be safeguarded by rescue_pages() when the previous chunks are +* lost. +*/ +static inline int adjust_rala_accelerated (unsigned long ra_max, +unsigned long *ra_size,unsigned long *la_size) +{ +if (*ra_size <= *la_size) +return 0 ; + + *la_size = (*ra_size - *la_size) * readahead_ratio /100 ; + *ra_size = *la_size *2 ; + +if (*ra_size > ra_max) + *ra_size = ra_max; +if (*la_size > *ra_size) + *la_size = *ra_size; + +return 1 ; +} + +/* +* Main function for page context based read-ahead. +*/ +static inline int +try_context_based_readahead (struct address_space *mapping, +struct file_ra_state *ra, +struct page *prev_page,struct page *page, +unsigned long index, +unsigned long ra_min,unsigned long ra_max) +{ +unsigned long ra_index; +unsigned long la_size; +unsigned long ra_size; +unsigned long remain_pages; + +/* Where to start read-ahead? +* NFSv3 daemons may process adjecent requests in parallel, +* leading to many locally disordered, globally sequential reads. +* So do not require nearby history pages to be present or accessed. +*/ +if (page) { + ra_index =first_absent_page (mapping, index, ra_max *5 /4 ); +if (unlikely (!ra_index)) +return -1 ; + }else if (!prev_page) { + ra_index =first_absent_page_bw (mapping, index, ra_min); +if (index - ra_index > ra_min) +return 0 ; + ra_min += index - ra_index; + index = ra_index; + }else + ra_index = index; + + ra_size =query_page_cache (mapping, &remain_pages, + index -1 , ra_min, ra_max); + + la_size = ra_index - index; +if (readahead_ratio <VM_READAHEAD_PROTECT_RATIO && + remain_pages <= la_size && la_size >1 ) { +rescue_pages (page, la_size); +return -1 ; + } + +if (ra_size == index) { +if (!adjust_rala_accelerated (ra_max, &ra_size, &la_size)) +return -1 ; +set_ra_class (ra,RA_CLASS_CONTEXT_ACCELERATED ); + }else { +if (!adjust_rala (ra_max, &ra_size, &la_size)) +return -1 ; +set_ra_class (ra,RA_CLASS_CONTEXT ); + } + +ra_state_init (ra, index, ra_index); +ra_state_update (ra, ra_size, la_size); + +return 1 ; +} +/* * ra_size is mainly determined by:
@@ -1180,7 +1514,7 @@ page_cache_readahead_adaptive(struct add* Context based sequential read-ahead. */ ret =try_context_based_readahead (mapping, ra, prev_page, page, - index, size, ra_min, ra_max); + index, ra_min, ra_max);if (ret >0 )return ra_dispatch (ra, mapping, filp);if (ret <0 )
--
PATCH 10/13 readahead: other methods
In reply to Wu FengguangVarious read-ahead strategies for:
- fresh read from start of file
- backward prefetching
- seek and read one record pattern(db workload)
- quick recover from thrashing
---
mm/readahead.c | 108 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 108 insertions(+)
mm/readahead.c +108 -0
@@ -1414,6 +1414,114 @@ try_context_based_readahead(struct addrereturn 1 ; } +/* +* Read-ahead on start of file. +* +* It is most important for small files. +* 1. Set a moderate large read-ahead size; +* 2. Issue the next read-ahead request as soon as possible. +* +* But be careful, there are some applications that dip into only the very head +* of a file. The most important thing is to prevent them from triggering the +* next (much larger) read-ahead request, which leads to lots of cache misses. +* Two pages should be enough for them, correct me if I'm wrong. +*/ +static inline unsigned long +newfile_readahead (struct address_space *mapping, +struct file *filp,struct file_ra_state *ra, +unsigned long req_size,unsigned long ra_min) +{ +unsigned long ra_size; +unsigned long la_size; + +if (req_size > ra_min) + req_size = ra_min; + + ra_size =4 * req_size; + la_size =2 * req_size; + +set_ra_class (ra,RA_CLASS_NEWFILE ); +ra_state_init (ra,0 ,0 ); +ra_state_update (ra, ra_size, la_size); + +return ra_dispatch (ra, mapping, filp); +} + +/* +* Backward prefetching. +* No look ahead and thrashing threshold estimation for stepping backward +* pattern: should be unnecessary. +*/ +static inline int +try_read_backward (struct file_ra_state *ra, +unsigned long begin_index,unsigned long end_index, +unsigned long ra_size, +unsigned long ra_min,unsigned long ra_max) +{ +if (ra_size > ra_max || end_index > ra->prev_page) +return 0 ; + +if (ra_has_index (ra, ra->prev_page)) { +if (end_index > ra->la_index) +return 0 ; + ra_size +=2 *ra_cache_hit (ra,0 ); + end_index = ra->la_index; + }else { + ra_size += ra_min; + end_index = ra->prev_page; + } + +if (ra_size > ra_max) + ra_size = ra_max; + +if (end_index > begin_index + ra_size) +return 0 ; + + begin_index = end_index - ra_size; + +set_ra_class (ra,RA_CLASS_BACKWARD ); +ra_state_init (ra, begin_index, begin_index); +ra_state_update (ra, ra_size,0 ); + +return 1 ; +} + +/* +* If there is a previous sequential read, it is likely to be another +* sequential read at the new position. +* Databases are known to have this seek-and-read-one-record pattern. +*/ +static inline int +try_random_readahead (struct file_ra_state *ra,unsigned long index, +unsigned long ra_size,unsigned long ra_max) +{ +unsigned long hit0 =ra_cache_hit (ra,0 ); +unsigned long hit1 =ra_cache_hit (ra,1 ) + hit0; +unsigned long hit2 =ra_cache_hit (ra,2 ); +unsigned long hit3 =ra_cache_hit (ra,3 ); + +if (!ra_has_index (ra, ra->prev_page)) +return 0 ; + +if (index == ra->prev_page +1 ) {/* read after thrashing */ + ra_size = hit0; +set_ra_class (ra,RA_CLASS_RANDOM_THRASHING ); +ra_account (ra,RA_EVENT_READAHEAD_THRASHING , + ra->readahead_index - index); + }else if (ra_size < hit1 &&/* read after seeking */ + hit1 > hit2 /2 && + hit2 > hit3 /2 && + hit3 > hit1 /2 ) { + ra_size =min (ra_max, hit1); +set_ra_class (ra,RA_CLASS_RANDOM_SEEK ); + }else +return 0 ; + +ra_state_init (ra, index, index); +ra_state_update (ra, ra_size,0 ); + +return 1 ; +}/* * ra_size is mainly determined by:
--
PATCH 11/13 readahead: thrashing protection
In reply to Wu FengguangIt tries to identify and protect live pages (sequential pages that are going to be accessed in the near future) on vmscan time.
Almost all live pages can be sorted out and saved. The only problem is that relavant pages can be mutilated into tiny chunks and ignored as random pages.
The cost: dead pages that won't be read will be kept in inactive_list for another round. Hopefully there won't be much.
This feature is greatly demanded by file servers, though should be useless for desktop. It saves the case when one fast reader flushes the cache and strips the read-ahead size of slow readers, or the case where caching is just a wasting of memory. Then you can raise readahead_ratio to 200 or more to enforce a larger read-ahead size and strip the useless cached data out of lru.
Its use is currently limited, for the context based method is not ready for the case. This problem will be solved when the timing info of evicted pages are available. It can also be extended to further benefit large memory systems. I'll leave them as future work.
---
include/linux/mm.h | 2 include/linux/page-flags.h | 2 mm/page_alloc.c | 2 mm/readahead.c | 319 ++++++++++++++++++++++++++++++++++++++++++++- mm/vmscan.c | 8 + 5 files changed, 332 insertions(+), 1 deletion(-)
include/linux/page-flags.h +2 -0
@@ -107,6 +107,8 @@ struct page_state {unsigned long pgfree;/* page freeings */ unsigned long pgactivate;/* pages moved inactive->active */ unsigned long pgdeactivate;/* pages moved active->inactive */ +unsigned long pgkeephot;/* pages sent back to active */ +unsigned long pgkeepcold;/* pages sent back to inactive */ unsigned long pgfault;/* faults (major+minor) */ unsigned long pgmajfault;/* faults (major only) */
include/linux/mm.h +2 -0
@@ -954,6 +954,8 @@ page_cache_readahead_adaptive(struct addunsigned long first_index, unsigned long index, unsigned long last_index); void fastcall ra_access(struct file_ra_state *ra, struct page *page); +int rescue_ra_pages (struct list_head *page_list,struct list_head *save_list); +/* Do stack extension */ extern int expand_stack (struct vm_area_struct *vma,unsigned long address);
mm/page_alloc.c +2 -0
@@ -2389,6 +2389,8 @@ static char *vmstat_text[] = {"pgfree" ,"pgactivate" ,"pgdeactivate" , +"pgkeephot" , +"pgkeepcold" ,"pgfault" , "pgmajfault",
mm/vmscan.c +8 -0
@@ -370,6 +370,7 @@ static pageout_t pageout(struct page *pareturn PAGE_CLEAN ; } +extern int readahead_ratio; DECLARE_PER_CPU(unsigned long, smooth_aging);/*
@@ -380,10 +381,14 @@ static int shrink_list(struct list_headLIST_HEAD (ret_pages);struct pagevec freed_pvec;int pgactivate =0 ; +int pgkeep =0 ;int reclaimed =0 ;cond_resched (); +if (readahead_ratio >=VM_READAHEAD_PROTECT_RATIO ) +rescue_ra_pages (page_list, &ret_pages); +pagevec_init (&freed_pvec,1 );while (!list_empty (page_list)) {struct address_space *mapping;
@@ -569,11 +574,13 @@ keep_locked: keep:list_add (&page->lru, &ret_pages);BUG_ON (PageLRU (page)); + pgkeep++; }list_splice (&ret_pages, page_list);if (pagevec_count (&freed_pvec))__pagevec_release_nonlru (&freed_pvec);mod_page_state (pgactivate, pgactivate); +mod_page_state (pgkeepcold, pgkeep - pgactivate); sc->nr_reclaimed += reclaimed;return reclaimed; }
@@ -837,6 +844,7 @@ refill_inactive_zone(struct zone *zone,mod_page_state_zone (zone, pgrefill, pgscanned);mod_page_state (pgdeactivate, pgdeactivate); +mod_page_state (pgkeephot, pgmoved); }/*
mm/readahead.c +318 -1
328 lines not on this page · raw message
--
PATCH 12/13 readahead: events accounting
In reply to Wu FengguangA debugfs file named `readahead' is created according to advices from J?rn Engel, Andrew Morton and Ingo Oeser. It yields to much better readability than the preious /proc/vmstat interface :)
It reveals various read-ahead activities/events, and is vital to the testing.
This is a trimmed down output on my PC:
# cat /debugfs/readahead [table requests] total newfile state context contexta cache_miss 234 50 10 54 0 read_random 116 37 4 21 0 io_congestion 0 0 0 0 0 io_cache_hit 61 0 14 37 0 io_block 4994 1576 232 120 2 readahead 1812 1522 171 107 2 lookahead 596 442 76 78 0 lookahead_hit 214 89 70 32 1 readahead_eof 1208 1080 95 21 2 readahead_shrink 0 0 0 0 0 readahead_thrash 0 0 0 0 0 readahead_rescue 1275 0 0 0 0
[table pages] total newfile state context contexta
cache_miss 401 98 50 87 0 read_random 120 37 4 21 0 io_congestion 0 0 0 0 0 io_cache_hit 872 0 412 448 0 io_block 22833 3857 12177 3680 21 readahead 15081 3827 9683 1484 21 readahead_hit 12684 3441 8164 1001 22 lookahead 8191 936 6926 329 0 lookahead_hit 8140 230 6368 1542 0 readahead_eof 5192 1955 2777 373 21 readahead_shrink 0 0 0 0 0 readahead_thrash 0 0 0 0 0 readahead_rescue 13656 0 0 0 0
[table summary] total newfile state context contexta
random_rate 6% 2% 2% 16% 0% ra_hit_rate 84% 89% 84% 67% 100% la_hit_rate 35% 20% 90% 40% 100% avg_ra_size 8 3 56 14 7 avg_la_size 14 2 90 4 0
---
mm/readahead.c | 194 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 192 insertions(+), 2 deletions(-)
mm/readahead.c +192 -2
@@ -47,6 +47,9 @@ enum ra_class {enum ra_event {RA_EVENT_CACHE_MISS ,/* read cache misses */ RA_EVENT_READRANDOM ,/* random reads */ +RA_EVENT_IO_CONGESTION ,/* io congestion */ +RA_EVENT_IO_CACHE_HIT ,/* canceled io due to cache hit */ +RA_EVENT_IO_BLOCK ,/* read on locked page */ RA_EVENT_READAHEAD ,/* read-ahead issued */ RA_EVENT_READAHEAD_HIT ,/* read-ahead page hit */
@@ -65,6 +68,177 @@ enum ra_event {*/ #define DEBUG_READAHEAD #ifdef DEBUG_READAHEAD +#include <linux/jiffies.h> +#include <linux/debugfs.h> +#include <linux/seq_file.h> +#include <linux/init.h> + +static char *ra_class_name[] = { +"total" , +"newfile" , +"state" , +"context" , +"contexta" , +"backward" , +"onthrash" , +"onraseek" , +"none" , +}; + +static char *ra_event_name[] = { +"cache_miss" , +"read_random" , +"io_congestion" , +"io_cache_hit" , +"io_block" , +"readahead" , +"readahead_hit" , +"lookahead" , +"lookahead_hit" , +"readahead_eof" , +"readahead_shrink" , +"readahead_thrash" , +"readahead_rescue" , +}; + +static unsigned long ra_event_count[RA_CLASS_END +1 ][RA_EVENT_END ][2 ]; + +static inline void ra_account (struct file_ra_state *ra, +enum ra_event e,int pages) +{ +enum ra_class c; + + c = (ra ? ra->flags &RA_CLASS_MASK :RA_CLASS_END ); +if (e ==RA_EVENT_READAHEAD_HIT && pages <0 ) { + c = (ra->flags >>RA_CLASS_SHIFT ) &RA_CLASS_MASK ; + pages = -pages; + } +if (!c) + c =RA_CLASS_END ; +BUG_ON (c >RA_CLASS_END ); + + ra_event_count[c][e][0 ] +=1 ; + ra_event_count[c][e][1 ] += pages; +} + +static int ra_account_show (struct seq_file *s,void *_) +{ +int i; +int c; +int e; +static char event_fmt[] ="%-16s" ; +static char class_fmt[] ="%11s" ; +static char item_fmt[] ="%11lu" ; +static char percent_format[] ="%10lu%%" ; +static char *table_name[] = { +"[table requests]" , +"[table pages]" , +"[table summary]" }; + +for (i =0 ; i <=1 ; i++) { +for (e =0 ; e <RA_EVENT_END ; e++) { + ra_event_count[0 ][e][i] =0 ; +for (c =1 ; c <=RA_CLASS_END ; c++) + ra_event_count[0 ][e][i] += + ra_event_count[c][e][i]; + } + +seq_printf (s, event_fmt, table_name[i]); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, class_fmt, ra_class_name[c]); +seq_puts (s,"\n" ); + +for (e =0 ; e <RA_EVENT_END ; e++) { +if (e ==RA_EVENT_READAHEAD_HIT && i ==0 ) +continue ; + +seq_printf (s, event_fmt, ra_event_name[e]); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, item_fmt, + ra_event_count[c][e][i]); +seq_puts (s,"\n" ); + } +seq_puts (s,"\n" ); + } + +seq_printf (s, event_fmt, table_name[2 ]); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, class_fmt, ra_class_name[c]); +seq_puts (s,"\n" ); + +seq_printf (s, event_fmt,"random_rate" ); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, percent_format, + (ra_event_count[c][RA_EVENT_READRANDOM ][0 ] *100 ) / + (ra_event_count[c][RA_EVENT_READRANDOM ][0 ] + + ra_event_count[c][RA_EVENT_READAHEAD ][0 ] +1 )); +seq_puts (s,"\n" ); + +seq_printf (s, event_fmt,"ra_hit_rate" ); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, percent_format, + (ra_event_count[c][RA_EVENT_READAHEAD_HIT ][1 ] *100 ) / + (ra_event_count[c][RA_EVENT_READAHEAD ][1 ] +1 )); +seq_puts (s,"\n" ); + +seq_printf (s, event_fmt,"la_hit_rate" ); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, percent_format, + (ra_event_count[c][RA_EVENT_LOOKAHEAD_HIT ][0 ] *100 ) / + (ra_event_count[c][RA_EVENT_LOOKAHEAD ][0 ] +1 )); +seq_puts (s,"\n" ); + +seq_printf (s, event_fmt,"avg_ra_size" ); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, item_fmt, + (ra_event_count[c][RA_EVENT_READAHEAD ][1 ] + + ra_event_count[c][RA_EVENT_READAHEAD ][0 ] /2 ) / + (ra_event_count[c][RA_EVENT_READAHEAD ][0 ] +1 )); +seq_puts (s,"\n" ); + +seq_printf (s, event_fmt,"avg_la_size" ); +for (c =0 ; c <=RA_CLASS_END ; c++) +seq_printf (s, item_fmt, + (ra_event_count[c][RA_EVENT_LOOKAHEAD ][1 ] + + ra_event_count[c][RA_EVENT_LOOKAHEAD ][0 ] /2 ) / + (ra_event_count[c][RA_EVENT_LOOKAHEAD ][0 ] +1 )); +seq_puts (s,"\n" ); + +return 0 ; +} + +static struct dentry *readahead_dentry; + +static int ra_debug_open (struct inode *inode,struct file *file) +{ +return single_open (file, ra_account_show,NULL ); +} + +static ssize_t ra_debug_write (struct file *file,const char __user *buf, +size_t size,loff_t *offset) +{ +if (file->f_dentry == readahead_dentry) +memset (ra_event_count,0 ,sizeof (ra_event_count)); +return 1 ; +} + +static struct file_operations ra_debug_fops = { + .owner =THIS_MODULE , + .open = ra_debug_open, + .write = ra_debug_write, + .read = seq_read, + .llseek = seq_lseek, + .release = single_release, +}; + +static int __init readahead_init (void ) +{ + readahead_dentry =debugfs_create_file ("readahead" , +0644 ,NULL ,NULL , &ra_debug_fops); +return 0 ; +} + +module_init(readahead_init) #define dprintk(args...) \ if (readahead_ratio & 1) printk(KERN_DEBUG args)
@@ -73,6 +247,10 @@ enum ra_event {#else /* DEBUG_READAHEAD */ +static inline void ra_account (struct file_ra_state *ra, +enum ra_event e,int pages) +{ +} #define dprintk(args...) do {} while(0) #define ddprintk(args...) do {} while(0)
@@ -945,6 +1123,8 @@ static int ra_dispatch(struct file_ra_stra_account (ra,RA_EVENT_READAHEAD_EOF , actual);if (la_size)ra_account (ra,RA_EVENT_LOOKAHEAD , la_size); +if (ra_size > actual) +ra_account (ra,RA_EVENT_IO_CACHE_HIT , ra_size - actual);ra_account (ra,RA_EVENT_READAHEAD , actual); dprintk("readahead-%s(ino=%lu, index=%lu, ra=%lu+%lu-%lu) = %d\n",
@@ -1577,8 +1757,11 @@ page_cache_readahead_adaptive(struct addif (page) {if (!TestClearPageReadahead (page))return 0 ; -if (bdi_read_congested (mapping->backing_dev_info)) +if (bdi_read_congested (mapping->backing_dev_info)) { +ra_account (ra,RA_EVENT_IO_CONGESTION , + end_index - index);return 0 ; + } }if (page)
@@ -1665,8 +1848,15 @@ void fastcall ra_access(struct file_ra_s (1 << PG_referenced)))return ; -if (!ra_has_index (ra, page->index)) +if (ra_has_index (ra, page->index)) { +if (PageLocked (page)) +ra_account (ra,RA_EVENT_IO_BLOCK , + ra->readahead_index - page->index); + }else { +if (PageLocked (page)) +ra_account (0 ,RA_EVENT_IO_BLOCK ,1 );return ; + } ra->cache_hit++;
--
PATCH 13/13 readahead: page aging accounting
In reply to Wu FengguangThe accuracy of stateful thrashing-threshold estimation depends largely on the measurement of cold page aging speed.
A file named pageaging' is created in debugfs to monitor the trace of two measurement variables: per-zone nr_page_aging' and per-cpu `smooth_aging'. Their values and the jiffies are recorded each time one of them has a delta of 1, 1/2, 1/4, 1/16, 1/256, 1/4096 (nr_inactive + nr_free).
Sample series of collected data shows that smooth_aging is more stable in small sampling granularity:
time dt page_aging8 smooth_aging8 872765 26 520056 33 653782 163 872791 12 520089 132 653945 51 872803 4 520221 132 653996 66 872807 17 520353 165 654062 107 872824 22 520518 99 654169 74 872846 372 520617 99 654243 78 873218 294 520716 99 654321 73 873512 196 520815 99 654394 130 873708 15 520914 231 654524 28 873723 15 521145 198 654552 9 873738 881 521343 99 654561 182 874619 700 521442 0 654743 198 875319 384 521442 66 654941 110 875703 2119 521508 99 655051 1632 877822 3960 521607 0 656683 980 881782 904 521607 0 657663 216
time dt page_aging1 smooth_aging1
-90822 12418 5775 12999 33302 10767 -78404 17510 18774 10303 44069 10345 -60894 24757 29077 9871 54414 14615 -36137 19194 38948 10404 69029 13726 -16943 19636 49352 10440 82755 12865 2693 16299 59792 12453 95620 10734 18992 19851 72245 10073 106354 15960 38843 16099 82318 10767 122314 14059 54942 16094 93085 10041 136373 12117 71036 19888 103126 12595 148490 16155 90924 18452 115721 9782 164645 11705 109376 22395 125503 10214 176350 13679 131771 19310 135717 10759 190029 11843 151081 20793 146476 10699 201872 12595 171874 22308 157175 10321 214467 13157 194182 17954 167496 10773 227624 14803 212136 19946 178269 10554 242427 13391 232082 21051 188823 11179 255818 11783
---
mm/readahead.c | 134 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 133 insertions(+), 1 deletion(-)
mm/readahead.c +133 -1
@@ -207,11 +207,130 @@ static int ra_account_show(struct seq_fireturn 0 ; } +/* +* Measure the aging progress of cold pages over time. +*/ +#define AGING_INFO_SIZE (1 << 8) +#define AGING_INFO_MASK (AGING_INFO_SIZE - 1) +static int aging_info_shift[] = {0 ,1 ,2 ,4 ,8 ,12 }; +#define AGING_INFO_SHIFTS (sizeof(aging_info_shift)/\ + sizeof(aging_info_shift[0])) +static int aging_info_index[AGING_INFO_SHIFTS ]; +static unsigned long aging_info[AGING_INFO_SIZE ][AGING_INFO_SHIFTS *3 ]; +static spinlock_t aging_info_lock =SPIN_LOCK_UNLOCKED ; + +static unsigned long nr_free_inactive (void ); +static unsigned long nr_page_aging (void ); +static unsigned long nr_smooth_aging (void ); + +static void collect_aging_info (void ) +{ +int i; +unsigned long mem; +unsigned long page_aging; +unsigned long smooth_aging; + + mem =nr_free_inactive (); + page_aging =nr_page_aging (); + smooth_aging =nr_smooth_aging (); + +spin_lock_irq (&aging_info_lock); + +for (i =AGING_INFO_SHIFTS -1 ; i >=0 ; i--) { +if (smooth_aging - aging_info[aging_info_index[i]][i*3 +2 ] + + page_aging - aging_info[aging_info_index[i]][i*3 +1 ] > +2 * (mem >> aging_info_shift[i])) { + aging_info_index[i]++; + aging_info_index[i] &=AGING_INFO_MASK ; + aging_info[aging_info_index[i]][i*3 ] = jiffies; + aging_info[aging_info_index[i]][i*3 +1 ] = page_aging; + aging_info[aging_info_index[i]][i*3 +2 ] = smooth_aging; + }else +break ; + } + +spin_unlock_irq (&aging_info_lock); +} + +static void *aginginfo_start (struct seq_file *s,loff_t *pos) +{ +int n = *pos; +int i; + +spin_lock_irq (&aging_info_lock); + +if (!n) { +for (i =0 ; i <AGING_INFO_SHIFTS ; i++) { +seq_printf (s,"%12s %10s %18s%d %18s%d\t" ,"time" ,"dt" , +"page_aging" , aging_info_shift[i], +"smooth_aging" , aging_info_shift[i]); + } +seq_puts (s,"\n" ); + } + +if (++n <AGING_INFO_SIZE ) +return (void *)n; +else +return NULL ; +} + +static void *aginginfo_next (struct seq_file *s,void *p,loff_t *pos) +{ +int n = (int )p; + + ++*pos; +return (void *)(++n <AGING_INFO_SIZE ? n :0 ); +} + +static void aginginfo_stop (struct seq_file *s,void *p) +{ +spin_unlock_irq (&aging_info_lock); +} + +static int aginginfo_show (struct seq_file *s,void *p) +{ +int n = (int )p; +int i; +int index0; +int index1; +long time; +unsigned long nr1; +unsigned long nr2; + +for (i =0 ; i <AGING_INFO_SHIFTS ; i++) { + index0 = aging_info_index[i] + n; + index1 = aging_info_index[i] + n +1 ; + index0 &=AGING_INFO_MASK ; + index1 &=AGING_INFO_MASK ; + time = aging_info[index0][i*3 ]; + nr1 = aging_info[index1][i*3 +1 ] - aging_info[index0][i*3 +1 ]; + nr2 = aging_info[index1][i*3 +2 ] - aging_info[index0][i*3 +2 ]; +seq_printf (s,"%12ld %10lu %10lu %8lu %10lu %8lu\t" , + time, aging_info[index1][i*3 ] - time, + aging_info[index0][i*3 +1 ], nr1, + aging_info[index0][i*3 +2 ], nr2); + } +seq_puts (s,"\n" ); + +return 0 ; +} +static struct dentry *readahead_dentry; +static struct dentry *pageaging_dentry; + +struct seq_operations aginginfo_ops = { + .start = aginginfo_start, + .next = aginginfo_next, + .stop = aginginfo_stop, + .show = aginginfo_show, +};static int ra_debug_open (struct inode *inode,struct file *file) { -return single_open (file, ra_account_show,NULL ); +if (file->f_dentry == readahead_dentry) +return single_open (file, ra_account_show,NULL ); +else +return seq_open (file, &aginginfo_ops); } static ssize_t ra_debug_write(struct file *file, const char __user *buf,
@@ -231,10 +350,20 @@ static struct file_operations ra_debug_f .release = single_release, }; +static struct file_operations aginginfo_fops = { + .open = ra_debug_open, + .read = seq_read, + .llseek = seq_lseek, + .release = seq_release, +}; + +static int __init readahead_init (void ) { readahead_dentry =debugfs_create_file ("readahead" ,0644 ,NULL ,NULL , &ra_debug_fops); + pageaging_dentry =debugfs_create_file ("pageaging" , +0644 ,NULL ,NULL , &aginginfo_fops);return 0 ; }
@@ -1219,6 +1348,9 @@ static inline unsigned long compute_thraelse *remain =0 ; +#ifdef DEBUG_READAHEAD +collect_aging_info (); +#endif ddprintk("compute_thrashing_threshold: " "ra=%lu=%lu*%lu/%lu, remain %lu for %lu\n", ra_size, stream_shift, global_size, global_shift,
--
Wu,
I will run some standard benchmarks and post the results soon. The benchmarks I have are (1) DSS (TPCH) (2) iozone (3) sysbench (4) tiobench
It will take atleast a week, since some disks on my machine have to be replaced and the setup has to be remade.
RP
Quoted 15 lines · Wu Fengguang
This version of adaptive read-ahead patch features various clean ups. Changes include:
- rewrote context based method to make it clean and robust
- improved accuracy of stateful thrashing threshold estimation
-nr_page_agingmade right
- sorted out the thrashing protection logic
- enhanced debug/accounting facilitiesRegards,
Wu Fengguang
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Quoted 8 lines · Ram Pai
Wu,
I will run some standard benchmarks and post the results soon. The benchmarks I have are (1) DSS (TPCH) (2) iozone (3) sysbench (4) tiobench
It will take atleast a week, since some disks on my machine have to be replaced and the setup has to be remade.
Thank you for taking the trouble. In the meantime I will prepare a patch against the stable 2.6.14, so that you can comfortably run vanilla 2.6.14 as the reference point.
Regards,
Wu Fengguang