WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH 2/6] libxl: portiblity fixes

Christoph Egger writes ("[Xen-devel] [PATCH 2/6] libxl: portiblity fixes"):
> --- a/tools/libxl/libxl_osdeps.h      Tue Jul 27 13:48:38 2010 +0200
> +++ b/tools/libxl/libxl_osdeps.h      Tue Jul 27 13:53:12 2010 +0200
> @@ -23,6 +23,8 @@
>  
>  #define _GNU_SOURCE
>  
> +#include <libxl_internal.h>
> +

This is wrong.  libxl_osdeps.h should not include libxl_internal.h.

> +/* libxl_blktap_enabled:
> + *    return true if blktap/blktap2 support is available.
> + */
> +int libxl_blktap_enabled(struct libxl_ctx *ctx);

This is not what libxl_osdeps.h is for.  These kind of functions can
be declared in a new section in libxl_internal.h.

Also, you should divide your patches conceptually, rather than
according to which files they touch.

This patch is wrong because it introduces a couple of function
declarations but it does not introduce the definitions; your later
patch which introduces the definitions is wrong because it introduces
some functions which are intended to replace existing code, but the
patch does not replace the existing code and the new functions are not
called anywhere in that patch.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>