[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState |
Date: |
Tue, 30 Oct 2018 15:00:30 +0100 |
User-agent: |
NeoMutt/20180716 |
On Tue, Oct 30, 2018 at 02:32:40PM +0100, Philippe Mathieu-Daudé wrote:
> Hi Gerd,
>
> On 30/10/18 12:13, Gerd Hoffmann wrote:
> > Indicates support state for somerhing (device, backend, subsystem, ...)
>
> "something"
Oops, I'll fix.
> > +##
> > +# @SupportState:
> > +#
> > +# Indicate Support level of qemu devices, backends, subsystems, ...
> > +#
> > +# Since: 3.2
> > +##
> > +{ 'enum': 'SupportState',
> > + 'data': [ 'unknown',
>
> 'unknown' is scary and should be fixed.
'unknown' maps to "0" due to being first in list, so this is what you
get when it isn't explicitly set to something else. Which make sense
IMHO.
> > + 'supported',
> > + 'maintained',
> > + 'odd-fixes',
>
> All those fit in 'supported'
>
> > + 'orphan',
> > + 'obsolete',
> > + 'deprecated' ] }
>
> And all those should appear as 'deprecated' IMHO.
See minutes on deprecation discussion. Seems there is agreement we
need something more finegrained than "supported" and "deprecated".
cheers,
Gerd
- [Qemu-ppc] [PATCH 0/4] Introducing QemuSupportState, Gerd Hoffmann, 2018/10/30
- [Qemu-ppc] [PATCH 2/4] add QemuSupportState to DeviceClass, Gerd Hoffmann, 2018/10/30
- [Qemu-ppc] [PATCH 4/4] switch machine types to QemuSupportState, Gerd Hoffmann, 2018/10/30
- [Qemu-ppc] [PATCH 1/4] add QemuSupportState, Gerd Hoffmann, 2018/10/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Philippe Mathieu-Daudé, 2018/10/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState,
Gerd Hoffmann <=
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Philippe Mathieu-Daudé, 2018/10/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Cornelia Huck, 2018/10/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Philippe Mathieu-Daudé, 2018/10/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Eduardo Habkost, 2018/10/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Cornelia Huck, 2018/10/31
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, John Snow, 2018/10/31
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Eduardo Habkost, 2018/10/31
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, John Snow, 2018/10/31
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, Eduardo Habkost, 2018/10/31
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/4] add QemuSupportState, John Snow, 2018/10/31