[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5.1 1/8] xen: link against xentoolcore
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH v5.1 1/8] xen: link against xentoolcore |
Date: |
Thu, 26 Oct 2017 15:41:18 -0700 (PDT) |
User-agent: |
Alpine 2.10 (DEB 1266 2009-07-14) |
On Fri, 20 Oct 2017, Ian Jackson wrote:
> From: Anthony PERARD <address@hidden>
>
> Xen libraries in 4.10 include a new xentoolcore library. This
> contains the xentoolcore_restrict_all function which we are about to
> want to use.
>
> Signed-off-by: Ian Jackson <address@hidden>
> ---
> v5: More truthful commit message.
> ---
> configure | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/configure b/configure
> index fd7e3a5..6f691df 100755
> --- a/configure
> +++ b/configure
> @@ -2072,7 +2072,7 @@ if test "$xen" != "no" ; then
> $($pkg_config --modversion xencontrol | sed 's/\./ /g') )"
> xen=yes
> xen_pc="xencontrol xenstore xenguest xenforeignmemory xengnttab"
> - xen_pc="$xen_pc xenevtchn xendevicemodel"
> + xen_pc="$xen_pc xenevtchn xendevicemodel xentoolcore"
> QEMU_CFLAGS="$QEMU_CFLAGS $($pkg_config --cflags $xen_pc)"
> libs_softmmu="$($pkg_config --libs $xen_pc) $libs_softmmu"
> LDFLAGS="$($pkg_config --libs $xen_pc) $LDFLAGS"
> @@ -2104,18 +2104,20 @@ EOF
> cat > $TMPC <<EOF &&
> #undef XC_WANT_COMPAT_MAP_FOREIGN_API
> #include <xenforeignmemory.h>
> +#include <xentoolcore.h>
> int main(void) {
> xenforeignmemory_handle *xfmem;
>
> xfmem = xenforeignmemory_open(0, 0);
> xenforeignmemory_map2(xfmem, 0, 0, 0, 0, 0, 0, 0);
> + xentoolcore_restrict_all(0);
>
> return 0;
> }
> EOF
> - compile_prog "" "$xen_libs -lxendevicemodel $xen_stable_libs"
> + compile_prog "" "$xen_libs -lxendevicemodel $xen_stable_libs
> -lxentoolcore"
> then
> - xen_stable_libs="-lxendevicemodel $xen_stable_libs"
> + xen_stable_libs="-lxendevicemodel $xen_stable_libs -lxentoolcore"
Is it on purpose that -lxentoolcore is at the end of this string rather
than before $xen_stable_libs?
In any case
Acked-by: Stefano Stabellini <address@hidden>
> xen_ctrl_version=41000
> xen=yes
> elif
> --
> 2.1.4
>
- [Qemu-devel] [PATCH v5.1 4/8] xen: destroy_hvm_domain: Move reason into a variable, (continued)
- [Qemu-devel] [PATCH v5.1 2/8] xen: restrict: use xentoolcore_restrict_all, Ian Jackson, 2017/10/20
- Re: [Qemu-devel] [PATCH v5.1 2/8] xen: restrict: use xentoolcore_restrict_all, Stefano Stabellini, 2017/10/26
- [Qemu-devel] [PATCH v5.1 7/8] os-posix: Provide new -runas <uid>:<gid> facility, Ian Jackson, 2017/10/20
- Re: [Qemu-devel] [PATCH v5.1 7/8] os-posix: Provide new -runas <uid>:<gid> facility, Anthony PERARD, 2017/10/24
- Re: [Qemu-devel] [PATCH v5.1 7/8] os-posix: Provide new -runas <uid>:<gid> facility, Ian Jackson, 2017/10/24
- Re: [Qemu-devel] [PATCH v5.1 7/8] os-posix: Provide new -runas <uid>:<gid> facility, Stefano Stabellini, 2017/10/26
- [Qemu-devel] [PATCH v5.1 5/8] xen: move xc_interface compatibility fallback further up the file, Ian Jackson, 2017/10/20
- Re: [Qemu-devel] [PATCH v5.1 5/8] xen: move xc_interface compatibility fallback further up the file, Stefano Stabellini, 2017/10/26
- Re: [Qemu-devel] [PATCH v5.1 1/8] xen: link against xentoolcore,
Stefano Stabellini <=
- Re: [Qemu-devel] [PATCH v5.1 1/8] xen: link against xentoolcore, Ian Jackson, 2017/10/27
Re: [Qemu-devel] [PATCH v5 0/8] xen: xen-domid-restrict improvements, no-reply, 2017/10/21