Problème pthread_cancel et printf

Bonjour,

Je débute en threads C, et j'ai un petit problème d'annulation de threads.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

#include <stdio.h> #include <unistd.h> #include <pthread.h>   void * my_routine(void *arg) { int i; for (i = 0; i < 10; i++) { printf("%d\n", i); } return NULL; }   int main(int argc, char *argv[]) { pthread_t thread; int ret = pthread_create(&thread, NULL, my_routine, NULL); usleep(20); pthread_cancel(thread); pthread_join(thread, NULL); return 0; }

---

De temps en temps, le printf n'affiche pas la bonne valeur au moment de l'annulation :

Code:

---

1 2 3 4 5 6

0 1 2 3 4 4

---

Comment l'éviter ?

pthread_cancel serait-il l'équivalent de Thread.stop() en Java : il ne faut jamais l'utiliser ?

Bonjour,

ça me semble plus être un problème au niveau du printf utilisé dans un environnement multi-thread, mais je n'en suis pas certain, quelque chose du genre le cancel arrive en plein milieu du printf ...

sinon en ce qui concerne ton code, juste quelques remarques :

int main(char *argv[]) { n'est pas correct. Si tu n'utilises pas les arguments passés en ligne de commande int main(void) {, ou si tu les utilises int main(int argc, char *argv[]) {. :) tu fais du java toi

int ret = pthread_create(&thread, NULL, my_routine, NULL); c'est toujours mieux de vérifier la valeur de retour de ce genre d'appel et gérer les erreurs éventuelles.

Code:

---

1 2 3 4 5 6

void * my_routine(void *arg) { int i; for (i = 0; i < 10; i++) { printf("%d\n", i); } }

---

Cette fonction doit retourner une valeur et aucun return ... le comportement de ton programme n'est pas défini dans ce cas. Ici tu peux simplement rajouter un return NULL;.

Je ne connais pas les problèmes liés à l'utilisation de Thread.stop() en java, mais avec les pthreads il n'y a, a priori, pas de problèmes pour utiliser pthread_cancel ... peut-être des détails (comme tous les threads ne sont pas interruptibles) mais ils sont documentés dans la man page.

Citation:

---

Envoyé par kwariz

int main(char *argv[]) { n'est pas correct. Si tu n'utilises pas les arguments passés en ligne de commande int main(void) {, ou si tu les utilises int main(int argc, char *argv[]) {. :) tu fais du java toi

---

Oui, faute d'inattention, j'ai oublié le argc, normalement je mets argc et argv. Je ne savais pas qu'on pouvait mettre int main(void) (par contre je savais qu'on pouvait mettre int main() en C k&r). J'ai corrigé.

Citation:

---

Envoyé par kwariz

int ret = pthread_create(&thread, NULL, my_routine, NULL); c'est toujours mieux de vérifier la valeur de retour de ce genre d'appel et gérer les erreurs éventuelles.

---

Tout-à-fait. Mais pour un bête code de test, la flemme ^^

J'ai ajouté un usleep(500000) (500ms) à la fin. Ainsi, on peut voir que le numéro dupliqué est affiché après la pause, comme si quelque chose restait quelque part après l'annulation du printf, et qu'il était flushé à la fin…

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#include <stdio.h> #include <unistd.h> #include <pthread.h>   void * my_routine(void *arg) { int i; for (i = 0; i < 10; i++) { printf("%d\n", i); } return NULL; }   int main(void) { pthread_t thread; int ret = pthread_create(&thread, NULL, my_routine, NULL); usleep(20); pthread_cancel(thread); pthread_join(thread, NULL); usleep(500000); return 0; }

---

Oui, ce «bug» provient du fait que tu interromps printff en plein milieu de son boulot.
Essaye par exemple dans ta boucle for de rendre ton printf insensibles aux interruptions et d'ajouter un [codeinline]usleep[codeinline] pour ralentir le tout et avoir*un point où on peut traiter le cancel :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void * my_routine(void *arg) { int i; for (i = 0; i < 1000; i++) { pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, NULL); printf("%d\n", i); pthread_setcancelstate(PTHREAD_CANCEL_ENABLE, NULL); usleep(2); }   return NULL; }

---

Rallonge aussi le usleep du main.

Je comprends bien, mais pourquoi printf est-il un cancellation point s'il ne faut pas l'utiliser tel quel même quand le canceltype du thread est PTHREAD_CANCEL_DEFERRED?

printf n'est pas un cancellation point mais en contient un, certainement un write.
Enfin je suppose en décryptant la doc posix concernant ce point.
Dans celle-ci on a une liste de fonctions qui sont des cancellation points obligatoirement (printf n'en fait pas partie), et d'autres qui peuvent en être un (dont printf).

Merci beaucoup, c'est plus clair.

Cependant, "qui peuvent en être un": soit c'est en un, soit non. Dans le premier cas, ça doit laisser l'état cohérent lors de l'annulation. Dans le second cas, pas de problèmes.

Ça "en contient un" est sans doute la raison technique, mais est-ce le comportement attendu ?

Je pense qu'il faut comprendre le "peut en être un" par "selon l'implémentation, c'en est un"

Oui, mais si ça en est un, ça doit en être un "vrai", pas un qui laisse l'état inconsistant.

Citation:

---

Envoyé par ®om

Ça "en contient un" est sans doute la raison technique, mais est-ce le comportement attendu ?

---

C'est une bonne question ... à laquelle je n'ai pas vraiment de réponse.
Tout comme toi je trouve le comportement au minimum étrange et inattendu dans mon esprit du coup effectivement j'assimilerais ça effectivement à un «bug».
En gros le printf est interrompu en plein boulot, et le buffer de sorti est marqué sale (=à émettre) avant d'avoir pu être rempli correctement. Du coup au flush suivant le buffer est émis alors qu'il est dans un état indéterminé. Ce qui explique le comportement «le doublon n'est émis qu'à la fin du programme si aucun flush n'est fait avant», «il n'est pas émis si on provoque un sigfault délibérément après le join» ...
Maintenant pour savoir si c'est correct (en un sens), il faudrait faire le test en monothread avec des signaux qui interrompent un printf :) si le comportement est identique alors oui c'est normal (ce qui peut sembler ridicule).

Pour info, je n'ai pas trouvé d'équivalent dans la bug db de glibc.