All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tamas Lengyel <tamas.lengyel@zentific.com>
To: Tim Deegan <tim@xen.org>
Cc: "Tian, Kevin" <kevin.tian@intel.com>,
	"wei.liu2@citrix.com" <wei.liu2@citrix.com>,
	Ian Campbell <ian.campbell@citrix.com>,
	Razvan Cojocaru <rcojocaru@bitdefender.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Jun Nakajima <jun.nakajima@intel.com>,
	"Dong, Eddie" <eddie.dong@intel.com>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	Steven Maresca <steve@zentific.com>,
	Andres Lagar-Cavilla <andres@lagarcavilla.org>,
	Jan Beulich <JBeulich@suse.com>,
	"rshriram@cs.ubc.ca" <rshriram@cs.ubc.ca>,
	Keir Fraser <keir@xen.org>,
	Daniel De Graaf <dgdegra@tycho.nsa.gov>,
	"yanghy@cn.fujitsu.com" <yanghy@cn.fujitsu.com>
Subject: Re: [PATCH V7 07/12] xen: Introduce monitor_op domctl
Date: Thu, 26 Mar 2015 12:47:07 +0100	[thread overview]
Message-ID: <CABfawh=riOZy=KcWZGm=Sj41bgBGQ5k-+hg2=rFrmmxv4opSDg@mail.gmail.com> (raw)
In-Reply-To: <20150326114418.GF6519@deinos.phlegethon.org>


[-- Attachment #1.1: Type: text/plain, Size: 6505 bytes --]

On Thu, Mar 26, 2015 at 12:44 PM, Tim Deegan <tim@xen.org> wrote:

> At 11:13 +0000 on 26 Mar (1427368409), Tim Deegan wrote:
> > At 12:08 +0100 on 26 Mar (1427371713), Tamas K Lengyel wrote:
> > > Thanks, I'll be sending v8 of the series today. Are there any plans to
> > > merge the early parts of the series that have been acked while these
> > > patches at the end receive final touch-ups?
> >
> > Yes, I'm going to apply 0-6 now.
>
> ...by which I meant of course 1-6. :)
>
> Unfortunately, with those applied I get a build failure in the stubdom
> build:
>
> gcc -isystem
> /local/scratch/tdeegan/xen.git/stubdom/../extras/mini-os/include
> -D__MINIOS__ -DHAVE_LIBC -isystem
> /local/scratch/tdeegan/xen.git/stubdom/../extras/mini-os/include/posix
> -isystem /local/scratch/tdeegan/xen.git/stubdom/../tools/xenstore/include
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/../extras/mini-os/include/x86
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/../extras/mini-os/include/x86/x86_32
> -U __linux__ -U __FreeBSD__ -U __sun__ -nostdinc -isystem
> /local/scratch/tdeegan/xen.git/stubdom/../extras/mini-os/include/posix
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/cross-root-i686/i686-xen-elf/include
> -isystem /usr/lib/gcc/x86_64-linux-gnu/4.9/include -isystem
> /local/scratch/tdeegan/xen.git/stubdom/lwip-x86_32/src/include -isystem
> /local/scratch/tdeegan/xen.git/stubdom/lwip-x86_32/src/include/ipv4
> -I/local/scratch/tdeegan/xen.git/stubdom/include
> -I/local/scratch/tdeegan/xen.git/stubdom/../xen/include -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include
> -D__MINIOS__ -DHAVE_LIBC -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include/posix
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../tools/xenstore/include
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include/x86
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include/x86/x86_32
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include
> -D__MINIOS__ -DHAVE_LIBC -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include/posix
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../tools/xenstore/include
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include/x86
> -isystem
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../extras/mini-os/include/x86/x86_32
> -O1 -fno-omit-frame-pointer  -m32 -march=i686 -m32 -march=i686 -g
> -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes
> -Wdeclaration-after-statement -Wno-unused-but-set-variable
> -Wno-unused-local-typedefs   -fno-stack-protector -fno-exceptions -O1
> -fno-omit-frame-pointer  -m32 -march=i686 -m32 -march=i686 -g
> -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes
> -Wdeclaration-after-statement -Wno-unused-but-set-variable
> -Wno-unused-local-typedefs   -O0 -g3 -D__XEN_TOOLS__ -MMD -MF .build.d
> -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls
> -mno-tls-direct-seg-refs  -I../../xen/common/libelf -Werror
> -Wmissing-prototypes -I. -I./include
> -I/local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../tools/include
> -pthread -O1 -fno-omit-frame-pointer  -m32 -march=i686 -m32 -march=i686 -g
> -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes
> -Wdeclaration-after-statement -Wno-unused-but-set-variable
> -Wno-unused-local-typedefs   -O0 -g3 -D__XEN_TOOLS__ -MMD -MF
> .xc_mem_paging.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
> -fno-optimize-sibling-calls -mno-tls-direct-seg-refs
> -I../../xen/common/libelf -Werror -Wmissing-prototypes -I. -I./include
> -I/local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../tools/include
> -pthread -include
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../tools/config.h
> -c -o xc_mem_paging.o xc_mem_paging.c
> xc_mem_paging.c:64:5: error: conflicting types for 'xc_mem_paging_nominate'
>  int xc_mem_paging_nominate(xc_interface *xch, domid_t domain_id, uint64_t
> gfn)
>      ^
> In file included from xc_private.h:33:0,
>                  from xc_mem_paging.c:24:
> xenctrl.h:2272:5: note: previous declaration of 'xc_mem_paging_nominate'
> was here
>  int xc_mem_paging_nominate(xc_interface *xch, domid_t domain_id,
>      ^
> xc_mem_paging.c:71:5: error: conflicting types for 'xc_mem_paging_evict'
>  int xc_mem_paging_evict(xc_interface *xch, domid_t domain_id, uint64_t
> gfn)
>      ^
> In file included from xc_private.h:33:0,
>                  from xc_mem_paging.c:24:
> xenctrl.h:2274:5: note: previous declaration of 'xc_mem_paging_evict' was
> here
>  int xc_mem_paging_evict(xc_interface *xch, domid_t domain_id, unsigned
> long gfn);
>      ^
> xc_mem_paging.c:78:5: error: conflicting types for 'xc_mem_paging_prep'
>  int xc_mem_paging_prep(xc_interface *xch, domid_t domain_id, uint64_t gfn)
>      ^
> In file included from xc_private.h:33:0,
>                  from xc_mem_paging.c:24:
> xenctrl.h:2275:5: note: previous declaration of 'xc_mem_paging_prep' was
> here
>  int xc_mem_paging_prep(xc_interface *xch, domid_t domain_id, unsigned
> long gfn);
>      ^
> xc_mem_paging.c:85:5: error: conflicting types for 'xc_mem_paging_load'
>  int xc_mem_paging_load(xc_interface *xch, domid_t domain_id,
>      ^
> In file included from xc_private.h:33:0,
>                  from xc_mem_paging.c:24:
> xenctrl.h:2276:5: note: previous declaration of 'xc_mem_paging_load' was
> here
>  int xc_mem_paging_load(xc_interface *xch, domid_t domain_id,
>      ^
> /local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32/../../tools/Rules.mk:107:
> recipe for target 'xc_mem_paging.o' failed
> make[3]: *** [xc_mem_paging.o] Error 1
> make[3]: Leaving directory
> '/local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32'
> Makefile:146: recipe for target 'build' failed
> make[2]: *** [build] Error 2
> make[2]: Leaving directory
> '/local/scratch/tdeegan/xen.git/stubdom/libxc-x86_32'
> Makefile:344: recipe for target 'libxc-x86_32/libxenctrl.a' failed
> make[1]: *** [libxc-x86_32/libxenctrl.a] Error 2
> make[1]: Leaving directory '/local/scratch/tdeegan/xen.git/stubdom'
> Makefile:104: recipe for target 'install-stubdom' failed
> make: *** [install-stubdom] Error 2
>

Ack, the stubdom build has not been in my build script so this side all
went unnoticed thus far. Looking into it now.

Thanks,
Tamas

[-- Attachment #1.2: Type: text/html, Size: 7174 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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

  reply	other threads:[~2015-03-26 11:47 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-12 17:58 [PATCH V7 00/12] xen: Clean-up of mem_event subsystem Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 01/12] xen/mem_event: Cleanup of mem_event structures Tamas K Lengyel
2015-03-26 10:27   ` Jan Beulich
2015-03-12 17:58 ` [PATCH V7 02/12] xen/mem_event: Cleanup mem_event names in rings, functions and domctls Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 03/12] xen/mem_paging: Convert mem_event_op to mem_paging_op and cleanup Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 04/12] xen: Rename mem_event to vm_event Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 05/12] tools/tests: Clean-up tools/tests/xen-access Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 06/12] x86/hvm: factor out and rename vm_event related functions Tamas K Lengyel
2015-03-26 10:31   ` Jan Beulich
2015-03-12 17:58 ` [PATCH V7 07/12] xen: Introduce monitor_op domctl Tamas K Lengyel
2015-03-26 10:50   ` Jan Beulich
2015-03-26 11:08     ` Tamas K Lengyel
2015-03-26 11:13       ` Tim Deegan
2015-03-26 11:44         ` Tim Deegan
2015-03-26 11:47           ` Tamas Lengyel [this message]
2015-03-26 15:01           ` Tamas K Lengyel
2015-03-26 12:30     ` Tamas K Lengyel
2015-03-26 12:55       ` Jan Beulich
2015-03-26 13:51         ` Tamas Lengyel
2015-03-12 17:58 ` [PATCH V7 08/12] xen/vm_event: Deprecate VM_EVENT_FLAG_DUMMY flag Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 09/12] xen/vm_event: Decouple vm_event and mem_access Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 10/12] xen/vm_event: Relocate memop checks Tamas K Lengyel
2015-03-19 10:18   ` Tim Deegan
2015-03-12 17:58 ` [PATCH V7 11/12] xen/xsm: Split vm_event_op into three separate labels Tamas K Lengyel
2015-03-12 17:58 ` [PATCH V7 12/12] xen/vm_event: Add RESUME option to vm_event_op domctl Tamas K Lengyel
2015-03-26 10:56   ` Jan Beulich
2015-03-26 11:29     ` Tamas K Lengyel
2015-03-26 11:45       ` Jan Beulich
2015-03-26 11:48         ` Andrew Cooper
2015-03-26 12:53           ` Jan Beulich
2015-03-26 11:51         ` Tim Deegan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CABfawh=riOZy=KcWZGm=Sj41bgBGQ5k-+hg2=rFrmmxv4opSDg@mail.gmail.com' \
    --to=tamas.lengyel@zentific.com \
    --cc=JBeulich@suse.com \
    --cc=andres@lagarcavilla.org \
    --cc=dgdegra@tycho.nsa.gov \
    --cc=eddie.dong@intel.com \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jun.nakajima@intel.com \
    --cc=keir@xen.org \
    --cc=kevin.tian@intel.com \
    --cc=rcojocaru@bitdefender.com \
    --cc=rshriram@cs.ubc.ca \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=steve@zentific.com \
    --cc=tim@xen.org \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    --cc=yanghy@cn.fujitsu.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.