xrun: Add locking to pause/resume operations

Add locking to pause and resume operations to prevent container->status
from getting out of sync with the actual state of the container.

Signed-off-by: Mykyta Poturai <[email protected]>

xrun/src/xrun.c

@@ -96,6 +96,7 @@ struct container {
 	char dt_image[CONFIG_XRUN_MAX_PATH_SIZE];
 	bool has_dt_image;
 	enum container_status status;
+	struct k_mutex lock;
 	int refcount;
 };
 
@@ -234,6 +235,7 @@ static struct container *register_container_id(const char *container_id)
 
 	strncpy(container->container_id, container_id, CONTAINER_NAME_SIZE);
 	container->domid = next_domid++;
+	k_mutex_init(&container->lock);
 
 	sys_slist_append(&container_list, &container->node);
 	container->refcount = 1;
@@ -631,6 +633,7 @@ int xrun_pause(const char *container_id)
 	if (!container) {
 		return -EINVAL;
 	}
+	k_mutex_lock(&container->lock, K_FOREVER);
 
 	ret = domain_pause(container->domid);
 	if (ret) {
@@ -639,6 +642,7 @@ int xrun_pause(const char *container_id)
 
 	container->status = PAUSED;
 out:
+	k_mutex_unlock(&container->lock);
 	put_container(container);
 	return ret;
 }
@@ -651,6 +655,7 @@ int xrun_resume(const char *container_id)
 	if (!container) {
 		return -EINVAL;
 	}
+	k_mutex_lock(&container->lock, K_FOREVER);
 
 	ret = domain_unpause(container->domid);
 	if (ret) {
@@ -659,6 +664,7 @@ int xrun_resume(const char *container_id)
 
 	container->status = RUNNING;
 out:
+	k_mutex_unlock(&container->lock);
 	put_container(container);
 	return ret;
 }