Changeset 568 for trunk/kernel/fs/vfs.h

Timestamp:

Oct 5, 2018, 12:02:49 AM (5 years ago)

Author:

alain

Message:

omplete restructuration of kernel locks.

File:

• trunk/kernel/fs/vfs.h (modified) (5 diffs)

trunk/kernel/fs/vfs.h

@@ -30 +30 @@
 #include <hal_atomic.h>
 #include <remote_rwlock.h>
-#include <remote_spinlock.h>
-#include <spinlock.h>
+#include <remote_busylock.h>
+#include <busylock.h>
 #include <list.h>
 #include <xlist.h>
-#include <slist.h>
 #include <bits.h>
 #include <xhtab.h>
@@ -105 +104 @@
 
 /******************************************************************************************
- * This structure defines informations common to all inodes and dentries
- * of a given file system. As it is declared a global variable in the kdata segment,
- * it is replicated in all clusters and handled as private by each OS intance.
+ * This structure defines a VFS context, that contains informations common to all inodes
+ * and dentries for a given file system. As it is declared as a global variable in the
+ * kdata segment, it is handled as private by each OS intance in a given cluster.
  *****************************************************************************************/
 
@@ -134 +133 @@
         uint32_t       cluster_size;             /*! cluster size on device (bytes)          */
         xptr_t         vfs_root_xp;              /*! extended pointer on VFS root inode      */
-    spinlock_t     lock;                     /*! lock protecting inum allocator          */
+    busylock_t     lock;                     /*! lock protecting inum allocator          */
     uint32_t       bitmap[BITMAP_SIZE(CONFIG_VFS_MAX_INODES)];  /* inum allocator        */
     void         * extend;                   /*! FS specific context extension           */
@@ -143 +142 @@
  * This structure define a VFS inode.
  * It contains an extended pointer on the parent dentry, and (for directory only)
- * an hash table xhtab refering all children dentries.
- * The <parent> inode is unique for a directory (not hard links for directories). 
+ * an hash table (xhtab) registering all children dentries.
+ * The <parent> inode is unique for a directory (no hard links for directories). 
  * For a file, the parent field points to the first dentry who created this inode.
- * Syncrhonisation:
- * - the main_lock (spinlock) is used during the inode tree traversal or for inode
- *   modification (add/remove children).
- * - the data_lock (rwlock) is used during read/write accesses to the data stored
- *   in the mapper. 
- * - the mapper lock (rwlock) is only used during the radix tree traversal to return
- *   the relevant page for read/write.
+ * Synchronisation:
+ * - the main_lock (remote_busylock) is used during the inode tree traversal,
+ *   or for inode modification (add/remove children).
+ * - the data_lock (remote_rwlock) is used during read/write accesses to the data
+ *   stored in the mapper. 
+ * - the mapper lock (remote rwlock) is only used during the radix tree traversal 
+ *   to return the relevant page for read/write.
  *****************************************************************************************/
 
@@ -195 +194 @@
         xhtab_t            children;         /*! embedded xhtab of children dentries         */
         remote_rwlock_t    data_lock;        /*! protect read/write to data and to size      */
-        remote_spinlock_t  main_lock;        /*! protect inode tree traversal and modifs     */
+        remote_busylock_t  main_lock;        /*! protect inode tree traversal and modifs     */
         list_entry_t       list;             /*! member of set of inodes in same cluster     */
         xlist_entry_t      wait_root;        /*! root of threads waiting on this inode       */