WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH][RESEND] Porting Mini-OS to ia64

With respect to your patch. I would prefer to see the following
directory structure:
mini-os/include/x86
mini-os/include/x86/x86_32
mini-os/include/x86/x86_64
If we find that there is only one file in x86_32/x86_64 it would be
better to scrap the two directories and put the files in x86.

Continuing, architecture dependent .c files should end up in:
mini-os/arch/x86
mini-os/arch/x86/x86_32
mini-os/arch/x86/x86_64

I moved only few files and changed the Makefile. In the next step I will look into the files for special defines for x86_32 and x86_64.
I hope this patch is ok now.

The patch still introduces
mini-os/include/x86_32 and mini-os/include/x86_64.

I would prefer both x86_32 and x86_64 to be subdirectories of mini-os/include/x86.

Also, I would replace "=" with "+=" when you assign to EXTRA_INC/SRC in mini-os/Makefile.

Cheers
Gregor

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel