[libvirt] [PATCH v8 1/8] parallels: add driver skeleton

Daniel P. Berrange berrange at redhat.com
Wed Jul 18 19:15:18 UTC 2012


On Wed, Jul 04, 2012 at 09:42:51PM +0400, Dmitry Guryanov wrote:
> Parallels Virtuozzo Server is a cloud-ready virtualization
> solution that allows users to simultaneously run multiple virtual
> machines and containers on the same physical server.
> 
> Current name of this product is Parallels Server Bare Metal and
> more information about it can be found here -
> http://www.parallels.com/products/server/baremetal/sp/.
> 
> This first patch adds driver, which can report node info only.
> 
> Signed-off-by: Dmitry Guryanov <dguryanov at parallels.com>
> ---
> changes in v8:
>   * removed patch for file mingw32-libvirt.spec.in
>   * version change to 0.10.0

> diff --git a/src/parallels/parallels_driver.h b/src/parallels/parallels_driver.h
> new file mode 100644
> index 0000000..c04db35
> --- /dev/null
> +++ b/src/parallels/parallels_driver.h

> +# define parallelsError(code, ...)                                         \
> +        virReportErrorHelper(VIR_FROM_TEST, code, __FILE__,         \
> +                             __FUNCTION__, __LINE__, __VA_ARGS__)

As of latest GIT today, you shouldn't define this any
more. Just call 'virReportError' throughout your
code instead of 'parallelsError'. The new generic macro
does exactly the same as what you've done here, assuming
VIR_FROM_THIS is set currectly in each source file.

Regards,
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 :|




More information about the libvir-list mailing list