From dedad4e3d2ed1d6ecdb89422fc0baa028021f19d Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Wed, 29 Jul 2015 11:09:45 +0100 Subject: Unbuffer all tests that rely on stdio This forces all tests that rely on stdio to be unbuffered, like interrupt.exp was adjusted in 6f98576f. To recap, in some scenarios, GDB or GDBserver can be spawned with input _not_ connected to a tty, and then tests that rely on stdio fail with timeouts, because the inferior's stdout and stderr streams end up fully buffered. Calling gdb_unbuffer_output forces output to be unbuffered. See https://sourceware.org/ml/gdb-patches/2015-02/msg00809.html and https://sourceware.org/ml/gdb-patches/2015-02/msg00819.html. Tested on x86_64 Fedora 20, native, and against a remote gdbserver board file that connects to the target with ssh, with and without -t (create pty). gdb/testsuite/ChangeLog: 2015-07-29 Pedro Alves * gdb.base/call-ar-st.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/call-rt-st.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/call-strs.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/call-strs.exp: Adjust to step over the gdb_unbuffer_output call. * gdb.base/catch-gdb-caused-signals.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/dprintf.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/ending-run.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/run.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/shlib-call.exp: Adjust to step over the gdb_unbuffer_output call. * gdb.base/shmain.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/sizeof.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. * gdb.base/varargs.c: Include "../lib/unbuffer_output.c". (main): Rename to ... (test): ... this. (main): Reimplement. * gdb.base/varargs.exp: Run to test instead of to main. * gdb.mi/mi-dprintf.c: Include "../lib/unbuffer_output.c". (main): Call gdb_unbuffer_output. --- gdb/testsuite/ChangeLog | 33 +++++++++++++++++++++++ gdb/testsuite/gdb.base/call-ar-st.c | 4 +++ gdb/testsuite/gdb.base/call-rt-st.c | 3 +++ gdb/testsuite/gdb.base/call-strs.c | 4 +++ gdb/testsuite/gdb.base/call-strs.exp | 3 +-- gdb/testsuite/gdb.base/catch-gdb-caused-signals.c | 4 +++ gdb/testsuite/gdb.base/dprintf.c | 4 +++ gdb/testsuite/gdb.base/ending-run.c | 4 +++ gdb/testsuite/gdb.base/run.c | 4 +++ gdb/testsuite/gdb.base/shlib-call.exp | 2 +- gdb/testsuite/gdb.base/shmain.c | 5 ++++ gdb/testsuite/gdb.base/sizeof.c | 4 +++ gdb/testsuite/gdb.base/sizeof.exp | 2 +- gdb/testsuite/gdb.base/varargs.c | 13 ++++++++- gdb/testsuite/gdb.base/varargs.exp | 2 +- gdb/testsuite/gdb.mi/mi-dprintf.c | 4 +++ 16 files changed, 89 insertions(+), 6 deletions(-) (limited to 'gdb') diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 9b4bf6b..072a108 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,5 +1,38 @@ 2015-07-29 Pedro Alves + * gdb.base/call-ar-st.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/call-rt-st.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/call-strs.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/call-strs.exp: Adjust to step over the + gdb_unbuffer_output call. + * gdb.base/catch-gdb-caused-signals.c: Include + "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/dprintf.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/ending-run.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/run.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/shlib-call.exp: Adjust to step over the + gdb_unbuffer_output call. + * gdb.base/shmain.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/sizeof.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + * gdb.base/varargs.c: Include "../lib/unbuffer_output.c". + (main): Rename to ... + (test): ... this. + (main): Reimplement. + * gdb.base/varargs.exp: Run to test instead of to main. + * gdb.mi/mi-dprintf.c: Include "../lib/unbuffer_output.c". + (main): Call gdb_unbuffer_output. + +2015-07-29 Pedro Alves + * gdb.mi/mi-dprintf.exp (mi_expect_dprintf): New procedure, factore out from mi_continue_dprintf. For call-style dprintfs, expect dprintf output out of $inferior_spawn_id. diff --git a/gdb/testsuite/gdb.base/call-ar-st.c b/gdb/testsuite/gdb.base/call-ar-st.c index 4b2ffb4..497af84 100644 --- a/gdb/testsuite/gdb.base/call-ar-st.c +++ b/gdb/testsuite/gdb.base/call-ar-st.c @@ -3,6 +3,8 @@ #include #include +#include "../lib/unbuffer_output.c" + /************************************************************************** * TESTS : * -- function arguments that are enumerated types @@ -912,6 +914,8 @@ int main () { static struct two_floats_t *f2; static struct two_floats_t *f3; + gdb_unbuffer_output (); + /* Initialize arrays */ for (index = 0; index < 120; index++) { diff --git a/gdb/testsuite/gdb.base/call-rt-st.c b/gdb/testsuite/gdb.base/call-rt-st.c index 944f1af..072ea86 100644 --- a/gdb/testsuite/gdb.base/call-rt-st.c +++ b/gdb/testsuite/gdb.base/call-rt-st.c @@ -2,6 +2,8 @@ #include #include +#include "../lib/unbuffer_output.c" + /************************************************************************** * TESTS : * function returning large structures, which go on the stack @@ -562,6 +564,7 @@ int main () { struct one_double_t *d1; struct two_floats_t *f3; + gdb_unbuffer_output (); /* Allocate space for large structures */ diff --git a/gdb/testsuite/gdb.base/call-strs.c b/gdb/testsuite/gdb.base/call-strs.c index c0b1823..03a8064 100644 --- a/gdb/testsuite/gdb.base/call-strs.c +++ b/gdb/testsuite/gdb.base/call-strs.c @@ -2,6 +2,8 @@ #include #include +#include "../lib/unbuffer_output.c" + char buf[100]; char bigbuf[1000]; char * s; @@ -47,6 +49,8 @@ link_malloc () int main() { + gdb_unbuffer_output (); + s = &buf[0]; strcpy(buf, "test string"); str_func("abcd", "efgh", "ijkl", "mnop", "qrst", "uvwx", "yz12"); diff --git a/gdb/testsuite/gdb.base/call-strs.exp b/gdb/testsuite/gdb.base/call-strs.exp index e11296f..96641e5 100644 --- a/gdb/testsuite/gdb.base/call-strs.exp +++ b/gdb/testsuite/gdb.base/call-strs.exp @@ -57,8 +57,7 @@ if ![runto_main] then { set prev_timeout $timeout set timeout 120 -#step -gdb_test "step" \ +gdb_test "next 2" \ "strcpy\\(buf, \"test string\"\\);" \ "step after assignment to s" diff --git a/gdb/testsuite/gdb.base/catch-gdb-caused-signals.c b/gdb/testsuite/gdb.base/catch-gdb-caused-signals.c index 769858c..a2cdcb0 100644 --- a/gdb/testsuite/gdb.base/catch-gdb-caused-signals.c +++ b/gdb/testsuite/gdb.base/catch-gdb-caused-signals.c @@ -20,11 +20,15 @@ #include #include +#include "../lib/unbuffer_output.c" + int main (void) { int i = 0; + gdb_unbuffer_output (); + i++; /* set dprintf here */ return 0; /* set breakpoint here */ } diff --git a/gdb/testsuite/gdb.base/dprintf.c b/gdb/testsuite/gdb.base/dprintf.c index 3fd3b3a..c0c3ce3 100644 --- a/gdb/testsuite/gdb.base/dprintf.c +++ b/gdb/testsuite/gdb.base/dprintf.c @@ -17,6 +17,8 @@ #include +#include "../lib/unbuffer_output.c" + static int g; void @@ -32,6 +34,8 @@ main (int argc, char *argv[]) { int loc = 1234; + gdb_unbuffer_output (); + /* Ensure these functions are available. */ printf ("kickoff %d\n", loc); fprintf (stderr, "also to stderr %d\n", loc); diff --git a/gdb/testsuite/gdb.base/ending-run.c b/gdb/testsuite/gdb.base/ending-run.c index 42f12c0..1e09f16 100644 --- a/gdb/testsuite/gdb.base/ending-run.c +++ b/gdb/testsuite/gdb.base/ending-run.c @@ -4,6 +4,8 @@ #include #include +#include "../lib/unbuffer_output.c" + int callee (int x) { int y = x * x; /* -break1- */ @@ -16,6 +18,8 @@ int main() int *p; int i; + gdb_unbuffer_output (); + p = (int *) malloc( 4 ); for (i = 1; i < 10; i++) diff --git a/gdb/testsuite/gdb.base/run.c b/gdb/testsuite/gdb.base/run.c index 0c62002..614b018 100644 --- a/gdb/testsuite/gdb.base/run.c +++ b/gdb/testsuite/gdb.base/run.c @@ -6,11 +6,15 @@ #include #include +#include "../lib/unbuffer_output.c" + int factorial (int); int main (int argc, char **argv, char **envp) { + gdb_unbuffer_output (); + #ifdef FAKEARGV printf ("%d\n", factorial (1)); /* commands.exp: hw local_var out of scope */ #else diff --git a/gdb/testsuite/gdb.base/shlib-call.exp b/gdb/testsuite/gdb.base/shlib-call.exp index 2ccaa39..f48b6fb 100644 --- a/gdb/testsuite/gdb.base/shlib-call.exp +++ b/gdb/testsuite/gdb.base/shlib-call.exp @@ -71,7 +71,7 @@ if ![runto_main] then { #step -over -gdb_test "next" "g = shr1\\(g\\);" "next to shr1" +gdb_test "next 2" "g = shr1\\(g\\);" "next to shr1" #print g diff --git a/gdb/testsuite/gdb.base/shmain.c b/gdb/testsuite/gdb.base/shmain.c index e36f84f..932b834 100644 --- a/gdb/testsuite/gdb.base/shmain.c +++ b/gdb/testsuite/gdb.base/shmain.c @@ -3,6 +3,8 @@ #include "ss.h" #include +#include "../lib/unbuffer_output.c" + extern int structarg(struct s); extern int pstructarg(struct s*); extern int shr1(int); @@ -31,6 +33,9 @@ int mainshr1(int g) int main() { struct s y; + + gdb_unbuffer_output (); + g = 1; g = shr1(g); g = shr2(g); diff --git a/gdb/testsuite/gdb.base/sizeof.c b/gdb/testsuite/gdb.base/sizeof.c index 9c67c03..eb21f22 100644 --- a/gdb/testsuite/gdb.base/sizeof.c +++ b/gdb/testsuite/gdb.base/sizeof.c @@ -1,5 +1,7 @@ #include +#include "../lib/unbuffer_output.c" + typedef char padding[16]; struct { @@ -98,6 +100,8 @@ fill_structs (void) int main () { + gdb_unbuffer_output (); + fill_structs (); printf ("sizeof (char) == %d\n", (int) sizeof (char)); diff --git a/gdb/testsuite/gdb.base/sizeof.exp b/gdb/testsuite/gdb.base/sizeof.exp index a237ee3..ce6b7ed 100644 --- a/gdb/testsuite/gdb.base/sizeof.exp +++ b/gdb/testsuite/gdb.base/sizeof.exp @@ -48,7 +48,7 @@ if ![runto_main] then { # Query GDB for the size of various types # -gdb_test "next" +gdb_test "next 2" set sizeof_char [get_sizeof "char" 1] set sizeof_short [get_sizeof "short" 2] diff --git a/gdb/testsuite/gdb.base/varargs.c b/gdb/testsuite/gdb.base/varargs.c index 1ad2ffc..4c0f165 100644 --- a/gdb/testsuite/gdb.base/varargs.c +++ b/gdb/testsuite/gdb.base/varargs.c @@ -8,6 +8,8 @@ #include #include +#include "../lib/unbuffer_output.c" + int find_max1(int, ...); int find_max2(int, int, ...); double find_max_double(int, double, ...); @@ -45,7 +47,9 @@ long double _Complex ldc4 = 4.0L + 4.0Li; #endif -int main() { +int +test (void) +{ c = -1; uc = 1; s = -2; @@ -70,6 +74,13 @@ int main() { return 0; } +int +main (void) +{ + gdb_unbuffer_output (); + test (); +} + /* Integer varargs, 1 declared arg */ int find_max1(int num_vals, ...) { diff --git a/gdb/testsuite/gdb.base/varargs.exp b/gdb/testsuite/gdb.base/varargs.exp index 96933bb..fe54314 100644 --- a/gdb/testsuite/gdb.base/varargs.exp +++ b/gdb/testsuite/gdb.base/varargs.exp @@ -66,7 +66,7 @@ if [gdb_skip_stdio_test "varargs.exp"] { return } -if ![runto_main] then { +if ![runto test] then { perror "couldn't run to breakpoint" continue } diff --git a/gdb/testsuite/gdb.mi/mi-dprintf.c b/gdb/testsuite/gdb.mi/mi-dprintf.c index 0b8fc82..2571e50 100644 --- a/gdb/testsuite/gdb.mi/mi-dprintf.c +++ b/gdb/testsuite/gdb.mi/mi-dprintf.c @@ -19,6 +19,8 @@ #include #include +#include "../lib/unbuffer_output.c" + static int g; void @@ -34,6 +36,8 @@ main (int argc, char *argv[]) { int loc = 1234; + gdb_unbuffer_output (); + /* Ensure these functions are available. */ printf ("kickoff %d\n", loc); fprintf (stderr, "also to stderr %d\n", loc); -- cgit v1.1