Changeset 238 for trunk/kernel/vfs/vfs.h

Timestamp:

Jul 19, 2017, 3:31:39 PM (7 years ago)

Author:

alain

Message:

Fixing bugs in vfs_lookup()

File:

• trunk/kernel/vfs/vfs.h (modified) (6 diffs)

trunk/kernel/vfs/vfs.h

@@ -399 +399 @@
 void vfs_inode_destroy( vfs_inode_t *  inode );  
 
+/******************************************************************************************
+ * This function scan an existing parent inode directory, identified by the <parent> 
+ * argument to find a directory entry identified by the <name> argument, and update 
+ * the remote child inode, identified by the <child_xp> argument.
+ * Depending on the file system type, it calls the relevant, FS specific function,
+ * to scan the directory, and set the "type", "size", and "extend" fields.
+ * It must be called by a thread running in the cluster containing the parent inode.
+ ******************************************************************************************
+ * @ parent    : local pointer on parent inode (directory).
+ * @ name      : child name.
+ * @ child_xp  : extended pointer on remote child inode (file or directory)
+ * @ return 0 if success / return ENOENT if not found.
+ *****************************************************************************************/
+error_t vfs_inode_load( vfs_inode_t * parent,
+                        char        * name,
+                        xptr_t        child_xp );
+
 /****************************************************************************************** 
  * This function atomically increment/decrement the inode refcount.
@@ -617 +634 @@
  * @ pathname    : path in kernel space (can be relative or absolute).
  * @ lookup_mode : flags defining the working mode (defined above in this file).
- * @ inode_xp    : [out] buffer for extended pointer on inode.
+ * @ inode_xp    : [out] buffer for extended pointer on searched inode.
  * @ return 0 if success / ENOENT if inode not found , EACCES if permissopn denied,
  *                        EAGAIN if a new complete lookup must be made
@@ -635 +652 @@
  * 2) The inode and its associated mapper are created in cluster identified by <child_cxy>.
  *    The new inode and the parent inode can have different FS types.
- * 3) The "child_xp" field in created dentry (pointing on thecreated inode) is updated.
+ * 3) The "child_xp" field in created dentry (pointing on the created inode) is updated.
  ******************************************************************************************
  * @ child_cxy  : target cluster for child inode.
@@ -666 +683 @@
  * This recursive function diplays a complete inode/dentry sub-tree.
  * Any inode can be selected as the sub-tree root.
- * TODO this function is not ptotected against a concurrent node removal...
+ * TODO this function is not protected against a concurrent inode/dentry removal...
  ******************************************************************************************
  * @ inode_xp   : extended pointer on sub-tree root inode.
@@ -857 +874 @@
 
 /******************************************************************************************
- * This function makes an I/O operation to move one page from VFS to a given mapper,
- * in case of MISS on the mapper cache.
+ * This function makes an I/O operation to move one page to/from device from/to the mapper.
+ * It is used in case of MISS on the mapper, or when a dirty page in the mapper must
+ * be updated in the File System.
  * Depending on the file system type, it calls the proper, FS specific function.
  * It must be executed by a thread running in the cluster containing the mapper.
@@ -864 +882 @@
  * It takes the mapper lock before launching the IO operation.
  ******************************************************************************************
- * @ page   : local pointer on the page descriptor. 
+ * @ page      : local pointer on the page descriptor.
+ * @ to_mapper : transfer direction.
  * @ returns 0 if success / return EINVAL if it cannot access the external device.
  *****************************************************************************************/
-error_t vfs_move_page_to_mapper( struct page_s * page );
-
-/******************************************************************************************
- * This function makes an I/0 operation to move one page from a given mapper to VFS,
- * when a dirty page in the mapper cache must be updated in the File System.
+error_t vfs_mapper_move_page( struct page_s * page,
+                              bool_t          to_mapper );
+
+/******************************************************************************************
+ * This function makes an I/O operation to move, from device to mapper, all pages covering
+ * a given inode, identified by the <inode> argument. It can be a directory or a file,
+ * but is mainly used to load (prefetch) a complete directory in the inode mapper.
  * Depending on the file system type, it calls the proper, FS specific function.
  * It must be executed by a thread running in the cluster containing the mapper.
- * The mapper pointer is obtained from the page descriptor.
+ * The mapper pointer is obtained from the inode descriptor.
  * It takes the mapper lock before launching the IO operation.
- * It does nothing if the page is not dirty. If the page is dirty, it clear the page
- * dirty bit, and remove the page from the PPM dirty list. 
- ******************************************************************************************
- * @ page   : local pointer on the page descriptor. 
- * @ returns 0 if success / return EINVAL if it cannot access the external device.
- *****************************************************************************************/
-error_t vfs_move_page_from_mapper( struct page_s * page );       
-
-
-
-
-
-
-/////////////////////////////////////////////////////////////////////////////////////////// 
-// These typedef define the FS specific operations that must be implemented by any 
-// specific file system to be supported by the ALMOS_MKH VFS.
-// These typedef are not actually used, and are only defined for documentation
-/////////////////////////////////////////////////////////////////////////////////////////// 
-
-typedef error_t (fs_init_t)          ( xptr_t vfs_root_xp );
-
-typedef error_t (fs_inode_extend_t)  ( struct vfs_inode_s * inode,
-                                       void               * extend );
- 
-typedef void    (fs_inode_release_t) ( struct vfs_inode_s * inode );
-
-typedef error_t (fs_write_page_t)    ( struct page_s * page );
-
-typedef error_t (fs_read_page_t)     ( struct page_s * page );
-
-
-
-
-
+ ******************************************************************************************
+ * @ inode  : local pointer on inode.
+ * @ return 0 if success / return EIO if device access failure.
+ *****************************************************************************************/
+error_t vfs_mapper_load_all( vfs_inode_t * inode );