Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Paul Koning <paul_koning@dell.com>
To: Joel Brobecker <brobecker@adacore.com>
Cc: "Frank Ch. Eigler" <fche@redhat.com>, gdb-patches@sourceware.org
Subject: Re: duplicated code in gdb and gdbserver
Date: Mon, 10 Jan 2011 15:50:00 -0000	[thread overview]
Message-ID: <48CC49A5-BD0F-4732-B4B3-B670D80258BB@dell.com> (raw)
In-Reply-To: <20110110140848.GB2331@adacore.com>


On Jan 10, 2011, at 9:08 AM, Joel Brobecker wrote:

>> A differently aimed step toward this could be to remove duplication
>> between gdb native and gdbserver by gradually *deprecating gdb native
>> support*.  A library API for the remote protocol would be a natural
>> non-symbolic/process-control programming interface.
> 
> I hope you are not suggesting that we actually do remove native support
> and force native support through gdbserver!

It does sound like that. 

A big problem is that gdbserver support exists only for a few target processors and operating systems.  But in principle it seems reasonable.  Then again, so does the other (making gdb native support be the guts of gdbserver).

I've looked at that since I'm interested in one of the operating systems that isn't well supported in gdbserver, and if the gdb native code could "just be made to work" that seems like a shortcut.  Unfortunately, as has been pointed out, the list of assumptions in the gdb native support seem to make that a large job at best.

Since gdbserver is small and, being a client-server application, is at least somewhat modular, it seems that Frank's suggestion makes sense.  No matter how you slice it, it would take a serious restructuring to have a single instance of the code.

	paul



  parent reply	other threads:[~2011-01-10 15:50 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-07 15:23 Yao Qi
2011-01-07 16:37 ` Doug Evans
2011-01-08  3:34   ` Yao Qi
2011-01-08  5:17     ` Joel Brobecker
2011-01-10 13:39   ` Frank Ch. Eigler
2011-01-10 14:09     ` Joel Brobecker
2011-01-10 15:10       ` Frank Ch. Eigler
2011-01-10 15:50       ` Paul Koning [this message]
2011-01-10 15:51     ` Doug Evans
2011-01-10 15:54       ` Frank Ch. Eigler
2011-01-10 16:35         ` Doug Evans
2011-01-10 19:02           ` Frank Ch. Eigler
2011-01-11 23:35             ` Joel Brobecker
2011-01-11 23:38           ` Joel Brobecker
2011-01-12  0:30             ` Frank Ch. Eigler
2011-01-12 17:54               ` Doug Evans
2011-01-12 18:06                 ` Doug Evans
2011-01-12 18:43                   ` Paul Koning
2011-01-12 19:04                     ` Doug Evans
2011-01-12 20:09                 ` Joel Brobecker
2011-01-12 20:48                 ` Frank Ch. Eigler
2011-01-14 17:04                   ` Doug Evans
2011-01-12 15:47           ` Tom Tromey
2011-01-07 17:17 ` Stan Shebs

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=48CC49A5-BD0F-4732-B4B3-B670D80258BB@dell.com \
    --to=paul_koning@dell.com \
    --cc=brobecker@adacore.com \
    --cc=fche@redhat.com \
    --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