WIP: user-space: avoid use of k_object_free() in user code

Add a warning and a comment to all places where k_object_free()
is called in code that may be run in user threads.

This will result in resource leaks, but there is currently no
immediate solution to how to free these objects from user threads.
We may need to revisit the partition of functionality between system
and user.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>

Author: kv2019i

src/audio/dai-zephyr.c

@@ -670,7 +670,12 @@ __cold void dai_common_free(struct dai_data *dd)
 	dai_release_llp_slot(dd);
 
 #ifdef CONFIG_SOF_USERSPACE_LL
-	k_object_free(dd->dai->lock);
+	/*
+	 * k_object_free() is not available to user-space, resources
+	 * are freed when thread terminates
+	 */
+	LOG_WRN("leaking semaphore as k_object_free() not available..");
+	/*k_object_free(dd->dai->lock);*/
 #endif
 
 	dai_put(dd->dai);

src/include/sof/audio/component_ext.h

@@ -55,7 +55,11 @@ static inline void comp_free(struct comp_dev *dev)
 	drv->ops.free(dev);
 
 #ifdef CONFIG_SOF_USERSPACE_LL
-	k_object_free(dev->list_mutex);
+	/*
+	 * k_object_free() is not available to user-space, resources
+	 * are freed when thread terminates
+	 */
+	k_object_release(dev->list_mutex);
 #endif
 }
 

src/schedule/zephyr_ll.c

@@ -452,7 +452,12 @@ static int zephyr_ll_task_sched_free(void *data, struct task *task)
 		k_sem_take(pdata->sem, K_USEC(LL_TIMER_PERIOD_US * 100));
 
 #ifdef CONFIG_SOF_USERSPACE_LL
-	k_object_free(pdata->sem);
+	/*
+	 * k_object_free() is not available to user-space, resources
+	 * are freed when thread terminates
+	 */
+	LOG_WRN("leaking semaphore as k_object_free() not available..");
+	/*k_object_free(pdata->sem);*/
 #endif
 
 	/* Protect against racing with schedule_task() */