RE: [Xen-ia64-devel] pv_ops: imntrinsic pv_ops

2008-04-07 Thread Dong, Eddie
Isaku Yamahata wrote: Hi Eddie. I commited some clean ups based on your patch. Could you please review it? It looks like you still prefer to use intermediate symbol paravirt_/ia64_native_xxx to wrap ia64_xxx. In some sense, when I saw many similar and bulk code in the patch, I feel

Re: [Xen-ia64-devel] pv_ops: imntrinsic pv_ops

2008-04-07 Thread Isaku Yamahata
On Mon, Apr 07, 2008 at 05:06:36PM +0800, Dong, Eddie wrote: It looks like you still prefer to use intermediate symbol paravirt_/ia64_native_xxx to wrap ia64_xxx. In some sense, when I saw many similar and bulk code in the patch, I feel dirty. I prefer we don't touch those

Re: [Xen-ia64-devel] pv_ops: imntrinsic pv_ops

2008-04-02 Thread Isaku Yamahata
On Wed, Apr 02, 2008 at 01:51:28PM +0800, Dong, Eddie wrote: Current definition of intrinsic APIs seems to be too expansive, this one give alternative way to do simply and reduce some changes. If this applies, further simplification can be applied. Thx, eddie Interesting approach. If we can

RE: [Xen-ia64-devel] pv_ops: imntrinsic pv_ops

2008-04-02 Thread Dong, Eddie
Isaku Yamahata wrote: On Wed, Apr 02, 2008 at 01:51:28PM +0800, Dong, Eddie wrote: Current definition of intrinsic APIs seems to be too expansive, this one give alternative way to do simply and reduce some changes. If this applies, further simplification can be applied. Thx, eddie

[Xen-ia64-devel] pv_ops: imntrinsic pv_ops

2008-04-01 Thread Dong, Eddie
Current definition of intrinsic APIs seems to be too expansive, this one give alternative way to do simply and reduce some changes. If this applies, further simplification can be applied. Thx, eddie Simplify intrinsic API handling. Signed-off-by: Yaozu (Eddie) Dong [EMAIL PROTECTED] diff