From: minyard@acm.org
To: qemu-devel@nongnu.org
Cc: bcketchum@gmail.com, hwd@huawei.com, afaerber@suse.de, mst@redhat.com
Subject: [Qemu-devel] [PATCH 0/7] Allow a client chardev to reconnect if disconnected
Date: Tue, 4 Mar 2014 18:38:50 -0600 [thread overview]
Message-ID: <1393979937-9082-1-git-send-email-minyard@acm.org> (raw)
This patch set was part of the IPMI patches, but people have asked it to be
separated out because it's useful by itself and the IPMI patches are stuck
in legal limbo.
I have left the patch to move allocating CharDriverState in one place
(patch 1). I didn't get much opinion either way, but I think it is
an improvement and a net reduction in code. I also added a patch (patch 6)
that goes another step there, it makes the error handling in
qmp_chardev_add() consistent and is also a net reduction in code.
The main patches to add the feature are patch 2 and patch 3.
The rest are little possible bug fixes that I noticed while working in the
code.
I went through all the comments and I believe I fixed everything.
Thanks to everyone that reviewed this before.
next reply other threads:[~2014-03-05 0:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-05 0:38 minyard [this message]
2014-03-05 0:38 ` [Qemu-devel] [PATCH 1/7] qemu-char: Allocate CharDriverState in qemu_chr_new_from_opts minyard
2014-03-05 0:38 ` [Qemu-devel] [PATCH 2/7] qemu-char: Allow a chardev to reconnect if disconnected minyard
2014-03-06 6:47 ` Weidong Huang
2014-03-06 17:18 ` Corey Minyard
2014-03-06 17:41 ` Michael S. Tsirkin
2014-03-06 7:43 ` Michael S. Tsirkin
2014-03-06 18:04 ` Corey Minyard
2014-03-06 18:43 ` Michael S. Tsirkin
2014-03-05 0:38 ` [Qemu-devel] [PATCH 3/7] qemu-char: Wait until socket connect to report connected minyard
2014-03-05 0:38 ` [Qemu-devel] [PATCH 4/7] qemu-char: remove free of chr from win_stdio_close minyard
2014-03-05 0:38 ` [Qemu-devel] [PATCH 5/7] qemu-char: Close fd at end of file minyard
2014-03-05 0:38 ` [Qemu-devel] [PATCH 6/7] qemu-char: Clean up error handling in qmp_chardev_add minyard
2014-03-05 0:38 ` [Qemu-devel] [PATCH 7/7] console: Don't use the console if NULL minyard
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=1393979937-9082-1-git-send-email-minyard@acm.org \
--to=minyard@acm.org \
--cc=afaerber@suse.de \
--cc=bcketchum@gmail.com \
--cc=hwd@huawei.com \
--cc=mst@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).