Get rid of unnecessary barrier. Improve comment.

[pintos-anon] / grading / vm / mmap-shuffle.c

diff --git a/grading/vm/mmap-shuffle.c b/grading/vm/mmap-shuffle.c
index 1eccc03849ee0ccc24aa7ad11afa477d59ea91d2..15a37f438f26feb8ad5a047ef5f81a972624fb87 100644 (file)
--- a/grading/vm/mmap-shuffle.c
+++ b/grading/vm/mmap-shuffle.c
@@ -8,7 +8,11 @@
 #include "../lib/arc4.h"
 #include "../lib/cksum.h"
 
-#define SIZE (63 * 1024)        /* Max file size. */
+/* This is the max file size for an older version of the Pintos
+   file system that had 126 direct blocks each pointing to a
+   single disk sector.  We could raise it now. */
+#define SIZE (126 * 512)
+
 static char *buf = (char *) 0x10000000;
 
 static struct arc4 *
@@ -68,7 +72,7 @@ main (void)
       return 1;
     }
 
-  if (!mmap (fd, buf, SIZE))
+  if (mmap (fd, buf) == MAP_FAILED)
     {
       printf ("(mmap-shuffle) mmap() failed\n");
       return 1;