[CRIU] [PATCH 01/20] x86: moved x86-specific files into the directory arch/x86.
Alexander Kartashov
alekskartashov at parallels.com
Wed Dec 19 09:45:18 EST 2012
On 12/19/2012 06:28 PM, Cyrill Gorcunov wrote:
> On Wed, Dec 19, 2012 at 06:15:14PM +0400, Alexander Kartashov wrote:
>> On 12/13/2012 10:53 AM, Cyrill Gorcunov wrote:
>>> as to "#include <asm/<file>.h>" Pavel mentioned -- I think we must not
>>> use global namespace hare but include header as local one, it
>> What do you think about the following proposition:
>>
>> * if the whole header is moved to arch/ then let it be accessible
>> in the global namespace (eg, linkage.h, atomic.h);
>> * it a header is split then let it be accessible in the namespace cr/.
> Саш, привет, поясни плиз, я что-то не совсем понял.
> Что я предлагаю: положить хэдеры в arch/x/include,
> если они зависят от архитектуры.
>
> Например, для types.h можно сделать как в ядре
>
> include/types.h
> ---------------
>
> #include <asm/types.h>
>
>
> arch/x86/include/types.h
> ------------------------
>
> typedef uint64_t u64;
> typedef int64_t s64;
> typedef unsigned int u32;
> typedef signed int s32;
> typedef unsigned short u16;
> typedef signed short s16;
> typedef unsigned char u8;
> typedef signed char s8;
>
> в самих сорцах делается
>
> #include "types.h"
>
> а компилер, в зав-ти от архитектуры выберет
> нужный asm/types.h
>
> Cyrill
Мне кажется, types.h --- плохой пример, поскольку если его
положить в arch/$(ARCH)/include/asm, то он будет пересекатсья
с системным заголовком. Кроме того, types.h только частично
зависит от архитектуры.
В целом, я с Вашим предложением согласен и хочу уточнить
его следующим образом:
1) Если заголовок полностью зависит от архитекутры, то пусть
он доступен в глобальном пространстве имен, т. е.
он будет перемещен в arch/$(ARCH)/include.
2) Если заголовок частично зависит от архитектуры, то его
машинно-зависимую часть перенести в arch/$(ARCH)/include/cr,
как например, в случае с types.h. asm/ не хочу использовать
по причине, описанной в примере с types.h.
--
Sincerely yours,
Alexander Kartashov
Intern
Core team
www.parallels.com
Skype: aleksandr.kartashov
Email: alekskartashov at parallels.com
More information about the CRIU
mailing list