From: Jim Blandy <jimb@codesourcery.com>
To: Emi SUZUKI <emi-suzuki@tjsys.co.jp>
Cc: gdb-patches@sourceware.org
Subject: Re: [RFC] checking the Z-packet support on gdbserver
Date: Tue, 18 Sep 2007 21:45:00 -0000 [thread overview]
Message-ID: <m38x73wsug.fsf@codesourcery.com> (raw)
In-Reply-To: <20070914.183913.226021396.emi-suzuki@tjsys.co.jp> (Emi SUZUKI's message of "Fri, 14 Sep 2007 18:39:13 +0900")
Emi SUZUKI <emi-suzuki@tjsys.co.jp> writes:
> Well, I understood casually that the answer of 'qSupported' packet
> would tell the support for the other 'q' packets from the current
> implementaion... According to the description in the info, definitely
> it would make sense. How about the attached?
Actually, thinking more carefully: what effect will this change have
on existing stubs that do support hardware watchpoints, but don't
include 'Z0+', etc. in their qSupported responses? Won't it mean that
new GDBs will decline to use hardware watchpoints with them?
It seems to me that, when GDB has no information about 'Zx'
availability, it should continue to assume that they are available and
then get errors if not, as it does now, but it should also work with
smarter stubs, like your patched GDB server, to provide a better user
experience.
So, would it make more sense for the initial state of the Z packets in
remote_protocol_features to be PACKET_SUPPORT_UNKNOWN, and for
gdbserver to transmit either Zx- or Zx+ as appropriate? Thus, in your
case, as soon as GDB connected to the target it would know that
hardware watchpoints weren't available, but on connection to some
older stub which said nothing about the Z packets in its qSupported
response (if it gave such a response at all), GDB would continue to
try hardware watchpoints.
next prev parent reply other threads:[~2007-09-18 21:45 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-13 12:09 [RFC] checking the Z-packet suppport " Emi SUZUKI
2007-09-13 17:05 ` Jim Blandy
2007-09-14 9:40 ` [RFC] checking the Z-packet support " Emi SUZUKI
2007-09-14 12:15 ` Daniel Jacobowitz
2007-09-18 21:17 ` Jim Blandy
2007-09-18 21:28 ` Daniel Jacobowitz
2007-09-18 21:45 ` Jim Blandy [this message]
2007-09-20 9:49 ` Emi SUZUKI
2007-09-14 12:15 ` [RFC] checking the Z-packet suppport " Daniel Jacobowitz
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=m38x73wsug.fsf@codesourcery.com \
--to=jimb@codesourcery.com \
--cc=emi-suzuki@tjsys.co.jp \
--cc=gdb-patches@sourceware.org \
/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