public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Erik Mouw <erik@harddisk-recovery.nl>
To: Matthew Wilcox <willy@debian.org>
Cc: Marcelo Tosatti <marcelo.tosatti@cyclades.com>,
	Xose Vazquez Perez <xose@wanadoo.es>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: two sym53c8xx.o modules
Date: Thu, 9 Oct 2003 15:11:52 +0200	[thread overview]
Message-ID: <20031009131152.GG11525@bitwizard.nl> (raw)
In-Reply-To: <20031009123857.GC27861@parcelfarce.linux.theplanet.co.uk>

On Thu, Oct 09, 2003 at 01:38:57PM +0100, Matthew Wilcox wrote:
> Is it really important to change this at this stage of 2.4?  For
> reference, sym1 has already gone away in 2.6, so it'll be incovenient
> for users no matter what you change.  Let's stick with the devil we know.

Yes it is. The _2 driver supports newer hardware (or better: can do
faster speeds on newer hardware), while the old driver doesn't. The old
driver, however, is more reliable in case of hardware errors. It's very
nice to be able to change drivers on the fly without having to figure
out the full path to the correct module.

Hmm, just tested the _2 driver and now I remember why I hardly use it
with flakey drives:

  root@dagobert:~ # scsiinfo -eXR /dev/sda 0 0 0 0 0 0 0 0 0 0 0 0 0 1
  <4>sym0:4:0:extraneous data discarded.
  <4>sym0:4:0:extraneous data discarded.
  <4>sym0:4:0:extraneous data discarded.
  <4>sym0:4:0:extraneous data discarded.
  <4>sym0:4:0:extraneous data discarded.
  Unable to store Read-Write Error Recovery Page 01h
  18 00 00 00 18 00 00 00 70 00 05 00 00 00 00 18 
  00 00 00 00 26 00 00 80 02 00 01 0a 00 00 00 00 
  00 00 00 00 00 01 00 00 00 00 00 00 00 00 00 00 

I haven't figured out what it could be yet, scsiinfo works perfectly
well with the old sym58xxx driver (and the aic7xxx driver, FWIW).

Anyway, here's a patch to rename sym53c8xx_2/sym53c8xx.o to
sym53c8xx_2/sym53c8xx_2.o. It's against 2.4.22, but should apply to all
2.4 kernels.


Erik

--- linux/drivers/scsi/Makefile.old	Tue Aug 26 12:16:09 2003
+++ linux/drivers/scsi/Makefile	Thu Oct  9 14:48:40 2003
@@ -97,7 +97,7 @@
 obj-$(CONFIG_SCSI_NCR53C7xx)	+= 53c7,8xx.o 
 subdir-$(CONFIG_SCSI_SYM53C8XX_2)	+= sym53c8xx_2
 ifeq ($(CONFIG_SCSI_SYM53C8XX_2),y)
-  obj-$(CONFIG_SCSI_SYM53C8XX_2)	+= sym53c8xx_2/sym53c8xx.o
+  obj-$(CONFIG_SCSI_SYM53C8XX_2)	+= sym53c8xx_2/sym53c8xx_2.o
 endif
 obj-$(CONFIG_SCSI_SYM53C8XX)	+= sym53c8xx.o 
 obj-$(CONFIG_SCSI_NCR53C8XX)	+= ncr53c8xx.o 
--- linux/drivers/scsi/sym53c8xx_2/Makefile.old	Sat Nov 10 00:22:54 2001
+++ linux/drivers/scsi/sym53c8xx_2/Makefile	Thu Oct  9 14:49:43 2003
@@ -1,14 +1,14 @@
-# File: drivers/sym53c8xx/Makefile
+# File: drivers/scsi/sym53c8xx_2/Makefile
 # Makefile for the NCR/SYMBIOS/LSI 53C8XX PCI SCSI controllers driver.
 
-list-multi := sym53c8xx.o
-sym53c8xx-objs := sym_fw.o sym_glue.o sym_hipd.o sym_malloc.o sym_misc.o sym_nvram.o
-obj-$(CONFIG_SCSI_SYM53C8XX_2) := sym53c8xx.o
+list-multi := sym53c8xx_2.o
+sym53c8xx_2-objs := sym_fw.o sym_glue.o sym_hipd.o sym_malloc.o sym_misc.o sym_nvram.o
+obj-$(CONFIG_SCSI_SYM53C8XX_2) := sym53c8xx_2.o
 
 EXTRA_CFLAGS += -I.
 
-sym53c8xx.o: $(sym53c8xx-objs)
-	$(LD) -r -o $@ $(sym53c8xx-objs)
+sym53c8xx_2.o: $(sym53c8xx_2-objs)
+	$(LD) -r -o $@ $(sym53c8xx_2-objs)
 
 include $(TOPDIR)/Rules.make
 


-- 
+-- Erik Mouw -- www.harddisk-recovery.nl -- 0800 220 20 20 --
| Eigen lab: Delftechpark 26, 2628 XH, Delft, Nederland
| Files foetsie, bestanden kwijt, alle data weg?!
| Blijf kalm en neem contact op met Harddisk-recovery.nl!

  reply	other threads:[~2003-10-09 13:12 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-10-09  0:43 two sym53c8xx.o modules Xose Vazquez Perez
2003-10-09 11:41 ` Marcelo Tosatti
2003-10-09 12:24   ` Erik Mouw
2003-10-09 12:38     ` Matthew Wilcox
2003-10-09 13:11       ` Erik Mouw [this message]
2003-10-09 13:21         ` Matthew Wilcox
2003-10-09 14:04           ` Erik Mouw
2003-10-09 15:44             ` Xose Vazquez Perez
2003-10-09 13:00     ` Marcelo Tosatti
2003-10-09 16:05   ` Xose Vazquez Perez

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=20031009131152.GG11525@bitwizard.nl \
    --to=erik@harddisk-recovery.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=marcelo.tosatti@cyclades.com \
    --cc=willy@debian.org \
    --cc=xose@wanadoo.es \
    /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