pintos-os.org Git - pintos-anon/blob - ta-advice/hw4.txt

   1 HW3 DESIGN SUMMARY
   2 ==================
   3
   4 OVERALL
   5
   6   DOCUMENTATION
   7     -20 Egregious violation of style versus design document example
   8     -10 Numerous excessively long lines (> 79 characters)
   9     -10 Numerous capitalization, punctuation, spelling, or grammar errors
  10
  11   DESIGN
  12     -10 Failure to check return value of malloc()
  13     -10 Use of ASSERT to check something that can actually fail, e.g. malloc()
  14
  15   CODING STYLE
  16     -10 Inconsistent or bad coding style: no indentation, cramming
  17         many statements into one line, other issues at TA's discretion
  18     -10 Numerous very long source code lines (> 100 characters)
  19     -10 Commented-out or #if'd out code makes real code hard to read
  20     -10 Many missing comments on structure, structure members,
  21         global or static variables, or function definitions
  22     -10 Function(s) should be decomposed for clarity [indicate function]
  23     -10 Cut-and-pasted code should be made into function [indicate where]
  24     -10 Uninformative or deceptive identifiers
  25
  26 Subtotal: /10 (not capped)
  27
  28 PROBLEM 1: INDEX AND EXTENSIBLE FILES
  29
  30   DOCUMENTATION
  31     -30 Grossly inaccurate: documentation has no resemblance to code
  32     -15 Important inaccuracies: documentation and code differ significantly
  33         [indicate how]
  34      -5 Minor inaccuracies: documentation and code differ [indicate how]
  35      -5 A1: Missing entirely/missing comments or purpose/too long
  36      -1 A1: "start" member retained in inode_disk but not used
  37      -2 A1: "unused" array in inode_disk should be used for block pointers
  38      -5 A2: Missing/non-responsive/too long/too short
  39      -2 A2: Didn't show work or provide enough information to reproduce
  40      -2 A2: Math doesn't add up
  41      -3 A2: Incorrect claim to have 128 indirect or doubly indirect blocks
  42      -5 A3: Missing/non-responsive/too long/too short
  43      -5 A4: Missing/non-responsive/too long/too short
  44      -5 A5: Missing/non-responsive/too long/too short
  45      +1 A5: Mentioned that locks and semaphores are FIFO ordered.
  46      -3 A5: Didn't describe one of two rules needed for fairness
  47      -5 A5: Didn't describe either of two rules needed for fairness
  48      -2 A5: Use of timer to ensure fairness
  49      -2 A5: Claim that "rare" events do not affect fairness
  50      -5 A6: Missing/non-responsive/too long/too short
  51      -3 A6: Mentioned only one reason
  52
  53   DESIGN
  54     -30 Not implemented
  55      -5 Global lock for file extension
  56      -5 Check on file length followed by obtaining lock fails to recheck length
  57     -15 Reads or writes wait for one another, even when file not being extended
  58
  59 Subtotal: /30 (capped at 0)
  60
  61 PROBLEM 2: SUBDIRECTORIES
  62
  63   DOCUMENTATION
  64     -30 Grossly inaccurate: documentation has no resemblance to code
  65     -15 Important inaccuracies: documentation and code differ significantly
  66         [indicate how]
  67      -5 Minor inaccuracies: documentation and code differ [indicate how]
  68      -5 B1: Missing entirely/missing comments or purpose/too long
  69      -1 B1: Not mentioned how files and directories are distinguished
  70      -2 B1: Global list of open directories lacks lock
  71      -2 B1: Global list of directories has no way to wait for directory access
  72      -5 B2: Missing/non-responsive/too long/too short
  73      -5 B3: Missing/non-responsive/too long/too short
  74      -5 B4: Missing/non-responsive/too long/too short
  75      -5 B5: Missing/non-responsive/too long/too short
  76      -3 B5: Claim that extra code is needed to avoid deleting parents of cwds
  77      -5 B6: Missing/non-responsive/too long/too short
  78      -3 B6: Mentioned only one reason
  79
  80   DESIGN
  81     -30 Not implemented
  82     -10 Use of string for current directory is slow, takes a lot of
  83         memory, and has undesirable semantics for directory removed
  84         and re-created
  85      -5 Use of sector number for current directory has bad semantics
  86         when a directory's inode is reused later
  87     -10 Operations on different directories must wait for one another
  88
  89 Subtotal: /30 (capped at 0)
  90
  91 PROBLEM 3: BUFFER CACHE
  92
  93   DOCUMENTATION
  94     -30 Grossly inaccurate: documentation has no resemblance to code
  95     -15 Important inaccuracies: documentation and code differ significantly
  96         [indicate how]
  97      -5 Minor inaccuracies: documentation and code differ [indicate how]
  98      -5 C1: Missing entirely/missing comments or purpose/too long
  99      -2 C1: Didn't drop inode_disk from struct inode
 100      -5 C2: Missing/non-responsive/too long/too short
 101      -5 C3: Missing/non-responsive/too long/too short
 102      -5 C4: Missing/non-responsive/too long/too short
 103      -5 C5: Missing/non-responsive/too long/too short
 104      -5 C6: Missing/non-responsive/too long/too short
 105      -2 C6: Omitted one of three workloads
 106      -4 C6: Omitted two of three workloads
 107
 108   DESIGN
 109     -30 Not implemented
 110      -3 Modifies all cache entries on block access or eviction
 111     -10 Metadata are preferred over data to the extent that the cache
 112         will contain only metadata in many cases
 113      -5 Busy waits in read-ahead
 114      -5 Creates new thread on most reads or on every read
 115      -5 In some cases operations on cached blocks may have to wait for
 116         I/O of unrelated blocks to complete
 117
 118 Subtotal: /30 (capped at 0)