[Xen-ia64-devel] pv_ops: kernel/inst_native.h

Isaku/Alex:
        There is a new file called kernel/inst_native.h to define those
pv MACROs for native. I would suggest we do following changes:
        1: Move it to public head files such as include/asm-ia64 at
least since some other files will use it too.
        2: Further thinking is that how we generate those dual compile
code? If we will use symbol link, then I would suggest we put it in
include/asm-ia64/native. Comments?

        3: How about this style if MACRO?
#define MOV_FROM_CR(reg,crx) \
        mov reg = cr.##crx


        Using this we can reduce the patch size & avoid mistake.

Thx, eddie

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

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-ia64-devel] pv_ops enable, Dong, Eddie Re: [Xen-ia64-devel] pv_ops enable, Alex Williamson [Xen-ia64-devel] pv_ops: kernel/inst_native.h, Dong, Eddie <= Re: [Xen-ia64-devel] pv_ops: kernel/inst_native.h, Isaku Yamahata

Previous by Date:

Re: [Xen-ia64-devel] Re: [patch] ia64: kexec: add __va_efi, Simon Horman

Next by Date:

RE: [Xen-ia64-devel] RE: pv_ops polish: config option & head file, Dong, Eddie

Previous by Thread:

Re: [Xen-ia64-devel] pv_ops enable, Alex Williamson

Next by Thread:

Re: [Xen-ia64-devel] pv_ops: kernel/inst_native.h, Isaku Yamahata

Indexes:

[Date] [Thread] [Top] [All Lists]