2.2.16pre7: Getting rid of some warnings

From: Horst von Brand (vonbrand@sleipnir.valparaiso.cl)
Date: Wed May 31 2000 - 21:29:08 EST


i686 UP here. Will take a similar look at sparc32 UP tomorrow.

The following patch fixes some warnings due to unused (really!)
variables. Some remain here on UP due to spinlock operations #defined as
nothing, but what the heck. One format (or perhaps variable definition?) is
wrong in drivers/block/raid0.c.

I'm running binutils-2.9.5.0.42, which complains all over the place about
stuff like:

   {standard input}:1890: Warning: using `%eax' instead of `%ax' due to `l' suffix

I remember there was an issue about non-optimal code from older binutils.
Are the required binutils-2.8.1.0.23 new enough to fix this now or do we
still have to wait?

There are a lot of complaints in arch/i386/kernel/bios.c like:

   {standard input}:1112: Warning: indirect lcall without `*'

Again, an old vs. new binutils issue, or should this be fixed?

What was the fix again for:

   {standard input}:15: Warning: Ignoring changed section attributes for .modinfo

which I'm getting for most all modules?

--- linux/fs/isofs/joliet.c~ Wed May 31 20:16:45 2000
+++ linux/fs/isofs/joliet.c Wed May 31 22:09:43 2000
@@ -19,8 +19,6 @@
             struct nls_table *nls)
 {
         unsigned char *ip, *op;
- unsigned char ch, cl;
- unsigned char *uni_page;
 
         ip = uni;
         op = ascii;
--- linux/fs/fat/dir.c~ Wed May 31 20:16:45 2000
+++ linux/fs/fat/dir.c Wed May 31 22:06:01 2000
@@ -68,8 +68,6 @@
             struct nls_table *nls)
 {
         unsigned char *ip, *op;
- unsigned char ch, cl;
- unsigned char *uni_page;
         unsigned short val;
 
         ip = uni;
--- linux/drivers/net/8390.c~ Tue Oct 26 21:53:40 1999
+++ linux/drivers/net/8390.c Wed May 31 21:59:20 2000
@@ -1089,7 +1089,6 @@
                                                                 int start_page)
 {
         long e8390_base = dev->base_addr;
- struct ei_device *ei_local = (struct ei_device *) dev->priv;
    
         outb_p(E8390_NODMA+E8390_PAGE0, e8390_base+E8390_CMD);
     
--- linux/drivers/block/raid0.c~ Tue Jan 4 15:12:14 2000
+++ linux/drivers/block/raid0.c Wed May 31 21:54:08 2000
@@ -184,7 +184,7 @@
 
   if (hash - data->hash_table > data->nr_zones)
   {
- printk(KERN_DEBUG "raid0_map: invalid block %ul\n", block);
+ printk(KERN_DEBUG "raid0_map: invalid block %l\n", block);
           return -1;
   }
 
--- linux/arch/i386/kernel/bios32.c~ Wed May 31 20:16:41 2000
+++ linux/arch/i386/kernel/bios32.c Wed May 31 21:48:04 2000
@@ -1055,7 +1055,6 @@
         /*
          * Find and scan busses behind ServerWorks north bridge chips
          */
- struct pci_bus *bus;
         unsigned char busno, busmax;
         pci_probe |= PCI_NO_PEER_FIXUP;
         pci_read_config_byte(d, 0x44, &busno);

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Jun 07 2000 - 21:00:11 EST