Discussion :

[PoZZyX]

Bonjour,

J'ai un petit souci en C sur linux avec la librairie pthread.

J'essaie de passer aux thread que je créé leur id (int), pour ce faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
main :
-----------
int n;
n = ...
pthread_create(&(clients[n].thread), NULL, handle_client, &n);
 
void* handle_client(void * n)
{
	int position_n = *((int *) n);
	....
}

Cela fonctionne 9x sur 10, mais j'ai l'impression que je reçois parfois la mauvaise valeur lors d'une série de créations (n varie à chaque fois).

Malheureusement je ne sais pas trop comment tester vu qu'un printf peut avoir lieu avant un autre.

J'ai cependant lu à quelque part qu'il fallait mettre de sleep après la création d'un thread ?!?! Est-ce correct ? est ce que mon problème peut venir de là ? ou de la façon par laquelle je passe mon paramètre ?

Merci d'avance pour votre aide !!!

[LittleWhite]

Bonjour,

J'ai un doute sur le morceau de code que vous avez envoyé. ( Du coup, j'aurai préférré le vrai, mais bon)

L'idée du sleep n'est peut être pas si mauvaise en y reflechissant, pourquoi?
Voici ce que peux faire le programme ->

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int n = 43;
 
// Création du thread on passe un pointeur sur n ( id = 1 )
// -> Oops mon système a donné la main au père
// Changement de n
// Création du deuxième thread ( id = 2 )
// 2 copie n dans la variable locale
// -> Mon système met en pause 2, et repasse sur 1
// 1 copie enfin n, qui, comme c'était un pointeur ... la valeur de n a été changé

Voici un cas d'erreur ( qui arrivera un cas sur 10 ... )
Je ne suis pas vraiment pour une solution avec un sleep. Juste l'utilisation de plus de variables peut être une bonne idée.

[Pouet_forever]

Tu dois sûrement essayer de modifier n dans plusieurs threads différents. Utilises les mutex pour être sûr que ta variable soit toujours 'valide'.

[PoZZyX]

Effectivement c'était plutôt :

Code :

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

1
2
3
4
5
6
7
 
while (1)
{
n = new_n(); // n tjs différent
// accept socket
pthread_create(&(clients[n].thread), NULL, handle_client, &n);
}

En ajoutant un mutex avant le n = new_n() et au début de ma fonction handle_client, il me semble que le problème ne survient plus.

J'avais déjà essayé l'idée du mutex, mais je l'avais mis APRES le new_n() et non pas AVANT comme maintenant.

En tout cas merci pour votre aide !!!