[binutils-gdb] new-ui command: gdb internal errors if input is already pending

[binutils-gdb] new-ui command: gdb internal errors if input is already pending

[sergiodj+buildbot]

*** TEST RESULTS FOR COMMIT 4295e285efa8193504ee08b9f633d9f8680bf181 ***

Author: Pedro Alves <palves@redhat.com>
Branch: master
Commit: 4295e285efa8193504ee08b9f633d9f8680bf181

new-ui command: gdb internal errors if input is already pending

I noticed that if input is already pending on the new-ui TTY, gdb
internal-errors.

E.g., create /dev/pts/2, and type anything there (even just <return>
is sufficient).

Now start GDB creating a new UI on that TTY, while at the same time,
running a synchronous execution command.  Something like:

$ gdb program -ex "new-ui console /dev/pts/2" -ex "start"

Back on /dev/pts/2, we get:

  (gdb) .../src/gdb/event-top.c:360: internal-error: double prompt
  A problem internal to GDB has been detected,
  further debugging may prove unreliable.

While the main UI was waiting for "start" to finish, gdb kepts pumping
events, including the input fd of the extra console.  The problem is
that stdin_event_handler doesn't restore the current UI back to what
it was, assuming that it's only ever called from the top level event
loop.  However, in this case, it's being called from the nested event
loop from within maybe_wait_sync_command_done.

When finally the "start" command is done, we reach the code that
prints the prompt in the main UI, just before starting the main event
loop.  Since now the current UI is pointing at the extra console (by
mistake), we find ourselves printing a double prompt on the extra
console.  This is caught by the assertion that fails, as shown above.

Since other event handlers also don't restore the UI (e.g., signal
event handlers), I think it's better if whatever is pumping events to
take care to restore the UI, if it cares.  That's what this patch
does.  New test included.

gdb/ChangeLog:
2016-09-06  Pedro Alves  <palves@redhat.com>

	* top.c (wait_sync_command_done): Don't assume current_ui doesn't
	change across events.  Restore the current UI before returning.
	(gdb_readline_wrapper): Restore the current UI before returning.

gdb/testsuite/ChangeLog:
2016-09-06  Pedro Alves  <palves@redhat.com>

	* gdb.base/new-ui-pending-input.c: New file.
	* gdb.base/new-ui-pending-input.exp: New file.
	* gdb.exp (clear_gdb_spawn_id): New procedure.
	(with_spawn_id): Check whether gdb_spawn_id exists before
	referencing it.  If gdb_spawn_id didn't exist on entry, clear it
	on exit.

Failures on Fedora-x86_64-native-gdbserver-m32, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-x86-64-4

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-gdbserver-m32/builds/4274>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/cgit/Fedora-x86_64-native-gdbserver-m32/.git/tree/?h=master&id=28b8ba1648efbc975667a6d9e8cb49d188b7bc8d>

*** Diff to previous build ***
============================
PASS -> FAIL: gdb.ada/win_fu_syms.exp: compilation foo.adb
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Fedora-x86_64-native-gdbserver-m32/xfails/master/xfail;hb=9fb6986>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Fedora-x86_64-native-gdbserver-m32/xfails/master/xfail.table;hb=9fb6986>

Failures on Fedora-x86_64-native-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-x86-64-2

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-gdbserver-m64/builds/4265>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-x86_64-native-extended-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-x86-64-4

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-extended-gdbserver-m64/builds/4295>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Ubuntu-AArch64-native-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	ubuntu-aarch64-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/445>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-s390x-m64, branch master

[sergiodj+buildbot]

Buildslave:
	marist-fedora-s390x

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-s390x-m64/builds/3764>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/cgit/Fedora-s390x-m64/.git/tree/?h=master&id=9ac769ab6ede49727f1b18646098f38767596a80>

*** Diff to previous build ***
============================
PASS -> FAIL: gdb.threads/watchpoint-fork.exp: child: multithreaded: breakpoint
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Fedora-s390x-m64/xfails/master/xfail;hb=5655ece>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Fedora-s390x-m64/xfails/master/xfail.table;hb=5655ece>

Failures on Debian-s390x-native-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	debian-jessie-s390x-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Debian-s390x-native-gdbserver-m64/builds/2898>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Debian-s390x-native-extended-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	debian-jessie-s390x-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Debian-s390x-native-extended-gdbserver-m64/builds/2905>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Debian-i686, branch master

[sergiodj+buildbot]

Buildslave:
	wildebeest-debian-wheezy-i686

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Debian-i686/builds/4087>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/cgit/Debian-i686/.git/tree/?h=master&id=e9d619524847d308a1901f0f971930f37dc98529>

*** Diff to previous build ***
============================
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: continue: setup: continue to breakpoint: run to breakpoint in thread 2
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: continue: setup: unbreak loop in thread 2
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: continue: continue
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Debian-i686/xfails/master/xfail;hb=b49a91b>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Debian-i686/xfails/master/xfail.table;hb=b49a91b>

Failures on Debian-i686-native-extended-gdbserver, branch master

[sergiodj+buildbot]

Buildslave:
	wildebeest-debian-wheezy-i686

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Debian-i686-native-extended-gdbserver/builds/4069>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/cgit/Debian-i686-native-extended-gdbserver/.git/tree/?h=master&id=d5639e2e95d7c48772f081e62c87125057f1907a>

*** Diff to previous build ***
============================
PASS -> FAIL: gdb.threads/forking-threads-plus-breakpoint.exp: cond_bp_target=0: detach_on_fork=on: displaced=off: inferior 1 exited
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: step: setup: continue to breakpoint: run to breakpoint in thread 2
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: step: setup: unbreak loop in thread 2
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: step: step
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: continue: setup: continue to breakpoint: run to breakpoint in thread 3
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: continue: setup: unbreak loop in thread 3
PASS -> FAIL: gdb.threads/multiple-step-overs.exp: displaced=on: continue: continue
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Debian-i686-native-extended-gdbserver/xfails/master/xfail;hb=b49a91b>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

	<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Debian-i686-native-extended-gdbserver/xfails/master/xfail.table;hb=b49a91b>

Failures on Debian-x86_64-native-extended-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	wildebeest-debian-wheezy-x86_64

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Debian-x86_64-native-extended-gdbserver-m64/builds/4118>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-ppc64be-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-ppc64be-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-ppc64be-m64/builds/4140>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-ppc64be-native-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-ppc64be-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-ppc64be-native-gdbserver-m64/builds/4133>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-ppc64be-native-extended-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-ppc64be-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-ppc64be-native-extended-gdbserver-m64/builds/4141>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-ppc64be-cc-with-index, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-ppc64be-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-ppc64be-cc-with-index/builds/4138>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

Failures on Fedora-ppc64le-native-extended-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-ppc64le-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-ppc64le-native-extended-gdbserver-m64/builds/4075>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

*** Internal error on buildslave (no space left on device). ***
*** Please report this to the buildslave owner (see <http://gdb-build.sergiodj.net//buildslaves/fedora-ppc64le-1>) ***

Failures on Fedora-ppc64le-native-gdbserver-m64, branch master

[sergiodj+buildbot]

Buildslave:
	fedora-ppc64le-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-ppc64le-native-gdbserver-m64/builds/4062>

Commit(s) tested:
	4295e285efa8193504ee08b9f633d9f8680bf181

Author(s) (in the same order as the commits):
	Pedro Alves <palves@redhat.com>

Subject:
	new-ui command: gdb internal errors if input is already pending

Testsuite log (gdb.sum and gdb.log) URL(s):
	<Error fetching commit ID for 4295e285efa8193504ee08b9f633d9f8680bf181>

*** Internal error on buildslave (no space left on device). ***
*** Please report this to the buildslave owner (see <http://gdb-build.sergiodj.net//buildslaves/fedora-ppc64le-1>) ***