The environment variable XENPAGING_POLICY_MRU_SIZE will change the mru_size in the policy at runtime. Specifying the mru_size at runtime allows the admin to keep more pages in memory so guests can make more progress. Its also good for development to reduce the value to put more pressure on the paging related code paths. Signed-off-by: Olaf Hering --- Note: some work is still required to actually get the value into the environment of the process. Maybe it will become an config option at some point. tools/xenpaging/policy_default.c | 23 ++++++++++++++++++----- tools/xenpaging/xenpaging.c | 8 ++++++++ tools/xenpaging/xenpaging.h | 1 + 3 files changed, 27 insertions(+), 5 deletions(-) --- xen-unstable.hg-4.1.22571.orig/tools/xenpaging/policy_default.c +++ xen-unstable.hg-4.1.22571/tools/xenpaging/policy_default.c @@ -26,11 +26,12 @@ #include "policy.h" -#define MRU_SIZE (1024 * 16) +#define DEFAULT_MRU_SIZE (1024 * 16) -static unsigned long mru[MRU_SIZE]; +static unsigned long *mru; static unsigned int i_mru; +static unsigned int mru_size; static unsigned long *bitmap; static unsigned long *unconsumed; static unsigned long current_gfn; @@ -57,7 +58,19 @@ int policy_init(xenpaging_t *paging) max_pages = paging->domain_info->max_pages; /* Initialise MRU list of paged in pages */ - for ( i = 0; i < MRU_SIZE; i++ ) + if ( paging->policy_mru_size > 0 ) + mru_size = paging->policy_mru_size; + else + mru_size = DEFAULT_MRU_SIZE; + + mru = malloc(sizeof(*mru) * mru_size); + if ( mru == NULL ) + { + rc = -ENOMEM; + goto out; + } + + for ( i = 0; i < mru_size; i++ ) mru[i] = INVALID_MFN; /* Don't page out page 0 */ @@ -100,12 +113,12 @@ void policy_notify_paged_out(unsigned lo void policy_notify_paged_in(unsigned long gfn) { - unsigned long old_gfn = mru[i_mru & (MRU_SIZE - 1)]; + unsigned long old_gfn = mru[i_mru & (mru_size - 1)]; if ( old_gfn != INVALID_MFN ) clear_bit(old_gfn, bitmap); - mru[i_mru & (MRU_SIZE - 1)] = gfn; + mru[i_mru & (mru_size - 1)] = gfn; i_mru++; } --- xen-unstable.hg-4.1.22571.orig/tools/xenpaging/xenpaging.c +++ xen-unstable.hg-4.1.22571/tools/xenpaging/xenpaging.c @@ -78,6 +78,7 @@ xenpaging_t *xenpaging_init(domid_t doma xenpaging_t *paging; xc_interface *xch; xentoollog_logger *dbg = NULL; + char *p; int rc; if ( getenv("XENPAGING_DEBUG") ) @@ -92,6 +93,13 @@ xenpaging_t *xenpaging_init(domid_t doma paging = malloc(sizeof(xenpaging_t)); memset(paging, 0, sizeof(xenpaging_t)); + p = getenv("XENPAGING_POLICY_MRU_SIZE"); + if ( p && *p ) + { + paging->policy_mru_size = atoi(p); + DPRINTF("Setting policy mru_size to %d\n", paging->policy_mru_size); + } + /* Open connection to xen */ paging->xc_handle = xch; --- xen-unstable.hg-4.1.22571.orig/tools/xenpaging/xenpaging.h +++ xen-unstable.hg-4.1.22571/tools/xenpaging/xenpaging.h @@ -45,6 +45,7 @@ typedef struct xenpaging { unsigned long *bitmap; mem_event_t mem_event; + int policy_mru_size; } xenpaging_t;