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

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

 



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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]