Changeset 81685dd9 in mainline for uspace/lib/drv/generic/driver.c

Timestamp:

2017-10-20T07:18:57Z (7 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

feab36ae

Parents:

04efacc

Message:

Add devctl unload-drv subcommand to manually unload a driver that is not in use.

File:

• uspace/lib/drv/generic/driver.c (modified) (6 diffs)

uspace/lib/drv/generic/driver.c

@@ -68 +68 @@
 FIBRIL_MUTEX_INITIALIZE(functions_mutex);
 
+FIBRIL_RWLOCK_INITIALIZE(stopping_lock);
+static bool stopping = false;
+
 static ddf_dev_t *create_device(void);
 static void delete_device(ddf_dev_t *);
@@ -127 +130 @@
         }
         
+        fibril_rwlock_read_lock(&stopping_lock);
+
+        if (stopping) {
+                fibril_rwlock_read_unlock(&stopping_lock);
+                async_answer_0(iid, EIO);
+                return;
+        }
+        
         ddf_dev_t *dev = create_device();
         if (!dev) {
+                fibril_rwlock_read_unlock(&stopping_lock);
                 free(dev_name);
                 async_answer_0(iid, ENOMEM);
@@ -148 +160 @@
         
         if (res != EOK) {
+                fibril_rwlock_read_unlock(&stopping_lock);
                 dev_del_ref(dev);
                 async_answer_0(iid, res);
@@ -156 +169 @@
         list_append(&dev->link, &devices);
         fibril_mutex_unlock(&devices_mutex);
+        fibril_rwlock_read_unlock(&stopping_lock);
         
         async_answer_0(iid, res);
@@ -282 +296 @@
         
         async_answer_0(iid, (sysarg_t) rc);
+}
+
+static void driver_stop(ipc_callid_t iid, ipc_call_t *icall)
+{
+        /* Prevent new devices from being added */
+        fibril_rwlock_write_lock(&stopping_lock);
+        stopping = true;
+
+        /* Check if there are any devices */
+        fibril_mutex_lock(&devices_mutex);
+        if (list_first(&devices) != NULL) {
+                /* Devices exist, roll back */
+                fibril_mutex_unlock(&devices_mutex);
+                stopping = false;
+                fibril_rwlock_write_unlock(&stopping_lock);
+                async_answer_0(iid, EBUSY);
+                return;
+        }
+
+        fibril_rwlock_write_unlock(&stopping_lock);
+
+        /* There should be no functions at this point */
+        fibril_mutex_lock(&functions_mutex);
+        assert(list_first(&functions) == NULL);
+        fibril_mutex_unlock(&functions_mutex);
+
+        /* Reply with success and terminate */
+        async_answer_0(iid, EOK);
+        exit(0);
 }
 
@@ -312 +355 @@
                 case DRIVER_FUN_OFFLINE:
                         driver_fun_offline(callid, &call);
+                        break;
+                case DRIVER_STOP:
+                        driver_stop(callid, &call);
                         break;
                 default: