From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52951) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WKzrf-0002DN-TB for qemu-devel@nongnu.org; Tue, 04 Mar 2014 19:39:24 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WKzrX-0008R2-Gm for qemu-devel@nongnu.org; Tue, 04 Mar 2014 19:39:15 -0500 Received: from mail-oa0-x233.google.com ([2607:f8b0:4003:c02::233]:63661) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WKzrX-0008Qy-Ah for qemu-devel@nongnu.org; Tue, 04 Mar 2014 19:39:07 -0500 Received: by mail-oa0-f51.google.com with SMTP id i4so331113oah.10 for ; Tue, 04 Mar 2014 16:39:06 -0800 (PST) Sender: Corey Minyard From: minyard@acm.org Date: Tue, 4 Mar 2014 18:38:50 -0600 Message-Id: <1393979937-9082-1-git-send-email-minyard@acm.org> Subject: [Qemu-devel] [PATCH 0/7] Allow a client chardev to reconnect if disconnected List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: bcketchum@gmail.com, hwd@huawei.com, afaerber@suse.de, mst@redhat.com 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.