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
next 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®