[C linux] Chiarimenti sulla wait()

[VegetaSSJ5]

Salve a tutti, ho bisogno di un chiarimento sulla funzione wait(). Da quello che ho capito un processo fa la fork e vuole aspettare la terminazione del figlio. Quando il figlio termina la propria esecuzione manda un SIGCHLD al padre. Se il padre fa una wait(NULL), riprende l'esecuzione qando il figlio ha terminato, giusto? Ho fatto un programmino che usa questa wait(), in patica il padre crea N figli e vuole riprendere a funzionare quando tutti i figli hanno terminato. Ho scritto questo pezzettino:

for (i=0; i<N; ++i) {
   pid_t pid=fork();
   if (pid==0) spesa();
}
while (j<N) {
   if ((i=(int)wait(NULL)) <=0) {
      perror("Errore nella wait");
      exit(-1);
   }
   printf("Il cliente %d ha terminato la sua esecuzione\n", i);
}
printf("\nTutti i clienti hanno terminato la propria esecuzione\n");

In fase d'esecuzione il programma termina perchè mi stampa il messaggio di errore della wait dicendo "Errore nella wait: No child processes".
Qualcuno mi sa aiutare??

[ilsensine]

La wait si trova sia nel path del padre che dei figli, mi sembra.

[VegetaSSJ5]

Originariamente inviato da ilsensine
La wait si trova sia nel path del padre che dei figli, mi sembra.

ilsensine chiaramente i codice non è quello , l'ho un po' semplificato io perchè in mezzo alle operazioni ci sono altre cose (semafori, memoria condivisa). il figlio esegue una exit(0) all'interno della funzione spesa().

[ilsensine]

Per caso tra le semplificazioni hai anche tolto l'inizializzazione e l'incremento di j?

[VegetaSSJ5]

Originariamente inviato da ilsensine
Per caso tra le semplificazioni hai anche tolto l'inizializzazione e l'incremento di j?

ilsensine ti riporto il codice di tutta la funzione

void genera_clienti (int N) {
	int i, j, numf, tipo, semaforo, bilancia, listaspesa[10];
	if ((semaforo= semget(5785, 10, IPC_CREAT | IPC_EXCL)) <0) {
		perror("Errore nella creazione del semaforo");
		exit(-1);
	}
	if ((bilancia= semget(3486, 1, IPC_CREAT | IPC_EXCL)) <0) {
		perror("Errore nella creazione del semaforo bilancia");
		exit(-1);
	}
	semun ctl_arg;
	ctl_arg.array=(unsigned short *) calloc (10, sizeof(unsigned short));
	if (semctl(semaforo, 0, SETALL, ctl_arg) <0) {
		perror("Errore nella chiamata a semctl -1-");
		exit(-1);
	}
	printf("\nSto generando i clienti...\n\n");
	ctl_arg.val=1;
	if (semctl(bilancia, 0, SETVAL, ctl_arg) <0) {
		perror("Errore nella chiamata a semctl per bilancia");
		exit(-1);
	}
	for (i=0; i<N; i++) {
		for (j=0; j<10; j++) listaspesa[j]=0;
		numf=rand()%10;
		if (numf<3) numf+=3;
		for (j=0; j<numf; j++) {
			tipo=rand()%10;
			while (listaspesa[tipo]!=0) (++tipo)%10;
			listaspesa[tipo]=rand()%10;
		}
		pid_t pid=fork();
		if (pid==0) {
			casse=(int *) shmat (fdcasse, 0, SHM_RND);
			printlistaspesa(listaspesa);
			spesa(listaspesa, semaforo, bilancia);
		}
		else sleep(1);
	}
	j=0;
	ctl_arg.array=(unsigned short *) calloc (10, sizeof(unsigned short));
	for (i=0; i<10; i++) ctl_arg.array[i]=1;
	if (semctl(semaforo, 0, SETALL, ctl_arg) <0) {
		perror("Errore nella chiamata a semctl -2-");
		exit(-1);
	}
	while (j<N) {
		if ((i=(int)wait(NULL)) <=0) {
			perror("Errore nella wait");
			exit(-1);
		}
		printf("Il cliente %d ha terminato la sua esecuzione\n", i);
		++j;
	}
	printf("\nTutti i clienti hanno terminato la propria esecuzione\n");
}

[ilsensine]

Sembra corretto; non vorrei che qualche figlio va in crash (anche se questo dovrebbe mandare in crash anche il padre, credo).

Puoi allegare il codice completo, così lo compilo & testo qui in locale?

[VegetaSSJ5]

ecco il programma. ti avverto comunque che il programma ancora non è terminato quindi mancano ancora delle cose importanti (come la deallocazione delle risorse dei semafori, infatti se il programma va in crash devi cambiare le key dei semafori e ricompilare o riavviare il sistema per far ripartire il programma). in pratica il fatto della wait mi serve perchè se non la facessi il padre terminerebbe la propria esecuzione e i figli sarebbero zombie, e questo non lo posso permettere perchè andrebbe contro le specifiche.

[VegetaSSJ5]

speriamo che il sensine non abbia scoperto che in realtà era un virus! a quest'ora dovrebbe già essergli scoppiato il pc!

[ilsensine]

Vedo errori un pò ovunque, tranne nel tuo wait. Mi puzza di buffer overflow o altre delizie, credo che faresti bene a controllare i vari passi.
Ad esempio qui

	if (semctl(semaforo, 0, SETALL, ctl_arg) <0) {
		perror("Errore nella chiamata a semctl -1-");
		exit(-1);
	}

Mi da un bel "permission denied" alla prima esecuzione, segno che c'è qualcosa che non va.
Sotto root non ho questo errore, il programma va avanti con errori nei punti più impensabili.

[ilsensine]

nb per evitare di riavviare ogni volta, credo che ti basti togliere IPC_EXCL nelle semget.

[VegetaSSJ5]

Originariamente inviato da ilsensine
nb per evitare di riavviare ogni volta, credo che ti basti togliere IPC_EXCL nelle semget.

azz! ti ringrazio!
per il resto hai qualche idea? anche a te ha dato quell'errore a runtime??

[ilsensine]

Argh!

if ((semaforo= semget(5785, 10, IPC_CREAT >>> | 0600 <<<)) <0) {
                perror("Errore nella creazione del semaforo");
                exit(-1);
        }
        if ((bilancia= semget(3486, 1, IPC_CREAT >>> | 0600 <<<)) <0) {
                perror("Errore nella creazione del semaforo bilancia");
                exit(-1);
        }

Nota che anche per il seme del semaforo puoi usare IPC_PRIVATE.

[VegetaSSJ5]

Originariamente inviato da ilsensine
Argh!

if ((semaforo= semget(5785, 10, IPC_CREAT >>> | 0600 <<<)) <0) {
                perror("Errore nella creazione del semaforo");
                exit(-1);
        }
        if ((bilancia= semget(3486, 1, IPC_CREAT >>> | 0600 <<<)) <0) {
                perror("Errore nella creazione del semaforo bilancia");
                exit(-1);
        }

Nota che anche per il seme del semaforo puoi usare IPC_PRIVATE.

intendi la key? quel | 0600 cosa provoca?

[ilsensine]

Originariamente inviato da VegetaSSJ5
intendi la key? quel | 0600 cosa provoca?

Che puoi USARE il semaforo, ad esempio?

[VegetaSSJ5]

Originariamente inviato da ilsensine
Che puoi USARE il semaforo, ad esempio?

dettagli...

[ilsensine]

Se crei il semaforo senza diritti di lettura/scrittura, come fai a modificarlo?
Lo fai già per la shmget, il principio è lo stesso.

Questa patch risolve il problema. Nota che con IPC_PRIVATE, non è necessario specificare IPC_CREAT:

--- main.c.org  2004-09-17 16:48:07.000000000 +0200
+++ main.c      2004-09-17 17:30:52.000000000 +0200
@@ -76,11 +76,11 @@

 void genera_clienti (int N) {
        int i, j, numf, tipo, semaforo, bilancia, listaspesa[10];
-       if ((semaforo= semget(5785, 10, IPC_CREAT | IPC_EXCL)) <0) {
+       if ((semaforo= semget(IPC_PRIVATE, 10, 0600)) <0) {
                perror("Errore nella creazione del semaforo");
                exit(-1);
        }
-       if ((bilancia= semget(3486, 1, IPC_CREAT | IPC_EXCL)) <0) {
+       if ((bilancia= semget(IPC_PRIVATE, 1, 0600)) <0) {
                perror("Errore nella creazione del semaforo bilancia");
                exit(-1);
        }

[VegetaSSJ5]

grazie ilsensine!

Originariamente inviato da VegetaSSJ5
per il resto hai qualche idea? anche a te ha dato quell'errore a runtime??

[ilsensine]

grazie ilsensine!

Con la patch non ti funziona? A me non capita quell'errore.

[VegetaSSJ5]

in pratica mi dà l'errore su quella wait quando termina un processo figlio...

[ilsensine]

Questo è il mio output con parametro "25"...