public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
To: jyoung5-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	kvm-ppc-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: [PATCH] Enhnace kvm-userspace configure script for powerpc
Date: Tue, 08 Jan 2008 16:19:03 -0600	[thread overview]
Message-ID: <4783F6D7.1040508@us.ibm.com> (raw)
In-Reply-To: <1199830415.12840.1.camel@thinkpad>

Jerone Young wrote:
> My upstream must be out of date. But the way upstream seems really
> awful. If I mistype any valid option then I don't even get an error.
> It's passed as a qemu option. This does not seem to be a great idea,
> causes more headaches.
>   

Sure, and QEMU will complain about it.

Regards,

Anthony Liguori

>
> On Tue, 2008-01-08 at 16:03 -0600, Anthony Liguori wrote:
>   
>> Jerone Young wrote:
>>     
>>> # HG changeset patch
>>> # User Jerone Young <jyoung5-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
>>> # Date 1199829366 21600
>>> # Node ID 5d3976c5b77e42ad1f63d02271a7ff2086cabb6c
>>> # Parent  bff590d13f32071b50d79f511424821847ed5ba7
>>> Enhnace kvm-userspace configure script for powerpc
>>>
>>>
>>> This patch adds more options to compile qemu with and
>>> places a conditional for powerpc to compile qemu.
>>>
>>> Signed-off-by: Jerone Young <jyoung5-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
>>>
>>> diff --git a/configure b/configure
>>> --- a/configure
>>> +++ b/configure
>>> @@ -8,7 +8,10 @@ qemu_ldflags=
>>>  qemu_ldflags=
>>>  enable_alsa=
>>>  disable_vnc_tls=
>>> +disable_sdl=
>>> +disable_gfx_check=
>>>  disable_gcc_check=
>>> +enable_qemu_static=
>>>  cross_prefix=
>>>  arch=`uname -m`
>>>  target_exec=
>>> @@ -28,8 +31,11 @@ usage() {
>>>  	    --qemu-ldflags=LDFLAGS LDFLAGS to add to qemu configuration
>>>  	    --enable-alsa          enable alsa support for qemu
>>>  	    --disable-vnc-tls      disable vnc tls support for qemu
>>> +	    --disable-sdl          disable sdl support for qemu
>>> +	    --disable-gfx-check    disable graphics check for qemu
>>>  	    --disable-gcc-check    don't insist on gcc-3.x
>>>  	                           CAUTION: this will break running without kvm
>>> +	    --qemu-static          compile qemu to be a static executable
>>>  EOF
>>>      exit 1
>>>  }
>>> @@ -66,8 +72,17 @@ while [[ "$1" = -* ]]; do
>>>  	--disable-vnc-tls)
>>>  	    disable_vnc_tls=1
>>>  	    ;;
>>> +	--disable-sdl)
>>> +	    disable_sdl=1
>>> +	    ;;
>>> +	--disable-gfx-check)
>>> +	    disable_gfx_check=1
>>> +	    ;;
>>>  	--disable-gcc-check)
>>>  	    disable_gcc_check=1
>>> +	    ;;
>>>   
>>>       
>> I believe that unknown options are passed directly to QEMU so I don't 
>> think this is strictly necessary.  It should Just Work already.
>>
>> Regards,
>>
>> Anthony Liguori
>>
>>     
>>> +	--qemu-static)
>>> +	    enable_qemu_static=1
>>>  	    ;;
>>>  	--arch)
>>>  	    arch="$arg"
>>> @@ -106,6 +121,10 @@ if [ "$arch" = "ia64" ]; then
>>>      target_exec="ia64-softmmu"
>>>  fi
>>>  
>>> +if [ "$arch" = "powerpc" ]; then
>>> +    target_exec="ppcemb-softmmu"
>>> +fi
>>> +
>>>  #configure user dir
>>>  (cd user; ./configure --prefix="$prefix" --kerneldir="$libkvm_kerneldir" \
>>>            --arch="$arch" \
>>> @@ -120,8 +139,11 @@ fi
>>>      ${enable_alsa:+"--enable-alsa"} \
>>>      ${disable_vnc_tls:+"--disable-vnc-tls"} \
>>>      ${disable_gcc_check:+"--disable-gcc-check"} \
>>> +    ${disable_sdl:+"--disable-sdl"} \
>>> +    ${disable_gfx_check:+"--disable-gfx-check"} \
>>>      --prefix="$prefix" \
>>>      ${qemu_cc:+"--cc=$qemu_cc"} \
>>> +    ${enable_qemu_static:+"--static"} \
>>>      ${cross_prefix:+"--cross-prefix=$cross_prefix"} \
>>>      ${cross_prefix:+"--cpu=$arch"}
>>>  )
>>>
>>> -------------------------------------------------------------------------
>>> Check out the new SourceForge.net Marketplace.
>>> It's the best place to buy or sell services for
>>> just about anything Open Source.
>>> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
>>> _______________________________________________
>>> kvm-devel mailing list
>>> kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
>>> https://lists.sourceforge.net/lists/listinfo/kvm-devel
>>>   
>>>       
>
>
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> _______________________________________________
> kvm-devel mailing list
> kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> https://lists.sourceforge.net/lists/listinfo/kvm-devel
>   


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace

      reply	other threads:[~2008-01-08 22:19 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-08 22:00 [PATCH] Enhnace kvm-userspace configure script for powerpc Jerone Young
2008-01-08 22:03 ` Anthony Liguori
     [not found]   ` <4783F33D.30506-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
2008-01-08 22:13     ` Jerone Young
2008-01-08 22:19       ` Anthony Liguori [this message]

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=4783F6D7.1040508@us.ibm.com \
    --to=aliguori-r/jw6+rmf7hqt0dzr+alfa@public.gmane.org \
    --cc=jyoung5-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=kvm-ppc-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox