All of lore.kernel.org
 help / color / mirror / Atom feed
* pre12 fails to compile:  wakeup_bdflush issues
@ 2001-09-20 17:47 Ben Greear
  2001-09-20 18:30 ` David S. Miller
  0 siblings, 1 reply; 3+ messages in thread
From: Ben Greear @ 2001-09-20 17:47 UTC (permalink / raw)
  To: linux-kernel

I used the .config from RH's roswell beta.

I get this error:

sysrq.c:35: conflicting types for 'wakeup_bdflush'
/root/linux/include/linux/fs.h:1347: previous declaration of 'wakeup_bdflush'

One says it takes a void argument, the other  an int......

Ben

-- 
Ben Greear <greearb@candelatech.com>          <Ben_Greear@excite.com>
President of Candela Technologies Inc      http://www.candelatech.com
ScryMUD:  http://scry.wanfear.com     http://scry.wanfear.com/~greear

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: pre12 fails to compile: wakeup_bdflush issues
  2001-09-20 17:47 pre12 fails to compile: wakeup_bdflush issues Ben Greear
@ 2001-09-20 18:30 ` David S. Miller
  2001-09-24  6:37   ` ddkilzer
  0 siblings, 1 reply; 3+ messages in thread
From: David S. Miller @ 2001-09-20 18:30 UTC (permalink / raw)
  To: greearb; +Cc: linux-kernel

   From: Ben Greear <greearb@candelatech.com>
   Date: Thu, 20 Sep 2001 10:47:20 -0700

   I get this error:
   
   sysrq.c:35: conflicting types for 'wakeup_bdflush'
   /root/linux/include/linux/fs.h:1347: previous declaration of 'wakeup_bdflush'
   
   One says it takes a void argument, the other  an int......

The fix is simple:

--- drivers/char/sysrq.c.~1~	Wed Sep 19 14:30:53 2001
+++ drivers/char/sysrq.c	Thu Sep 20 11:29:30 2001
@@ -32,7 +32,6 @@
 
 #include <asm/ptrace.h>
 
-extern void wakeup_bdflush(int);
 extern void reset_vc(unsigned int);
 extern struct list_head super_blocks;
 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: pre12 fails to compile: wakeup_bdflush issues
  2001-09-20 18:30 ` David S. Miller
@ 2001-09-24  6:37   ` ddkilzer
  0 siblings, 0 replies; 3+ messages in thread
From: ddkilzer @ 2001-09-24  6:37 UTC (permalink / raw)
  To: David S. Miller; +Cc: linux-kernel, Ben Greear

What about the other two calls to wakeup_bdflush(0) in 
drivers/char/sysrq.c?  Should they be fixed using the patch below?

Dave


--- drivers/char/sysrq.c.orig	Sun Sep 23 17:24:55 2001
+++ drivers/char/sysrq.c	Mon Sep 24 01:11:07 2001
@@ -32,7 +32,6 @@
 
 #include <asm/ptrace.h>
 
-extern void wakeup_bdflush(int);
 extern void reset_vc(unsigned int);
 extern struct list_head super_blocks;
 
@@ -221,7 +220,7 @@
 static void sysrq_handle_sync(int key, struct pt_regs *pt_regs,
 		struct kbd_struct *kbd, struct tty_struct *tty) {
 	emergency_sync_scheduled = EMERG_SYNC;
-	wakeup_bdflush(0);
+	wakeup_bdflush();
 }
 static struct sysrq_key_op sysrq_sync_op = {
 	handler:	sysrq_handle_sync,
@@ -232,7 +231,7 @@
 static void sysrq_handle_mountro(int key, struct pt_regs *pt_regs,
 		struct kbd_struct *kbd, struct tty_struct *tty) {
 	emergency_sync_scheduled = EMERG_REMOUNT;
-	wakeup_bdflush(0);
+	wakeup_bdflush();
 }
 static struct sysrq_key_op sysrq_mountro_op = {
 	handler:	sysrq_handle_mountro,


On Thu, Sep 20, 2001 at 13:30:37 EST, David S. Miller wrote:

>    From: Ben Greear <greearb@candelatech.com>
>    Date: Thu, 20 Sep 2001 10:47:20 -0700
> 
>    I get this error:
>    
>    sysrq.c:35: conflicting types for 'wakeup_bdflush'
>    /root/linux/include/linux/fs.h:1347: previous declaration of 'wakeup_bdflush'
>    
>    One says it takes a void argument, the other an int......
> 
> The fix is simple:
> 
> --- drivers/char/sysrq.c.~1~ Wed Sep 19 14:30:53 2001
> +++ drivers/char/sysrq.c Thu Sep 20 11:29:30 2001
> @@ -32,7 +32,6 @@
>  
>  #include <asm/ptrace.h>
>  
> -extern void wakeup_bdflush(int);
>  extern void reset_vc(unsigned int);
>  extern struct list_head super_blocks;
>  


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2001-09-24  6:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-09-20 17:47 pre12 fails to compile: wakeup_bdflush issues Ben Greear
2001-09-20 18:30 ` David S. Miller
2001-09-24  6:37   ` ddkilzer

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.