[Qemu-devel] [PULL 0/2] chardev: fix "info chardev" output

[Qemu-devel] [PULL 0/2] chardev: fix "info chardev" output

[Gerd Hoffmann]

  Hi,

Tiny series, first reverting the broken patch from Lei Li,
then fixing it for real.

please pull,
  Gerd

The following changes since commit 6a4e17711442849bf2cc731ccddef5a2a2d92d29:

  Remove Sun4c, Sun4d and a few CPUs (2013-05-26 11:37:58 +0000)

are available in the git repository at:

  git://git.kraxel.org/qemu chardev.6

for you to fetch changes up to 60d95386abf1f5f37ded6a812d12ac458df2491b:

  chardev: fix "info chardev" output (2013-05-27 12:47:28 +0200)

----------------------------------------------------------------
Gerd Hoffmann (2):
      Revert "chardev: Get filename for new qapi backend"
      chardev: fix "info chardev" output

 qemu-char.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

[Qemu-devel] [PATCH 1/2] Revert "chardev: Get filename for new qapi backend"

[Gerd Hoffmann]

Does not handle chardevs created via chardev-add monitor command.

This reverts commit 2b220025993e76d4116781ca91a4fabc5ad9c722.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 qemu-char.c |    2 --
 1 file changed, 2 deletions(-)

diff --git a/qemu-char.c b/qemu-char.c
index 4f8382e..f825294 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -3276,7 +3276,6 @@ CharDriverState *qemu_chr_new_from_opts(QemuOpts *opts,
         ChardevReturn *ret = NULL;
         const char *id = qemu_opts_id(opts);
         const char *bid = NULL;
-        char *filename = g_strdup(qemu_opt_get(opts, "backend"));
 
         if (qemu_opt_get_bool(opts, "mux", 0)) {
             bid = g_strdup_printf("%s-base", id);
@@ -3309,7 +3308,6 @@ CharDriverState *qemu_chr_new_from_opts(QemuOpts *opts,
         }
 
         chr = qemu_chr_find(id);
-        chr->filename = filename;
 
     qapi_out:
         qapi_free_ChardevBackend(backend);
-- 
1.7.9.7

[Qemu-devel] [PATCH 2/2] chardev: fix "info chardev" output

[Gerd Hoffmann]

Fill unset CharDriverState->filename with the backend name, so
'info chardev' will return at least the chardev type.  Don't
touch it in case the chardev init function filled it already,
like the socket+pty chardevs do for example.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 qemu-char.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/qemu-char.c b/qemu-char.c
index f825294..d04b429 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -3801,6 +3801,9 @@ ChardevReturn *qmp_chardev_add(const char *id, ChardevBackend *backend,
         chr->label = g_strdup(id);
         chr->avail_connections =
             (backend->kind == CHARDEV_BACKEND_KIND_MUX) ? MAX_MUX : 1;
+        if (!chr->filename) {
+            chr->filename = g_strdup(ChardevBackendKind_lookup[backend->kind]);
+        }
         QTAILQ_INSERT_TAIL(&chardevs, chr, next);
         return ret;
     } else {
-- 
1.7.9.7

Re: [Qemu-devel] [SeaBIOS] [PULL 0/2] chardev: fix "info chardev" output

[Gerd Hoffmann]

On 05/28/13 12:27, Gerd Hoffmann wrote:
>   Hi,
> 
> Tiny series, first reverting the broken patch from Lei Li,
> then fixing it for real.

Oops, scratch that.  Resent an old series due to 'git send-email
00*.patch' in the wrong directory.

sorry,
  Gerd

Re: [Qemu-devel] [PULL 0/2] chardev: fix "info chardev" output

[Anthony Liguori]

Pulled.  Thanks.

Regards,

Anthony Liguori

Re: [Qemu-devel] [PULL 0/2] chardev: fix "info chardev" output

[Anthony Liguori]

Pulled.  Thanks.

Regards,

Anthony Liguori

Re: [Qemu-devel] [PULL 0/2] chardev: fix "info chardev" output

[Eric Blake]

[-- Attachment #1: Type: text/plain, Size: 1043 bytes --]

On 05/28/2013 12:34 AM, Gerd Hoffmann wrote:
>   Hi,
> 
> Tiny series, first reverting the broken patch from Lei Li,
> then fixing it for real.
> 
> please pull,
>   Gerd
> 
> The following changes since commit 6a4e17711442849bf2cc731ccddef5a2a2d92d29:
> 
>   Remove Sun4c, Sun4d and a few CPUs (2013-05-26 11:37:58 +0000)
> 
> are available in the git repository at:
> 
>   git://git.kraxel.org/qemu chardev.6
> 
> for you to fetch changes up to 60d95386abf1f5f37ded6a812d12ac458df2491b:
> 
>   chardev: fix "info chardev" output (2013-05-27 12:47:28 +0200)
> 
> ----------------------------------------------------------------
> Gerd Hoffmann (2):
>       Revert "chardev: Get filename for new qapi backend"
>       chardev: fix "info chardev" output
> 

Series: Reviewed-by: Eric Blake <eblake@redhat.com>

>  qemu-char.c |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> 
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 621 bytes --]

[Qemu-devel] [PULL 0/2] chardev: fix "info chardev" output

[Gerd Hoffmann]

  Hi,

Tiny series, first reverting the broken patch from Lei Li,
then fixing it for real.

please pull,
  Gerd

The following changes since commit 6a4e17711442849bf2cc731ccddef5a2a2d92d29:

  Remove Sun4c, Sun4d and a few CPUs (2013-05-26 11:37:58 +0000)

are available in the git repository at:

  git://git.kraxel.org/qemu chardev.6

for you to fetch changes up to 60d95386abf1f5f37ded6a812d12ac458df2491b:

  chardev: fix "info chardev" output (2013-05-27 12:47:28 +0200)

----------------------------------------------------------------
Gerd Hoffmann (2):
      Revert "chardev: Get filename for new qapi backend"
      chardev: fix "info chardev" output

 qemu-char.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)