Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Zhang, Xiantao wrote: > Sorry for confusing you. Is it clear now? Thanks, yea this makes your intention clear to me. I think what you describe is an odd way to initialize both modules, and I'd like to suggest to try to come up with the reverse approach where kvm.ko loads your other module via r

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Avi Kivity wrote: > Zhang, Xiantao wrote: >> >> Hi Avi, >> Thanks, But seems it is very hard to implement it through a.o, >> because all functions in this part should be relocated to another >> separate address space in whole. If it is implemented as a >> module(ko), we can relocate it as a

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Zhang, Xiantao wrote: That answer confuses me. kvm_arch_init() for ia64 will reside in kvm.ko just like kvm_init(), right? So THIS_MODULE will be the same in kvm_arch_init() and kvm_init(). So which module are you giving as argument? >>> Sorry for confusing yo

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Avi Kivity wrote: > Zhang, Xiantao wrote: >> Carsten Otte wrote: >> >>> Zhang, Xiantao wrote: >>> No, THIS_MODULE just stands for itself. But for IA64, we need another module, and it will register some information to kvm module. So, we have to keep it. >>> That answer confus

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Zhang, Xiantao wrote: > Carsten Otte wrote: > >> Zhang, Xiantao wrote: >> >>> No, THIS_MODULE just stands for itself. But for IA64, we need another >>> module, and it will register some information to kvm module. So, we >>> have to keep it. >>> >> That answer confuses me. kvm_arch_in

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Carsten Otte wrote: > Zhang, Xiantao wrote: >> No, THIS_MODULE just stands for itself. But for IA64, we need another >> module, and it will register some information to kvm module. So, we >> have to keep it. > That answer confuses me. kvm_arch_init() for ia64 will reside in > kvm.ko just like kvm_i

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Zhang, Xiantao wrote: > No, THIS_MODULE just stands for itself. But for IA64, we need another > module, and it will register some information to kvm module. So, we have > to keep it. That answer confuses me. kvm_arch_init() for ia64 will reside in kvm.ko just like kvm_init(), right? So THIS_MODULE

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Christian Borntraeger wrote: > Am Freitag, 23. November 2007 schrieb Zhang, Xiantao: >> >> From: Zhang Xiantao <[EMAIL PROTECTED]> >> Date: Fri, 23 Nov 2007 07:28:35 +0800 >> Subject: [PATCH] Add the arg module for kvm_arch_into >> Add the arg module for kvm_arch_into, since some archs may need >>

Re: [kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

Am Freitag, 23. November 2007 schrieb Zhang, Xiantao: > > From: Zhang Xiantao <[EMAIL PROTECTED]> > Date: Fri, 23 Nov 2007 07:28:35 +0800 > Subject: [PATCH] Add the arg module for kvm_arch_into > Add the arg module for kvm_arch_into, since some archs may > need module info. > Signed-off-by: Zhang

[kvm-devel] [PATCH] Add the arg "module" for kvm_arch_init

From: Zhang Xiantao <[EMAIL PROTECTED]> Date: Fri, 23 Nov 2007 07:28:35 +0800 Subject: [PATCH] Add the arg module for kvm_arch_into Add the arg module for kvm_arch_into, since some archs may need module info. Signed-off-by: Zhang Xiantao <[EMAIL PROTECTED]> --- drivers/kvm/kvm.h |4 +++-