Re: [Xen-devel] [libvirt bisection] complete build-amd64-libvirt

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2014-07-11 at 09:23 +0200, Olaf Hering wrote:
> On Fri, Jul 11, xen.org wrote:
> 
> >   commit 4e7e5e9f7b29ef1c6924116e045a9b2c4bd3ea1a
> >   Author: Olaf Hering <olaf@xxxxxxxxx>
> >   Date:   Mon Jul 7 17:05:17 2014 +0200
> >   
> >       libxl: add discard support to libxl_device_disk
> 
> This used to work for me. What has changed?
> 
> libxl/libxl_conf.c: In function 'libxlDiskSetDiscard':
> libxl/libxl_conf.c:724:19: error: conversion to incomplete type
> make[3]: *** [libxl/libvirt_driver_libxl_impl_la-libxl_conf.lo] Error 1
> 
>  718 static int
>  719 libxlDiskSetDiscard(libxl_device_disk *x_disk, int discard)
>  720 {
>  721     if (!x_disk->readwrite)
>  722         return 0;
>  723 #if defined(LIBXL_HAVE_LIBXL_DEVICE_DISK_DISCARD_ENABLE)
>  724     switch ((enum virDomainDiskDiscard)discard) {

The enum is declared as 
typedef enum {
        ....
} virDomainDiskDiscard;

So using the enum tag in the cast is wrong.

commit 5a2bd4c9171d65a7d2ce836bff5047bccddf10f0
Author: Julio Faracco <jcfaracco@xxxxxxxxx>
Date:   Sat May 31 21:22:30 2014 -0300

    conf: more enum cleanups in "src/conf/domain_conf.h"
    
looks to have converted everything like this.

-----8<-------------------

From 2702cbbffd40384571b39bb6da21d76fc7b8ccd6 Mon Sep 17 00:00:00 2001
From: Ian Campbell <ian.campbell@xxxxxxxxxx>
Date: Fri, 11 Jul 2014 14:38:30 +0100
Subject: [PATCH] libxl: Correct cast to virDomainDiskDiscard enum.

This was converted to a typedef in 5a2bd4c9171d "conf: more enum
cleanups in "src/conf/domain_conf.h"" causing:
    libxl/libxl_conf.c: In function 'libxlDiskSetDiscard':
    libxl/libxl_conf.c:724:19: error: conversion to incomplete type

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 src/libxl/libxl_conf.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c
index 0b4a0b5..f620d47 100644
--- a/src/libxl/libxl_conf.c
+++ b/src/libxl/libxl_conf.c
@@ -721,7 +721,7 @@ libxlDiskSetDiscard(libxl_device_disk *x_disk, int discard)
     if (!x_disk->readwrite)
         return 0;
 #if defined(LIBXL_HAVE_LIBXL_DEVICE_DISK_DISCARD_ENABLE)
-    switch ((enum virDomainDiskDiscard)discard) {
+    switch ((virDomainDiskDiscard)discard) {
     case VIR_DOMAIN_DISK_DISCARD_DEFAULT:
     case VIR_DOMAIN_DISK_DISCARD_LAST:
         break;
-- 
1.7.10.4




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel