[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect |
Date: |
Fri, 14 Sep 2012 11:17:36 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Orit Wasserman <address@hidden> wrote:
> getaddrinfo can give us a list of addresses, but we only try to
> connect to the first one. If that fails we never proceed to
> the next one. This is common on desktop setups that often have ipv6
> configured but not actually working.
>
> To fix this make inet_connect_nonblocking retry connection with a different
> address.
> callers on inet_nonblocking_connect register a callback function that will
> be called when connect opertion completes, in case of failure the fd will have
> a negative value
>
> Signed-off-by: Orit Wasserman <address@hidden>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Just thinking out loud to be if I understood this correctly
> + do {
> + rc = getsockopt(s->fd, SOL_SOCKET, SO_ERROR, (void *) &val,
> &valsize);
> + } while (rc == -1 && socket_error() == EINTR);
"rc"" return the error code of getsockopt()
and "val" returns the error code of the socket if there is one.
> +
> + /* update rc to contain error details */
> + if (!rc && val) {
> + rc = -val;
> + }
If getsockopt() succeeds, we "reuse" "rc" error code to have the socket
error code. If you have to resent this, could we improve the comment here?
I have to go to the manual page of getsockopt() that don't likes
SO_ERROR to try to understand what this completely un-intuitive (at
least for me) two lines of code do.
> +
> + /* connect error */
> + if (rc < 0) {
> + closesocket(s->fd);
> + s->fd = rc;
> + }
If there is any error (getsockopt or in the socket), we just close the
fd and update the error code.
Head hurts.
Thanks, Juan.
- [Qemu-devel] [PATCH v3 0/3] nonblocking connect address handling cleanup, Orit Wasserman, 2012/09/13
- [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Orit Wasserman, 2012/09/13
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect,
Juan Quintela <=
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Amos Kong, 2012/09/19
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Orit Wasserman, 2012/09/20
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Amos Kong, 2012/09/20
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Orit Wasserman, 2012/09/23
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Amos Kong, 2012/09/24
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Orit Wasserman, 2012/09/24
- Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Amos Kong, 2012/09/24
Re: [Qemu-devel] [PATCH v3 3/3] Fix address handling in inet_nonblocking_connect, Michael S. Tsirkin, 2012/09/20