Carsten Otte wrote:
> Zhang, Xiantao wrote:
>> According to our previous discuss,  we proposed a source layout,
>> which contains an "include" directory to hold header files for all
>> archs under drivers/kvm/, and kvm_arch.h will finally go into
>> drivers/kvm/include/kvm-x86/(linked as kvm when compile).
> Right. The thing is, I've started a new header for this purpose
> yesterday. And this should be in the _same_ header, no matter where
> it'll end up. It is the x86 specific header file, currently named
> drivers/kvm/x86.h, which needs to be renamed/moved in the future.

Agree. future rename or remove operation is needed.

>> So, every
>> architecture can defines its own kvm_arch.h for their arch, and
>> compile will choose it per ARCH when compile time. But for now, we
>> can just put it here before another real new arch in.  Then, we can
>> remove x86.h, since it is not so common for all archs.  :)
>> BTW, header files should be managed with a uniform method, because
>> possible archs, such as IA64, maybe need many ones.
> That's fine with me. But prior to that we'll need to split x86 so that
> it can be relocated in its arch directory different from the common
> kvm location. And until we're there, we use x86.h as a place to store
> x86 specific header content.

OK, I will change it to x86.h, but we also renamed it to such
"kvm_arch.h", because kvm.h will includes it. 

> so long,
> Carsten

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Reply via email to