Re: [Xenomai] FreeRTOS skin #2

[Philippe Gerum <rpm@xenomai.org>]

On 07/09/2014 09:42 AM, Philippe Gerum wrote:
> On 07/01/2014 06:48 PM, Matthias Schneider wrote:
>> Please find enclosed another calling the check_processors() only when 
>> starting
>> the scheduler and remembering its output. It is also adapted to the 
>> latest changes.
>> Any comments will be appreciated.
>>
> 
> - clusters allowing key dups are available using the *_dup() call form

I just noticed that you would need uniquement labels for pushing your tasks to the registry anyway, so the following patch on top of the previous one would likely do:

diff --git a/lib/freertos/task.c b/lib/freertos/task.c
index d201711..d005163 100644
--- a/lib/freertos/task.c
+++ b/lib/freertos/task.c
@@ -426,10 +426,17 @@ static signed portBASE_TYPE __xTaskGenericCreate(struct freertos_task *task,
 
 	generate_name(task->name, (const char *)name, &task_namegen);
 
-	ret = cluster_addobj_dup(&freertos_task_table, task->name, &task->cobj);
-	if (__bt(ret)) {
-		threadobj_uninit(&task->thobj);
-		return pdFALSE;
+	for (;;) {
+		ret = cluster_addobj(&freertos_task_table, task->name, &task->cobj);
+		if (ret == -EEXIST) {
+			generate_name(task->name, NULL, &task_namegen);
+			continue;
+		}
+		if (__bt(ret)) {
+			threadobj_uninit(&task->thobj);
+			return pdFALSE;
+		}
+		break;
 	}
 
 	registry_init_file_obstack(&task->fsobj, &registry_ops);

-- 
Philippe.