Skip to contentSkip to conversation

Small fixes for -rc1 kernel

PATCH Paul P Komkoff Jr 2 participants 4 messages Jul 11, 2002 – Jul 12, 2002

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

I'm going to repeat myself.

But Marcelo seems to ignore this from me so maybe from yours ...

1. (found using kbuild 2.5) when binfmt_elf is in module, it won't load due to unresolved symbols. Two solutions - remove M choice or export symbols. I dunno how much will be broken by making elf modular ...

2. Long lasting issue with wan/comx and proc_get_inode

3. disable_ide_dma in dmi_scan unused now

4. undeclared function

5. unused locals

6. unused label

Next will be -ac specific part ...

# 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 v2.4.19-rc1 -> 1.558
# drivers/char/ip2/i2ellis.h 1.2 -> 1.3
# fs/proc/Makefile 1.2 -> 1.3
# arch/i386/kernel/Makefile 1.3 -> 1.4
# fs/dnotify.c 1.3 -> 1.4
# fs/proc/inode.c 1.5 -> 1.6
# arch/i386/kernel/dmi_scan.c 1.21 -> 1.22
# arch/i386/kernel/smpboot.c 1.9 -> 1.10
# drivers/char/tpqic02.c 1.8 -> 1.9
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 02/06/24 [email protected] 1.552
# Makefile:
# Changed EXTRAVERSION to -rc1
# --------------------------------------------
# 02/07/11 [email protected] 1.553
# Fix unresolved symbols in modular binfmt_elf
# --------------------------------------------
# 02/07/11 [email protected] 1.554
# Make drivers/net/wan/comx happy about proc_get_inode
# --------------------------------------------
# 02/07/11 [email protected] 1.555
# Disable disable_ide_dma - it is of no use for now and making gcc cry # --------------------------------------------
# 02/07/11 [email protected] 1.556

# declare iiEllisCleanup before using and make gcc shut up about it
# --------------------------------------------
# 02/07/11	[email protected]	1.557
# Two unused local variables in tpqic02
# --------------------------------------------
# 02/07/11	[email protected]	1.558
# One unused local label in dnotify
# --------------------------------------------
#
diff -Nru a/arch/i386/kernel/Makefile b/arch/i386/kernel/Makefile
--- a/arch/i386/kernel/Makefile	Thu Jul 11 23:38:30 2002
+++ b/arch/i386/kernel/Makefile	Thu Jul 11 23:38:30 2002
@@ -14,7 +14,7 @@

O_TARGET := kernel.o

-export-objs := mca.o mtrr.o msr.o cpuid.o microcode.o i386_ksyms.o
+export-objs := mca.o mtrr.o msr.o cpuid.o microcode.o i386_ksyms.o smpboot.o

obj-y := process.o semaphore.o signal.o entry.o traps.o irq.o vm86.o \
ptrace.o i8259.o ioport.o ldt.o setup.o time.o sys_i386.o \

arch/i386/kernel/dmi_scan.c +5 -0
diff -Nru a/arch/i386/kernel/dmi_scan.c b/arch/i386/kernel/dmi_scan.c
@@ -185,11 +185,15 @@
 #define NO_MATCH	{ NONE, NULL}
 #define MATCH(a,b)	{ a, b }
 
+#if 0
+
 /*
  *	We have problems with IDE DMA on some platforms. In paticular the
  *	KT7 series. On these it seems the newer BIOS has fixed them. The
  *	rule needs to be improved to match specific BIOS revisions with
  *	corruption problems
+ *
+ *	FIXME: Either remove or reenable (Stingray)
  */ 
  
 static __init int disable_ide_dma(struct dmi_blacklist *d)
@@ -204,6 +208,7 @@
 #endif	
 	return 0;
 }
+#endif
 
 /* 
  * Reboot options and system auto-detection code provided by
arch/i386/kernel/smpboot.c +3 -0
diff -Nru a/arch/i386/kernel/smpboot.c b/arch/i386/kernel/smpboot.c
@@ -33,6 +33,7 @@
  */
 
 #include <linux/config.h>
+#include <linux/module.h>
 #include <linux/init.h>
 
 #include <linux/mm.h>
@@ -1222,3 +1223,5 @@
 smp_done:
 	zap_low_mappings();
 }
+
+EXPORT_SYMBOL(smp_num_siblings);
drivers/char/ip2/i2ellis.h +1 -0
diff -Nru a/drivers/char/ip2/i2ellis.h b/drivers/char/ip2/i2ellis.h
@@ -568,6 +568,7 @@
 // the board again (step 1 not needed).
 
 static void iiEllisInit(void);
+static void iiEllisCleanup(void);
 static int iiSetAddress(i2eBordStrPtr, int, delayFunc_t );
 static int iiReset(i2eBordStrPtr);
 static int iiResetDelay(i2eBordStrPtr);
drivers/char/tpqic02.c +0 -2
diff -Nru a/drivers/char/tpqic02.c b/drivers/char/tpqic02.c
@@ -1815,7 +1815,6 @@
 static ssize_t qic02_tape_read(struct file *filp, char *buf, size_t count,
 			       loff_t * ppos)
 {
-	int err;
 	kdev_t dev = filp->f_dentry->d_inode->i_rdev;
 	unsigned short flags = filp->f_flags;
 	unsigned long bytes_todo, bytes_done, total_bytes_done = 0;
@@ -2009,7 +2008,6 @@
 static ssize_t qic02_tape_write(struct file *filp, const char *buf,
 				size_t count, loff_t * ppos)
 {
-	int err;
 	kdev_t dev = filp->f_dentry->d_inode->i_rdev;
 	unsigned short flags = filp->f_flags;
 	unsigned long bytes_todo, bytes_done, total_bytes_done = 0;
fs/dnotify.c +1 -1
diff -Nru a/fs/dnotify.c b/fs/dnotify.c
@@ -135,7 +135,7 @@
 	}
 	if (changed)
 		redo_inode_mask(inode);
-out:
+
 	write_unlock(&dn_lock);
 }
 
fs/proc/Makefile +1 -1
diff -Nru a/fs/proc/Makefile b/fs/proc/Makefile
@@ -9,7 +9,7 @@
 
 O_TARGET := proc.o
 
-export-objs := root.o
+export-objs := root.o inode.o
 
 obj-y    := inode.o root.o base.o generic.o array.o \
 		kmsg.o proc_tty.o proc_misc.o kcore.o
fs/proc/inode.c +4 -1
diff -Nru a/fs/proc/inode.c b/fs/proc/inode.c
@@ -14,6 +14,7 @@
 #include <linux/locks.h>
 #include <linux/limits.h>
 #define __NO_VERSION__
+#include <linux/config.h>
 #include <linux/module.h>
 #include <linux/smp_lock.h>
 
@@ -174,7 +175,9 @@
 out_fail:
 	de_put(de);
 	goto out;
-}			
+}
+
+EXPORT_SYMBOL(proc_get_inode);
 
 struct super_block *proc_read_super(struct super_block *s,void *data, 
 				    int silent)
Signature · 2 lines
Paul P 'Stingray' Komkoff 'Greatest' Jr /// (icq)23200764 /// (http)stingr.net
  When you're invisible, the only one really watching you is you (my keychain)
#1 lore raw
Quoted 1 line

2. Long lasting issue with wan/comx and proc_get_inode

comx is broken. exporting proc_get_inode is still ABSOLUTELY wrong.

#2 lore raw

Replying to Christoph Hellwig:

Quoted 3 lines
Quoted 1 line

2. Long lasting issue with wan/comx and proc_get_inode

comx is broken. exporting proc_get_inode is still ABSOLUTELY wrong.

And who should fix it? I just don't understand the whole meaning of that piece of code which uses proc_get_inode (yet)

Signature · 2 lines
Paul P 'Stingray' Komkoff 'Greatest' Jr /// (icq)23200764 /// (http)stingr.net
  When you're invisible, the only one really watching you is you (my keychain)
#3 lore raw
Quoted 2 lines · Paul P Komkoff Jr

And who should fix it? I just don't understand the whole meaning of that piece of code which uses proc_get_inode (yet)

fix the driver to use proper procfs interfaces.