Check input interrupt first when reading packet
authorYao Qi <yao.qi@linaro.org>
Fri, 8 Jan 2016 11:06:00 +0000 (11:06 +0000)
committerYao Qi <yao.qi@linaro.org>
Fri, 8 Jan 2016 11:06:00 +0000 (11:06 +0000)
commit5a0dd67a459338efb77f8d82bb3650d801ff0dd5
tree78dd87d22334677264120c614ec3e861629908fd
parent83c3256ef5afe10625fd55dda2df5fc56b9b7393
Check input interrupt first when reading packet

Hi,
I see timeout in one of several runs of random-signal.exp like this,

 $ (set -e; while true; do make check RUNTESTFLAGS="--target_board=native-gdbserver random-signal.exp"; done)

In about every five runs, we can see a fail,

PASS: gdb.base/random-signal.exp: continue
^CFAIL: gdb.base/random-signal.exp: stop with control-c (timeout)

after some investigation, I find '\003' may be discarded by GDBserver when
it is expecting '$'.  In GDB side, both normal packets and '\003' are sent
via function send, but GDBserver may receive them at any time, that is to
say, in the receive buffer in GDBserver, '\003' may appear before or after
normal packet.  However, current GDBserver doesn't handle this case.

With this patch applied, I don't see this fail in multiple runs.
Although there is still timeout fail, that is a different problem, the
next patch will fix it.

gdb/gdbserver:

2016-01-08  Yao Qi  <yao.qi@linaro.org>

* remote-utils.c (getpkt): If c is '\003', call target hook
request_interrupt.
gdb/gdbserver/ChangeLog
gdb/gdbserver/remote-utils.c