Changeset 56214383 in mainline for uspace/srv/bd/hr/raid5.c

Timestamp:

2025-03-29T12:09:58Z (3 months ago)

Author:

Miroslav Cimerman <mc@…>

Children:

49de61c

Parents:

8a65373

Message:

hr: util: hr_util_add_hotspare()

File:

• uspace/srv/bd/hr/raid5.c (modified) (3 diffs)

uspace/srv/bd/hr/raid5.c

@@ -150 +150 @@
 errno_t hr_raid5_add_hotspare(hr_volume_t *vol, service_id_t hotspare)
 {
-        HR_DEBUG("hr_raid5_add_hotspare()\n");
+        HR_DEBUG("%s()", __func__);
 
         fibril_mutex_lock(&vol->lock);
-        fibril_mutex_lock(&vol->hotspare_lock);
-
-        if (vol->hotspare_no >= HR_MAX_HOTSPARES) {
-                HR_ERROR("hr_raid5_add_hotspare(): cannot add more hotspares "
-                    "to \"%s\"\n", vol->devname);
-                fibril_mutex_unlock(&vol->lock);
-                return ELIMIT;
-        }
-
-        vol->hotspares[vol->hotspare_no].svc_id = hotspare;
-
-        vol->hotspare_no++;
-
-        hr_update_hotspare_status(vol, vol->hotspare_no - 1, HR_EXT_HOTSPARE);
+
+        errno_t rc = hr_util_add_hotspare(vol, hotspare);
+        if (rc != EOK)
+                goto end;
 
         /*
@@ -184 +174 @@
         }
 
-        fibril_mutex_unlock(&vol->hotspare_lock);
+end:
         fibril_mutex_unlock(&vol->lock);
 
-        return EOK;
+        return rc;
 }
 
@@ -770 +760 @@
         hr_extent_t *rebuild_ext = &vol->extents[bad];
 
-        rc = block_init(rebuild_ext->svc_id);
-        if (rc != EOK) {
-                HR_ERROR("hr_raid5_rebuild(): initing (%lu) failed, "
-                    "aborting rebuild\n", rebuild_ext->svc_id);
-                goto end;
-        }
-
         HR_DEBUG("hr_raid5_rebuild(): starting rebuild on (%lu)\n",
             rebuild_ext->svc_id);