[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC 3/5] x86: split PV dom0 builder to domain_build_pv.c



>>> On 03.03.17 at 14:35, <wei.liu2@xxxxxxxxxx> wrote:
> On Fri, Mar 03, 2017 at 04:15:19AM -0700, Jan Beulich wrote:
>> >>> On 03.03.17 at 12:06, <roger.pau@xxxxxxxxxx> wrote:
>> > On Fri, Mar 03, 2017 at 09:41:09AM +0000, Wei Liu wrote:
>> However, with ...
>> 
>> >> diff --git a/xen/arch/x86/domain_build_pv.c 
>> >> b/xen/arch/x86/domain_build_pv.c
>> > 
>> > I would place this in arch/xen/x86/pv/domain_build.c
>> 
>> ... this the case becomes less clear: Personally I'm not a fan of
>> #include "../file.h", so I wouldn't be certain which of the two is
>> the less undesirable place for the header.
>> 
> 
> But in this case I would prefer to move the to xen/include/xen and named
> in dom0_build.h.

Well, it's x86_specific, so if anything include/asm-x86/.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.