pintos-os.org Git - pintos-anon/blob - grading/vm/review.txt

   1 TESTCASES [[/10]]
   2 ------------------
   3   -3 Not explaining random vs. LRU algorithm
   4   -3 Not testing/explaining heavily paging program
   5   -3 Not testing/explaining mmap
   6   -1 No other test cases
   7
   8
   9 DESIGN [[/40]]
  10 --------------
  11
  12 DESIGNDOC:
  13   -10 Doesn't discuss page table design
  14   -10 Doesn't discuss swap design
  15   -5 Doesn't discuss page replacement algorithm
  16   -5 Doesn't explain how memory mapping works
  17
  18 Overall:
  19   -1 Gratuitous use of malloc() (e.g. for allocating a list or a lock)
  20   -1 Inappropriate use of ASSERT (e.g. to verify that malloc() succeeded)
  21
  22 Swap File Design
  23   -10 Doesn't allow out of order pages within swap disk
  24   -2 Does not synchronize data structure (give name of data structure here)
  25
  26 Page Table / Page Replacement
  27   -10 Doesn't use an inverted, hashed, or other efficient page table design
  28   -5 Uses random/sequential page replacement
  29   +2 Unusually clever page replacement
  30   +5 Sharing code pages
  31   -2 Does not synchronize data structure (give name of data structure here)
  32
  33 Demand Paging / Memory Mapping
  34   -10 Loading is not at all lazy
  35   -5 Loading is not entirely lazy
  36   -5 Stack does not grow downward from top of address space
  37   -5 Stack does not grow automatically
  38   -2 Does not synchronize data structure (give name of data structure here)
  39
  40
  41 STYLE [[/10]]
  42 -------------
  43   -5...-10 Fixing code after submission
  44   -5 Doesn't compile as submitted
  45   +1...+5 Cool test programs etc.
  46
  47
  48 COMMENTS
  49 --------
  50