[PATCH] misc: Removal of final callers using fastcall

[PATCH] misc: Removal of final callers using fastcall

[Harvey Harrison]

fastcall expands to empty everywhere, remove it.

Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
---
Andrew, I'm not sure who is best to hit with these final dribs and
drabs removing fastcall.  Once all of these have hit Linus' tree
I will send a final patch deleting the include/linux/linkage.h
definitions as well as any remaining occurances.

 arch/powerpc/sysdev/uic.c |    2 +-
 arch/sparc64/kernel/irq.c |    2 +-
 include/asm-arm/mutex.h   |    6 +++---
 mm/page_alloc.c           |    2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/powerpc/sysdev/uic.c b/arch/powerpc/sysdev/uic.c
index 847a549..bbb6caf 100644
--- a/arch/powerpc/sysdev/uic.c
+++ b/arch/powerpc/sysdev/uic.c
@@ -188,7 +188,7 @@ static struct irq_chip uic_irq_chip = {
  * instead of the end, to keep the window in which we can miss an
  * interrupt as small as possible.
  */
-void fastcall handle_uic_irq(unsigned int irq, struct irq_desc *desc)
+void handle_uic_irq(unsigned int irq, struct irq_desc *desc)
 {
 	unsigned int cpu = smp_processor_id();
 	struct irqaction *action;
diff --git a/arch/sparc64/kernel/irq.c b/arch/sparc64/kernel/irq.c
index 30431bd..5ec06c8 100644
--- a/arch/sparc64/kernel/irq.c
+++ b/arch/sparc64/kernel/irq.c
@@ -522,7 +522,7 @@ static struct irq_chip sun4v_virq = {
 	.set_affinity	= sun4v_virt_set_affinity,
 };
 
-static void fastcall pre_flow_handler(unsigned int virt_irq,
+static void pre_flow_handler(unsigned int virt_irq,
 				      struct irq_desc *desc)
 {
 	struct irq_handler_data *data = get_irq_chip_data(virt_irq);
diff --git a/include/asm-arm/mutex.h b/include/asm-arm/mutex.h
index cb29d84..020bd98 100644
--- a/include/asm-arm/mutex.h
+++ b/include/asm-arm/mutex.h
@@ -24,7 +24,7 @@
  * reattempted until it succeeds.
  */
 static inline void
-__mutex_fastpath_lock(atomic_t *count, fastcall void (*fail_fn)(atomic_t *))
+__mutex_fastpath_lock(atomic_t *count, void (*fail_fn)(atomic_t *))
 {
 	int __ex_flag, __res;
 
@@ -44,7 +44,7 @@ __mutex_fastpath_lock(atomic_t *count, fastcall void (*fail_fn)(atomic_t *))
 }
 
 static inline int
-__mutex_fastpath_lock_retval(atomic_t *count, fastcall int (*fail_fn)(atomic_t *))
+__mutex_fastpath_lock_retval(atomic_t *count, int (*fail_fn)(atomic_t *))
 {
 	int __ex_flag, __res;
 
@@ -70,7 +70,7 @@ __mutex_fastpath_lock_retval(atomic_t *count, fastcall int (*fail_fn)(atomic_t *
  * better generated assembly.
  */
 static inline void
-__mutex_fastpath_unlock(atomic_t *count, fastcall void (*fail_fn)(atomic_t *))
+__mutex_fastpath_unlock(atomic_t *count, void (*fail_fn)(atomic_t *))
 {
 	int __ex_flag, __res, __orig;
 
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index e8206c4..a8b08f9 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -1440,7 +1440,7 @@ try_next_zone:
 /*
  * This is the 'heart' of the zoned buddy allocator.
  */
-struct page * fastcall
+struct page *
 __alloc_pages(gfp_t gfp_mask, unsigned int order,
 		struct zonelist *zonelist)
 {
-- 
1.5.3.7.2212.gd092

Re: [PATCH] misc: Removal of final callers using fastcall

[Andrew Morton]

On Wed, 12 Dec 2007 15:38:26 -0800 Harvey Harrison <harvey.harrison@gmail.com> wrote:

> Andrew, I'm not sure who is best to hit with these final dribs and
> drabs removing fastcall.  Once all of these have hit Linus' tree
> I will send a final patch deleting the include/linux/linkage.h
> definitions as well as any remaining occurances.

Yes, that's a good approach, thanks.  Wait until the tree is fastcall-clean
and then kill the definition(s).

I think I skipped rather a lot of remove-fastcall patches because a)
suitable maintainers were cc'ed and b) I was going through a
suicidal-over-bug-reports phase.

Please keep them coming - I've always disliked fastcall.

Re: [PATCH] misc: Removal of final callers using fastcall

[Harvey Harrison]

On Thu, 2007-12-20 at 18:30 -0800, Andrew Morton wrote:
> On Wed, 12 Dec 2007 15:38:26 -0800 Harvey Harrison <harvey.harrison@gmail.com> wrote:
> 
> > Andrew, I'm not sure who is best to hit with these final dribs and
> > drabs removing fastcall.  Once all of these have hit Linus' tree
> > I will send a final patch deleting the include/linux/linkage.h
> > definitions as well as any remaining occurances.
> 
> Yes, that's a good approach, thanks.  Wait until the tree is fastcall-clean
> and then kill the definition(s).
> 
> I think I skipped rather a lot of remove-fastcall patches because a)
> suitable maintainers were cc'ed and b) I was going through a
> suicidal-over-bug-reports phase.
> 
> Please keep them coming - I've always disliked fastcall.

Once I see these have hit the main tree, I'll send patch getting any
more that have snuck in for the next rc.  After that there should be
few enough left that I can send you a small patch for the next rc
with the definition removal as well.

I'll keep on top of these.

Harvey