qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: lcapitulino@redhat.com
Subject: [Qemu-devel] [FOR 0.12 PATCH 00/18] QError conversions and more
Date: Mon,  7 Dec 2009 21:36:58 +0100	[thread overview]
Message-ID: <1260218236-22143-1-git-send-email-armbru@redhat.com> (raw)

This patch series converts monitor commands eject, change, closefd,
getfd to QError.

There's a simple bonus fix in PATCH 3.

Finally, PATCH 18 puts a human-readable error description into the QMP
error response.  For background, see the lengthy thread ending at
http://lists.gnu.org/archive/html/qemu-devel/2009-11/msg01221.html

Series should be applied after Luiz's work.

Luiz Capitulino (2):
  QError: new class for device encrypted errors
  monitor: do_cont(): Don't ask for passwords

Markus Armbruster (16):
  monitor: Fix double-prompt after "change vnc passwd BLA"
  QError: Put error definitions in alphabetical order
  QError: New QERR_DEVICE_LOCKED
  QError: New QERR_DEVICE_NOT_REMOVABLE
  monitor: convert do_eject() to QError
  QError: New QERR_INVALID_BLOCK_FORMAT
  QError: New QERR_SET_PASSWD_FAILED
  QError: New QERR_VNC_SERVER_FAILED
  monitor: convert do_change() to QObject, QError
  QError: New QERR_FD_NOT_FOUND
  monitor: convert do_closefd() to QError
  QError: New QERR_FD_NOT_SUPPLIED
  New QERR_INVALID_PARAMETER
  QError: New QERR_TOO_MANY_FILES
  monitor: convert do_getfd() to QError
  QMP: add human-readable description to error response

 QMP/qmp-spec.txt |    5 ++-
 monitor.c        |   55 +++++++++++++++++++++++-----------
 qemu-monitor.hx  |    3 +-
 qerror.c         |   85 +++++++++++++++++++++++++++++++++++++++++++-----------
 qerror.h         |   56 ++++++++++++++++++++++++++++-------
 5 files changed, 155 insertions(+), 49 deletions(-)

             reply	other threads:[~2009-12-07 20:37 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-07 20:36 Markus Armbruster [this message]
2009-12-07 20:36 ` [Qemu-devel] [FOR 0.12 PATCH 01/18] QError: new class for device encrypted errors Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 02/18] monitor: do_cont(): Don't ask for passwords Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 03/18] monitor: Fix double-prompt after "change vnc passwd BLA" Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 04/18] QError: Put error definitions in alphabetical order Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 05/18] QError: New QERR_DEVICE_LOCKED Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 06/18] QError: New QERR_DEVICE_NOT_REMOVABLE Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 07/18] monitor: convert do_eject() to QError Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 08/18] QError: New QERR_INVALID_BLOCK_FORMAT Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 09/18] QError: New QERR_SET_PASSWD_FAILED Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 10/18] QError: New QERR_VNC_SERVER_FAILED Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 11/18] monitor: convert do_change() to QObject, QError Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 12/18] QError: New QERR_FD_NOT_FOUND Markus Armbruster
2009-12-08 11:51   ` [Qemu-devel] " Luiz Capitulino
2009-12-08 12:25     ` Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 13/18] monitor: convert do_closefd() to QError Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 14/18] QError: New QERR_FD_NOT_SUPPLIED Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 15/18] New QERR_INVALID_PARAMETER Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 16/18] QError: New QERR_TOO_MANY_FILES Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 17/18] monitor: convert do_getfd() to QError Markus Armbruster
2009-12-07 20:37 ` [Qemu-devel] [FOR 0.12 PATCH 18/18] QMP: add human-readable description to error response Markus Armbruster
2009-12-08 12:11   ` [Qemu-devel] " Luiz Capitulino
2009-12-08 12:25     ` Daniel P. Berrange
2009-12-08 14:11       ` Luiz Capitulino
2009-12-08 14:50         ` Markus Armbruster
2009-12-08 12:38     ` Paolo Bonzini
2009-12-08 12:57       ` Markus Armbruster
2009-12-08 12:48     ` Markus Armbruster
2009-12-08 13:18     ` Anthony Liguori
2009-12-08 14:41       ` Luiz Capitulino
2009-12-08 11:49 ` [Qemu-devel] Re: [FOR 0.12 PATCH 00/18] QError conversions and more Luiz Capitulino

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=1260218236-22143-1-git-send-email-armbru@redhat.com \
    --to=armbru@redhat.com \
    --cc=lcapitulino@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).