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

Re: [Xen-devel] [PATCH 3/8] libxc: allow xc_get/set_hvm_param() to get/set 64-bit values



On Thu, 2014-06-12 at 12:05 +0100, David Vrabel wrote:
> On 12/06/14 11:01, Ian Campbell wrote:
> > 
> > On Tue, 2014-06-10 at 19:07 +0100, David Vrabel wrote:
> >> HVMOP_get_param and HVMOP_set_param take a uint32_t for the parameter
> >> index and a uint64_t for the value.  So, make the corresponding libxc
> >> function take the same types.
> >>
> >> Signed-off-by: David Vrabel <david.vrabel@xxxxxxxxxx>
> > 
> > Removes a pleasing number of casts!
> > 
> > I was about to ack and apply but for a stubdom build:
> >         xc_domain_save.c: In function âxc_domain_saveâ:
> >         xc_domain_save.c:1736:26: error: passing argument 4 of 
> > âxc_get_hvm_paramâ from incompatible pointer type [-Werror]
> >         In file included from xc_private.h:32:0,
> >                          from xc_domain_save.c:30:
> >         xenctrl.h:1812:5: note: expected âuint64_t *â but argument is of 
> > type âlong unsigned int *â
> >         xc_domain_save.c:1748:26: error: passing argument 4 of 
> > âxc_get_hvm_paramâ from incompatible pointer type [-Werror]
> >         In file included from xc_private.h:32:0,
> >                          from xc_domain_save.c:30:
> >         xenctrl.h:1812:5: note: expected âuint64_t *â but argument is of 
> > type âlong unsigned int *â
> >         cc1: all warnings being treated as errors
> >         make[3]: *** [xc_domain_save.o] Error 1
> >         make[3]: *** Waiting for unfinished jobs....
> >         make[3]: *** wait: No child processes.  Stop.
> >         make[2]: *** [build] Error 2
> 
> I think you have a stale xc_domain_save.c (and presumably other files as
> well) in your stubdom build.  The stubdom build works for me.

My pre-commit script include a "git clean -q -f -dx" which I've just
confirmed cleans all of these out for me. I've just run it again with
your patch applied and got the same result. Full log attached.

$ find -name xc_domain_save.c | xargs md5sum
5c20ab040ac5c9d9a9df4467f15e3ad5  ./stubdom/libxc-x86_64/xc_domain_save.c
5c20ab040ac5c9d9a9df4467f15e3ad5  ./stubdom/libxc-x86_32/xc_domain_save.c
5c20ab040ac5c9d9a9df4467f15e3ad5  ./tools/libxc/xc_domain_save.c

That file is also attached.

Ian.

Attachment: COMMITTER.x86_64.LOG
Description: Text Data

Attachment: xc_domain_save.c
Description: Text Data

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.