All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Egger <Christoph.Egger@amd.com>
To: qemu-devel@nongnu.org
Cc: quintela@redhat.com
Subject: Re: [Qemu-devel] [PATCH 11/19] configure xen in a single place
Date: Thu, 2 Jul 2009 17:24:19 +0200	[thread overview]
Message-ID: <200907021724.19681.Christoph.Egger@amd.com> (raw)
In-Reply-To: <9ff404766d8d5b7411bb7f154646c98d51c573a1.1246545655.git.quintela@redhat.com>

On Thursday 02 July 2009 16:46:35 quintela@redhat.com wrote:
> From: Juan Quintela <quintela@redhat.com>
>
>
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> ---
>  configure |   17 +++++++----------
>  1 files changed, 7 insertions(+), 10 deletions(-)
>
> diff --git a/configure b/configure
> index 5b35b69..56e4d38 100755
> --- a/configure
> +++ b/configure
> @@ -1991,11 +1991,6 @@ case "$target_cpu" in
>        echo "KVM_CFLAGS=$kvm_cflags" >> $config_mak
>        echo "#define CONFIG_KVM 1" >> $config_h
>      fi
> -    if test "$xen" = "yes" -a "$target_softmmu" = "yes";
> -    then
> -      echo "CONFIG_XEN=y" >> $config_mak
> -      echo "#define CONFIG_XEN 1" >> $config_h
> -    fi
>      target_phys_bits=32
>    ;;
>    x86_64)
> @@ -2014,11 +2009,6 @@ case "$target_cpu" in
>        echo "KVM_CFLAGS=$kvm_cflags" >> $config_mak
>        echo "#define CONFIG_KVM 1" >> $config_h
>      fi
> -    if test "$xen" = "yes" -a "$target_softmmu" = "yes"
> -    then
> -      echo "CONFIG_XEN=y" >> $config_mak
> -      echo "#define CONFIG_XEN 1" >> $config_h
> -    fi
>      target_phys_bits=64
>    ;;
>    alpha)
> @@ -2174,6 +2164,13 @@ esac
>  if [ $target_phys_bits -lt $hostlongbits ] ; then
>    target_phys_bits=$hostlongbits
>  fi
> +case "$target_cpu" in
> +  i386|x86_64)

If 'target_cpu' is set by 'uname -m' then this has to be

+  i386|x86_64|amd64)

or you will break *BSD, otherwise.

> +    if test "$xen" = "yes" -a "$target_softmmu" = "yes" ; then
> +      echo "CONFIG_XEN=y" >> $config_mak
> +      echo "#define CONFIG_XEN 1" >> $config_h
> +    fi
> +esac
>  echo "HWLIB=../libhw$target_phys_bits/libqemuhw$target_phys_bits.a" >>
> $config_mak echo "#define TARGET_PHYS_ADDR_BITS $target_phys_bits" >>
> $config_h echo "subdir-$target: subdir-libhw$target_phys_bits" >>
> $config_host_mak



-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Thomas M. McCoy, Giuliano Meroni
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632

  reply	other threads:[~2009-07-02 15:24 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-02 14:46 [Qemu-devel] [PATCH 00/19] More Build System cleanups (v2) quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 01/19] fix XEN Build quintela
2009-07-02 15:14   ` Anthony Liguori
2009-07-02 15:48     ` Avi Kivity
2009-07-02 17:12     ` [Qemu-devel] " Juan Quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 02/19] fix sparc not solaris build quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 03/19] Remove duplicated definition quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 04/19] use block-nested-y for files inside block/ quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 05/19] use nwfpe-obj-y for consistence quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 06/19] Remove unused Makefile variable quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 07/19] already defined several lines before in block-obj-y quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 08/19] configure already knows what TARGET_BASE_ARCH we need, no need to put the logic in Makefile.target quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 09/19] TARGET_ARCH2 is already known at configure time and it is called target_cpu Remove re-construction " quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 10/19] chang ifdefs by obj- quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 11/19] configure xen in a single place quintela
2009-07-02 15:24   ` Christoph Egger [this message]
2009-07-02 15:48     ` Paul Brook
2009-07-02 17:06     ` [Qemu-devel] " Juan Quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 12/19] configure kvm " quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 13/19] configure kqemu " quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 14/19] BASE_LDFLAGS is not used anywhere else quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 15/19] TARGET_ARCH == TARGET_BASE_ARCH in m68k and arm quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 16/19] Refactor a single rule to generate all qemu-* executables quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 17/19] qemu binary don't exist anymore. I guess not much people is compiling on alpha quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 18/19] We can wrap OS_CFLAGS/OS_LDFLAGS in CFLAGS/LDFLAGS at configure time quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 19/19] We can wrap ARCH_CFLAGS/ARCH_LDFLAGS " quintela

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=200907021724.19681.Christoph.Egger@amd.com \
    --to=christoph.egger@amd.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.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.