Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Paul Fertser <fercerpav@gmail.com>
To: Jan Kratochvil <jan.kratochvil@redhat.com>
Cc: gdb-patches@sourceware.org, Kai Tietz <ktietz@redhat.com>
Subject: Re: [PATCH] Add IPv6 support for remote TCP connections
Date: Sun, 09 Feb 2014 09:53:00 -0000	[thread overview]
Message-ID: <20140209095308.GH2320@home.lan> (raw)
In-Reply-To: <20140209083056.GA32481@host2.jankratochvil.net>

Hi,

On Sun, Feb 09, 2014 at 09:30:56AM +0100, Jan Kratochvil wrote:
> and currently a different patch is on
> 	jankratochvil/ipv6
> 	git://sourceware.org/git/archer.git
> 
> It should also implement IPv6 in gdbserver (which you may not need).

That was my other concern indeed. I am afraid your implementation
relies on IPv4-mapped IPv6 which is not supported on OpenBSD at all
(and also on older windows versions including wxp and w2k3) and can be
disabled by system administrators on other systems.

I think the way I modified net_open() behaviour matches the current
code closer, i.e. in case the target didn't bind its socket yet in my
version it'll will cycle through all the available addresses for 15
seconds and it looks like in your version it'll try connecting once to
all potential addresses, get "connection refused" and give up.

> The problem is that MinGW (=MS-Windows) port does not have getaddrinfo, there
> are multiple way how to deal with it, the most clean should be via extending
> gdb/gnulib/ :

Do you take into account that there're currently two competing
solutions for providing windows support: MinGW and MinGW-w64? The
former declares tools availability as their aim, but the latter also
strives for C99 and some POSIX compatibility. There's also Cygwin
which is, of course, even closer to traditional systems. All major
distros (that provide cross-compiling toolchain for windows) switched
to MinGW-w64 by now.

I've just tried cross-compiling code with getaddrinfo with mingw-w64
without any replacements and it was built (and run with wine) just
fine.

> But then mingw build fails.  I tried to fix it in archer/jankratochvil/ipv6
> ( git://sourceware.org/git/archer.git ) but I got lost in what is the right way
> so I guess you should do it from scratch on your own.  I tried it more
> a quick&dirty way but it probably needs a more clean thorough rework IMO.

https://www.gnu.org/software/gnulib/manual/html_node/getaddrinfo.html
implies all compatibility issues should be solved with it. So is it
not so, and there's a bug in gnulib?

> So due to buggy MinGW normal OSes still can't use IPv6 after 7 years while IPv4
> even no longer exists on some networks.

Probably it's time to change that finally, especially given MinGW-w64
seems to work fine?

-- 
Be free, use free (http://www.gnu.org/philosophy/free-sw.html) software!
mailto:fercerpav@gmail.com


  reply	other threads:[~2014-02-09  9:53 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-08 16:54 Paul Fertser
2014-02-09  8:31 ` Jan Kratochvil
2014-02-09  9:53   ` Paul Fertser [this message]
2014-02-09 13:05     ` Jan Kratochvil
2014-02-09 16:35       ` Eli Zaretskii
2014-02-09 16:47         ` Jan Kratochvil
2014-02-09 17:08           ` Paul Fertser
2014-02-09 17:29             ` Jan Kratochvil
     [not found]               ` <1392033768-16793-1-git-send-email-fercerpav@gmail.com>
2014-02-10 16:53                 ` [PATCH v2] " Eli Zaretskii
2014-02-10 17:02                   ` Paul Fertser
2014-02-10 17:45                     ` Eli Zaretskii
2014-02-10 19:58                       ` getaddrinfo available on all GDB hosts? [Re: [PATCH v2] Add IPv6 support for remote TCP connections] Jan Kratochvil
2014-02-11  3:42                         ` Joel Brobecker
2014-02-11 20:05                           ` Jan Kratochvil
2014-02-11 20:33                             ` Stan Shebs
2014-02-12  3:04                               ` Joel Brobecker
2014-02-12  3:00                             ` Joel Brobecker
2014-02-12 12:10                               ` Jan Kratochvil
2014-02-13  7:37                                 ` Joel Brobecker
2014-02-13 10:44                                   ` [patch] [sim] --disable-sim on ppc* by default (for AIX) [Re: getaddrinfo available on all GDB hosts?] Jan Kratochvil
2014-02-13 10:51                                     ` Pedro Alves
2014-02-13 11:04                                       ` Jan Kratochvil
2014-02-13 11:39                                         ` Pedro Alves
2014-02-13 11:51                                           ` Joel Brobecker
2014-02-13 12:08                                           ` [cancel] " Jan Kratochvil
2014-02-13 11:59                                     ` Joel Brobecker
2014-02-13 14:26                                 ` getaddrinfo available on all GDB hosts? [Re: [PATCH v2] Add IPv6 support for remote TCP connections] Tom Tromey
2014-02-13 11:36                           ` Jan Kratochvil
2014-02-13 11:55                             ` Joel Brobecker
2014-02-11 19:48                         ` [PATCH v3] Add IPv6 support for outgoing remote TCP connections Paul Fertser
2014-02-12 16:53                           ` Jan Kratochvil
2014-02-12 17:32                             ` Paul Fertser
2014-02-12 20:10                               ` Jan Kratochvil
2015-03-22 16:39             ` [PATCH] Add IPv6 support for " Jan Kratochvil
2015-03-22 16:54               ` Eli Zaretskii
2015-03-22 17:09                 ` Jan Kratochvil
2015-03-22 17:48                   ` Eli Zaretskii
2015-03-23 18:02                     ` Pedro Alves
2015-03-23 18:42                       ` Eli Zaretskii
2015-03-23 19:12                         ` Joel Brobecker
2015-03-23 19:18                           ` Eli Zaretskii
2015-04-13 15:12                             ` Pedro Alves
2015-04-13 15:27                               ` Eli Zaretskii
2015-04-13 16:10                                 ` Pedro Alves
2015-03-23 19:19                         ` Pedro Alves
2015-03-23 19:03               ` Corinna Vinschen
2015-03-23 19:12                 ` Pedro Alves
2014-02-09 16:29     ` Eli Zaretskii
2014-02-09 16:25   ` Eli Zaretskii

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=20140209095308.GH2320@home.lan \
    --to=fercerpav@gmail.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=ktietz@redhat.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