Discussion :

[pascal_06]

Bonjour à toutes et à tous,
je suis en train de suivre un cours d'initiation à la programmation système, notamment les pthread sous linux.
Nous avons vu un simple programme réalisant la somme de n premiers nombres dans deux thread, et à la fin le produit des deux résultats dans le thread principal:

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 <unistd.h>
#include <pthread.h>
 
void *run(void *arg) {
 
int i, n = *((int*)arg);
int *s=malloc(sizeof(int));
*s=0;  
if (n>0){
  for (i=1;i<=n;i++) *s+=i;
}
return s;
}
 
 
int main(void) {
  pthread_t thread1, thread2;
  int n1=3, n2=4;
  int *r1, *r2;
 
  pthread_create( &thread1, NULL, run, &n1);
  pthread_create( &thread2, NULL, run, &n2);
 
  pthread_join( thread1, (void**)&r1);
  pthread_join( thread2, (void**)&r2);
 
  printf("Resultat %d\n", *r1 * *r2);
 
  return EXIT_SUCCESS;
}

Je n'arrive pas à comprendre l'utilité de :

Code :

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

(void**)&r1

.

Pourriez vous m'aider s'il vous plait ?

Merci par avance !

[Matt_Houston]

Commenter les casts et compiler (ici avec -pedantic-errors ou -Wall -Wextra -Werror pour être strict) te donne la réponse :

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
pthread_void.c: In function ‘main’:
pthread_void.c:26:38: error: passing argument 2 of ‘pthread_join’ from incompatible pointer type
   pthread_join( thread1, /*(void**)*/&r1);
                                      ^
In file included from pthread_void.c:4:0:
/usr/include/pthread.h:156:5: note: expected ‘void **’ but argument is of type ‘int **’
 int pthread_join (pthread_t, void **);
     ^
pthread_void.c:27:38: error: passing argument 2 of ‘pthread_join’ from incompatible pointer type
   pthread_join( thread2, /*(void**)*/&r2);
                                      ^
In file included from pthread_void.c:4:0:
/usr/include/pthread.h:156:5: note: expected ‘void **’ but argument is of type ‘int **’
 int pthread_join (pthread_t, void **);
     ^

La fonction de jointure attend un pointeur vers une variable de type « pointeur générique », pour pouvoir y stocker la valeur de retour de la fonction exécutée par le thread. Or en C, si n'importe quel pointeur vers un type « objet » (à savoir, tout sauf une fonction) est convertible de manière bidirectionnelle en void *, ce n'est pas le cas pour un « pointeur de pointeur » en void **. Le compilateur a donc ici besoin d'une confirmation explicite de la part de l'utilisateur sous la forme d'un cast pour effectuer la conversion (int **) -> (void **).

[foetus]

10 lignes de code et : 2 fuites mémoires, 1 test malloc oublié, 2 tests NULL oubliés dans le main + 1 autre pour le paramètre reçu (sans parler du fait que le malloc il faut mieux le mettre dans le if)

Belle perf.

[pascal_06]

J'ai bien précisé que je suivais un cours d'initiation, ce qui implique que je sois "débutant" en la matière. Donc merci pour tes remarques, j'en parlerai au prof

La fonction de jointure attend un pointeur vers une variable de type « pointeur générique », pour pouvoir y stocker la valeur de retour de la fonction exécutée par le thread. Or en C, si n'importe quel pointeur vers un type « objet » (à savoir, tout sauf une fonction) est convertible de manière bidirectionnelle en void *, ce n'est pas le cas pour un « pointeur de pointeur » en void **. Le compilateur a donc ici besoin d'une confirmation explicite de la part de l'utilisateur sous la forme d'un cast pour effectuer la conversion (int **) -> (void **).

Bonjour,
merci pour ta réponse et l'astuce de compilation.
Je vais étudier ça en détail, ça me parait assez sibyllin cette affaire