mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Erik McKee <camhanaich99@yahoo.com>
To: linux-kernel@vger.kernel.org, torvalde@transmeta.com,
	kernel-newbies@vger.kernel.org, davej@suse.de
Subject: [ERROR][PATCH] smbfs compilation in 2.5.21
Date: Sat, 15 Jun 2002 21:28:31 -0700 (PDT)	[thread overview]
Message-ID: <20020616042831.19489.qmail@web14204.mail.yahoo.com> (raw)

The attatched patch fixes smbfs compilation in 2.5.  The problem was that the
paranoia macro required a format string, and one or more substitution elements.
 4 callers only used the string with no substitution.  This patch defines a
paranoia2 which allows for not substitutions ;)

Compilation tested only, as i can not run latest 2.5 due to raid5 not
compiling, but this has to be ok, since it doesn't change any code paths.

Erik McKee

# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
#	           ChangeSet	1.484   -> 1.485  
#	fs/smbfs/smb_debug.h	1.2     -> 1.3    
#	     fs/smbfs/proc.c	1.20    -> 1.21   
#	     fs/smbfs/sock.c	1.7     -> 1.8    
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 02/06/15	root@camhanaich.local	1.485
# sock.c:
#   Same edit as proc.c
# proc.c:
#   This makes use of the PARANOIA2 to allow compilation
# smb_debug.h:
#   Added PARANOIA2 to handle cases where there are no
#   format specifiers in the string
# --------------------------------------------
#
diff -Nru a/fs/smbfs/proc.c b/fs/smbfs/proc.c
--- a/fs/smbfs/proc.c	Sat Jun 15 23:12:04 2002
+++ b/fs/smbfs/proc.c	Sat Jun 15 23:12:05 2002
@@ -872,7 +872,7 @@
 		goto out;
 	}
 	if (smb_valid_packet(s->packet) != 0) {
-		PARANOIA("invalid packet!\n");
+		PARANOIA2("invalid packet!\n");
 		goto out;
 	}
 
diff -Nru a/fs/smbfs/smb_debug.h b/fs/smbfs/smb_debug.h
--- a/fs/smbfs/smb_debug.h	Sat Jun 15 23:12:04 2002
+++ b/fs/smbfs/smb_debug.h	Sat Jun 15 23:12:04 2002
@@ -12,8 +12,10 @@
  */
 #ifdef SMBFS_PARANOIA
 # define PARANOIA(f, a...) printk(KERN_NOTICE "%s: " f, __FUNCTION__, ## a)
+# define PARANOIA2(f) printk(KERN_NOTICE "%s: "f, __FUNCTION__)
 #else
 # define PARANOIA(f, a...) do { ; } while(0)
+# defind PARANOIA2(f) do { ; } while(0)
 #endif
 
 /* lots of debug messages */
diff -Nru a/fs/smbfs/sock.c b/fs/smbfs/sock.c
--- a/fs/smbfs/sock.c	Sat Jun 15 23:12:05 2002
+++ b/fs/smbfs/sock.c	Sat Jun 15 23:12:05 2002
@@ -125,7 +125,7 @@
 
 		result = -EIO;
 		if (job->sk->dead) {
-			PARANOIA("sock dead!\n");
+			PARANOIA2("sock dead!\n");
 			break;
 		}
 
@@ -189,7 +189,7 @@
 	{
 #ifdef SMBFS_PARANOIA
 		if (!smb_valid_socket(file->f_dentry->d_inode))
-			PARANOIA("bad socket!\n");
+			PARANOIA2("bad socket!\n");
 #endif
 		return SOCKET_I(file->f_dentry->d_inode);
 	}
@@ -299,7 +299,7 @@
 		VERBOSE("closing socket %p\n", server_sock(server));
 #ifdef SMBFS_PARANOIA
 		if (server_sock(server)->sk->data_ready == smb_data_ready)
-			PARANOIA("still catching keepalives!\n");
+			PARANOIA2("still catching keepalives!\n");
 #endif
 		server->sock_file = NULL;
 		fput(file);
@@ -585,7 +585,7 @@
 	return result;
 
 out_no_mem:
-	PARANOIA("couldn't allocate data area\n");
+	PARANOIA2("couldn't allocate data area\n");
 	result = -ENOMEM;
 	goto out;
 out_too_long:



__________________________________________________
Do You Yahoo!?
Yahoo! - Official partner of 2002 FIFA World Cup
http://fifaworldcup.yahoo.com

             reply	other threads:[~2002-06-16  4:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-16  4:28 Erik McKee [this message]
2002-06-16 12:59 ` Urban Widmark
2002-06-16 16:24   ` Erik McKee
2002-06-16 17:13     ` Urban Widmark
2002-06-16 18:26   ` Linus Torvalds
2002-06-16 18:40     ` Urban Widmark

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020616042831.19489.qmail@web14204.mail.yahoo.com \
    --to=camhanaich99@yahoo.com \
    --cc=davej@suse.de \
    --cc=kernel-newbies@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalde@transmeta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®