[patch] do not allocate code_gen buffer on ia64

[patch] do not allocate code_gen buffer on ia64

[Jes Sorensen]

[-- Attachment #1: Type: text/plain, Size: 260 bytes --]

Avi Kivity wrote:
> Jes Sorensen wrote:
>> Hi,
>>
>> This one limits the code_gen_buffer_size on ia64, phys_mem_size/4
>> really gets out of hand when you boot say a 64GB guest.
> 
> ia64 doesn't codegen; why not set it to zero?

How about this one then?

Jes

[-- Attachment #2: 1200-ia64-code-gen-buffer.patch --]
[-- Type: text/plain, Size: 626 bytes --]

Do not allocate a code_gen buffer on ia64 given it doesn't support
code generation.

Signed-off-by: Jes Sorensen <jes@sgi.com>

---
 qemu/exec.c |    4 ++++
 1 file changed, 4 insertions(+)

Index: kvm-userspace.git/qemu/exec.c
===================================================================
--- kvm-userspace.git.orig/qemu/exec.c
+++ kvm-userspace.git/qemu/exec.c
@@ -407,6 +407,10 @@
 
 static void code_gen_alloc(unsigned long tb_size)
 {
+#ifdef TARGET_IA64
+	return;
+#endif
+
 #ifdef USE_STATIC_CODE_GEN_BUFFER
     code_gen_buffer = static_code_gen_buffer;
     code_gen_buffer_size = DEFAULT_CODE_GEN_BUFFER_SIZE;

Re: [patch] do not allocate code_gen buffer on ia64

[Avi Kivity]

Jes Sorensen wrote:
> Avi Kivity wrote:
>> Jes Sorensen wrote:
>>> Hi,
>>>
>>> This one limits the code_gen_buffer_size on ia64, phys_mem_size/4
>>> really gets out of hand when you boot say a 64GB guest.
>>
>> ia64 doesn't codegen; why not set it to zero?
>
> How about this one then?
>

Applied, thanks.  Note qemu uses 4 spaces for intedenation.  Talk to 
your editor.

-- 
error compiling committee.c: too many arguments to function

Re: [patch] do not allocate code_gen buffer on ia64

[Jes Sorensen]

Avi Kivity wrote:
> Applied, thanks.  Note qemu uses 4 spaces for intedenation.  Talk to 
> your editor.

Even when it's a double indentation, ie 8 spaces?

Thats just plain sicko :-(

Jes

Re: [patch] do not allocate code_gen buffer on ia64

[Avi Kivity]

Jes Sorensen wrote:
> Avi Kivity wrote:
>> Applied, thanks.  Note qemu uses 4 spaces for intedenation.  Talk to 
>> your editor.
>
> Even when it's a double indentation, ie 8 spaces?
>

Yes.

> Thats just plain sicko :-(

It's the only scheme that works 100% reliably.

-- 
error compiling committee.c: too many arguments to function

Re: [patch] do not allocate code_gen buffer on ia64

[Jes Sorensen]

Avi Kivity wrote:
> Jes Sorensen wrote:
>> Avi Kivity wrote:
>>> Applied, thanks.  Note qemu uses 4 spaces for intedenation.  Talk to 
>>> your editor.
>>
>> Even when it's a double indentation, ie 8 spaces?
> Yes.

:-(

Well then we really should add something like this to every file in
qemu, since this behavior is so non standard from what any normal editor
does per default.

/* -*- mode: c; c-basic-offset: 4; indent-tabs-mode: nil -*- */

>> Thats just plain sicko :-(
> 
> It's the only scheme that works 100% reliably.

Ehm, maybe, but then that makes 8 space (tab) indention work 110%
reliable :-)

Cheers,
Jes

Re: [patch] do not allocate code_gen buffer on ia64

[Avi Kivity]

Jes Sorensen wrote:
> Avi Kivity wrote:
>> Jes Sorensen wrote:
>>> Avi Kivity wrote:
>>>> Applied, thanks.  Note qemu uses 4 spaces for intedenation.  Talk 
>>>> to your editor.
>>>
>>> Even when it's a double indentation, ie 8 spaces?
>> Yes.
>
> :-(
>
> Well then we really should add something like this to every file in
> qemu, since this behavior is so non standard from what any normal editor
> does per default.
>
> /* -*- mode: c; c-basic-offset: 4; indent-tabs-mode: nil -*- */
>

I taught emacs to do this locally.  It would be great if qemu upstream 
adopts it.

>>> Thats just plain sicko :-(
>>
>> It's the only scheme that works 100% reliably.
>
> Ehm, maybe, but then that makes 8 space (tab) indention work 110%
> reliable :-)

Drunk on Documentation/CodingStyle kool-aid, I see.

-- 
error compiling committee.c: too many arguments to function

Re: [patch] do not allocate code_gen buffer on ia64

[Anthony Liguori]

Avi Kivity wrote:
> Jes Sorensen wrote:
>> Avi Kivity wrote:
>>> Jes Sorensen wrote:
>>>> Hi,
>>>>
>>>> This one limits the code_gen_buffer_size on ia64, phys_mem_size/4
>>>> really gets out of hand when you boot say a 64GB guest.
>>>
>>> ia64 doesn't codegen; why not set it to zero?
>>
>> How about this one then?
>>
>
> Applied, thanks.  Note qemu uses 4 spaces for intedenation.  Talk to 
> your editor.

It would seem better to replace #ifdef TARGET_IA64 with if (kvm_enabled()).

If QEMU ever got proper ia64 target support, that's going to be an ugly 
bug to find.

Regards,

Anthony Liguori

Re: [patch] do not allocate code_gen buffer on ia64

[Avi Kivity]

Anthony Liguori wrote:
>
> It would seem better to replace #ifdef TARGET_IA64 with if 
> (kvm_enabled()).
>

Right.  Committed.

-- 
error compiling committee.c: too many arguments to function