Проблемы с запуском 2 дочерних процессов

Question

В этой программе mulpro c. c я пытаюсь запустить исполняемые файлы из двух программ, которые я создал (одна подсчитывает количество буквенных c символов в файле, а другая - пять определенных c специальные символы). Я пытаюсь создать родительский процесс (в данном случае просто mulpro c. c), который запускает эти две программы, каждая в своем собственном дочернем процессе, поэтому просто создайте два дочерних процесса из родительского процесса. Каждая из двух программ имеет свой собственный вывод, но вокруг соответствующих выводов я также хочу вывести по два сообщения для каждой, с указанием, когда она начинается и когда заканчивается. Однако при каждой попытке я получаю неверные и разные результаты (я не хочу публиковать их все здесь). Выходы двух моих программ даже записываются между собой, поэтому я полагаю, что могу использовать функцию waitpid () неправильно. Более того, у меня возникают проблемы с распечаткой правильных PID дочерних процессов в родительском процессе. Пожалуйста, игнорируйте две метки с номерами «Дочерний процесс» в целях отладки, поскольку они неуместны. Вот mulpro c. c ...

#include <stdio.h>
#include "count.h"
#include <string.h>
#include <stdlib.h>
#include <ctype.h>
#include <unistd.h> 
#include <stdbool.h>
#include <dirent.h>
#include <errno.h>
#include <sys/types.h>
#include <sys/wait.h>

int main( int argc, char *argv[] )
{       
    pid_t pid1;
    pid_t pid2;
    int status1;
    int status2;

    pid1 = fork();

    if ( pid1 < 0 )
    {
        perror( "ERROR! Fork failed!" );
        exit(1);
    }

    if ( pid1 == 0 )
    {
        // CHILD PROCESS CODE

        printf( "\nChild Process 2:\npid :%d\nppid:%d\n\n", getpid(), getppid() );

        printf( "CHILD <PID: %d> process is executing testspecial program!\n", getpid() );

        char *specialchar[] = { "./testspecial" , NULL };
        execv( specialchar[0], specialchar );   
    }

    if ( pid1 > 0 )
    {

        pid2 = fork();

        if ( pid2 == 0 )
        {
            // ANOTHER CHILD PROCESS CODE

            printf( "\nChild Process 1:\npid :%d\nppid:%d\n\n", getpid(), getppid() );

            printf( "CHILD <PID: %d> process is executing testalphabet program!\n", getpid() );

            char *alphabetchar[] = { "./testalphabet" , NULL };
            execv( alphabetchar[0], alphabetchar );
        }

        else if ( pid2 > 0 )
        {
            // PARENT PROCESS CODE

            printf( "\nParent Process:\npid:%d\nppid :%d\n", getpid(), getppid() );

            // if child1 terminated...

            if ( waitpid( pid1, &status1, 0 ) == pid1 )
            {
                printf( "CHILD <PID: %d> process has done with testalphabet program! See the results above!\n", getpid() );
            }

            // if child2 terminated...

            if ( waitpid( pid2, &status2, 0 ) == pid2 )
            {
                printf( "CHILD <PID: %d> process has done with testspecial program! See the results above!\n", getpid() );
            }
        }

    }


    return 0;
    
}

и вот только ОДИН из неправильных выходов ...

Parent Process:
pid:3166
ppid :3149

Child Process 1:
pid :3168
ppid:3166

CHILD <PID: 3168> process is executing testalphabet program!

Child Process 2:
pid :3167
ppid:3166

CHILD <PID: 3167> process is executing testspecial program!
A -> 0
B -> 0
C -> 0
D -> 0
E -> 0
F -> 0
G -> 0
H -> 3
I -> 0
J -> 0
K -> 0
L -> 0
M -> 0
N -> 0
O -> 0
P -> 0
Q -> 0
, -> 1
R -> 0
S -> 0
T -> 0
. -> 1
U -> 0
: -> 1
V -> 0
; -> 1
W -> 0
! -> 1
X -> 0
Y -> 0
Z -> 3
a -> 0
b -> 0
c -> 0
d -> 0
e -> 0
f -> 0
g -> 0
h -> 3
i -> 0
j -> 0
k -> 0
CHILD <PID: 3166> process has done with testalphabet program! See the results above!
l -> 0
m -> 0
n -> 0
o -> 0
p -> 0
q -> 0
r -> 0
s -> 0
t -> 0
u -> 0
v -> 0
w -> 0
x -> 0
y -> 0
z -> 0
CHILD <PID: 3166> process has done with testspecial program! See the results above!

Я ХОЧУ, чтобы на выходе что-то выглядело вот так ...

CHILD <PID: 3168> process is executing testalphabet program!
A -> 0
B -> 0
C -> 0
...
...
...
x -> 0
y -> 0
z -> 0
CHILD <PID: 3168> process has done with testalphabet program! See the results above!
CHILD <PID: 3167> process is executing testspecial program!
, -> 1
. -> 1
: -> 1
; -> 1
! -> 1
CHILD <PID: 3167> process has done with testspecial program! See the results above!

Очень признателен, если бы кто-нибудь мог исправить мои ошибки здесь ... Я новичок в использовании подобных системных вызовов, так что, надеюсь, я не слишком много трачу в этот экземпляр.

Answer 1

Если вы не хотите, чтобы процессы выполнялись параллельно, вам нужно дождаться завершения первого из них sh, прежде чем запускать второй. Так написано в псевдокоде, а не так:

pid1 = launch_proc1();
pid2 = launch_proc2();

waitpid(pid1);
waitpid(pid2);

Вам нужно сделать:

pid1 = launch_proc1();
waitpid(pid1);

pid2 = launch_proc2();    
waitpid(pid2);

Как преобразовать псевдокод в правильный c остается в качестве упражнения читателю.

РЕДАКТИРОВАТЬ: Некоторые пояснения по launc_procX:

launc_procX должны реализовывать комбинацию fork / exec.

int launc_proc1()
{
    int pid = fork();

    if ( pid < 0 )
    {
        perror( "ERROR! Fork failed!" );
        exit(EXIT_FAILURE);
    }

    else if (pid == 0)
    {
        // CHILD PROCESS CODE

        printf( "\nChild Process 2:\npid :%d\nppid:%d\n\n", getpid(), getppid() );

        printf( "CHILD <PID: %d> process is executing testspecial program!\n", getpid() );

        char *specialchar[] = { "./testspecial" , NULL };
        execv( specialchar[0], specialchar );

        // Handle failure of exec (this is important !!!!!)
        perror( "ERROR! exec failed!" );
        exit(EXIT_FAILURE);
    }
    // because execv is supposed to never return,
    // only parent is able to reach this point

    return pid;
}

Естественно, вы можете объединить launch_proc1 и launch_proc2 в один int launch_proc(const char *cmd)

Answer 2

Если вы не хотите, чтобы выходные данные двух дочерних процессов смешивались, в данном случае , дождитесь завершения первого дочернего процесса, а затем запустите следующий. Если вы хотите, чтобы выходные данные двух параллельных процессов не смешивались, как правило, создайте специальный процесс с именем, скажем, print_process. Вместо того, чтобы печатать напрямую, процесс должен отправить сообщение print_process. И print_process должен писать на терминале.

Answer 3

@ HAL9000 прав,

но для краткости перефразирую:

после вашего первого форка нужно только поставить:

waitpid(YOUR_CHILD_PID, NULL, 0);

в родительском процессе ПЕРЕД повторной вилкой.

обычно процессы могут быть созданы в al oop. где после каждой вилки в родительском процессе (где pid> 0) вы ждете только что созданного дочернего процесса.

в противном случае возникает состояние гонки (гонка между вашими процессами), и результат это ваше неопределенное поведение.

вы получаете разные результаты каждый раз в зависимости от порядка, в котором ядро ​​планирует процессы.

Думаю, вы это знаете. основная проблема заключается в том, что вы создаете второй процесс до того, как дожидаетесь первого. поэтому порядок планирования процессов не определен.