Re: [ogfs-dev][patch] current opengfs + opendlm can works.

On Thu, 4 Mar 2004 11:24:07 -0800
"Cahill, Ben M" <[email protected]> wrote:

> This just removed white space, so I did not apply this patch.  Are you
> sure that's all you wanted to do?

I'm using Debian Woody, I can't sure that's is the bug with gcc.
 gcc -v
Reading specs from /usr/lib/gcc-lib/alpha-linux/2.95.4/specs
gcc version 2.95.4 20011002 (Debian prerelease)

I show you why me do it.
--- opendlm.h.org       2004-03-05 08:44:01.000000000 +0800
+++ opendlm.h   2004-03-05 08:44:48.000000000 +0800
@@ -106,7 +106,7 @@
         do {                                                          
\                 if(debug)                                             
\                         printk (KERN_DEBUG "%s %s: " format "\n",     
\-                                PROTO_NAME, __FUNCTION__, ## arg);    
\+                                PROTO_NAME, __FUNCTION__ , ## arg);   
\         } while(0)

Only add a space before ',', but the gcc precompile are different.

@@ -25951,7 +25951,7 @@
 void opendlm_stop(void)
 {
        printk("opendlm_stop\n");
-       do {    if(debug)       printk ("<7>"  "%s %s: "  "opendlm_stop"
 "\n","opendlm" ,  );  } while(0) ;
+       do {    if(debug)      printk ("<7>"  "%s %s: "  "opendlm_stop" 
"\n","opendlm" , __FUNCTION__ );     } while(0) ;       

when I change to gcc-3.0, that's ok, interesting.

-- 
Hu Gang / Steve
Linux Registered User 204016
GPG Public Key: http://soulinfo.com/~hugang/hugang.asc


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click