On Thu, 26 Jul 2012 18:59:30 +0200 Paolo Bonzini <pbonz...@redhat.com> wrote:
> Il 26/07/2012 18:49, Luiz Capitulino ha scritto: > > On Thu, 26 Jul 2012 17:41:01 +0200 > > Paolo Bonzini <pbonz...@redhat.com> wrote: > > > >> Il 26/07/2012 17:26, Kevin Wolf ha scritto: > >>>> The DeviceNotActive error is not a particularly good match, add > >>>>> a separate one. > >>>>> > >>>>> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > >>> Luiz, what do you think about this one? It seems to contradict the idea > >>> of having only few error classes and free form error descriptions. > >> > >> I agree, but that's what we have to live with for now... > > > > Why don't you add QERR_INVALID_STATE then? > > Because I do want a meaningful error message. It's fine with me adding QERR_BLOCK_JOB_NOT_ACTIVE then, as you've said, it's what we have today.
