Changeset 9fc1d36d in mainline

Timestamp:

2024-10-28T20:03:53Z (7 months ago)

Author:

Miroslav Cimerman <mc@…>

Children:

1903d77

Parents:

e47a032

Message:

hr: RAID 0 status handling

File:

• uspace/srv/bd/hr/raid0.c (modified) (5 diffs)

uspace/srv/bd/hr/raid0.c

@@ -74 +74 @@
 };
 
+static errno_t hr_raid0_check_vol_status(hr_volume_t *vol)
+{
+        if (vol->status == HR_VOL_ONLINE)
+                return EOK;
+        return EINVAL;
+}
+
+/*
+ * Update vol->status and return EOK if volume
+ * is usable
+ */
+static errno_t hr_raid0_update_vol_status(hr_volume_t *vol)
+{
+        for (size_t i = 0; i < vol->dev_no; i++) {
+                if (vol->extents[i].status != HR_EXT_ONLINE) {
+                        log_msg(LOG_DEFAULT, LVL_ERROR,
+                            "RAID 0 needs all disks to be ONLINE, marking "
+                            "\"%s\" (%lu) as FAULTY",
+                            vol->devname, vol->svc_id);
+                        vol->status = HR_VOL_FAULTY;
+                        return EINVAL;
+                }
+        }
+
+        vol->status = HR_VOL_ONLINE;
+        return EOK;
+}
+
 static errno_t hr_raid0_bd_open(bd_srvs_t *bds, bd_srv_t *bd)
 {
@@ -109 +137 @@
 
         fibril_mutex_lock(&vol->lock);
+
+        rc = hr_raid0_check_vol_status(vol);
+        if (rc != EOK) {
+                fibril_mutex_unlock(&vol->lock);
+                return EIO;
+        }
 
         left = cnt;
@@ -135 +169 @@
                         rc = EINVAL;
                 }
-                if (rc != EOK)
+
+                if (rc == ENOENT) {
+                        hr_update_ext_status(vol, extent, HR_EXT_MISSING);
+                        rc = EIO;
                         goto error;
+                } else if (rc != EOK) {
+                        hr_update_ext_status(vol, extent, HR_EXT_FAILED);
+                        rc = EIO;
+                        goto error;
+                }
 
                 left -= cnt;
@@ -148 +190 @@
 
 error:
+        (void) hr_raid0_update_vol_status(vol);
         fibril_mutex_unlock(&vol->lock);
         return rc;
@@ -196 +239 @@
                 return EINVAL;
         }
+
+        rc = hr_raid0_update_vol_status(new_volume);
+        if (rc != EOK)
+                return rc;
 
         bd_srvs_init(&new_volume->hr_bds);