From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44585) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V2jG2-0007AT-8Q for qemu-devel@nongnu.org; Fri, 26 Jul 2013 10:44:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V2jG1-0003ET-4Z for qemu-devel@nongnu.org; Fri, 26 Jul 2013 10:44:38 -0400 Received: from oxygen.pond.sub.org ([2a01:4f8:121:10e4::3]:57748) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V2jG0-0003E3-Uf for qemu-devel@nongnu.org; Fri, 26 Jul 2013 10:44:37 -0400 From: Markus Armbruster Date: Fri, 26 Jul 2013 16:44:31 +0200 Message-Id: <1374849874-25531-1-git-send-email-armbru@redhat.com> Subject: [Qemu-devel] [PATCH v2 0/3] qemu-char: ringbuf fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: aliguori@us.ibm.com, lilei@linux.vnet.ibm.com, lcapitulino@redhat.com, kraxel@redhat.com, akong@redhat.com Fix code to match documentation. The patches take pains to preserve command-line, HMP and QMP bug-compatibility. I'm not sure that's actually needed, because: * All: the documented way to use the driver never worked in any released version, and the way that worked was undocumented. * QMP: I doubt anyone used chardev-add / chardev-remove in anger, because serious use surely would've run into the bug fixed in commit commit 2ea3e2c "qemu-char: Fix ID reuse after chardev-remove for qapi-based init". I include the compatibility gunk anyway, so we can have a more informed discussion. You might want to apply this after my "Our QAPI parser is a hack, replace it" series, so the indented comment in PATCH 3/3 works by design rather than accident. v2: Trivially rebased Markus Armbruster (3): Revert "chardev: Make the name of memory device consistent" qemu-char: Register ring buffer driver with correct name "ringbuf" qapi: Rename ChardevBackend member "memory" to "ringbuf" qapi-schema.json | 12 +++++++----- qemu-char.c | 26 +++++++++++++++----------- qemu-options.hx | 6 +++--- 3 files changed, 25 insertions(+), 19 deletions(-) -- 1.7.11.7