Skip to content

Commit

Permalink
fuzz: run the main-loop in fork-server process
Browse files Browse the repository at this point in the history
Without this, the time since the last main-loop keeps increasing, as the
fuzzer runs. The forked children need to handle all the "past-due"
timers, slowing them down, over time. With this change, the
parent/fork-server process runs the main-loop, while waiting on the
child, ensuring that the timer events do not pile up, over time.

Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
Reviewed-by: Darren Kenny <darren.kenny@oracle.com>
Message-id: 20200512030133.29896-5-alxndr@bu.edu
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
  • Loading branch information
a1xndr authored and stefanhaRH committed Jun 5, 2020
1 parent 3b11322 commit dfd5ddb
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions tests/qtest/fuzz/i440fx_fuzz.c
Expand Up @@ -151,6 +151,7 @@ static void i440fx_fuzz_qos_fork(QTestState *s,
i440fx_fuzz_qos(s, Data, Size);
_Exit(0);
} else {
flush_events(s);
wait(NULL);
}
}
Expand Down
2 changes: 2 additions & 0 deletions tests/qtest/fuzz/virtio_net_fuzz.c
Expand Up @@ -122,6 +122,7 @@ static void virtio_net_fork_fuzz(QTestState *s,
flush_events(s);
_Exit(0);
} else {
flush_events(s);
wait(NULL);
}
}
Expand All @@ -134,6 +135,7 @@ static void virtio_net_fork_fuzz_check_used(QTestState *s,
flush_events(s);
_Exit(0);
} else {
flush_events(s);
wait(NULL);
}
}
Expand Down
2 changes: 2 additions & 0 deletions tests/qtest/fuzz/virtio_scsi_fuzz.c
Expand Up @@ -145,6 +145,7 @@ static void virtio_scsi_fork_fuzz(QTestState *s,
flush_events(s);
_Exit(0);
} else {
flush_events(s);
wait(NULL);
}
}
Expand All @@ -164,6 +165,7 @@ static void virtio_scsi_with_flag_fuzz(QTestState *s,
}
_Exit(0);
} else {
flush_events(s);
wait(NULL);
}
}
Expand Down

0 comments on commit dfd5ddb

Please sign in to comment.