From: Zhang, X. <xia...@in...> - 2007-11-08 07:22:05
|
Hi Avi, This series of patches are intended to further split x86 specific code from kvm_main.c, and make kvm_main.c arch-independent. =20 For easy review, I splitted them into small patches. With these patches, we almost finish the first stage work for code split.=20 [1/5] Remove unused header files include from kvm_main.c since the corresponding functions have been moved to x86.c [2/5] Move kvm_x86_ops definition to x86, and prepare for next step's work. [3/5] Using kvm_arch prefix to redefine kvm_x86_ops->ops in x86.c. and make kvm_x86_ops as x86_specific part. [4/5] Combine kvm_init and kvm_init_x86 into one kvm_init function, and meanwhile, remove module_init and module_exit function since kvm_init will called by svm and vmx. [5/5] Make kvm_init as arch-indepenent, through defining a void pointer type parameter . All archs will register their corresponding functions into kvm module. For example, x86 will use this parameter to register vmx/svm's ops.=20 CC to Hollis and Carste to check whether they block your sides. :)=20 Thanks=20 Xiantao |
From: Carsten O. <co...@de...> - 2007-11-08 12:06:48
|
Zhang, Xiantao wrote: > This series of patches are intended to further split x86 specific code > from kvm_main.c, and make kvm_main.c arch-independent. > For easy review, I splitted them into small patches. With these patches, > we almost finish the first stage work for code split. These patches don't apply on top of my git tree: Wende Patch 01-xiantao.patch an patching file drivers/kvm/kvm_main.c Hunk #1 succeeded at 17 with fuzz 2. Hunk #2 succeeded at 43 with fuzz 2. patching file drivers/kvm/x86.c Hunk #1 succeeded at 16 with fuzz 1. Hunk #2 FAILED at 26. 1 out of 2 hunks FAILED -- rejects in file drivers/kvm/x86.c |
From: Zhang, X. <xia...@in...> - 2007-11-08 12:59:52
|
Hi,Carste=20 Maybe some new commits block it. I will rebase them to latest commit, = and resend :) Thanks for you try. Xiantao -----Original Message----- From: Carsten Otte [mailto:co...@de...]=20 Sent: 2007=C4=EA11=D4=C28=C8=D5 20:06 To: Zhang, Xiantao Cc: Avi Kivity; kvm...@li...; Hollis Blanchard; = car...@de... Subject: Re: [PATCH]0/5 Patch to split x86 specific code from kvm_main.c Zhang, Xiantao wrote: > This series of patches are intended to further split x86 specific code > from kvm_main.c, and make kvm_main.c arch-independent. =20 > For easy review, I splitted them into small patches. With these = patches, > we almost finish the first stage work for code split.=20 These patches don't apply on top of my git tree: Wende Patch 01-xiantao.patch an patching file drivers/kvm/kvm_main.c Hunk #1 succeeded at 17 with fuzz 2. Hunk #2 succeeded at 43 with fuzz 2. patching file drivers/kvm/x86.c Hunk #1 succeeded at 16 with fuzz 1. Hunk #2 FAILED at 26. 1 out of 2 hunks FAILED -- rejects in file drivers/kvm/x86.c |
From: Zhang, X. <xia...@in...> - 2007-11-08 13:37:44
Attachments:
0001-Move-some-includes-to-x86.c-from-kvm_main.c-since-t.patch
0002-move-kvm_x86_ops-to-x86.c.patch
0003-Using-kvm_arch-prefix-to-redefine-kvm_x86_ops-ops-i.patch
0004-Combine-kvm_init-and-kvm_init_x86-into-one-kvm_init.patch
0005-Make-kvm_init-as-arch-indepenent-through-defining-a.patch
|
Rebased to latest commits.=20 Thanks=20 Xiantao -----Original Message----- From: kvm...@li... = [mailto:kvm...@li...] On Behalf Of Zhang, = Xiantao Sent: 2007=C4=EA11=D4=C28=C8=D5 21:00 To: car...@de... Cc: kvm...@li...; Hollis Blanchard; Avi Kivity Subject: Re: [kvm-devel] [PATCH]0/5 Patch to split x86 specific code = fromkvm_main.c Hi,Carste=20 Maybe some new commits block it. I will rebase them to latest commit, = and resend :) Thanks for you try. Xiantao -----Original Message----- From: Carsten Otte [mailto:co...@de...]=20 Sent: 2007=C4=EA11=D4=C28=C8=D5 20:06 To: Zhang, Xiantao Cc: Avi Kivity; kvm...@li...; Hollis Blanchard; = car...@de... Subject: Re: [PATCH]0/5 Patch to split x86 specific code from kvm_main.c Zhang, Xiantao wrote: > This series of patches are intended to further split x86 specific code > from kvm_main.c, and make kvm_main.c arch-independent. =20 > For easy review, I splitted them into small patches. With these = patches, > we almost finish the first stage work for code split.=20 These patches don't apply on top of my git tree: Wende Patch 01-xiantao.patch an patching file drivers/kvm/kvm_main.c Hunk #1 succeeded at 17 with fuzz 2. Hunk #2 succeeded at 43 with fuzz 2. patching file drivers/kvm/x86.c Hunk #1 succeeded at 16 with fuzz 1. Hunk #2 FAILED at 26. 1 out of 2 hunks FAILED -- rejects in file drivers/kvm/x86.c |