Re: [PATCH] esx: Move VMX handling code out of the driver directory

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

 



On 12/21/2010 09:03 AM, Matthias Bolte wrote:
> Now the VMware driver doesn't depend on the ESX driver anymore.
> 
> Add a WITH_VMX option that depends on WITH_ESX and WITH_VMWARE.
> Also add a libvirt_vmx.syms file.
> 
> Move some escaping functions from esx_util.c to vmx.c.
> 
> Adapt the test suite, ESX and VMware driver to the new code layout.

> --- a/src/esx/esx_driver.c

> @@ -88,7 +88,7 @@ struct _esxVMX_Data {
>   * Firstly this functions checks if the given file name contains a separator.
>   * If it doesn't then the referenced file is in the same directory as the .vmx
>   * file. The datastore name and directory of the .vmx file are passed to this
> - * function via the opaque paramater by the caller of esxVMX_ParseConfig.
> + * function via the opaque paramater by the caller of virVMXParseConfig.

As long as you're touching this line, s/paramater/parameter/

> +++ b/src/libvirt_vmx.syms
> @@ -0,0 +1,23 @@
> +#
> +# These symbols are dependent upon --with-esx via WITH_ESX or --with-vmware via WITH_VMWARE.
> +#
> +
> +# vmx.h
> +virVMXConvertToUTF8;
> +virVMXEscapeHex;
> +virVMXUnescapeHex;
> +virVMXParseConfig;

Do you want to sort this list?

> diff --git a/src/esx/esx_vmx.c b/src/vmx/vmx.c
> similarity index 79%
> rename from src/esx/esx_vmx.c
> rename to src/vmx/vmx.c
> index 5cbb835..6e3e9af 100644

'git diff' is so awesome with renames :)

> @@ -454,19 +456,23 @@ def->parallels[0]...
>  
>  */
>  
> -#define VIR_FROM_THIS VIR_FROM_ESX
> +#define VIR_FROM_THIS VIR_FROM_NONE

Bit of a shame that we lose information here, but not the end of the world.

ACK with the spelling and sorting nits addressed.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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