Changeset 7b084d3 in mainline

Timestamp:

2015-04-12T09:37:32Z (10 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c4f32fff

Parents:

3f04c28

Message:

Do not maintain a separate function soft state.

File:

• uspace/drv/block/ddisk/ddisk.c (modified) (19 diffs)

uspace/drv/block/ddisk/ddisk.c

@@ -94 +94 @@
 } ddisk_regs_t;
 
-struct ddisk_dev;
-
-typedef struct ddisk_fun {
-        ddf_fun_t *fun;
-        struct ddisk_dev *ddisk_dev;
-        
-        size_t blocks;
-
-        bd_srvs_t bds;
-} ddisk_fun_t;
-
-typedef struct ddisk_dev {
+typedef struct {
         fibril_mutex_t lock;
         fibril_condvar_t io_cv;
         bool io_busy;
         ddf_dev_t *dev;
+        ddf_fun_t *fun;
         ddisk_res_t ddisk_res;
-        ddisk_fun_t *ddisk_fun;         
         ddisk_regs_t *ddisk_regs;
         uintptr_t dma_buffer_phys;
         void *dma_buffer;
+        bd_srvs_t bds;
 } ddisk_dev_t;
 
@@ -206 +196 @@
             read, ba, buf);
 
-        if (ba >= ddisk_dev->ddisk_fun->blocks)
+        if (ba >= pio_read_32(&ddisk_dev->ddisk_regs->size) / DDISK_BLOCK_SIZE)
                 return ELIMIT;
 
@@ -239 +229 @@
     size_t size, bool is_read)
 {
-        ddisk_fun_t *ddisk_fun = (ddisk_fun_t *) bd->srvs->sarg;
+        ddisk_dev_t *ddisk_dev = (ddisk_dev_t *) bd->srvs->sarg;
         aoff64_t i;
         int rc;
@@ -247 +237 @@
 
         for (i = 0; i < cnt; i++) {
-                rc = ddisk_rw_block(ddisk_fun->ddisk_dev, is_read, ba + i,
+                rc = ddisk_rw_block(ddisk_dev, is_read, ba + i,
                     buf + i * DDISK_BLOCK_SIZE);
                 if (rc != EOK)
@@ -276 +266 @@
 int ddisk_bd_get_num_blocks(bd_srv_t *bd, aoff64_t *rnb)
 {
-        ddisk_fun_t *ddisk_fun = (ddisk_fun_t *) bd->srvs->sarg;
-
-        *rnb = ddisk_fun->blocks;
+        ddisk_dev_t *ddisk_dev = (ddisk_dev_t *) bd->srvs->sarg;
+
+        *rnb = pio_read_32(&ddisk_dev->ddisk_regs->size) / DDISK_BLOCK_SIZE;
         return EOK;     
 }
@@ -321 +311 @@
         int rc;
         ddf_fun_t *fun = NULL;
-        ddisk_fun_t *ddisk_fun = NULL;
 
         fun = ddf_fun_create(ddisk_dev->dev, fun_exposed, DDISK_FUN_NAME);
@@ -330 +319 @@
         }
 
-        /* Allocate soft state */
-        ddisk_fun = ddf_fun_data_alloc(fun, sizeof(ddisk_fun_t));
-        if (!ddisk_fun) {
-                ddf_msg(LVL_ERROR, "Failed allocating softstate.");
-                rc = ENOMEM;
-                goto error;
-        }
-
-        ddisk_fun->fun = fun;
-        ddisk_fun->ddisk_dev = ddisk_dev;
-
-        bd_srvs_init(&ddisk_fun->bds);
-        ddisk_fun->bds.ops = &ddisk_bd_ops;
-        ddisk_fun->bds.sarg = ddisk_fun;
-
         /* Set up a connection handler. */
         ddf_fun_set_conn_handler(fun, ddisk_bd_connection);
 
-        ddisk_fun->blocks = pio_read_32(&ddisk_dev->ddisk_regs->size) /
-            DDISK_BLOCK_SIZE;
-
         rc = ddf_fun_bind(fun);
         if (rc != EOK) {
@@ -359 +330 @@
 
         ddf_fun_add_to_category(fun, "bd");
-
-        ddisk_dev->ddisk_fun = ddisk_fun;
+        ddisk_dev->fun = fun;
+
         return EOK;
 error:
@@ -369 +340 @@
 }
 
-static int ddisk_fun_remove(ddisk_fun_t *ddisk_fun)
-{
-        int rc;
-
-        if (ddisk_fun->fun == NULL)
+static int ddisk_fun_remove(ddisk_dev_t *ddisk_dev)
+{
+        int rc;
+
+        if (ddisk_dev->fun == NULL)
                 return EOK;
 
-        ddf_msg(LVL_DEBUG, "ddisk_fun_remove(%p, '%s')", ddisk_fun,
+        ddf_msg(LVL_DEBUG, "ddisk_fun_remove(%p, '%s')", ddisk_dev,
             DDISK_FUN_NAME);
-        rc = ddf_fun_offline(ddisk_fun->fun);
+        rc = ddf_fun_offline(ddisk_dev->fun);
         if (rc != EOK) {
                 ddf_msg(LVL_ERROR, "Error offlining function '%s'.",
@@ -385 +356 @@
         }
 
-        rc = ddf_fun_unbind(ddisk_fun->fun);
+        rc = ddf_fun_unbind(ddisk_dev->fun);
         if (rc != EOK) {
                 ddf_msg(LVL_ERROR, "Failed unbinding function '%s'.",
@@ -392 +363 @@
         }
 
-        ddf_fun_destroy(ddisk_fun->fun);
-        ddisk_fun->fun = NULL;
+        ddf_fun_destroy(ddisk_dev->fun);
+        ddisk_dev->fun = NULL;
         rc = EOK;
 error:
@@ -399 +370 @@
 }
 
-static int ddisk_fun_unbind(ddisk_fun_t *ddisk_fun)
-{
-        int rc;
-
-        if (ddisk_fun->fun == NULL)
+static int ddisk_fun_unbind(ddisk_dev_t *ddisk_dev)
+{
+        int rc;
+
+        if (ddisk_dev->fun == NULL)
                 return EOK;
 
-        ddf_msg(LVL_DEBUG, "ddisk_fun_unbind(%p, '%s')", ddisk_fun,
+        ddf_msg(LVL_DEBUG, "ddisk_fun_unbind(%p, '%s')", ddisk_dev,
             DDISK_FUN_NAME);
-        rc = ddf_fun_unbind(ddisk_fun->fun);
+        rc = ddf_fun_unbind(ddisk_dev->fun);
         if (rc != EOK) {
                 ddf_msg(LVL_ERROR, "Failed unbinding function '%s'.",
@@ -415 +386 @@
         }
 
-        ddf_fun_destroy(ddisk_fun->fun);
-        ddisk_fun->fun = NULL;
+        ddf_fun_destroy(ddisk_dev->fun);
+        ddisk_dev->fun = NULL;
         rc = EOK;
+
 error:
         return rc;
@@ -483 +455 @@
         ddf_msg(LVL_NOTE, "Allocated DMA buffer at %p virtual and %p physical.",
             ddisk_dev->dma_buffer, (void *) ddisk_dev->dma_buffer_phys);
+
+        bd_srvs_init(&ddisk_dev->bds);
+        ddisk_dev->bds.ops = &ddisk_bd_ops;
+        ddisk_dev->bds.sarg = ddisk_dev;
 
         rc = ddisk_fun_create(ddisk_dev);
@@ -509 +485 @@
          */
         ddf_msg(LVL_NOTE,
-            "Device at %p with %zu blocks (%zuB) using interrupt %d",
-            (void *) ddisk_dev->ddisk_res.base, ddisk_dev->ddisk_fun->blocks,
-            ddisk_dev->ddisk_fun->blocks * DDISK_BLOCK_SIZE,
+            "Device at %p with %" PRIu32 " blocks (%" PRIu32
+            "B) using interrupt %d",
+            (void *) ddisk_dev->ddisk_res.base,
+            pio_read_32(&ddisk_dev->ddisk_regs->size) / DDISK_BLOCK_SIZE,
+            pio_read_32(&ddisk_dev->ddisk_regs->size),
             ddisk_dev->ddisk_res.irq);
 
         return EOK;
+
 error:
         if (ddisk_dev->ddisk_regs)
@@ -529 +508 @@
 
         if (!surprise)
-                rc = ddisk_fun_remove(ddisk_dev->ddisk_fun);
+                rc = ddisk_fun_remove(ddisk_dev);
         else
-                rc = ddisk_fun_unbind(ddisk_dev->ddisk_fun);
+                rc = ddisk_fun_unbind(ddisk_dev);
+
         if (rc != EOK) {
                 ddf_msg(LVL_ERROR, "Unable to cleanup function '%s'.",
@@ -554 +534 @@
 
         ddf_msg(LVL_DEBUG, "ddisk_dev_remove(%p)", dev);
-
         return ddisk_dev_remove_common(ddisk_dev, false);
 }
@@ -563 +542 @@
 
         ddf_msg(LVL_DEBUG, "ddisk_dev_gone(%p)", dev);
-
         return ddisk_dev_remove_common(ddisk_dev, true);
 }
@@ -582 +560 @@
 static void ddisk_bd_connection(ipc_callid_t iid, ipc_call_t *icall, void *arg)
 {
-        ddisk_fun_t *ddfun;
-
-        ddfun = (ddisk_fun_t *) ddf_fun_data_get((ddf_fun_t *)arg);
-        bd_conn(iid, icall, &ddfun->bds);
+        ddisk_dev_t *ddisk_dev;
+        ddf_fun_t *fun = (ddf_fun_t *) arg;
+
+        ddisk_dev = (ddisk_dev_t *) ddf_dev_data_get(ddf_fun_get_dev(fun));
+        bd_conn(iid, icall, &ddisk_dev->bds);
 }