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 _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel
