Changeset 800d188 in mainline for uspace/srv/bd/hr/io.c

Timestamp:

2025-04-15T07:00:31Z (2 months ago)

Author:

Miroslav Cimerman <mc@…>

Children:

29e7cc7

Parents:

ca7fa5b

git-author:

Miroslav Cimerman <mc@…> (2025-04-15 06:55:25)

git-committer:

Miroslav Cimerman <mc@…> (2025-04-15 07:00:31)

Message:

hr: metadata saving on stop

New bool argument for hr_metadata_save() to indicate if
we want callback too (on array stop we don't, so we won't
init a rebuild).

File:

• uspace/srv/bd/hr/io.c (modified) (2 diffs)

uspace/srv/bd/hr/io.c

@@ -45 +45 @@
 #include "var.h"
 
+static errno_t exec_io_op(hr_io_t *);
+
 errno_t hr_io_worker(void *arg)
 {
         hr_io_t *io = arg;
+
+        errno_t rc = exec_io_op(io);
+
+        /*
+         * We don't have to invalidate extents who got ENOMEM
+         * on READ/SYNC. But when we get ENOMEM on a WRITE, we have
+         * to invalidate it, because there could have been
+         * other writes, there is no way to rollback.
+         */
+        if (rc != EOK && (rc != ENOMEM || io->type == HR_BD_WRITE))
+                io->vol->state_callback(io->vol, io->extent, rc);
+
+        return rc;
+}
+
+errno_t hr_io_worker_basic(void *arg)
+{
+        hr_io_t *io = arg;
+
+        errno_t rc = exec_io_op(io);
+
+        return rc;
+}
+
+static errno_t exec_io_op(hr_io_t *io)
+{
+        size_t ext_idx = io->extent;
         hr_extent_t *extents = (hr_extent_t *)&io->vol->extents;
-        size_t ext_idx = io->extent;
         errno_t rc;
 
@@ -90 +118 @@
         HR_DEBUG("WORKER (%p) rc: %s\n", io, str_error(rc));
 
-        /*
-         * We don't have to invalidate extents who got ENOMEM
-         * on READ/SYNC. But when we get ENOMEM on a WRITE, we have
-         * to invalidate it, because there could have been
-         * other writes, there is no way to rollback.
-         */
-        if (rc != EOK && (rc != ENOMEM || io->type == HR_BD_WRITE))
-                io->state_callback(io->vol, io->extent, rc);
-
         return rc;
 }