Geert-san,

On Thu, 14 Dec 2006, Ishizaki Kou wrote:

quoted

--- /dev/null      Wed Dec 13 21:32:01 2006
+++ linux-powerpc-git/arch/powerpc/platforms/celleb/hvCall_macro.S	Wed Dec 13 18:37:14 2006

quoted

+#define	__define_beat_callup_handler_NOOUTS(name, id)		\
+_GLOBAL(beat_##name)										\
+													HCALL_MEDIUM;	     \
+													mflr		     r0;      \
+													std		     r0, 16(r1);     \
+													HCALL_INST_PRECALL;		       \
+													LOAD_CONST64(r11,id);		           \
+													HVSC;					         \
+													HCALL_INST_POSTCALL;				     \
+													ld						     r0, 16(r1);     \
+													mtlr						     r0;	         \
+													blr

`id' is always the name prefixed with `HV_'? In that case you can use
`HV_##name', so you don't have to pass the number.

It's in that case, since `id' is assumed to be a hypervisor-call number,
and defined in celleb/beat_syscall.h.

quoted

--- /dev/null	Wed Dec 13 21:32:02 2006
+++ linux-powerpc-git/arch/powerpc/platforms/celleb/hvCall.S	Wed Dec 13 18:37:14 2006

quoted

+__define_beat_callup_handler_OUT2(4, allocate_memory, (0))

                                                         ^^^
I.e., these can go away.

I'll modify them so.
These sources are generated from our 'hvcall table', so we have to
modify the generator.

Best Regards,
Kou Ishizaki