[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]