Changeset fc265b4 in mainline for uspace/lib/device/src/hr.c

Timestamp:

2025-05-07T10:41:29Z (5 weeks ago)

Author:

Miroslav Cimerman <mc@…>

Children:

af4ecb76

Parents:

ac4b70b

Message:

hr: IPC methods: don't create own hr_t session

File:

• uspace/lib/device/src/hr.c (modified) (13 diffs)

uspace/lib/device/src/hr.c

@@ -148 +148 @@
 }
 
-errno_t hr_auto_assemble(size_t *rassembled_cnt)
-{
-        hr_t *hr;
+errno_t hr_auto_assemble(hr_t *hr, size_t *rassembled_cnt)
+{
         errno_t rc;
         size_t assembled_cnt;
-
-        rc = hr_sess_init(&hr);
-        if (rc != EOK)
-                return rc;
 
         async_exch_t *exch = async_exchange_begin(hr->sess);
@@ -179 +174 @@
                 *rassembled_cnt = assembled_cnt;
 error:
-        hr_sess_destroy(hr);
         return rc;
 }
@@ -266 +260 @@
 }
 
-errno_t hr_stop(const char *devname)
-{
-        hr_t *hr;
+errno_t hr_stop(hr_t *hr, const char *devname)
+{
         errno_t rc;
         async_exch_t *exch;
@@ -277 +270 @@
                 return rc;
 
-        rc = hr_sess_init(&hr);
-        if (rc != EOK)
-                return rc;
-
         exch = async_exchange_begin(hr->sess);
         if (exch == NULL) {
@@ -290 +279 @@
         async_exchange_end(exch);
 error:
-        hr_sess_destroy(hr);
-        return rc;
-}
-
-errno_t hr_stop_all(void)
-{
-        hr_t *hr;
-        async_exch_t *exch;
-        errno_t rc;
-
-        rc = hr_sess_init(&hr);
-        if (rc != EOK)
-                return rc;
+        return rc;
+}
+
+errno_t hr_stop_all(hr_t *hr)
+{
+        async_exch_t *exch;
+        errno_t rc;
 
         exch = async_exchange_begin(hr->sess);
@@ -313 +296 @@
         async_exchange_end(exch);
 error:
-        hr_sess_destroy(hr);
-        return rc;
-}
-
-errno_t hr_fail_extent(const char *volume_name, unsigned long extent)
-{
-        hr_t *hr;
+        return rc;
+}
+
+errno_t hr_fail_extent(hr_t *hr, const char *volume_name, unsigned long extent)
+{
         errno_t rc;
         async_exch_t *exch;
@@ -328 +309 @@
                 return rc;
 
-        rc = hr_sess_init(&hr);
-        if (rc != EOK)
-                return rc;
-
         exch = async_exchange_begin(hr->sess);
         if (exch == NULL) {
@@ -341 +318 @@
         async_exchange_end(exch);
 error:
-        hr_sess_destroy(hr);
-        return rc;
-}
-
-errno_t hr_add_hotspare(const char *volume_name, const char *hotspare)
-{
-        hr_t *hr;
+        return rc;
+}
+
+errno_t hr_add_hotspare(hr_t *hr, const char *volume_name, const char *hotspare)
+{
         errno_t rc;
         async_exch_t *exch;
@@ -360 +335 @@
                 return rc;
 
-        rc = hr_sess_init(&hr);
-        if (rc != EOK)
-                return rc;
-
         exch = async_exchange_begin(hr->sess);
         if (exch == NULL) {
@@ -373 +344 @@
         async_exchange_end(exch);
 error:
-        hr_sess_destroy(hr);
-        return rc;
-}
-
-errno_t hr_print_status(void)
-{
-        hr_t *hr;
+        return rc;
+}
+
+errno_t hr_print_status(hr_t *hr)
+{
         errno_t rc, retval;
         async_exch_t *exch;
@@ -385 +354 @@
         size_t size, i;
         hr_vol_info_t *vols = NULL;
-
-        rc = hr_sess_init(&hr);
-        if (rc != EOK)
-                return rc;
 
         exch = async_exchange_begin(hr->sess);
@@ -429 +394 @@
 
         if (size == 0) {
-                printf("no active arrays\n");
+                printf("no active volumes\n");
                 goto error;
         }
@@ -440 +405 @@
 
 error:
-        hr_sess_destroy(hr);
         if (vols != NULL)
                 free(vols);