Pere sterile

**Treuze** · 13/05/2006, 19h50

Laors la je comprends pas, apprement mon pere ne fait pas de fils !

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
 
#include <sys/types.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
//#include "message.h"
#include <sys/ipc.h>
#include <sys/msg.h>
#include <signal.h>
#include <errno.h>
#include <sys/wait.h>
 
#define CLE_IPCMSG 123
#define CLE_SEM_ATELIER 456
 
struct message{
        long type;
        int pid;
        char message[20];
        };
 
pid_t pidaccueil;
int msgid;
struct message client;
 
void processusaccueil();
void clientarrive();
void processusclient();
int traitanterreur();
int traitanterreurmsg(){}
 
int main(int argc, char** argv[]){
 
 
    int pipe_accueil_mecano[2];
    int pipe_mecano_client[2];
 
 
 
    if (msgid = msgget(CLE_IPCMSG,IPC_CREAT | IPC_EXCL) == -1)
        traitanterreurmsg(msgid);
    /*if (semget(CLE_SEM_ATELIER,IPC_CREAT|IPC_EXCL) == -1)
        traitanterreur(1);
    if (pipe(pipe_accueil_mecano) == -1)
        traitanterreur(2);    
    if (pipe(pipe_mecano_client) == -1)
        traitanterreur(3);*/
 
 
    printf("Bienvenue au garage !!!\n");
 
    switch(fork()){
            case -1 :
                  traitanterreur(4);
                  break;
            case  0 :
                  processusaccueil();
                  break;
            }
 
    printf ("fils cree"); 
 
    switch(fork()){
            case -1 :
                  traitanterreur(6);
            case 0 :
                processusclient();
    }    
    wait(NULL);
    wait(NULL);
}
 
void processusaccueil(){
 
    struct sigaction act_accueil;
    sigset_t sig_accueil;
 
     act_accueil.sa_handler = clientarrive;
    sigemptyset(&act_accueil.sa_mask);
    act_accueil.sa_flags = SA_RESTART;
 
    if (sigaction(SIGUSR1,&act_accueil,NULL) == -1)
        traitanterreur(5); 
 
    sigemptyset(&sig_accueil);
 
    pidaccueil = getpid();
 
    while (1){
            sigsuspend(&sig_accueil);
         }
}
 
void clientarrive(){
            int erreur;
            int taille_message = sizeof(client) - sizeof(long);
            if (erreur = msgrcv(msgid,&client,taille_message,0,IPC_NOWAIT) == -1)
                traitanterreurmsg(erreur);
 
            printf("%d",client.pid);
}
 
void processusclient(){
        pid_t pid;
 
        pid = getpid();
 
        kill(pidaccueil,SIGUSR1);
}
 
int traitanterreur(int erreur){
    switch(erreur){
        case 1 :
            printf("Erreur lors de la creation de la semaphore\n");
            break;                   
         case 2 : 
            printf("Erreur lors de la creation du pipe accueil -> mecano\n");
            break;
        case 3 : 
            printf("Erreur lors de la creation du pipe mecano -> client\n");
            break;
        case 4 :
            printf("Erreur de la creation du processus accueil\n");
            break;
        case 5 :
            printf("Erreur lors de la def. du signal accueil\n");
            break;        
        }
    exit(erreur);
}

Alors voila je sais pas ... un fork tt bete ca devrait marcher portt

**Treuze** · 13/05/2006, 21h17

Est ce que qqun pourrait le compiler , histoire de voir si ca vient de chez moi ...

**Jack_serious** · 13/05/2006, 21h59

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
 
#include <sys/types.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
/*#include "message.h"*/
#include <sys/ipc.h>
#include <sys/msg.h>
#include <signal.h>
#include <errno.h>
#include <sys/wait.h>
/*
** Includes manquants
*/
#include <unistd.h>
#include <signal.h>
#include <sys/types.h>
#include <sys/select.h>
 
 
#define CLE_IPCMSG 123
#define CLE_SEM_ATELIER 456
 
struct message{
  long type;
  int pid;
  char message[20];
};
 
pid_t pidaccueil;
int msgid;
struct message client;
 
void processusaccueil();
void clientarrive();
void processusclient();
int traitanterreur();
int traitanterreurmsg(){}
 
/*
** Attention => prototype de main => int main(int argc, char **argv);
** et non int main(int argc, char **argv[]);
** mais ici on peut mettre void vu que les parametres ne sont pas utilises
*/
int main(void)
{
  int pipe_accueil_mecano[2];
  int pipe_mecano_client[2];
 
  if ((msgid = msgget(CLE_IPCMSG,IPC_CREAT | IPC_EXCL)) == -1)
    traitanterreurmsg(msgid);
  /*if (semget(CLE_SEM_ATELIER,IPC_CREAT|IPC_EXCL) == -1)
        traitanterreur(1);
    if (pipe(pipe_accueil_mecano) == -1)
        traitanterreur(2);
    if (pipe(pipe_mecano_client) == -1)
    traitanterreur(3);*/
 
  printf("Bienvenue au garage !!!\n");
  switch(fork())
    {
    case -1 :
      traitanterreur(4);
      break;
    case  0 :
      processusaccueil();
      break;
    }
  printf ("fils cree.\n");
  switch(fork())
    {
    case -1 :
      traitanterreur(6);
    case 0 :
      processusclient();
    }
  wait(NULL);
  wait(NULL);
  /*
  ** Avec un return c'est mieux.
  */
  return (EXIT_SUCCESS);
}
 
void processusaccueil()
{
  struct sigaction act_accueil;
  sigset_t sig_accueil;
 
  act_accueil.sa_handler = clientarrive;
  sigemptyset(&act_accueil.sa_mask);
  act_accueil.sa_flags = SA_RESTART;
  if (sigaction(SIGUSR1, &act_accueil, NULL) == -1)
    traitanterreur(5);
  sigemptyset(&sig_accueil);
  pidaccueil = getpid();
  while (1)
    {
      sigsuspend(&sig_accueil);
    }
  /*
  ** Toujours mettre un return
  */
  return ;
}
 
void clientarrive()
{
  int erreur;
  int taille_message = sizeof(client) - sizeof(long);
  if (erreur = msgrcv(msgid,&client,taille_message,0,IPC_NOWAIT) == -1)
    traitanterreurmsg(erreur);
  printf("%d\n", client.pid);
  return ;
}
 
void processusclient()
{
  pid_t pid;
 
  pid = getpid();
 
  kill(pidaccueil,SIGUSR1);
  return ;
}
 
int traitanterreur(int erreur)
{
  switch(erreur){
  case 1 :
    printf("Erreur lors de la creation de la semaphore\n");
    break;
  case 2 :
    printf("Erreur lors de la creation du pipe accueil -> mecano\n");
    break;
  case 3 :
    printf("Erreur lors de la creation du pipe mecano -> client\n");
    break;
  case 4 :
    printf("Erreur de la creation du processus accueil\n");
    break;
  case 5 :
    printf("Erreur lors de la def. du signal accueil\n");
    break;
  }
  exit(erreur);
  return (0);
}

Alors...

Avec les quelques petites modification au dessus, ca compile sans option.

Si je rajoute quelques avertissement a la compilation j'ai ca:

/usr/include/sys/ipc.h:25:3: warning: #warning "Files using this header must be compiled with _SVID_SOURCE or _XOPEN_SOURCE"
pere.c: In function ‘traitanterreurmsg’:
pere.c:37: warning: control reaches end of non-void function
pere.c: In function ‘main’:
pere.c:47: warning: unused variable ‘pipe_mecano_client’
pere.c:46: warning: unused variable ‘pipe_accueil_mecano’
pere.c: In function ‘processusaccueil’:
pere.c:86: error: storage size of ‘act_accueil’ isn’t known
pere.c:90: warning: implicit declaration of function ‘sigemptyset’
pere.c:91: error: ‘SA_RESTART’ undeclared (first use in this function)
pere.c:91: error: (Each undeclared identifier is reported only once
pere.c:91: error: for each function it appears in.)
pere.c:92: warning: implicit declaration of function ‘sigaction’
pere.c:98: warning: implicit declaration of function ‘sigsuspend’
pere.c:86: warning: unused variable ‘act_accueil’
pere.c: In function ‘clientarrive’:
pere.c:110: warning: suggest parentheses around assignment used as truth value
pere.c: In function ‘processusclient’:
pere.c:121: warning: implicit declaration of function ‘kill’

Ce qui n'est pas cool.

Pour le test en lui meme, avec le code ci dessus j'ai ca en sortie:

Bienvenue au garage !!!
fils cree.
User defined signal 1

Donc apparement le fils est bien cree.

**je®ome** · 13/05/2006, 22h06

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
 
jerome@linux:~/tmptest> gcc -Wall treuze.c -o treuze
treuze.c: In function ‘traitanterreurmsg’:
treuze.c:29: warning: control reaches end of non-void function
treuze.c: At top level:
treuze.c:31: warning: second argument of ‘main’ should be ‘char **’
treuze.c: In function ‘main’:
treuze.c:39: warning: suggest parentheses around assignment used as truth value
treuze.c:51: warning: implicit declaration of function ‘fork’
treuze.c:35: warning: unused variable ‘pipe_mecano_client’
treuze.c:34: warning: unused variable ‘pipe_accueil_mecano’
treuze.c:70: warning: control reaches end of non-void function
treuze.c: In function ‘processusaccueil’:
treuze.c:86: warning: implicit declaration of function ‘getpid’
treuze.c: In function ‘clientarrive’:
treuze.c:96: warning: suggest parentheses around assignment used as truth value
treuze.c:130:11: warning: no newline at end of file

Beaucoup d'erreurs sont à enlever
il manque des entêtes, unistd.h , sys/types.h

if (msgid = msgget(CLE_IPCMSG,IPC_CREAT | IPC_EXCL) == -1)

pas bien

if ((msgid = msgget(CLE_IPCMSG,IPC_CREAT | IPC_EXCL)) == -1)

plusieurs fois dans le code.
tester retour de wait et traîter dans le cas ou il est différent de -1

il ya encore d'autres choses à revoir dans le code

en espérant de ne pas avoir raconté trop de salades,
je te souhaite une bonne soirée

**Treuze** · 13/05/2006, 22h27

Merci , mais le programme n est pas termine , je voulais voir si ca marchait jusque la et le fait est que mon programme se compile mais le processus pere ne cree pas le fils !

**Treuze** · 14/05/2006, 15h03

Desoel pour le post precedent je n avais pas vu ta reponse Jack_serious

Mais je ne comprends pas ! quelle est la modification que tu as fait pour arriver a creer le fils ?

En tt cas merci , ca marche maintenant

**Treuze** · 14/05/2006, 17h07

Desoel pour le post precedent je n avais pas vu ta reponse Jack_serious

Mais je ne comprends pas ! quelle est la modification que tu as fait pour arriver a creer le fils ?

Pourquoi mettre des return alors que le type de retour est void ?

En tt cas merci , ca marche maintenant

**Jack_serious** · 14/05/2006, 17h16

Envoyé par Treuze

Desoel pour le post precedent je n avais pas vu ta reponse Jack_serious

Mais je ne comprends pas ! quelle est la modification que tu as fait pour arriver a creer le fils ?

Pourquoi mettre des return alors que le type de retour est void ?

Pour indiquer explicitement la sortie de la fonction.

En pratique je ne sais pas si ca pose un probleme, mais un compilateur bien regle hurle systematiquement quand on oublie de mettre un return a la fin d'une fonction.

Ca fait partie des regles de codage: toute fonction se termine par un return.

Si tu le ne met pas, ton compilateur le rajouteras de toute facon.

Le return indique a la machine de restaurer la pile a la sortie d'une fonction, pour revenir a la fonction precedente dans l etat ou elle etait quand on l a quittee (liberation des variables locales, etc...).

Donc il vaut mieux le mettre de facon explicite. C'est plus propre.

**Treuze** · 14/05/2006, 18h24

Merci pour tes explications

J ai un nouveau probleme (enfin c est surement tjrs le meme...)

Lors de la creation de mes deux processus fils , seul le 2 eme qui appelle le processusclient() fonctionne

En effet, le printf("processus accueil lance") ne marche pas :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
 
 
#include <sys/types.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <signal.h>
#include <errno.h>
#include <sys/wait.h>
#include <unistd.h>
#include <signal.h>
#include <sys/types.h>
#include <sys/select.h>
 
 
#define CLE_IPCMSG 123
#define CLE_SEM_ATELIER 456
 
#define LECTURE 0
#define ECRITURE 1
 
struct message{
  long type;
  int pid;
  char message[20];
};
 
pid_t pidaccueil;
 
int msgid;
struct message client;
int pipe_accueil_mecano[2];
int pipe_mecano_client[2];
int pipe_client_accueil[2];
 
void processusaccueil();
void clientarrive();
void processusclient();
int traitanterreur();
int traitanterreurmsg(){}
 
int main()
{
 
  if ((msgid = msgget(CLE_IPCMSG,IPC_CREAT | IPC_EXCL)) == -1)
        traitanterreurmsg(msgid);
  if (pipe(pipe_accueil_mecano) == -1)
        traitanterreur(2);
  if (pipe(pipe_mecano_client) == -1)
        traitanterreur(3);
  if (pipe(pipe_client_accueil) == -1)
    traitanterreur(6);
 
  printf("Bienvenue au garage !!!\n");
  switch(fork())
    {
    case -1 :
      traitanterreur(4);
      break;
 
    case  0 :
      processusaccueil();
      break;
    }
 
  switch(fork())
    {
    case -1 :
      traitanterreur(6);
      break;
 
    case 0 :
      processusclient();
      break;
    }
 
  return (EXIT_SUCCESS);
}
 
void processusaccueil()
{
  printf("processus accueil lance");
  pidaccueil = getpid();
 
  clientarrive();
 
  return ;
}
 
void clientarrive()
{
  int erreur;
  int taille_message = sizeof(client) - sizeof(long);
  int pidclient;
 
    sleep (5);    
    printf("je lis");
    close(pipe_client_accueil[ECRITURE]);
 
    read(pipe_client_accueil[LECTURE],&pidclient,sizeof(int));
    close(pipe_client_accueil[LECTURE]);
    printf("j ai fini de lire");
 
      if (erreur = msgrcv(msgid,&client,taille_message,0,IPC_NOWAIT) == -1)
            traitanterreurmsg(erreur);
 
      if (client.pid == 0){
        close(pipe_accueil_mecano[LECTURE]);
        write(pipe_accueil_mecano[ECRITURE],&pidclient,sizeof(int));
        close(pipe_accueil_mecano[ECRITURE]); 
        printf("%d",pidclient);
          }
 
      else {
        msgsnd(msgid,&pidclient,sizeof(int),IPC_NOWAIT);
        }
 
  return ;
}
 
void processusclient()
{
  int pid;
 
  pid = getpid();
  printf("j ecris");
  close (pipe_client_accueil[LECTURE]);
  write (pipe_client_accueil[ECRITURE],&pid,sizeof(int));
  close (pipe_client_accueil[ECRITURE]);
  printf("j ai fini d ecrire");
 
  return ;
}
 
int traitanterreur(int erreur)
{
  switch(erreur){
  case 1 :
    printf("Erreur lors de la creation de la semaphore\n");
    break;
  case 2 :
    printf("Erreur lors de la creation du pipe accueil -> mecano\n");
    break;
  case 3 :
    printf("Erreur lors de la creation du pipe mecano -> client\n");
    break;
  case 4 :
    printf("Erreur de la creation du processus accueil\n");
    break;
  case 5 :
    printf("Erreur lors de la def. du signal accueil\n");
    break;
  }
  exit(erreur);
  return (0);
}

**Jack_serious** · 14/05/2006, 18h46

Hihihi ca c'est un des problemes les plus recurents et qui peuvent te faire chercher pendant tres tres longtemps...

Si si il s'ecrit. Mais il reste coince dans le buffer c'est tout.

Essaie de changer cette ligne :

Code :

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

1
2
 
printf("processus accueil lance");

Par ca:

Code :

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

1
2
 
printf("processus accueil lance.\n");

Tu verras ca marche nettement mieux.

Explication:
Tout ce que tu ecris avec printf n'est pas affiche directement a l'ecran.
Il faut savoir que les appels systemes (lecture sur le clavier, ecriture sur l'ecran, etc...) sont les instructions qui prennent le plus de temps.
En consequence, pour rendre le programme plus rapide, l'ecriture sur l'ecran est "bufferisee". C'est a dire que le texte reste dans un buffer, jusqu'a ce que le buffer soit videe, et l'ecriture effectivement faite.
Trois choses declenchent le vidage du buffer d'ecriture:

Buffer plein (non controlable)
Envoi d'un retour a la ligne ('\n') dans le buffer.
Vidage manuel du buffer => fflush(stdout);

En l'occurence le texte etait bien ecrit, mais restait dans le buffer.

Deux choses encore:
Le compilateur me donne nettement moins de warnings qu'avant:

Envoyé par gcc

In file included from pere.c:5:
/usr/include/sys/ipc.h:25:3: warning: #warning "Files using this header must be compiled with _SVID_SOURCE or _XOPEN_SOURCE"
pere.c: In function ‘traitanterreurmsg’:
pere.c:40: warning: control reaches end of non-void function
pere.c: In function ‘clientarrive’:
pere.c:95: warning: suggest parentheses around assignment used as truth value

Le premier je n'ai pas reussi a le virer. Desole.

Pour les deux autres:
Remplacer:

Code :

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

1
2
 
int traitanterreurmsg(){}

par

Code :

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

1
2
 
int traitanterreurmsg(){return (0);}

Et

Code :

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

1
2
 
  if (erreur = msgrcv(msgid,&client,taille_message,0,IPC_NOWAIT) == -1)

Par

Code :

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

1
2
 
  if ((erreur = msgrcv(msgid,&client,taille_message,0,IPC_NOWAIT)) == -1)

La condition etait bonne, mais le compilateur previent juste que ca prete a confusion.

**Treuze** · 14/05/2006, 19h03

Alors la je tombe sur le c** , depuis hier je cherche ou peut etre l erreur et ca vient juste d un \n !!

Merci bcp pour tes explications et tt le temps que ca m'evitera de perdre

Pere sterile

C

Discussions similaires

Partager

Partager