Changes in uspace/srv/fs/tmpfs/tmpfs_ops.c [472c09d:19f857a] in mainline

File:

• uspace/srv/fs/tmpfs/tmpfs_ops.c (modified) (15 diffs)

uspace/srv/fs/tmpfs/tmpfs_ops.c

@@ -29 +29 @@
 /** @addtogroup fs
  * @{
- */ 
+ */
 
 /**
@@ -40 +40 @@
 #include "../../vfs/vfs.h"
 #include <ipc/ipc.h>
+#include <macros.h>
+#include <limits.h>
 #include <async.h>
 #include <errno.h>
 #include <atomic.h>
 #include <stdlib.h>
-#include <string.h>
+#include <str.h>
 #include <stdio.h>
 #include <assert.h>
@@ -93 +95 @@
 }
 
-static size_t tmpfs_size_get(fs_node_t *fn)
+static aoff64_t tmpfs_size_get(fs_node_t *fn)
 {
         return TMPFS_NODE(fn)->size;
@@ -168 +170 @@
                     (nodep->index == key[NODES_KEY_INDEX]));
         default:
-                abort();
-        }
+                assert((keys == 1) || (keys == 2));
+        }
+
+        return 0;
 }
 
@@ -439 +443 @@
 {
         dev_handle_t dev_handle = (dev_handle_t) IPC_GET_ARG1(*request);
-        
-        /* Accept the mount options */
+        fs_node_t *rootfn;
+        int rc;
+        
+        /* Accept the mount options. */
         char *opts;
-        int rc = async_string_receive(&opts, 0, NULL);
-        
+        rc = async_data_write_accept((void **) &opts, true, 0, 0, 0, NULL);
         if (rc != EOK) {
                 ipc_answer_0(rid, rc);
+                return;
+        }
+
+        /* Check if this device is not already mounted. */
+        rc = tmpfs_root_get(&rootfn, dev_handle);
+        if ((rc == EOK) && (rootfn)) {
+                (void) tmpfs_node_put(rootfn);
+                free(opts);
+                ipc_answer_0(rid, EEXIST);
                 return;
         }
@@ -456 +470 @@
         }
 
-        fs_node_t *rootfn;
         rc = tmpfs_root_get(&rootfn, dev_handle);
         assert(rc == EOK);
@@ -498 +511 @@
 void tmpfs_read(ipc_callid_t rid, ipc_call_t *request)
 {
-        dev_handle_t dev_handle = (dev_handle_t)IPC_GET_ARG1(*request);
-        fs_index_t index = (fs_index_t)IPC_GET_ARG2(*request);
-        off_t pos = (off_t)IPC_GET_ARG3(*request);
-
+        dev_handle_t dev_handle = (dev_handle_t) IPC_GET_ARG1(*request);
+        fs_index_t index = (fs_index_t) IPC_GET_ARG2(*request);
+        aoff64_t pos =
+            (aoff64_t) MERGE_LOUP32(IPC_GET_ARG3(*request), IPC_GET_ARG4(*request));
+        
         /*
          * Lookup the respective TMPFS node.
@@ -517 +531 @@
         tmpfs_node_t *nodep = hash_table_get_instance(hlp, tmpfs_node_t,
             nh_link);
-
+        
         /*
          * Receive the read request.
@@ -524 +538 @@
         size_t size;
         if (!async_data_read_receive(&callid, &size)) {
-                ipc_answer_0(callid, EINVAL);   
+                ipc_answer_0(callid, EINVAL);
                 ipc_answer_0(rid, EINVAL);
                 return;
@@ -531 +545 @@
         size_t bytes;
         if (nodep->type == TMPFS_FILE) {
-                bytes = max(0, min(nodep->size - pos, size));
+                bytes = min(nodep->size - pos, size);
                 (void) async_data_read_finalize(callid, nodep->data + pos,
                     bytes);
@@ -537 +551 @@
                 tmpfs_dentry_t *dentryp;
                 link_t *lnk;
-                int i;
+                aoff64_t i;
                 
                 assert(nodep->type == TMPFS_DIRECTORY);
@@ -547 +561 @@
                  */
                 for (i = 0, lnk = nodep->cs_head.next;
-                    i < pos && lnk != &nodep->cs_head;
+                    (i < pos) && (lnk != &nodep->cs_head);
                     i++, lnk = lnk->next)
                         ;
@@ -572 +586 @@
 void tmpfs_write(ipc_callid_t rid, ipc_call_t *request)
 {
-        dev_handle_t dev_handle = (dev_handle_t)IPC_GET_ARG1(*request);
-        fs_index_t index = (fs_index_t)IPC_GET_ARG2(*request);
-        off_t pos = (off_t)IPC_GET_ARG3(*request);
-
+        dev_handle_t dev_handle = (dev_handle_t) IPC_GET_ARG1(*request);
+        fs_index_t index = (fs_index_t) IPC_GET_ARG2(*request);
+        aoff64_t pos =
+            (aoff64_t) MERGE_LOUP32(IPC_GET_ARG3(*request), IPC_GET_ARG4(*request));
+        
         /*
          * Lookup the respective TMPFS node.
@@ -636 +651 @@
 void tmpfs_truncate(ipc_callid_t rid, ipc_call_t *request)
 {
+        dev_handle_t dev_handle = (dev_handle_t) IPC_GET_ARG1(*request);
+        fs_index_t index = (fs_index_t) IPC_GET_ARG2(*request);
+        aoff64_t size =
+            (aoff64_t) MERGE_LOUP32(IPC_GET_ARG3(*request), IPC_GET_ARG4(*request));
+        
+        /*
+         * Lookup the respective TMPFS node.
+         */
+        unsigned long key[] = {
+                [NODES_KEY_DEV] = dev_handle,
+                [NODES_KEY_INDEX] = index
+        };
+        link_t *hlp = hash_table_find(&nodes, key);
+        if (!hlp) {
+                ipc_answer_0(rid, ENOENT);
+                return;
+        }
+        tmpfs_node_t *nodep = hash_table_get_instance(hlp, tmpfs_node_t,
+            nh_link);
+        
+        if (size == nodep->size) {
+                ipc_answer_0(rid, EOK);
+                return;
+        }
+        
+        if (size > SIZE_MAX) {
+                ipc_answer_0(rid, ENOMEM);
+                return;
+        }
+        
+        void *newdata = realloc(nodep->data, size);
+        if (!newdata) {
+                ipc_answer_0(rid, ENOMEM);
+                return;
+        }
+        
+        if (size > nodep->size) {
+                size_t delta = size - nodep->size;
+                memset(newdata + nodep->size, 0, delta);
+        }
+        
+        nodep->size = size;
+        nodep->data = newdata;
+        ipc_answer_0(rid, EOK);
+}
+
+void tmpfs_close(ipc_callid_t rid, ipc_call_t *request)
+{
+        ipc_answer_0(rid, EOK);
+}
+
+void tmpfs_destroy(ipc_callid_t rid, ipc_call_t *request)
+{
         dev_handle_t dev_handle = (dev_handle_t)IPC_GET_ARG1(*request);
         fs_index_t index = (fs_index_t)IPC_GET_ARG2(*request);
-        size_t size = (off_t)IPC_GET_ARG3(*request);
-
-        /*
-         * Lookup the respective TMPFS node.
-         */
+        int rc;
+
         link_t *hlp;
         unsigned long key[] = {
@@ -655 +720 @@
         tmpfs_node_t *nodep = hash_table_get_instance(hlp, tmpfs_node_t,
             nh_link);
-
-        if (size == nodep->size) {
-                ipc_answer_0(rid, EOK);
-                return;
-        }
-
-        void *newdata = realloc(nodep->data, size);
-        if (!newdata) {
-                ipc_answer_0(rid, ENOMEM);
-                return;
-        }
-        if (size > nodep->size) {
-                size_t delta = size - nodep->size;
-                memset(newdata + nodep->size, 0, delta);
-        }
-        nodep->size = size;
-        nodep->data = newdata;
-        ipc_answer_0(rid, EOK);
-}
-
-void tmpfs_close(ipc_callid_t rid, ipc_call_t *request)
-{
-        ipc_answer_0(rid, EOK);
-}
-
-void tmpfs_destroy(ipc_callid_t rid, ipc_call_t *request)
-{
-        dev_handle_t dev_handle = (dev_handle_t)IPC_GET_ARG1(*request);
-        fs_index_t index = (fs_index_t)IPC_GET_ARG2(*request);
-        int rc;
-
-        link_t *hlp;
-        unsigned long key[] = {
-                [NODES_KEY_DEV] = dev_handle,
-                [NODES_KEY_INDEX] = index
-        };
-        hlp = hash_table_find(&nodes, key);
-        if (!hlp) {
-                ipc_answer_0(rid, ENOENT);
-                return;
-        }
-        tmpfs_node_t *nodep = hash_table_get_instance(hlp, tmpfs_node_t,
-            nh_link);
         rc = tmpfs_destroy_node(FS_NODE(nodep));
         ipc_answer_0(rid, rc);