Update docs to say that a pointer to an inode should not limit you to

64 open files.

diff --git a/TODO b/TODO
index d23010673f370b5af9acf3295b22118e82c44bc0..737f9af26227388ed3510825290a22f94bff7055 100644 (file)
--- a/TODO
+++ b/TODO
@@ -58,29 +58,6 @@ priority again to a level that's lower than a still active donation.
 
  - Godmar
 
-From: Godmar Back <godmar@gmail.com>
-Subject: project 4 question/comment regarding caching inode data
-To: Ben Pfaff <blp@cs.stanford.edu>
-Date: Sat, 14 Jan 2006 15:59:33 -0500
-
-Ben,
-
-in section 6.3.3 in the P4 FAQ, you write:
-
-"You can store a pointer to inode data in struct inode, if you want,"
-
-Should you point out that if they indeed do that, they likely wouldn't
-be able to support more than 64 open inodes systemwide at any given
-point in time.
-
-(This seems like a rather strong limitation; do your current tests
-open more than 64 files?
-It would also point to an obvious way to make the projects harder by
-specifically disallowing that inode data be locked in memory during
-the entire time an inode is kept open.)
-
- - Godmar
-
 From: Godmar Back <godmar@gmail.com>
 Subject: on caching in project 4
 To: Ben Pfaff <blp@cs.stanford.edu>

diff --git a/doc/filesys.texi b/doc/filesys.texi
index 1f10143cfaf9030a977df98e7bb71cae6306acf0..f2c7da9159880a3e0d0d11fac9dd6989c26b126b 100644 (file)
--- a/doc/filesys.texi
+++ b/doc/filesys.texi
@@ -424,8 +424,10 @@ corresponding sector from disk when it's created.  Keeping extra
 copies of inodes would subvert the 64-block limitation that we place
 on your cache.
 
-You can store a pointer to inode data in @struct{inode}, if you want,
-and you can store other information to help you find the inode when you
+You can store a pointer to inode data in @struct{inode}, but it you do
+so you should carefully make sure that this does not limit your OS to 64
+simultaneously open files.
+You can also store other information to help you find the inode when you
 need it.  Similarly, you may store some metadata along each of your 64
 cache entries.