Changeset fb06476f in mainline for uspace/lib

Timestamp:

2025-04-21T13:20:05Z (6 months ago)

Author:

Miroslav Cimerman <mc@…>

Children:

c6d2af8

Parents:

5cb1631

Message:

hr: use enum for RAID layouts

Location:

uspace/lib/device

Files:

• include/hr.h (modified) (3 diffs)
• src/hr.c (modified) (2 diffs)

uspace/lib/device/include/hr.h

@@ -55 +55 @@
 } hr_level_t;
 
-/*
- * SNIA
- * Common RAID Disk Data Format
- * Specification
- * Version 2.0 Revision 19
- */
-#define HR_RLQ_RAID4_0  0x00 /* RAID-4 Non-Rotating Parity 0 */
-#define HR_RLQ_RAID4_N  0x01 /* RAID-4 Non-Rotating Parity N */
-#define HR_RLQ_RAID5_0R 0x00 /* RAID-5 Rotating Parity 0 with Data Restart */
-#define HR_RLQ_RAID5_NR 0x02 /* RAID-5 Rotating Parity N with Data Restart */
-#define HR_RLQ_RAID5_NC 0x03 /* RAID-5 Rotating Parity N with Data Continuation */
+typedef enum hr_layout {
+        HR_RLQ_NONE     = 0,
+        HR_RLQ_RAID4_0,         /* RAID-4 Non-Rotating Parity 0 */
+        HR_RLQ_RAID4_N,         /* RAID-4 Non-Rotating Parity N */
+        HR_RLQ_RAID5_0R,        /* RAID-5 Rotating Parity 0 with Data Restart */
+        HR_RLQ_RAID5_NR,        /* RAID-5 Rotating Parity N with Data Restart */
+        HR_RLQ_RAID5_NC         /* RAID-5 Rotating Parity N with Data Continuation */
+} hr_layout_t;
 
 typedef enum hr_vol_status {
@@ -119 +116 @@
         HR_METADATA_GEOM_MIRROR = 1,
         HR_METADATA_GEOM_STRIPE = 2,
-        HR_METADATA_LAST_DUMMY  = 3
+        HR_METADATA_LAST_DUMMY
 } hr_metadata_type_t;
 
@@ -132 +129 @@
 extern const char       *hr_get_vol_status_msg(hr_vol_status_t);
 extern const char       *hr_get_ext_status_msg(hr_ext_status_t);
-extern const char       *hr_get_layout_str(hr_level_t, uint8_t);
+extern const char       *hr_get_layout_str(hr_layout_t);
 extern const char       *hr_get_metadata_type_str(hr_metadata_type_t);
 

uspace/lib/device/src/hr.c

@@ -207 +207 @@
         if (vol_info->level == HR_LVL_4 || vol_info->level == HR_LVL_5) {
                 printf("layout: %s\n",
-                    hr_get_layout_str(vol_info->level, vol_info->layout));
+                    hr_get_layout_str(vol_info->layout));
         }
         if (vol_info->level == HR_LVL_0 || vol_info->level == HR_LVL_4) {
@@ -428 +428 @@
 }
 
-const char *hr_get_layout_str(hr_level_t level, uint8_t layout)
-{
-        switch (level) {
-        case HR_LVL_4:
-                switch (layout) {
-                case HR_RLQ_RAID4_0:
-                        return "RAID-4 Non-Rotating Parity 0";
-                case HR_RLQ_RAID4_N:
-                        return "RAID-4 Non-Rotating Parity N";
-                default:
-                        return "Invalid RAID 4 layout";
-                }
-        case HR_LVL_5:
-                switch (layout) {
-                case HR_RLQ_RAID5_0R:
-                        return "RAID-5 Rotating Parity 0 with Data Restart";
-                case HR_RLQ_RAID5_NR:
-                        return "RAID-5 Rotating Parity N with Data Restart";
-                case HR_RLQ_RAID5_NC:
-                        return "RAID-5 Rotating Parity N with Data Continuation";
-                default:
-                        return "Invalid RAID 5 layout";
-                }
+const char *hr_get_layout_str(hr_layout_t layout)
+{
+        switch (layout) {
+        case HR_RLQ_NONE:
+                return "RAID layout not set";
+        case HR_RLQ_RAID4_0:
+                return "RAID-4 Non-Rotating Parity 0";
+        case HR_RLQ_RAID4_N:
+                return "RAID-4 Non-Rotating Parity N";
+        case HR_RLQ_RAID5_0R:
+                return "RAID-5 Rotating Parity 0 with Data Restart";
+        case HR_RLQ_RAID5_NR:
+                return "RAID-5 Rotating Parity N with Data Restart";
+        case HR_RLQ_RAID5_NC:
+                return "RAID-5 Rotating Parity N with Data Continuation";
         default:
-                return "Invalid RAID level";
+                return "Invalid RAID layout";
         }
 }