Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Yao Qi <yao@codesourcery.com>
To: <gdb-patches@sourceware.org>
Cc: Hui Zhu <hui_zhu@mentor.com>
Subject: Re: [RFC] Autoload-breakpoints new version [3/9] notification async
Date: Wed, 08 Aug 2012 13:26:00 -0000	[thread overview]
Message-ID: <1866631.Ri6KkAT9H2@qiyao.dyndns.org> (raw)
In-Reply-To: <5020BF3A.8030809@mentor.com>

On Tuesday, August 07, 2012 03:09:46 PM Hui Zhu wrote:
> This patch is the extend for the notification function.
> Current notification function cannot handle the packet when GDB doesn't want
> to send or receive packet from the remote target. This patch do the extend
> make GDB can handle the notification even if it doesn't send or receive
> packet.

I am trying to understand what current GDB can do, and what patched GDB can 
do.  If I understand it correctly,

  - current GDB only handle notification when it sends or receives packets
  - patched GDB can handle notification at any time when notification comes 
in.

Looks this patch changes the way of GDB handling notifications (from "only-on-
send-receive" handling to "anytime" handling), but has nothing to do with how 
notification is sent out from remote stub.  If so, why do you add 
"NotificationAsync" in remote_supported_packet?  Remote supported packets are 
used to tell GDB what feature remote stub has or doesn't have.  In other 
words, "NotificationAsync" is a feature of GDB, instead of remote stub, so we 
don't need "NotificationAsync".

I applied patch 1/9 and 3/9, and get following regressions in gdbserver with 
async mode on.

  FAIL: gdb.mi/mi-nonstop-exit.exp: finished exec continue (unknown output 
after running)
  FAIL: gdb.mi/mi-nonstop-exit.exp: finished exec continue (2) (unknown output 
after running)
  FAIL: gdb.mi/mi-nsmoribund.exp: resume all, waiting for program exit 
(timeout)

-- 
Yao (齐尧)


  reply	other threads:[~2012-08-08 13:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-07  7:10 Hui Zhu
2012-08-08 13:26 ` Yao Qi [this message]
2012-08-11 15:20   ` Hui Zhu
2012-08-13  2:25     ` Yao Qi
2012-08-14 12:01       ` Hui Zhu
2012-08-15 14:23         ` Hui Zhu
2012-08-23 10:44           ` Hui Zhu
2012-08-23 11:20             ` Pedro Alves
2012-08-23 12:46           ` Yao Qi
2012-08-23 14:30             ` Hui Zhu

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=1866631.Ri6KkAT9H2@qiyao.dyndns.org \
    --to=yao@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=hui_zhu@mentor.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