Changeset 629 for trunk/kernel/fs

Timestamp:

May 17, 2019, 9:27:04 AM (5 years ago)

Author:

alain

Message:

Remove the "giant" rwlock protecting the GPT, and
use the GPT_LOCKED attribute in each PTE to prevent
concurrent modifications of one GPT entry.
The version number has been incremented to 2.1.

Location:

trunk/kernel/fs

Files:

• fatfs.h (modified) (2 diffs)
• vfs.c (modified) (3 diffs)

trunk/kernel/fs/fatfs.h

@@ -181 +181 @@
  *
  * WARNING 2 : Most fields are constant values, but the <free_cluster_hint>,
- * <free_clusters>, <dirty_page_min>, <dirty_page_max>, <lock>, and the <fs_info_buffer>
- * are shared variables, that can be modified by any thread running in any cluster.
- * The <fs_info_buffer> contains a copy of the FS_INFO sector, and is only allocated in
- * the FAT cluster (i.e. in cluster 0). It is used by all to synchronously update the 
- * free clusters info on IOC device.
+ * <free_clusters>, <lock>, and the <fs_info_buffer> are shared variables, 
+ * that can be modified by any thread running in any cluster. The <fs_info_buffer> 
+ * contains a copy of the FS_INFO sector, and is only allocated in the FAT cluster 
+ * (cluster 0). It is used to synchronously update the free clusters info on IOC device.
  *  => For all these variables, only the values stored in the FAT cluster must be used.
  ****************************************************************************************/
@@ -202 +201 @@
 
     /* shared variables (only the copy in FAT cluster must be used)                     */
-    uint32_t            dirty_page_min;        /*! min dirty page index in FAT mapper   */
-    uint32_t            dirty_page_max;        /*! max dirty page index in FAT mapper   */
     uint32_t            free_cluster_hint;     /*! cluster[hint+1] is the first free    */
     uint32_t            free_clusters;         /*! free clusters number                 */

trunk/kernel/fs/vfs.c

@@ -2608 +2608 @@
 #if (DEBUG_VFS_LOOKUP & 1)
 if( DEBUG_VFS_LOOKUP < cycle )
-printk("\n[%s] thread[%x,%x] created missing inode <%s> in cluster %x\n",
+printk("\n[%s] thread[%x,%x] created missing inode for <%s> in cluster %x\n",
 __FUNCTION__, process->pid, this->trdid, name, child_cxy );
 #endif
@@ -2629 +2629 @@
 
                 // when the missing dentry is not in the parent mapper,
-                // it is a new dentry that must be registered in parent directory mapper
+                // a new dentry must be registered in parent directory mapper
                 if ( error )
                 {
@@ -2814 +2814 @@
 
     // 1. allocate one free cluster in file system to child inode,
-    // and update the File Allocation Table in both the TAF mapper and IOC device.
+    // and update the File Allocation Table in both the FAT mapper and IOC device.
     // It depends on the child inode FS type.
     vfs_ctx_t * ctx = hal_remote_lpt( XPTR( child_cxy , &child_ptr->ctx ) );