Discussion :

[isildur37]

Je suis très peu expérimenté dans la programmation c, et pour la fac les profs nous demande d'implémenter le problème des fumeurs (synchronisation de processus via des sémaphores).
Je pense que le problème vient d'une mauvaise connaissance du langage, mais malgré plusieurs essais, je ne trouve pas mon erreur, si vous pouviez m'aider.

Voici donc le debut code

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 <sys/types.h>
#include <time.h>
#include <unistd.h>
#include <stdlib.h>
#include <semaphore.h>       
#include <assert.h>
#include <errno.h>
#include <signal.h>
 
 
sem_t elements [3];	// les elements necessaires au fumeur sont stockés sous forme de tableau de semaphores, remplacer le type int par un type de semaphores, pui effectuer des modifs dans up et dans down
 
void init_sem(){	//initialise notre tableau de semaphore (toutes les semaphores a 1)
	sem_init (&elements[0],1,0); 
	sem_init (&elements[1],1,0);
	sem_init (&elements[2],1,0);
};
 
 
void down(int a){
	sem_wait(&elements[a]);
};
 
void up(int a){
	sem_post(&elements[a]);
};

Et voici l'erreur qui m'est renvoyée quand je compile
(commande tapée "gcc fum2.c") et j'ai testé aussi "g++ fum2.cpp" (en renommant le .c en .cpp, la meme erreur est sortie

Code :

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

1
2
3
4
5
6
7
8
9
10
 
/tmp/ccpdibJc.o: In function `init_sem':
fum2.c:(.text+0x1e): undefined reference to `sem_init'
fum2.c:(.text+0x3a): undefined reference to `sem_init'
fum2.c:(.text+0x56): undefined reference to `sem_init'
/tmp/ccpdibJc.o: In function `down':
fum2.c:(.text+0x76): undefined reference to `sem_wait'
/tmp/ccpdibJc.o: In function `up':
fum2.c:(.text+0x91): undefined reference to `sem_post'
collect2: ld returned 1 exit status

[ALT]

À première vue, je dirais : La variable sem_init est-elle déclarée ?

[isildur37]

Voila ce que dit la documentation concernant la librairie semaphore.h

http://brickos.sourceforge.net/docs/...aphore_8h.html

j'en ai déduis qu'il suffisait d'appeler la fonction

Code :

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

1
2
3
4
5
6
7
8
9
 
int 	sem_init (sem_t *sem, int pshared, unsigned int value)
 	Initialize a semaphore.
 
int 	sem_wait (sem_t *sem)
 	Wait for semaphore (blocking). 
 
int 	sem_post (sem_t *sem)
 	Post a semaphore.

PS: tout le debut du code est copié-collé, donc non je n'ai pas declaré de variable sem_....

[ALT]

Alors essaye sans l'espace entre le nom de la fonction & la parenthèse ouvrante.
Boarf, ce n'est pas que j'y croie beaucoup, mais bon.

[isildur37]

Merci quand meme, j'ai testé, sa ne change rien, toujours les memes erreurs.

[matafan]

Il faut que tu te link avec la libpthread. Ajoutes -lpthread dans ta ligne de commande gcc :

Code :

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

gcc -lpthread fum2.c

[dapounet]

Bonjour,

Il faut que tu te link avec la libpthread. Ajoutes -lpthread dans ta ligne de commande gcc :

Code :

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

gcc -lpthread fum2.c

Il y vraiment des systèmes où les fonctions realtime sont dans la bibliothèque des pthreads ? Je pencherais plutôt pour -lrt, sinon vérifier dans la man page d'une des fonctions.

[matafan]

Oui effectivement -lrt c'est mieux, même si -lpthread résoud indirectement le problème. man sem_overview dit :

Code :

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

1
2
3
   Linking
       Programs  using  the POSIX semaphores API must be compiled with cc -lrt
       to link against the real-time library, librt.

[Sve@r]

Et voici l'erreur qui m'est renvoyée quand je compile
(commande tapée "gcc fum2.c") et j'ai testé aussi "g++ fum2.cpp" (en renommant le .c en .cpp, la meme erreur est sortie

Code :

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

1
2
3
4
5
6
7
8
9
10
 
/tmp/ccpdibJc.o: In function `init_sem':
fum2.c:(.text+0x1e): undefined reference to `sem_init'
fum2.c:(.text+0x3a): undefined reference to `sem_init'
fum2.c:(.text+0x56): undefined reference to `sem_init'
/tmp/ccpdibJc.o: In function `down':
fum2.c:(.text+0x76): undefined reference to `sem_wait'
/tmp/ccpdibJc.o: In function `up':
fum2.c:(.text+0x91): undefined reference to `sem_post'
collect2: ld returned 1 exit status

Parce que ta commande de compilation "gcc" demande la création de l'exécutable complet et qu'il manque plein de trucs pour faire de ce bout de code un exécutable.
Te manque la fonction main() et les librairies qui contiennent le code des fonctions sem_init et sem_wait.

Si tu veux juste vérifier la syntaxe de ton code, ou le compiler comme module qui sera inclus plus tard, faut que tu fasses "gcc -c fum2.c". Si ça marche, ça te génèrera un "fum2.o" que tu pourras inclure plus tard avec un autre source par une commande de compilation adéquate.

Ici les cours et tutoriels pour apprendre C : http://c.developpez.com/cours/

[isildur37]

Je vous remercie tous de vos réponses. Finalement la prof s'est aperçue que personne ne connaissait suffisamment le c pour faire ce code... Et ce que vous me dite la au niveau de la compilation, j'en avais jamais entendu parler, mais jpense que sa va me servir, je prend note, et encore merci.

@Sver, j'ai tout le code, mais j'ai juste mis les declarations puisque je pense que les erreur venaient de la. Il y a bien un main() dans le programme complet