From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7456 invoked by alias); 9 Oct 2006 19:01:53 -0000 Received: (qmail 7446 invoked by uid 22791); 9 Oct 2006 19:01:52 -0000 X-Spam-Check-By: sourceware.org Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.31.1) with ESMTP; Mon, 09 Oct 2006 19:01:48 +0000 Received: from drow by nevyn.them.org with local (Exim 4.54) id 1GX0Nj-0005ur-Tq; Mon, 09 Oct 2006 15:01:43 -0400 Date: Mon, 09 Oct 2006 19:01:00 -0000 From: Daniel Jacobowitz To: Jan Kratochvil Cc: Eli Zaretskii , gdb-patches@sourceware.org Subject: Re: [patch] IPv6 support for gdbserver Message-ID: <20061009190143.GA22487@nevyn.them.org> Mail-Followup-To: Jan Kratochvil , Eli Zaretskii , gdb-patches@sourceware.org References: <20060927163337.GA27149@host0.dyn.jankratochvil.net> <20060927182038.GA5635@nevyn.them.org> <20060927185547.GA13544@host0.dyn.jankratochvil.net> <20060927190611.GA7326@nevyn.them.org> <20060930152757.GA27372@host0.dyn.jankratochvil.net> <20061008190239.GA29584@host0.dyn.jankratochvil.net> <20061009141634.GA392@host0.dyn.jankratochvil.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20061009141634.GA392@host0.dyn.jankratochvil.net> User-Agent: Mutt/1.5.13 (2006-08-11) X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-10/txt/msg00078.txt.bz2 On Mon, Oct 09, 2006 at 04:16:34PM +0200, Jan Kratochvil wrote: > > And second, isn't there a better way of specifying two descriptors? I > > find the "fdNM" method inelegant, and the limitation of a single-digit > > descriptor it requires too high a price to pay. I hope we can come up > > with a better method. > > Designed for trivia C parsing as the IPv6 support was considered too > complicated by Daniel Jacobowitz and the string parsing in C is a lot of lines. > The new syntax looks like that of socat(1): > socat EXEC:'gdbserver fdin=3,fdout=4 emacs foo.txt',fdin=3,fdout=4 TCP-LISTEN:2345 We're a GNU program. We should use the standard GNU style for command line options wherever possible. Anyway, I don't think we need another update of this patch; let's get answers to the big picture questions first. I've asked the general questions I want feedback about on the gdb@ mailing list, in another message. Then, if it needs any big reworking, I'll volunteer to finish it. We've made you jump through enough hoops already! I'd prefer to avoid the "tcp6:" syntax if we can, and have things work transparently. It's been about a year since I last looked at IPv6 support in anything, though. Do you think this is feasible? In particular, "target remote hostname:port" ought to work fine for IPv6. > > > * gdb.texinfo (Using the gdbserver program): Remove "host:port". > > > > Why? I think back-compatibility is important. > > It looks broken to me to support IPv4 without supporting IPv6, moreover if the > IPv6 functions to use are simpler. I hope you are aware of the former full > IPv6 support posted here: > http://sourceware.org/ml/gdb-patches/2006-09/msg00216.html > > > The second patch "gdb-cvs-IPv6-hostport-minus.patch" was not updated for this > one as it looks as the IPv4 legacy code is not going to be dropped even with > the socat(1) support there. Plenty of people use gdbserver with IPv4, and up until now, no one has even asked us for IPv6 support. I think it's absurd to say that IPv6 is as essential as IPv4 support here. Maybe I was wrong, and we should go ahead and add IPv6 directly to gdbserver. I need to understand the usefulness first. Let's give it a few days. -- Daniel Jacobowitz CodeSourcery