X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Ffilesys%2Ffilesys.c;h=7a53f5ffd924ca7b1b384fd9b46b3256e14f30e6;hb=fd2a5afa946474ba0839de0e9da238dbaecbd6a5;hp=cae53eac492d42cea94a34b5b9da173bdb6034a3;hpb=f09512e71e50d7be09249e7b87b270906ed8e499;p=pintos-anon

diff --git a/src/filesys/filesys.c b/src/filesys/filesys.c
index cae53ea..7a53f5f 100644
--- a/src/filesys/filesys.c
+++ b/src/filesys/filesys.c
@@ -1,274 +1,103 @@
-#include "filesys.h"
-#include "bitmap.h"
-#include "debug.h"
-#include "directory.h"
-#include "disk.h"
-#include "file.h"
-#include "filehdr.h"
-#include "lib.h"
-
-#define FREE_MAP_SECTOR 0
-#define ROOT_DIR_SECTOR 1
-
-#define NUM_DIR_ENTRIES 10
-#define ROOT_DIR_FILE_SIZE (sizeof (struct dir_entry) * NUM_DIR_ENTRIES)
-
-struct disk *filesys_disk;
-
-static struct file free_map_file, root_dir_file;
-
-static void
-do_format (void)
-{
-  struct bitmap free_map;
-  struct filehdr *map_hdr, *dir_hdr;
-  struct dir dir;
-
-  printk ("Formatting filesystem...");
-
-  /* Create the initial bitmap and reserve sectors for the
-     free map and root directory file headers. */
-  if (!bitmap_init (&free_map, disk_size (filesys_disk)))
-    PANIC ("bitmap creation failed--disk is too large");
-  bitmap_mark (&free_map, FREE_MAP_SECTOR);
-  bitmap_mark (&free_map, ROOT_DIR_SECTOR);
-
-  /* Allocate data sector(s) for the free map file
-     and write its file header to disk. */
-  map_hdr = filehdr_allocate (&free_map, bitmap_storage_size (&free_map));
-  if (map_hdr == NULL)
-    PANIC ("free map creation failed--disk is too large");
-  filehdr_write (map_hdr, FREE_MAP_SECTOR);
-  filehdr_destroy (map_hdr);
-
-  /* Allocate data sector(s) for the root directory file
-     and write its file header to disk. */
-  dir_hdr = filehdr_allocate (&free_map, ROOT_DIR_FILE_SIZE);
-  if (dir_hdr == NULL)
-    PANIC ("root directory creation failed");
-  filehdr_write (dir_hdr, ROOT_DIR_SECTOR);
-  filehdr_destroy (dir_hdr);
-
-  /* Write out the free map now that we have space reserved
-     for it. */
-  if (!file_open (&free_map_file, FREE_MAP_SECTOR))
-    PANIC ("can't open free map file");
-  bitmap_write (&free_map, &free_map_file);
-  bitmap_destroy (&free_map);
-  file_close (&free_map_file);
-
-  /* Write out the root directory in the same way. */
-  if (!file_open (&root_dir_file, ROOT_DIR_SECTOR))
-    PANIC ("can't open root directory");
-  if (!dir_init (&dir, NUM_DIR_ENTRIES))
-    PANIC ("can't initialize root directory");
-  dir_write (&dir, &root_dir_file);
-  dir_destroy (&dir);
-  file_close (&free_map_file);
-
-  printk ("done.\n");
-}
-
+#include "filesys/filesys.h"
+#include <debug.h>
+#include <stdio.h>
+#include <string.h>
+#include "filesys/file.h"
+#include "filesys/free-map.h"
+#include "filesys/inode.h"
+#include "filesys/directory.h"
+
+/* Partition that contains the file system. */
+struct block *fs_device;
+
+static void do_format (void);
+
+/* Initializes the file system module.
+   If FORMAT is true, reformats the file system. */
 void
 filesys_init (bool format) 
 {
-  filesys_disk = disk_get (0, 1);
-  if (filesys_disk == NULL)
-    PANIC ("hd0:1 (hdb) not present, filesystem initialization failed");
+  fs_device = block_get_role (BLOCK_FILESYS);
+  if (fs_device == NULL)
+    PANIC ("No file system device found, can't initialize file system.");
+
+  inode_init ();
+  free_map_init ();
 
   if (format) 
     do_format ();
-  
-  if (!file_open (&free_map_file, FREE_MAP_SECTOR))
-    PANIC ("can't open free map file");
-  if (!file_open (&root_dir_file, ROOT_DIR_SECTOR))
-    PANIC ("can't open root dir file");
-}
-
-bool
-filesys_create (const char *name, off_t initial_size) 
-{
-  struct dir dir;
-  struct bitmap free_map;
-  disk_sector_no hdr_sector;
-  struct filehdr *filehdr;
-  bool success = false;
 
-  /* Read the root directory. */
-  if (!dir_init (&dir, NUM_DIR_ENTRIES))
-    return false;
-  dir_read (&dir, &root_dir_file);
-  if (dir_lookup (&dir, name, NULL)) 
-    goto exit1;
-
-  /* Allocate a block for the file header. */
-  if (!bitmap_init (&free_map, disk_size (filesys_disk)))
-    goto exit1;
-  bitmap_read (&free_map, &free_map_file);
-  hdr_sector = bitmap_find_and_set (&free_map);
-  if (hdr_sector == BITMAP_ERROR)
-    goto exit2;
-
-  /* Add the file to the directory. */
-  if (!dir_add (&dir, name, hdr_sector))
-    goto exit2;
-
-  /* Allocate space for the file. */
-  filehdr = filehdr_allocate (&free_map, initial_size);
-  if (filehdr == NULL)
-    goto exit2;
-
-  /* Write everything back. */
-  filehdr_write (filehdr, hdr_sector);
-  dir_write (&dir, &root_dir_file);
-  bitmap_write (&free_map, &free_map_file);
-
-  success = true;
-
-  /* Clean up. */
-  filehdr_destroy (filehdr);
- exit2:
-  bitmap_destroy (&free_map);
- exit1:
-  dir_destroy (&dir);
-
-  return success;
+  free_map_open ();
 }
 
-bool
-filesys_open (const char *name, struct file *file)
+/* Shuts down the file system module, writing any unwritten data
+   to disk. */
+void
+filesys_done (void) 
 {
-  struct dir dir;
-  disk_sector_no hdr_sector;
-  bool success = false;
-
-  if (!dir_init (&dir, NUM_DIR_ENTRIES))
-    return false;
-  dir_read (&dir, &root_dir_file);
-  if (dir_lookup (&dir, name, &hdr_sector))
-    success = file_open (file, hdr_sector);
-  
-  dir_destroy (&dir);
-  return success;
+  free_map_close ();
 }
-
+
+/* Creates a file named NAME with the given INITIAL_SIZE.
+   Returns true if successful, false otherwise.
+   Fails if a file named NAME already exists,
+   or if internal memory allocation fails. */
 bool
-filesys_remove (const char *name) 
+filesys_create (const char *name, off_t initial_size) 
 {
-  struct dir dir;
-  disk_sector_no hdr_sector;
-  struct filehdr *filehdr;
-  struct bitmap free_map;
-  bool success = false;
-
-  /* Read the root directory. */
-  if (!dir_init (&dir, NUM_DIR_ENTRIES))
-    return false;
-  dir_read (&dir, &root_dir_file);
-  if (!dir_lookup (&dir, name, &hdr_sector))
-    goto exit1;
-
-  /* Read the file header. */
-  filehdr = filehdr_read (hdr_sector);
-  if (filehdr == NULL)
-    goto exit1;
-
-  /* Allocate a block for the file header. */
-  if (!bitmap_init (&free_map, disk_size (filesys_disk)))
-    goto exit2;
-  bitmap_read (&free_map, &free_map_file);
-
-  /* Deallocate. */
-  filehdr_deallocate (filehdr, &free_map);
-  bitmap_reset (&free_map, hdr_sector);
-  dir_remove (&dir, name);
-
-  /* Write everything back. */
-  bitmap_write (&free_map, &free_map_file);
-  dir_write (&dir, &root_dir_file);
-
-  success = true;
-
-  /* Clean up. */
-  bitmap_destroy (&free_map);
- exit2:
-  filehdr_destroy (filehdr);
- exit1:
-  dir_destroy (&dir);
+  block_sector_t inode_sector = 0;
+  struct dir *dir = dir_open_root ();
+  bool success = (dir != NULL
+                  && free_map_allocate (1, &inode_sector)
+                  && inode_create (inode_sector, initial_size)
+                  && dir_add (dir, name, inode_sector));
+  if (!success && inode_sector != 0) 
+    free_map_release (inode_sector, 1);
+  dir_close (dir);
 
   return success;
 }
 
-bool
-filesys_list (void) 
+/* Opens the file with the given NAME.
+   Returns the new file if successful or a null pointer
+   otherwise.
+   Fails if no file named NAME exists,
+   or if an internal memory allocation fails. */
+struct file *
+filesys_open (const char *name)
 {
-  struct dir dir;
+  struct dir *dir = dir_open_root ();
+  struct inode *inode = NULL;
 
-  if (!dir_init (&dir, NUM_DIR_ENTRIES))
-    return false;
-  dir_read (&dir, &root_dir_file);
-  dir_list (&dir);
-  dir_destroy (&dir);
+  if (dir != NULL)
+    dir_lookup (dir, name, &inode);
+  dir_close (dir);
 
-  return true;
+  return file_open (inode);
 }
 
+/* Deletes the file named NAME.
+   Returns true if successful, false on failure.
+   Fails if no file named NAME exists,
+   or if an internal memory allocation fails. */
 bool
-filesys_dump (void) 
+filesys_remove (const char *name) 
 {
-  struct bitmap free_map;
-  struct dir dir;  
+  struct dir *dir = dir_open_root ();
+  bool success = dir != NULL && dir_remove (dir, name);
+  dir_close (dir); 
 
-  printk ("Free map:\n");
-  if (!bitmap_init (&free_map, disk_size (filesys_disk)))
-    return false;
-  bitmap_read (&free_map, &free_map_file);
-  bitmap_dump (&free_map);
-  bitmap_destroy (&free_map);
-  printk ("\n");
-  
-  if (!dir_init (&dir, NUM_DIR_ENTRIES))
-    return false;
-  dir_read (&dir, &root_dir_file);
-  dir_dump (&dir);
-  dir_destroy (&dir);
-
-  return true;
-}
-
-static void must_succeed_function (int, int) NO_INLINE;
-
-static void 
-must_succeed_function (int line_no, int success) 
-{
-  if (!success)
-    PANIC ("filesys_self_test: operation failed on line %d", line_no);
+  return success;
 }
-
-#define MUST_SUCCEED(EXPR) must_succeed_function (__LINE__, EXPR)
-
-void
-filesys_self_test (void)
+
+/* Formats the file system. */
+static void
+do_format (void)
 {
-  static const char s[] = "This is a test string.";
-  struct file file;
-  char s2[sizeof s];
-
-  MUST_SUCCEED (filesys_create ("foo", sizeof s));
-  MUST_SUCCEED (filesys_open ("foo", &file));
-  MUST_SUCCEED (file_write (&file, s, sizeof s) == sizeof s);
-  MUST_SUCCEED (file_tell (&file) == sizeof s);
-  MUST_SUCCEED (file_length (&file) == sizeof s);
-  file_close (&file);
-
-  MUST_SUCCEED (filesys_open ("foo", &file));
-  MUST_SUCCEED (file_read (&file, s2, sizeof s2) == sizeof s2);
-  MUST_SUCCEED (memcmp (s, s2, sizeof s) == 0);
-  MUST_SUCCEED (file_tell (&file) == sizeof s2);
-  MUST_SUCCEED (file_length (&file) == sizeof s2);
-  file_close (&file);
-
-  MUST_SUCCEED (filesys_remove ("foo"));
-
-  printk ("filesys: self test ok\n");
+  printf ("Formatting file system...");
+  free_map_create ();
+  if (!dir_create (ROOT_DIR_SECTOR, 16))
+    PANIC ("root directory creation failed");
+  free_map_close ();
+  printf ("done.\n");
 }