Re: Linux v2.5.50

[Adrian Bunk]

On Wed, Nov 27, 2002 at 03:07:38PM -0800, Linus Torvalds wrote:

>...
> Summary of changes from v2.5.49 to v2.5.50
> ============================================
>...
> Doug Ledford <[e-mail]>:
>...
>   o Convert from host->host_queue to host->my_devices list usage Add in
>     usage of new same_target_siblings list Add
>     scsi_release_commandblocks() call to scsi_free_sdev() Make all scsi
>     device freeing use scsi_free_sdev()
>...

The change to eata_pio_proc.c contains a typo:

<--  snip  -->

...
  gcc -Wp,-MD,drivers/scsi/.eata_pio.o.d -D__KERNEL__ -Iinclude -Wall 
-Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
-fomit-frame-pointer -pipe -mpreferred-stack-boundary=2 -march=k6 -Iarch/i386/mach-generic 
-nostdinc -iwithprefix include    -DKBUILD_BASENAME=eata_pio 
-DKBUILD_MODNAME=eata_pio 
  -c -o drivers/scsi/eata_pio.o drivers/scsi/eata_pio.c
In file included from drivers/scsi/eata_pio.c:94:
drivers/scsi/eata_pio_proc.c:88: macro `list_for_each_entry' used with only 2 args
In file included from drivers/scsi/eata_pio.c:94:
drivers/scsi/eata_pio_proc.c: In function `eata_pio_proc_info':
drivers/scsi/eata_pio_proc.c:88: parse error before `)'
...
make[2]: *** [drivers/scsi/eata_pio.o] Error 1

<--  snip  -->

The fix is simple:

--- linux-2.5.50/drivers/scsi/eata_pio_proc.c.old	2002-12-01 15:32:22.000000000 +0100
+++ linux-2.5.50/drivers/scsi/eata_pio_proc.c	2002-12-01 15:32:50.000000000 +0100
@@ -85,7 +85,7 @@
     len += size; 
     pos = begin + len;
     
-    list_for_each_entry(scd, &HBA_ptr->my_devices; siblings) {
+    list_for_each_entry(scd, &HBA_ptr->my_devices, siblings) {
 	    proc_print_scsidevice(scd, buffer, &size, len);
 	    len += size; 
 	    pos = begin + len;

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed