diff options
| author | Nick Bofferding <bofferdn@us.ibm.com> | 2019-04-16 16:13:05 -0500 |
|---|---|---|
| committer | Nicholas E. Bofferding <bofferdn@us.ibm.com> | 2019-04-17 08:26:18 -0500 |
| commit | 5002bdd789546e59f68924c5931dc0e35b668aed (patch) | |
| tree | 49edc522ec652567a5de83883a4933b49dfd1c65 | |
| parent | d4e167d5bd38e879da7f7dee391c9ca1f12b763c (diff) | |
| download | talos-hostboot-5002bdd789546e59f68924c5931dc0e35b668aed.tar.gz talos-hostboot-5002bdd789546e59f68924c5931dc0e35b668aed.zip | |
Reduce LRU RO constraint to allow RO page evictions
The current read only contraint used to determine when to cast out read only
pages exceeds the maximum LRU value, and thus the required page evictions never
occur. This change decreases the constraint so that it is one less than the
max LRU value.
Change-Id: I9e37067c4ed219458778edce6ab826ef481dbf21
CQ: FW690233
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/76054
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Glenn Miles <milesg@ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
| -rw-r--r-- | src/kernel/block.C | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/kernel/block.C b/src/kernel/block.C index 5b24ae1a6..b3fefa55a 100644 --- a/src/kernel/block.C +++ b/src/kernel/block.C @@ -386,8 +386,10 @@ void Block::castOutPages(uint64_t i_type) if((iv_baseAddr != VMM_ADDR_BASE_BLOCK) && // Skip base area (iv_baseAddr != VMM_ADDR_EXTEND_BLOCK)) // Skip extended memory. { + // NOTE: All LRU constraints must be < 7, since getLRU() only reports + // 3 bits worth of size (despite the uint8_t return type). size_t rw_constraint = 5; - size_t ro_constraint = 10; + size_t ro_constraint = 6; if(i_type == VmmManager::CRITICAL) { |
