From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Daniel P. Berrange" Subject: Re: [libvirt] [PATCH 3/7] kvmtool: Add new enums and error codes for the driver Date: Tue, 6 Dec 2011 14:47:51 +0000 Message-ID: <20111206144751.GE7937@redhat.com> References: <1321012626-31713-1-git-send-email-jyang@redhat.com> <1321012626-31713-5-git-send-email-jyang@redhat.com> Reply-To: "Daniel P. Berrange" Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Cc: libvir-list@redhat.com, kvm@vger.kernel.org, penberg@cs.helsinki.fi, levinsasha928@gmail.com, gorcunov@gmail.com, mingo@elte.hu, asias.hejun@gmail.com To: Osier Yang Return-path: Received: from mx1.redhat.com ([209.132.183.28]:5000 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933345Ab1LFOsc (ORCPT ); Tue, 6 Dec 2011 09:48:32 -0500 Content-Disposition: inline In-Reply-To: <1321012626-31713-5-git-send-email-jyang@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, Nov 11, 2011 at 07:57:02PM +0800, Osier Yang wrote: > --- > include/libvirt/virterror.h | 1 + > src/driver.h | 1 + > src/util/virterror.c | 3 +++ > 3 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/include/libvirt/virterror.h b/include/libvirt/virterror.h > index a8549b7..deda42d 100644 > --- a/include/libvirt/virterror.h > +++ b/include/libvirt/virterror.h > @@ -84,6 +84,7 @@ typedef enum { > VIR_FROM_LIBXL = 41, /* Error from libxenlight driver */ > VIR_FROM_LOCKING = 42, /* Error from lock manager */ > VIR_FROM_HYPERV = 43, /* Error from Hyper-V driver */ > + VIR_FROM_KVMTOOL = 44, /* Error from kvm tool driver */ > } virErrorDomain; > > > diff --git a/src/driver.h b/src/driver.h > index 4c14aaa..158a13c 100644 > --- a/src/driver.h > +++ b/src/driver.h > @@ -30,6 +30,7 @@ typedef enum { > VIR_DRV_VMWARE = 13, > VIR_DRV_LIBXL = 14, > VIR_DRV_HYPERV = 15, > + VIR_DRV_KVMTOOL = 16, > } virDrvNo; > > > diff --git a/src/util/virterror.c b/src/util/virterror.c > index 5006fa2..abb5b5a 100644 > --- a/src/util/virterror.c > +++ b/src/util/virterror.c > @@ -175,6 +175,9 @@ static const char *virErrorDomainName(virErrorDomain domain) { > case VIR_FROM_HYPERV: > dom = "Hyper-V "; > break; > + case VIR_FROM_KVMTOOL: > + dom = "KVMTOOL "; > + break; > } > return(dom); > } Trivial, ACK Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|