segmentation fault sur producteur/consommateur

segmentation fault sur producteur/consommateur - C - Programmation

Marsh Posté le 26-02-2007 à 10:19:57    

Bonjour,
 
Je dois réaliser un programme illustrant le problème du producteur/consommateur en C.
 
Avec un seul producteur et consommateur, je ne rencontre aucun problème mais avec plusieurs, j'obtiens une segmentation fault.
 
Voici mon code:

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <unistd.h>
  4. #include <stdlib.h>
  5. #include <sys/types.h>
  6. #include <sys/stat.h>
  7. #include <semaphore.h>
  8. #include <pthread.h>
  9. #include <fcntl.h>
  11. #define MAX 10
  13. int nb=0, buffer[10], cpt1=0, cpt2=0; //un nombre qui sera la ressource commune
  15. //Déclaration de sémaphores
  16. sem_t a; //EM entre producteurs ou consommateurs
  17. sem_t b; //attente de producteur par le consommateur
  18. sem_t c; //attente des producteurs en cas de buffer plein
  20. void *prod (void *d)
  21. {
  22.   sem_wait(&c);
  23.   sem_wait(&a);
  24.   printf("Je suis le producteur et je modifie la ressource commune\n" );
  25.   nb++; //modification de la ressource commune
  26.   printf("Je suis le producteur et je met la ressource commune dans le buffer\n" );
  27.   buffer[cpt1]=nb;
  28.   cpt1++;
  29.   if (cpt1 == MAX)
  30.   {
  31.     cpt1=0;
  32.   }
  33.   sem_post(&a);
  34.   sem_post(&b);
  35. }
  37. void *cons (void *d)
  38. {
  39.   sem_wait(&b);
  40.   sem_wait(&a);
  41.   printf("Je suis le consommateur et je prend la ressource commune dans le buffer\n" );
  42.   nb=buffer[cpt2];
  43.   printf("Je suis le consommateur et j'affiche le nombre qui est: %d\n", nb);
  44.   cpt2++;
  45.   if (cpt2 == MAX)
  46.   {
  47.     cpt2=0;
  48.   }
  49.   sem_post(&a);
  50.   sem_post(&c);
  51. }
  53. int main()
  54. {
  55.   pthread_t produ, conso, nbprod, nbconso;
  56.   int rc,t;
  58.   //initialisation des semaphores
  59.   sem_init(&a, 0, 1);
  60.   sem_init(&b, 0, 0);
  61.   sem_init(&c, 0, MAX);
  63.   for (t=0; t<10; t++)
  64.   {
  65.     buffer[t]=0;
  66.   }
  68.   printf("Entrer le nombre de producteurs\n" );
  69.   scanf("%d", nbprod);
  71.   printf("Entrer le nombre de consommateurs\n" );
  72.   scanf("%d", nbconso);
  74.   for (t=0; t<nbprod; t++)
  75.   {
  76.     rc = pthread_create(&produ, NULL, prod, 0);
  77.     if (rc)
  78.     {
  79.       printf("ERROR; return code from pthread_join() is %d\n", rc);
  80.     }
  81.   }
  83.   for (t=0; t<nbconso; t++)
  84.   {
  85.     rc = pthread_create(&conso, NULL, cons, 0);
  86.     if (rc)
  87.     {
  88.       printf("ERROR; return code from pthread_join() is %d\n", rc);
  89.     }
  90.   }
  92.   pthread_join(produ,0);
  93.   pthread_join(conso,0);
  94. }


 
Pouvez-vous m'aider svp?
 
Merci

Message cité 1 fois
Reply

Marsh Posté le 26-02-2007 à 10:19:57   

Reply

Marsh Posté le 26-02-2007 à 10:58:52    

#
 printf("Entrer le nombre de producteurs\n" );
#
 scanf("%d", nbprod);
#
 
#
 printf("Entrer le nombre de consommateurs\n" );
#
 scanf("%d", nbconso);
 
 
et on y croit très fort ...

Reply

Marsh Posté le 26-02-2007 à 11:10:50    

C'est vrai que un petit &, ça change tout!
 
Merci^^

Reply

Marsh Posté le 26-02-2007 à 11:40:19    

ça change rien. pthread_t n'est pas un int.

Reply

Marsh Posté le 26-02-2007 à 13:10:17    

Enfin, le principal, c'est que le programme fonctionne maintenant!
 
Mais il passe d'abord par tous les producteurs puis il fait tous les consommateurs, y aurait-il un moyen que ça se mélange un peu plus?

Reply

Marsh Posté le 26-02-2007 à 14:25:24    

ton programme ne fonctionne pas correctement avec ce mélange pthread/int. Corrige cela en premier. Et apprend à compiler avec des warnings.

Reply

Marsh Posté le 26-02-2007 à 19:15:11    

raks a écrit :

Je dois réaliser un programme illustrant le problème du producteur/consommateur en C.
 
Avec un seul producteur et consommateur, je ne rencontre aucun problème mais avec plusieurs, j'obtiens une segmentation fault.
 
Voici mon code:


Ton code ne compile pas. Corrections pour le rendre compilable :  

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <pthread.h>
  4. #include <semaphore.h>
  6. #ifdef WIN32
  8. #else
  10. #include <unistd.h>
  11. #include <stdlib.h>
  12. #include <sys/types.h>
  13. #include <sys/stat.h>
  14. #include <fcntl.h>
  16. #endif
  18. #define MAX 10
  20. int nb = 0, buffer[10], cpt1 = 0, cpt2 = 0; //un nombre qui sera la ressource commune
  22. //Déclaration de sémaphores
  23. sem_t a;                        //EM entre producteurs ou consommateurs
  24. sem_t b;                        //attente de producteur par le consommateur
  25. sem_t c;                        //attente des producteurs en cas de buffer plein
  27. void *prod (void *d)
  28. {
  29.    sem_wait (&c);
  30.    sem_wait (&a);
  31.    printf ("Je suis le producteur et je modifie la ressource commune\n" );
  32.    nb++;                        //modification de la ressource commune
  33.    printf
  34.       ("Je suis le producteur et je met la ressource commune dans le buffer\n" );
  35.    buffer[cpt1] = nb;
  36.    cpt1++;
  37.    if (cpt1 == MAX)
  38.    {
  39.       cpt1 = 0;
  40.    }
  41.    sem_post (&a);
  42.    sem_post (&b);
  44.    /* -ed- parametre non utilise. C'est louche... */
  45.    (void) d;
  47.    /* -ed- un thread doit retourner une adresse ou NULL */
  48.    return NULL;
  49. }
  51. void *cons (void *d)
  52. {
  53.    sem_wait (&b);
  54.    sem_wait (&a);
  55.    printf
  56.       ("Je suis le consommateur et je prend la ressource commune dans le buffer\n" );
  57.    nb = buffer[cpt2];
  58.    printf ("Je suis le consommateur et j'affiche le nombre qui est: %d\n",
  59.            nb);
  60.    cpt2++;
  61.    if (cpt2 == MAX)
  62.    {
  63.       cpt2 = 0;
  64.    }
  65.    sem_post (&a);
  66.    sem_post (&c);
  68.    (void) d;
  69.    return NULL;
  70. }
  72. /* -ed- manque void */
  73. int main (void)
  74. {
  75.    pthread_t produ, conso;
  77.    /* -ed- etait de type pthread_t... */
  78.    int nbprod, nbconso;
  79.    int rc, t;
  81.    //initialisation des semaphores
  82.    sem_init (&a, 0, 1);
  83.    sem_init (&b, 0, 0);
  84.    sem_init (&c, 0, MAX);
  86.    for (t = 0; t < 10; t++)
  87.    {
  88.       buffer[t] = 0;
  89.    }
  91.    printf ("Entrer le nombre de producteurs\n" );
  92.    /* -ed-
  93.       scanf("%d", nbprod);
  95.       scanf() avec "%d" attend l'adrese d'un int */
  96.    scanf ("%d", &nbprod);
  98.    printf ("Entrer le nombre de consommateurs\n" );
  99.    scanf ("%d", &nbconso);
  101.    for (t = 0; t < nbprod; t++)
  102.    {
  103.       rc = pthread_create (&produ, NULL, prod, 0);
  104.       if (rc)
  105.       {
  106.          printf ("ERROR; return code from pthread_join() is %d\n", rc);
  107.       }
  108.    }
  110.    for (t = 0; t < nbconso; t++)
  111.    {
  112.       rc = pthread_create (&conso, NULL, cons, 0);
  113.       if (rc)
  114.       {
  115.          printf ("ERROR; return code from pthread_join() is %d\n", rc);
  116.       }
  117.    }
  119.    pthread_join (produ, 0);
  120.    pthread_join (conso, 0);
  122.    /* -ed- main() retourne une valeur au système. 0 est correct (OK) */
  123.    return 0;
  124. }


Joli cas de blocage... (Win32)


Entrer le nombre de producteurs
2
Entrer le nombre de consommateurs
3
Je suis le producteur et je modifie la ressource commune
Je suis le producteur et je met la ressource commune dans le buffer
Je suis le producteur et je modifie la ressource commune
Je suis le producteur et je met la ressource commune dans le buffer
Je suis le consommateur et je prend la ressource commune dans le buffer
Je suis le consommateur et j'affiche le nombre qui est: 1
Je suis le consommateur et je prend la ressource commune dans le buffer
Je suis le consommateur et j'affiche le nombre qui est: 2


Je te conseille ceci :  
 
http://mapage.noos.fr/emdel/pthreads.htm


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed