[CRIU] [PATCH 01/20] x86: moved x86-specific files into the directory arch/x86.

[Alexander Kartashov]

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