From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: qemu-devel@nongnu.org, "Markus Armbruster" <armbru@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Gerd Hoffmann" <kraxel@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v5 0/9] Enable passing pre-opened chardev socket FD
Date: Mon, 12 Mar 2018 13:14:45 +0000 [thread overview]
Message-ID: <20180312131445.GS3493@redhat.com> (raw)
In-Reply-To: <c9bbe182-4a13-0063-4ce8-98151f1bd228@redhat.com>
On Mon, Mar 12, 2018 at 08:12:44AM -0500, Eric Blake wrote:
> On 03/12/2018 07:49 AM, Daniel P. Berrangé wrote:
> > An update of:
> >
> > v1: https://lists.gnu.org/archive/html/qemu-devel/2017-12/msg04618.html
> > v2: https://lists.gnu.org/archive/html/qemu-devel/2017-12/msg04706.html
> > v3: https://lists.gnu.org/archive/html/qemu-devel/2017-12/msg04892.html
> > v4: https://lists.gnu.org/archive/html/qemu-devel/2018-02/msg00950.html
> >
> > This enables fixing a long standing problem that libvirt has with
> > starting up QEMU. It has to busy-wait retrying connect() on the QMP
> > monitor socket until QEMU finally creates & listens on it, but at same
> > time must be careful to not wait forever if QEMU exits.
> >
> > With this patch series, libvirt can simply pass in a pre-opened UNIX domain
> > socket file descriptor, which it can immediately connect to with no busy-wait.
> >
> > NB, this will generate one expected failure with patchew / checkpatch.pl
>
> >
> > Changed in v5:
> >
> > - Docs typo (Eric)
> > - Fix errno setting in int parsing code (Eric)
> >
>
> > Daniel P. Berrangé (9):
> > char: don't silently skip tn3270 protocol init when TLS is enabled
> > cutils: add qemu_strtoi & qemu_strtoui parsers for int/unsigned int
> > types
> > sockets: pull code for testing IP availability out of specific test
> > sockets: strengthen test suite IP protocol availability checks
> > sockets: move fd_is_socket() into common sockets code
> > sockets: check that the named file descriptor is a socket
> > sockets: allow SocketAddress 'fd' to reference numeric file
> > descriptors
> > char: refactor parsing of socket address information
> > char: allow passing pre-opened socket file descriptor at startup
>
> Whose tree should this go through? It's got some QAPI impact, so I'm okay
> including it in my QAPI pull request later today, if there is no better
> tree...
These days I usually send PR for stuff touching sockets code myself. So
unless someone else strongly prefers to take it via their tree for sake
of any easy of conflict / merge resolution, I can do a PR once acked.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2018-03-12 13:15 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-12 12:49 [Qemu-devel] [PATCH v5 0/9] Enable passing pre-opened chardev socket FD Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 1/9] char: don't silently skip tn3270 protocol init when TLS is enabled Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 2/9] cutils: add qemu_strtoi & qemu_strtoui parsers for int/unsigned int types Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 3/9] sockets: pull code for testing IP availability out of specific test Daniel P. Berrangé
2018-03-12 17:46 ` Daniel P. Berrangé
2018-03-12 18:08 ` Eric Blake
2018-03-12 19:18 ` Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 4/9] sockets: strengthen test suite IP protocol availability checks Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 5/9] sockets: move fd_is_socket() into common sockets code Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 6/9] sockets: check that the named file descriptor is a socket Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 7/9] sockets: allow SocketAddress 'fd' to reference numeric file descriptors Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 8/9] char: refactor parsing of socket address information Daniel P. Berrangé
2018-03-12 12:49 ` [Qemu-devel] [PATCH v5 9/9] char: allow passing pre-opened socket file descriptor at startup Daniel P. Berrangé
2018-03-12 13:12 ` [Qemu-devel] [PATCH v5 0/9] Enable passing pre-opened chardev socket FD Eric Blake
2018-03-12 13:14 ` Daniel P. Berrangé [this message]
2018-03-12 14:01 ` Eric Blake
2018-03-12 14:41 ` Eric Blake
2018-03-12 17:57 ` no-reply
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=20180312131445.GS3493@redhat.com \
--to=berrange@redhat.com \
--cc=armbru@redhat.com \
--cc=eblake@redhat.com \
--cc=kraxel@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.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;
as well as URLs for NNTP newsgroup(s).