Patch?: linux-2.5.59/sound/soundcore.c referenced non-existant errno variable

[Adam J. Richter]

This is a MIME-formatted message.  If you see this text it means that your
E-mail software does not support MIME-formatted messages.

--=_courier-533-1042848724-0001-2
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

	linux-2.5.59/sound/sound_firmware.c attempts to use the
user level system call interface from the kernel, which I understand
works on i386 and perhaps all architectures, but requires a variable
named "errno."  (Actually, it mixed things like close() and
sys_close(), but that's beside the point.)

	I could just declare a "static int errno;" in the file, but I
thought it might be helpful to use filp_open, vfs_read and filp_close
instead, which shaves a few cpu cycles and avoids the need to allocate
a file descriptor number.  I think that this also seems to be the
convention for other kernel facilities that want to open their own
files briefly.  I've attached a patch below.  The patch shrinks
sound_firmware.c by six lines.  I have only verified that the patch
compiles and that the soundcore module now loads.

	Comments are welcome.  If this patch looks good, then I'd
appreciate it if someone more connected to sound development would
integrate it and forward it to Linus.

-- 
Adam J. Richter     __     ______________   575 Oroville Road
[e-mail]     \ /                  Milpitas, California 95035
[phone]         | g g d r a s i l   United States of America
                         "Free Software For The Rest Of Us."

--=_courier-533-1042848724-0001-2
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="fw.diff"

--- linux-2.5.59/sound/sound_firmware.c	2003-01-16 18:21:34.000000000 -0800
+++ linux/sound/sound_firmware.c	2003-01-17 15:38:38.000000000 -0800
@@ -9,41 +9,35 @@
 
 static int do_mod_firmware_load(const char *fn, char **fp)
 {
-	int fd;
+	struct file *file;
 	long l;
 	char *dp;
 
-	fd = open(fn, 0, 0);
-	if (fd == -1)
+	file = filp_open(fn, O_RDONLY, 0);
+	if (!file)
 	{
 		printk(KERN_INFO "Unable to load '%s'.\n", fn);
 		return 0;
 	}
-	l = lseek(fd, 0L, 2);
+	l = file->f_dentry->d_inode->i_size;
 	if (l <= 0 || l > 131072)
 	{
 		printk(KERN_INFO "Invalid firmware '%s'\n", fn);
-		sys_close(fd);
+		filp_close(file, NULL);
 		return 0;
 	}
-	lseek(fd, 0L, 0);
 	dp = vmalloc(l);
-	if (dp == NULL)
-	{
-		printk(KERN_INFO "Out of memory loading '%s'.\n", fn);
-		sys_close(fd);
-		return 0;
-	}
-	if (read(fd, dp, l) != l)
-	{
+	if (dp != NULL) {
+		if (vfs_read(file, dp, l, &file->f_pos) == l) {
+			filp_close(file, NULL);
+			*fp = dp;
+			return (int) l;
+		}
 		printk(KERN_INFO "Failed to read '%s'.\n", fn);
 		vfree(dp);
-		sys_close(fd);
-		return 0;
 	}
-	close(fd);
-	*fp = dp;
-	return (int) l;
+	filp_close(file, NULL);
+	return 0;
 }
 
 /**

--=_courier-533-1042848724-0001-2--