[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error
From: |
Jamie Lokier |
Subject: |
Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error |
Date: |
Sun, 18 Oct 2009 05:28:26 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
Paolo Bonzini wrote:
> On 10/15/2009 09:08 PM, Luiz Capitulino wrote:
> >{ "error": { "code": 12
> > "desc": "device already open",
> > "data": { "bus": 0, "address": 12 } } }
> >
> > Note that this also can be reused by any bus, as the "data" information
> >is built at error time and can contain anything.
>
> The "desc" is not even necessary on the wire.
When you send an error code that that client doesn't know yet (because
you can't update every client immediately), it'll be very helpful to
users to see "device already open" instead of "unknown error 12".
-- Jamie
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, (continued)
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Gerd Hoffmann, 2009/10/16
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/16
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Paolo Bonzini, 2009/10/16
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Anthony Liguori, 2009/10/16
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/16
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Paolo Bonzini, 2009/10/16
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Anthony Liguori, 2009/10/16
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Paolo Bonzini, 2009/10/16
- Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error,
Jamie Lokier <=
- Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Jamie Lokier, 2009/10/18
[Qemu-devel] [PATCH 7/9] qdev: Use QError for not found error, Luiz Capitulino, 2009/10/13
- Re: [Qemu-devel] [PATCH 7/9] qdev: Use QError for not found error, Markus Armbruster, 2009/10/13
- Re: [Qemu-devel] [PATCH 7/9] qdev: Use QError for not found error, Luiz Capitulino, 2009/10/14
- Re: [Qemu-devel] [PATCH 7/9] qdev: Use QError for not found error, Daniel P. Berrange, 2009/10/19
- Re: [Qemu-devel] [PATCH 7/9] qdev: Use QError for not found error, Gerd Hoffmann, 2009/10/19
- Re: [Qemu-devel] [PATCH 7/9] qdev: Use QError for not found error, Daniel P. Berrange, 2009/10/19
- [Qemu-devel] Re: [PATCH 7/9] qdev: Use QError for not found error, Paolo Bonzini, 2009/10/19