Discussion :

[ikuzar]

Bonjour,
j'aimerais savoir comment doit - on procéder pour récuperer par l'intérmédiaire de pthread_join(thread, val_ret) une valeur de retour envoyé par pthread_exit(val_ret). Dans mon cas la valeur à recuperer est un entier.

Voici les syntaxes exactes:

Code :

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

1
2
pthread_exit(void *retval);
pthread_join(pthread th, void  **thread_return);

Voici à peu près ce que je dois faire ( est ce que c'est correcte ? )

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void *my_thread(void *vargp)
{
    int retval = 42
    pthread_exit(&retval);
}
int main()
{
    int *i;
    pthread_t tid;
    pthread_create(&tid, NULL, my_thread, NULL);
    pthread_join(tid, &i);
    printf("%d\n",i);
}

Merci d'avance pour votre aide

[nicolas.sitbon]

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
26
27
28
29
30
31
32
#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
 
static void * my_thread(void * vargp)
{
   int * retval = malloc(sizeof *retval);
 
   if(retval != NULL)
   {
      *retval = 42;
   }
 
   pthread_exit(&retval);
}
 
int main(void)
{
   int * i = NULL;
   pthread_t tid;
 
   pthread_create(&tid, NULL, my_thread, NULL);
   pthread_join(tid, (void**)&i);
 
   if(i != NULL)
   {
      printf("%d\n", *i);
      free(i), i = NULL;
   }
 
   return 0;
}

Cordialement.

[Médinoc]

Tu as oublié le free après le printf...
Et il faudra sans doute un cast lors de l'appel à pthread_join().

Edit: Corrigé depuis.

[nicolas.sitbon]

Tu as oublié le free après le printf...

Et il faudra sans doute un cast lors de l'appel à pthread_join().

Et oui, voilà ce qu'il se passe quand on ne prend même pas la peine de compiler son code...
Merci Médinoc.

[ikuzar]

Voici le programme qui compile sans erreur ni warning et qui affiche bien la valeur voulue : 42

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
26
27
28
#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
 
static void * my_thread(void * vargp)
{
  int *retval = malloc(sizeof retval);
 
   if(retval != NULL)
   {
     *retval = 42;
   }
   //pthread_exit(void *retval);
   pthread_exit(retval);
}
 
int main(void)
{
   int * i = NULL;
   pthread_t tid;
 
   pthread_create(&tid, NULL, my_thread, NULL);
   // pthread_join(pthread thread,void **thread_return);
   pthread_join(tid,(void **) &i);
   printf("%d\n", *i);
   free(i);
   return 0;
}

Tu as oublié le free après le printf...
Et il faudra sans doute un cast lors de l'appel à pthread_join().

Edit: Corrigé depuis.

- pourquoi doit on faire un cast lors de l'appel à pthread_join( ) ? Ne devrait - on pas le faire aussi dans pthread_exit( ) ...

quand je fais ceci :

Code :

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

1
2
3
4
5
6
static void * my_thread(void * vargp)
{
  int retval = 42; 
   //pthread_exit(void *retval);
   pthread_exit(&retval);
}

ca compile MAIS a l'execution ca ne donne pas 42, ca donne -1208470568 puis une erreur de segmentation à cause de free(i);

- pourquoi obtient on -1208470568 au lieu de 42 ? pthread_exit( void *retval ) attends en paramètre l'adresse d'un int ( c'est ce que j'ai fait ), non ?

Merci d'avance pour votre aide .

[nicolas.sitbon]

- pourquoi doit on faire un cast lors de l'appel à pthread_join( ) ? Ne devrait - on pas le faire aussi dans pthread_exit( ) ...

Parce que pthread_join() attend en second paramètre un (void**) et qu'on lui passe un (int**).
pthread_exit() n'en a pas besoin dans la mesure où son argument est un pointeur générique (void*).

quand je fais ceci :

Code :

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

1
2
3
4
5
6
static void * my_thread(void * vargp)
{
  int retval = 42; 
   //pthread_exit(void *retval);
   pthread_exit(&retval);
}

ca compile MAIS a l'execution ca ne donne pas 42, ca donne -1208470568 puis une erreur de segmentation à cause de free(i);

- pourquoi obtient on -1208470568 au lieu de 42 ?

Tu retournes une adresse sur la pile, le soucis c'est qu'au retour de ta fonction cette adresse n'a plus aucune signification, donc comportement indéterminé. Voilà pourquoi je fais une allocation dynamique.

pthread_exit( void *retval ) attends en paramètre l'adresse d'un int ( c'est ce que j'ai fait ), non ?

pthread_exit() prend en paramètre un pointeur générique, donc elle n'attend pas spécialement l'adresse d'un int, ça peut être l'adresse de n'importe quel objet. ton soucis, c'est que ton objet n'existe plus quand tu retournes dans ta fonction main().