MPS issue job004103

TitleNo smoke test coverage of mapping failure in pagesMarkAllocated
Assigned userGareth Rees
DescriptionIn the VM arena class, pagesMarkAllocated has a failure case with a TODO:

  /* region from basePI to j needs deallocating */
  /* TODO: Consider making pages spare instead, then purging. */

Before implementing the TODO, we'd like to be confident that this code is exercised by a test case. But the smoke tests passes even if this block of code is replaced with NOTREACHED.
AnalysisThis is because the smoke test cases don't set a commit limit. There used to be a commit limit test in amcss and a couple of other tests, but it was removed in change 189781 [2].
How foundinspection
Created byGareth Rees
Created on2018-08-01 11:42:07
Last modified byGareth Rees
Last modified on2018-08-01 12:15:07
History2018-08-01 GDR Created.