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 (per problem):
  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   -3 Holds a global lock while accessing the disk
  20   -1 Gratuitous use of malloc() (e.g. for allocating a list or a lock)
  21   -1 Inappropriate use of ASSERT (e.g. to verify that malloc() succeeded)
  22
  23 Swap File Design
  24   -10 Doesn't allow out of order pages within swap disk
  25   -2 Does not synchronize data structure (give name of data structure here)
  26
  27 Page Table / Page Replacement
  28   -10 Doesn't use an inverted, hashed, or other efficient page table design
  29   -5 Uses random/sequential page replacement
  30   +2 Unusually clever page replacement
  31   +5 Sharing code pages
  32   -2 Does not synchronize data structure (give name of data structure here)
  33
  34 Demand Paging / Memory Mapping
  35   -10 Loading is not at all lazy
  36   -5 Loading is not entirely lazy
  37   -5 Stack does not grow downward from top of address space
  38   -5 Stack does not grow automatically
  39   -2 Does not synchronize data structure (give name of data structure here)
  40
  41
  42 STYLE [[/10]]
  43 -------------
  44   -5...-10 Fixing code after submission
  45   -5 Doesn't compile as submitted
  46   +1...+5 Cool test programs etc.
  47
  48
  49 COMMENTS
  50 --------
  51