|
Re: [PATCH] kvm: initialize kvm_arch_ops in kvm_init(): msg#00274emulators.kvm.devel
Yoshimi Ichiyanagi wrote: >> This bit is unnecessary, no? I think kvm_init() will only be called >> after the module is loaded, at which point kvm_arch_ops is initialized >> > >from the .bss section. > > > Logically it's not required, however it's better to initialize > explicitly in general. > > Personally I agree, but Linux coding style does not. > By the way, the previous patch didn't fix the problem completely. > You can't load the supported arch's module > if you load the unsupported arch's module beforehand. > > The following patch will fix this problem. > Please take a look. > Applied, thanks. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV |
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| Previous by Date: | Re: [PATCH 0/8] KVM updates for 2.6.20-rc2: 00274, Ingo Molnar |
|---|---|
| Next by Date: | Re: [PATCH 0/8] KVM updates for 2.6.20-rc2: 00274, Avi Kivity |
| Previous by Thread: | Re: [PATCH] kvm: initialize kvm_arch_ops in kvm_init()i: 00274, Yoshimi Ichiyanagi |
| Next by Thread: | Exception 13 problem: 00274, Jun Koi |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
| News | FAQ | advertise |