Domanda SystemCall C

[N0body]

Buona sera,
Ho delle difficoltà riguardo questo programma.
Non riesco a capire perché quando eseguo mi stampa prima il terzo figlio e poi il resto nel giusto modo,e soprattutto perché alla stampa dei pid figli mi escono altri numeri.
Grazio tutte le persone per l'interessamento e il tempo dedicatomi.
Ps. Scusate per la mancata indentation.

Spoiler

1. [HASHTAG]#include[/HASHTAG] <stdlib.h>
2. [HASHTAG]#include[/HASHTAG] <stdio.h>
3. [HASHTAG]#include[/HASHTAG] <unistd.h>
4. [HASHTAG]#include[/HASHTAG] <sys/wait.h>
5. 
   
6. int main(){
7. int pid_1,pid_2,pid_3;
8. pid_1=fork();
9. int p1=0,p2=0,p3=0;
10. if(pid_1 < 0){
11. printf("Creazione fallita\n");
12. exit(-1);
13. }else if(pid_1 == 0){
14. printf("Pid primo figlio %d\n",getpid());
15. printf("La differenza e' %d\n",getpid()-getppid());
16. p1=getpid();
17. exit(0);
18. }
19. else if(pid_1>0){
20. wait(NULL);
21. pid_2=fork();
22. if(pid_2 <0){
23. printf("errore\n");
24. exit(0);
25. }else if(pid_2 == 0){
26. p2=getpid();
27. execlp("/bin/echo","echo","\nsono il secondo figlio",NULL);
28. exit(0);
29. }else if(pid_2>0){
30. wait(NULL);
31. pid_3 = fork();
32. if(pid_3<0){
33. printf("Errore\n");
34. exit(0);
35. }else if (pid_3 == 0){
36. printf("Pid terzo figlio %d\n",getpid());
37. execlp("/bin/ls","ls",NULL);
38. p3=getpid();
39. exit(0);
40. }else{
41. wait(NULL);
42. printf("Pid padre %d\n",getppid());
43. printf("I figli sono %d %d %d\n",pid_1,pid_2,pid_3);
44. exit(0);
45. }
46. }
47. }
48. return 0;
49. }

Aggiornamento: problema risolto e funzionante.

 

[St3ve]

Questa istruzione è sbagliata:

printf("I figli sono %d %d %d\n",p1,p2,p3);

Perché gli assegnamenti che hai fatto alle variabili p1, p2 e p3 erano su aree di memoria differenti, ogni nuovo processo ha il proprio spazio di indirizzamento. I valori che vedi a schermo variano perché le variabili non sono inizializzate, ma se le inizializzi subito a 0, vedrai che non cambiano.

L'ordine delle print a me sempra ok. Se a te sembra sbagliato prova a dirci cosa ti aspettavi di vedere (e perché ti aspettavi di vederlo così) e cosa ottieni.

 

[N0body]

Ti ringrazio per la risposta veloce,e la gentilezza.
Allego uno screen,mi aspetto il pid dei figli e l'ordine come l'ho scritto nel programma ovvero primo figlio,secondo figlio e terzo figlio. Purtroppo sono ancora inesperto su questo tipo di argomento e sto cercando di approfondire,grazie per la pazienza.
http://imgur.com/a/HR85R

 

[St3ve]

Ok, si... hai ragione. Togli tutti i wait(NULL) che hai e poi mettine uno sopra ogni fork(), l'unica fork che non deve avere sopra il wait è la prima visto che ovviamente non deve aspettare nessuno.

 

[N0body]

http://imgur.com/a/Bt3WO
Rimane in attesa e ripete una ls,come puoi vedere dallo screen. E sto provando ad inserire anche un echo sul secondo figlio,non capisco perchè rimane in attesa nonostante le execlp fanno terminare i figli e c'è un exit(0) sul padre alla fine.

 

[St3ve]

Serve un wait anche dentro l'else, prima che il padre stampi:

1. padre crea figlio 1
   
2. figlio 1 stampa ed esce
   
3. padre aspetta e crea figlio 2
4. figlio 2 stampa ed esce
   
5. padre aspetta e crea figlio 3
6. figlio 3 stampa ed esce
   
7. padre aspetta, stampa ed esce

Senza quest'ultimo wait il padre non aspetta al punto 7, quindi non puoi predire chi stampa per prima. Le wait prima delle fork servono per dire "aspetta che muore un figlio prima di creare il successivo" (almeno i due non si pestano i piedi su chi deve stampare prima), mentre l'ultimo wait serve per dire "aspetta che muore anche l'ultimo figlio prima di eseguire la print del padre".

Di ls ce n'è solo una 2, fatta dal terzo figlio. Non capisco cosa intendi con "ripete una ls", ma ora dovrebbe essere a posto.

 

[N0body]

Ti ringrazio infinitamente per la pazienza ed il tempo dedicatomi. Ora modifico il codice nel post di creazione e inserisco il sorgente funzionante.