Discussion :

[firejocker]

Bonjour,

j'ai un soucis avec le passage en parametre d'une fonction,
la doc de la fonction est :

OSTaskCreate
-------------

Purpose To create a task.

Syntax :

unsigned char OSTaskCreate (void (*task) (void * pd), void *pdata, unsigned char *pstk, unsigned long stk_size, unsigned char prio);


where, task is a pointer to the task's code.

pdata is a pointer to an optional data area, which can be used to pass parameters to the task when it is created.

pstk is a pointer to the task's top of stack. The stack is used to store local
variables, function parameters, return addresses, and CPU registers during an
interrupt. The size of this stack is defined by the task requirements and the
anticipated interrupt nesting. Determining the size of the stack involves
knowing how many bytes are required for storage of local variables for the task itself, all nested functions, as well as requirements for interrupts (accountingfor nesting).

prio is the task priority. A unique priority number must be assigned to each
task; the lower the number, the higher the priority.
Example OSTaskCreate (beep_task, (void *)0, beep_stk, 256, 10);

/* create a beep_task with priority 10 */
static unsigned char beep_stk [256];
Description This function allows an application to create a task. The task is managed by
μC/OS. Tasks can be created prior to the start of multitasking or by a running
task. Note that a task cannot be created by an ISR.

Lorsque j'essai d'appeler une fonction simple, avec un seul parametre, pas de probleme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
void TaskReader(char ch[10]);
.....
.....
char s[]="test";
OSTaskCreate ((void *)TaskReader, (void *)(s), (void *)Stack, STK_SIZE, 14);

ce code marche nickel,

par contre avec :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
void TaskReader(int a, char ch[10]);
....
....
int a = 3;
char s[]="test";
OSTaskCreate ((void *)TaskReader, (void *)(a, s), (void *)Stack, STK_SIZE, 14);

ca, ca plante completement et je vois pas comment l'ecrire, j'ai essayé de passer par une structure qui contiendrai mes 2 parametres, erreur !

merci.

[homeostasie]

Tu peux passer un tableau de tes paramètres. Puis récupérer dans ta fonction l'adresse du premier élément.
Je pense qu'avec une structure, ca devrait aussi bien fonctionner.
Pourrais tu montrer comment tu t'y es pris?

[firejocker]

je m'y suis pris comme je l'ai mis en code dans moi precedent post,

pour la structure, j'avais fais ca :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
struct maStruct{
    int a;
    char s[10];
};
 
struct maStruct param;
 
param.a = 321;
strcpy(param.s, "test");
 
OSTaskCreate ((void *)TaskReader, (void *)param, (void *)Stack, STK_SIZE, 14);

Tu peux passer un tableau de tes paramètres.

Comment ca ?

[homeostasie]

Faut que tu déclares ta structure à passer en paramètre en tant que pointeeur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
struct maStruct{
    int a;
    char s[10];
};

struct maStruct *param;

param->a = 321;
strcpy(param->s, "test");

OSTaskCreate ((void *)TaskReader, (void *)param, (void *)Stack, STK_SIZE, 14);

Je pense que ca devrait le faire!

[DaZumba]

Je pense que ca devrait le faire!

En utilisant un pointeur non-initialise? Ca m'etonnerait...

[homeostasie]

En utilisant un pointeur non-initialise? Ca m'etonnerait...

Oui oui, pardon, c'est clair!

Plutot cela:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
struct maStruct{
    int a;
    char s[10];
};

struct maStruct param;

param.a = 321;
strcpy(param.s, "test");

OSTaskCreate ((void *)TaskReader, &param, (void *)Stack, STK_SIZE, 14);

Dans la fonction OSTaskCreate, tu fais ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
struct maStruct *param = pData;

[firejocker]

nan, ca donne rien, et j'ai essayé plein de mix de code et ca donne rien.

et meme en rajoutant avant un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

param = (struct maStruct *) malloc(sizeof(struct maStruct));

ce que j'ai essayé :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
OSTaskCreate ((void *)TaskReader, (void *)param, (void *)Stack, STK_SIZE, 14);
OSTaskCreate ((void *)TaskReader, param, (void *)Stack, STK_SIZE, 14);
OSTaskCreate ((void *)TaskReader, &param, (void *)Stack, STK_SIZE, 14);
...

Dans la fonction OSTaskCreate, tu fais ceci:

J'y ai pas acces, c'est une fonction qui appartient à une librairie proprietaire, je code en embarque dans un terminal de lecture de code barre portatif

[homeostasie]

Ce que j'ai fait tout à l'heure(transmettre par référence), ca doit fonctionner, sinon tu peux faire ainsi:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
struct maStruct{
    int a;
    char s[10];
};

struct maStruct *param = malloc(sizeof(maStruct));;

param->a = 321;
strcpy(param->s, "test");

OSTaskCreate ((void *)TaskReader, param, (void *)Stack, STK_SIZE, 14);

Puis dans OSTaskCreate (...):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
unsigned char OSTaskCreate(...,void* param1,...)
{
    maStruct *param = param1;
    ...
}

[firejocker]

Je n'ai pas acces aux sources de OSTaskCreate.

[homeostasie]

Je n'ai pas acces aux sources de OSTaskCreate.

Oui j'ai lu ta remarque après mon post...
Hum hum, en fait, dans ce cas là, je ne sais pas vraiment.
Perso, je tenterais ainsi:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
int tab[3] = {0};
OSTaskCreate ((void *)TaskReader, (void*)tab, (void *)Stack, STK_SIZE, 14);

Ce que je trouve bizarre, c'est que tu ne peux même pas voir le code de la tâche que tu crées...Car les paramètres que tu décides de transmettre sont en fonction de ce qu'elle a besoin de recevoir.

[Médinoc]

Note: De manière habituelle, on évite de caster un pointeur de fonction. La fonction pointée DOIT avoir le bon prototype, sinon c'est la porte ouverte aux erreurs comme celle vue avec deux paramètres.

Et aussi, on évite de caster les retours de malloc et on évite de caster en void* : Un cast n'est nécessaire que dans des cas particuliers (passage par un intptr_t, etc.)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
struct maStruct{
	int a;
	char s[10];
};
 
void TaskReader(void *pVoid)
{
	struct maStruct *pParam = pVoid;
 
	//Utilisation de la structure
	//...
 
	//À la fin, on la libère
	free(pParam);
}
 
void Fonction(void)
{
	struct maStruct const param = { 321, "test" };
	struct maStruct *pParam = malloc(sizeof(*pParam));
 
	memcpy(pParam, &param, sizeof(param));
 
	OSTaskCreate(TaskReader, pParam, Stack, STK_SIZE, 14);
}

[homeostasie]

Medinoc, dans son cas, il ne peut modifier le code de la tâche qu'il veut lancer... C'est à dire faire cela:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
void TaskReader(void *pVoid)
{
	struct maStruct *pParam = pVoid;

	//Utilisation de la structure
	//...

	//À la fin, on la libère
	free(pParam);
}

[Médinoc]

Si, il a parfaitement accès aux sources de sa tâche puisqu'il y avait ajouté un paramètre.
C'est la fonction OSTaskCreate dont il ne possède pas les sources, ce qui - je pense - est normal.

[firejocker]

Yes, le code de Médinoc fonctionne nickel, c'est pas con du tout ca !

Medinoc, dans son cas, il ne peut modifier le code de la tâche qu'il veut lancer... C'est à dire faire cela:

Si si, TaskReader c'est ma fonction que j'appel, c'etait OSTaskCreate que je ne pouvais modifier.

Merci à vous deux !

edit :

C'est la fonction OSTaskCreate dont il ne possède pas les sources, ce qui - je pense - est normal.

Ouep, c'est normal, c'est une fonction de mon terminal, c'est comme si je voulais modifier le printf du c

[homeostasie]

Arf, sourire, c'est moi dès le début qui t'es mal renseigné, c'était bien dans cette fonction qu'il fallait faire la modif...
Hum hum, je sais ce qu'il me reste à faire, partir en we!

Bon we

[firejocker]

lol

merci encore et bon we