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

Re: [Xen-devel] xen-unstable build fails


  • To: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
  • From: Shriram R <rshriram@xxxxxxxxx>
  • Date: Tue, 4 May 2010 23:31:20 -0700
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Tue, 04 May 2010 23:32:10 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=Ics2n3gzhNElhsebru0UJ2MhM6LIoxsgrVdoGCafgNEsK85TSIrd7VbJbxiBZidFbP AC93PIAGi5dInvqaw64m5s1LAAXtH0ZMMO1Ep2vWQnRG0CpBT2e1fv189Blu5l45aitS WDZPThlE408LJG9t1o/4udNL6qXviZPZ1i8uo=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

I thing Yang's patch is still not applied. And the build error is my mistake.. I missed the xen-hptool.c file

changeset 21259:f7605c6c9548 changes the global suspend event channel lock file
to a per-domain lock file, but do not update xc_suspend_evtchn_release() call in
misc/hp-tool.c. So I got a build error when building tools. The patch fix this.

Signed-off-by: Yang Hongyang <yanghy@xxxxxxxxxxxxxx>

diff -r efa1b905d893 tools/misc/xen-hptool.c
--- a/tools/misc/xen-hptool.c   Tue May 04 13:59:55 2010 +0100
+++ b/tools/misc/xen-hptool.c   Wed May 05 19:56:51 2010 +0800
@@ -133,7 +133,7 @@

 failed:
    if (suspend_evtchn != -1)
-        xc_suspend_evtchn_release(xce, suspend_evtchn);
+        xc_suspend_evtchn_release(xce, domid, suspend_evtchn);

    return -1;
 }
@@ -228,7 +228,7 @@
                                mfn, domid);
                    }
                    xc_domain_resume(xc_fd, domid, 1);
-                    xc_suspend_evtchn_release(xce, suspend_evtchn);
+                    xc_suspend_evtchn_release(xce, domid, suspend_evtchn);
                    xc_evtchn_close(xce);
                }
                break;


On Tue, May 4, 2010 at 10:46 PM, Juergen Gross <juergen.gross@xxxxxxxxxxxxxx> wrote:
Hi,

latest xen-unstable doesn't build on my machine:

gcc  -O1 -fno-omit-frame-pointer -fno-optimize-sibling-calls -m64 -g -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wno-unused-value -Wdeclaration-after-statement  -D__XEN_TOOLS__ -MMD -MF .xen-hptool.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  -Werror -I ../../tools/python/xen/lowlevel/xc -I ../../tools/libxc -I ../../tools/include -c -o xen-hptool.o xen-hptool.c
xen-hptool.c: In function ‘suspend_guest’:
xen-hptool.c:136: error: too few arguments to function ‘xc_suspend_evtchn_release’
xen-hptool.c: In function ‘hp_mem_offline_func’:
xen-hptool.c:231: error: too few arguments to function ‘xc_suspend_evtchn_release’
make[3]: *** [xen-hptool.o] Error 1
make[3]: Leaving directory `/root/xen-unstable.hg/tools/misc'


Juergen

--
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@xxxxxxxxxxxxxx
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

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



--
perception is but an offspring of its own self
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

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