Changeset 656 for trunk/kernel/libk/list.h

Timestamp:

Dec 6, 2019, 12:07:51 PM (4 years ago)

Author:

alain

Message:

Fix several bugs in the FATFS and in the VFS,
related to the creation of big files requiring
more than 4 Kbytes (one cluster) on device.

File:

• trunk/kernel/libk/list.h (modified) (7 diffs)

trunk/kernel/libk/list.h

@@ -1 +1 @@
 /*
- * list.h - Double circular linked list
+ * list.h - Local double circular linked list, using local pointers.
  *
  * Authors Ghassan Almaless  (2008,2009,2010,2011,2012)
@@ -91 +91 @@
 
 /***************************************************************************
- * This macro returns t pointer on the first element of a list. 
+ * This macro returns a pointer on the first element of a list. 
  ***************************************************************************
  * @ root     : pointer on the list root
@@ -171 +171 @@
                                    list_entry_t * entry )
 {
-    list_entry_t * next = root->next;  
-
-        entry->next = next;
+    list_entry_t * first = root->next;  
+
+        entry->next = first;
         entry->pred = root;
   
-        root->next = entry;
-        next->pred = entry;
+        root->next  = entry;
+        first->pred = entry;
 }
 
@@ -190 +190 @@
                                   list_entry_t * entry )
 {
-    list_entry_t * pred = root->pred;
+    list_entry_t * last = root->pred;
 
         entry->next = root;
-        entry->pred = pred;
+        entry->pred = last;
   
         root->pred = entry;
-        pred->next = entry;
+        last->next = entry;
 }
 
@@ -366 +366 @@
                                           list_entry_t * entry )
 {
-    list_entry_t * next = hal_remote_lpt( XPTR( cxy , &root->next ) );
+    list_entry_t * first = hal_remote_lpt( XPTR( cxy , &root->next ) );
         
-        hal_remote_spt( XPTR( cxy , &entry->next ) , next );
+        hal_remote_spt( XPTR( cxy , &entry->next ) , first );
         hal_remote_spt( XPTR( cxy , &entry->pred ) , root );
   
-        hal_remote_spt( XPTR( cxy , &root->next ) , entry );
-        hal_remote_spt( XPTR( cxy , &next->pred ) , entry );
+        hal_remote_spt( XPTR( cxy , &root->next )  , entry );
+        hal_remote_spt( XPTR( cxy , &first->pred ) , entry );
 }
 
@@ -387 +387 @@
                                          list_entry_t * entry )
 {
-    list_entry_t * pred = hal_remote_lpt( XPTR( cxy , &root->pred ) );
+    list_entry_t * last = hal_remote_lpt( XPTR( cxy , &root->pred ) );
         
         hal_remote_spt( XPTR( cxy , &entry->next ) , root );
-        hal_remote_spt( XPTR( cxy , &entry->pred ) , pred );
+        hal_remote_spt( XPTR( cxy , &entry->pred ) , last );
   
         hal_remote_spt( XPTR( cxy , &root->pred ) , entry );
-        hal_remote_spt( XPTR( cxy , &pred->next ) , entry );
+        hal_remote_spt( XPTR( cxy , &last->next ) , entry );
 }
 
@@ -401 +401 @@
  ***************************************************************************
  * @ cxy     : remote list cluster identifier
- * @ entry   : pointer on the entry to be removed.
+ * @ entry   : local pointer on the remote entry to be removed.
  **************************************************************************/
 static inline void list_remote_unlink( cxy_t          cxy,