[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info |
Date: |
Mon, 28 May 2012 16:06:30 -0300 |
On Mon, 28 May 2012 16:36:16 +0300
Alon Levy <address@hidden> wrote:
> On Mon, May 28, 2012 at 10:13:28AM -0300, Luiz Capitulino wrote:
> > On Mon, 28 May 2012 12:08:13 +0300
> > Alon Levy <address@hidden> wrote:
> >
> > > On Fri, May 25, 2012 at 11:43:11AM -0300, Luiz Capitulino wrote:
> > > > On Thu, 24 May 2012 19:22:52 +0300
> > > > Alon Levy <address@hidden> wrote:
> > > >
> > > > > For all devices print id, mode and guest_bug status.
> > > >
> > > > Is qxl really tied to spice? In the meaning that it's impossible to use
> > > > it
> > > > without spice? Wouldn't it be better to have 'info display' instead?
> > >
> > > Would a patch implementing 'info display' require me to implement it for
> > > all displays?
> >
> > Would be nice, even if we start with very basic information.
>
> How would that work? I have QXLInfo that only makes sense when the
> information is about a qxl device. Can't have opaque data in a QMP
> response, so would this be a "info display qxl" "info display cirrus"
> etc. or "info qxl"?
You could show what's available and/or being used currently.
To be honest, I'm not very familiar with any of those drivers and don't
know if/how management would use that information. I suggested 'info display'
because it seemed more natural to me (vs. having this info tied to 'info
spice').
- [Qemu-devel] [PATCH] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/24
- Re: [Qemu-devel] [PATCH] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/24
- [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/24
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Luiz Capitulino, 2012/05/25
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/28
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Luiz Capitulino, 2012/05/28
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/28
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info,
Luiz Capitulino <=
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Gerd Hoffmann, 2012/05/29
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/29
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Luiz Capitulino, 2012/05/29
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/29
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Luiz Capitulino, 2012/05/29
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Alon Levy, 2012/05/30
- Re: [Qemu-devel] [PATCH v2] hmp/qxl: info spice: add qxl info, Luiz Capitulino, 2012/05/30