From: Yao Qi <yao@codesourcery.com>
To: Marc Khouzam <marc.khouzam@ericsson.com>
Cc: "'gdb-patches@sourceware.org'" <gdb-patches@sourceware.org>
Subject: Re: [RFC 0/2, gdbserver] Set linux target in async mode in default
Date: Tue, 18 Sep 2012 14:36:00 -0000 [thread overview]
Message-ID: <5058868F.7070703@codesourcery.com> (raw)
In-Reply-To: <F7CE05678329534C957159168FA70DEC5E11A0DDE0@EUSAACMS0703.eamcs.ericsson.se>
On 09/18/2012 10:12 PM, Marc Khouzam wrote:
> To clarify, after this patch we can still use "-gdb-set target-async off"
> to turn off async mode right?
>
This patch doesn't change the behaviour from the GDB's and Eclipse's
perspective. We can still set target-async on or off in GDB side as needed.
IIUC, the term "async mode" in GDBserver is different from its
counterpart in GDB. In GDBserver, "async mode" means an "async event
loop" which is used for non-stop and notification. If we look at RSP,
there is no command to turn "async mode" on or off directly in GDBserver
(QNonStop can start async mode on linux target), the state of "async
mode" of GDBserver is unknown to GDB, and vice versa.
In current GDBserver/linux, we have only two modes, 'non-stop with
async-event-loop' and 'all-stop without async-event-loop'. This patch
only changes the latter to 'all-stop with async-event-loop', so that
async notification can work on top it. At the same time, GDB still
works in both "target-async on" and "target-async off" with this patched
GDbserver.
> Eclipse currently uses non-async mode for all-stop and some code
> is based on that assumption (e.g., interrupting the target)
> so we need to keep non-async available.
Forgot to mention in my mail that I run testsuite with
{native-gdbserver, native-extended-gdbserver} x {async, sync} (in GDB
side), and no regression.
--
Yao
next prev parent reply other threads:[~2012-09-18 14:36 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-18 9:49 Yao Qi
2012-09-18 9:49 ` [PATCH 2/2] gdbserver:Remove async from target_ops Yao Qi
2012-09-18 9:49 ` [PATCH 1/2] gdbserver:Set linux target in async mode in default Yao Qi
2012-09-18 14:12 ` [RFC 0/2, gdbserver] Set " Marc Khouzam
2012-09-18 14:36 ` Yao Qi [this message]
2012-09-18 14:39 ` Marc Khouzam
2012-09-25 16:23 ` Pedro Alves
2012-09-27 3:16 ` Yao Qi
2012-10-12 12:00 ` Yao Qi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5058868F.7070703@codesourcery.com \
--to=yao@codesourcery.com \
--cc=gdb-patches@sourceware.org \
--cc=marc.khouzam@ericsson.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox