[PATCH] [0/10]Patches to split qemu-kvm.c

[PATCH] [0/10]Patches to split qemu-kvm.c

[Zhang, Xiantao]

Hi, Avi
This series of patches are introduced to split qemu-kvm.c, and make it
arch-independent. 
In these patches, one file called qemu-kvm-x86.c is created. For
arch-specific stuff, I also use kvm_arch prefix to hold arch-specific
code. 
Tested on x86 and x86_64 platform.

[1/10]kvm: qemu : preparation for qemu-kvm.c split.
This patch adds qemu-kvm-x86.c to hold x86-specific code.

[2/10]kvm: qemu: split kvm_qemu_create_context.
This patche splits kvm_qemu_create_context and moves x86-specific
to qemu-kvm-x86.c. Temporarily set kvm_has_msr_star to a gloabal
variant.

[3/10] kvm: qemu : Moving load_regs and save_regs to qemu-kvm-x86.c
In this patch, using kvm_arch prefix to define load and save regs.
load_regs->kvm_arch_load_regs
save_regs->kvm_arch_save_regs.
Temporarily set lm_capable_kernel to a global variant.
Moving routines related to load & save regs to qemu-kvm-x86.c

[4/10] kvm: qemu: Split kvm_qemu_init_env.
Using kvm_arch_qemu_init_env to hold x86-specific stuff.
Moving corresponding static funtions to qemu-kvm-x86.c
Changing back lm_capable_kernel to static variants.

[5/10] kvm: qemu, Add kvm_arch_halt to handle arch-specific halt
operations.
In this patch, one arch-specific interface kvm_arch_halt is added to
handle arch halts.

[6/10]kvm: qemu: using kvm_arch_pre_kvm_run to hold x86-specific code.
In this patch, it use kvm_arch_pre_kvm_run to hold
x86-specific code for pre_kvm_run;

[7/10] kvm: qemu: using kvm_arch_post_kvm_run to hold x86-specific code.
    In this patch, it use kvm_arch_post_kvm_run to hold
    x86-specific code for post_kvm_run;

[8/10] kvm: qemu: Using to kvm_arch_has_work to handle arch-specific
has_work check.
In this patch, kvm_arch_has_work is introduced to hanlde arch-specific
check.

[9/10]kvm: qemu: Using kvm_arch_try_push_interrupts to hold
arch-sepcific stuff.
In this patch, kvm_arch_try_push_interrupts is used to handle
arch-specific
operations for try_push_interrupts.

[10/10] kvm: qemu: Using kvm_arch_update_regs_for_sipi to hold
arch-specific sipi operations.
In this patch, kvm_arch_update_regs_for_sipi is introduced for handling
arch-specific
update_regs once sipi received.

Signed-off-by: Zhang Xiantao <xiantao.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4

Re: [PATCH] [0/10]Patches to split qemu-kvm.c

[Avi Kivity]

Zhang, Xiantao wrote:
> Hi, Avi
> This series of patches are introduced to split qemu-kvm.c, and make it
> arch-independent. 
> In these patches, one file called qemu-kvm-x86.c is created. For
> arch-specific stuff, I also use kvm_arch prefix to hold arch-specific
> code. 
> Tested on x86 and x86_64 platform.
>
>   

I feel a little bad about it. qemu-kvm.c is not well written, and this 
makes fixing it more difficult.

However, it's too late to cry about it now, so I will apply the patches 
once they have been reviewed.

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


-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4

Re: [PATCH] [0/10]Patches to split qemu-kvm.c

[Zhang, Xiantao]

Avi Kivity wrote:
> Zhang, Xiantao wrote:
>> Hi, Avi
>> This series of patches are introduced to split qemu-kvm.c, and make
>> it arch-independent. In these patches, one file called
>> qemu-kvm-x86.c is created. For arch-specific stuff, I also use
>> kvm_arch prefix to hold arch-specific code. Tested on x86 and x86_64
>> platform. 
>> 
>> 
> 
> I feel a little bad about it. qemu-kvm.c is not well written, and this
> makes fixing it more difficult.

I also have the same feeling when splitting it. Maybe we can improve it
later. 

> However, it's too late to cry about it now, so I will apply the
> patches once they have been reviewed.

Thanks. It is very helpful to check-in our ia64 code. :)

-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4

Re: [PATCH] [0/10]Patches to split qemu-kvm.c

[Avi Kivity]

Zhang, Xiantao wrote:
> Hi, Avi
> This series of patches are introduced to split qemu-kvm.c, and make it
> arch-independent. 
> In these patches, one file called qemu-kvm-x86.c is created. For
> arch-specific stuff, I also use kvm_arch prefix to hold arch-specific
> code. 
> Tested on x86 and x86_64 platform.
>   

Applied and pushed all; thanks.

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


-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4