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

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


  • To: <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "Dong, Eddie" <eddie.dong@xxxxxxxxx>
  • Date: Mon, 17 Mar 2008 11:39:45 +0800
  • Delivery-date: Sun, 16 Mar 2008 20:44:36 -0700
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AciF5/UZ1ZlBEgPUTkeomiyAOalrFgB9/v8g
  • Thread-topic: 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


 


Rackspace

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