public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [patch] aic79xx hostraid support
@ 2004-08-21 13:32 Arjan van de Ven
  2004-08-21 14:22 ` Arjan van de Ven
  0 siblings, 1 reply; 7+ messages in thread
From: Arjan van de Ven @ 2004-08-21 13:32 UTC (permalink / raw)
  To: linux-scsi

Hi,

Below is a patch ported to 2.6.8 by John A. Hull (john_hull@dell.com) to add
support for the aic79xx hostraid family support. Please consider.

Greetings,
    Arjan van de Ven

diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx.h linux/drivers/scsi/aic7xxx/aic79xx.h
--- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx.h	2004-08-14 07:37:25.000000000 +0200
+++ linux/drivers/scsi/aic7xxx/aic79xx.h	2004-08-21 15:29:58.977305228 +0200
@@ -375,7 +375,8 @@ typedef enum {
 	AHD_RESET_POLL_ACTIVE = 0x200000,
 	AHD_UPDATE_PEND_CMDS  = 0x400000,
 	AHD_RUNNING_QOUTFIFO  = 0x800000,
-	AHD_HAD_FIRST_SEL     = 0x1000000
+	AHD_HAD_FIRST_SEL     = 0x1000000,
+ 	AHD_HOSTRAID_BOARD    = 0x2000000
 } ahd_flag;
 
 /************************* Hardware  SCB Definition ***************************/
diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_osm_pci.c linux/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
--- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_osm_pci.c	2004-08-14 07:36:33.000000000 +0200
+++ linux/drivers/scsi/aic7xxx/aic79xx_osm_pci.c	2004-08-21 15:27:18.902556250 +0200
@@ -64,6 +64,10 @@ static struct pci_device_id ahd_linux_pc
 		0x9005, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
 		PCI_CLASS_STORAGE_SCSI << 8, 0xFFFF00, 0
 	},
+	{
+		0x9005, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
+		PCI_CLASS_STORAGE_RAID << 8, 0xFFFF00, 0
+	},
 	{ 0 }
 };
 
diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_pci.c linux/drivers/scsi/aic7xxx/aic79xx_pci.c
--- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_pci.c	2004-08-14 07:36:57.000000000 +0200
+++ linux/drivers/scsi/aic7xxx/aic79xx_pci.c	2004-08-21 15:27:18.903556130 +0200
@@ -65,10 +65,10 @@ ahd_compose_id(u_int device, u_int vendo
 }
 
 #define ID_ALL_MASK			0xFFFFFFFFFFFFFFFFull
-#define ID_ALL_IROC_MASK		0xFFFFFF7FFFFFFFFFull
+#define ID_ALL_IROC_MASK		0xFF7FFFFFFFFFFFFFull
 #define ID_DEV_VENDOR_MASK		0xFFFFFFFF00000000ull
 #define ID_9005_GENERIC_MASK		0xFFF0FFFF00000000ull
-#define ID_9005_GENERIC_IROC_MASK	0xFFF0FF7F00000000ull
+#define ID_9005_GENERIC_IROC_MASK	0xFF70FFFF00000000ull
 
 #define ID_AIC7901			0x800F9005FFFF9005ull
 #define ID_AHA_29320A			0x8000900500609005ull
@@ -92,6 +92,8 @@ ahd_compose_id(u_int device, u_int vendo
 #define ID_AIC7902_PCI_REV_B0		0x10
 #define SUBID_HP			0x0E11
 
+#define DEVID_9005_HOSTRAID(id) ((id) & 0x80)
+
 #define DEVID_9005_TYPE(id) ((id) & 0xF)
 #define		DEVID_9005_TYPE_HBA		0x0	/* Standard Card */
 #define		DEVID_9005_TYPE_HBA_2EXT	0x1	/* 2 External Ports */
@@ -134,18 +136,18 @@ struct ahd_pci_identity ahd_pci_ident_ta
 		"Adaptec 29320ALP Ultra320 SCSI adapter",
 		ahd_aic7901_setup
 	},
-	/* aic7901A based controllers */
+	/* aic7902 based controllers */	
 	{
 		ID_AHA_29320,
 		ID_ALL_MASK,
 		"Adaptec 29320 Ultra320 SCSI adapter",
-		ahd_aic7901A_setup
+		ahd_aic7902_setup
 	},
 	{
 		ID_AHA_29320B,
 		ID_ALL_MASK,
 		"Adaptec 29320B Ultra320 SCSI adapter",
-		ahd_aic7901A_setup
+		ahd_aic7902_setup
 	},
 	{
 		ID_AHA_29320LP,
@@ -153,7 +155,6 @@ struct ahd_pci_identity ahd_pci_ident_ta
 		"Adaptec 29320LP Ultra320 SCSI adapter",
 		ahd_aic7901A_setup
 	},
-	/* aic7902 based controllers */	
 	{
 		ID_AHA_39320,
 		ID_ALL_MASK,
@@ -196,22 +197,10 @@ struct ahd_pci_identity ahd_pci_ident_ta
 		"Adaptec (HP OEM) 39320D Ultra320 SCSI adapter",
 		ahd_aic7902_setup
 	},
-	{
-		ID_AHA_29320,
-		ID_ALL_MASK,
-		"Adaptec 29320 Ultra320 SCSI adapter",
-		ahd_aic7902_setup
-	},
-	{
-		ID_AHA_29320B,
-		ID_ALL_MASK,
-		"Adaptec 29320B Ultra320 SCSI adapter",
-		ahd_aic7902_setup
-	},
 	/* Generic chip probes for devices we don't know 'exactly' */
 	{
-		ID_AIC7901 & ID_DEV_VENDOR_MASK,
-		ID_DEV_VENDOR_MASK,
+		ID_AIC7901 & ID_9005_GENERIC_MASK,
+		ID_9005_GENERIC_MASK,
 		"Adaptec AIC7901 Ultra320 SCSI adapter",
 		ahd_aic7901_setup
 	},
@@ -294,6 +283,12 @@ ahd_find_pci_device(ahd_dev_softc_t pci)
 				 subdevice,
 				 subvendor);
 
+	/*
+	 * Controllers, mask out the IROC/HostRAID bit
+	 */
+	
+	full_id &= ID_ALL_IROC_MASK;
+
 	for (i = 0; i < ahd_num_pci_devs; i++) {
 		entry = &ahd_pci_ident_table[i];
 		if (entry->full_id == (full_id & entry->id_mask)) {
@@ -313,12 +308,21 @@ ahd_pci_config(struct ahd_softc *ahd, st
 	u_long		 l;
 	u_int		 command;
 	uint32_t	 devconfig;
+	uint16_t	 device; 
 	uint16_t	 subvendor; 
 	int		 error;
 
 	shared_scb_data = NULL;
 	ahd->description = entry->name;
 	/*
+	 * Record if this is a HostRAID board.
+	 */
+	device = ahd_pci_read_config(ahd->dev_softc,
+				     PCIR_DEVICE, /*bytes*/2);
+	if (DEVID_9005_HOSTRAID(device))
+		ahd->flags |= AHD_HOSTRAID_BOARD;
+
+	/*
 	 * Record if this is an HP board.
 	 */
 	subvendor = ahd_pci_read_config(ahd->dev_softc,

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [patch] aic79xx hostraid support
  2004-08-21 13:32 [patch] aic79xx hostraid support Arjan van de Ven
@ 2004-08-21 14:22 ` Arjan van de Ven
  2004-08-23 13:45   ` Luben Tuikov
  0 siblings, 1 reply; 7+ messages in thread
From: Arjan van de Ven @ 2004-08-21 14:22 UTC (permalink / raw)
  To: linux-scsi

On Sat, Aug 21, 2004 at 04:22:06PM +0200, Arjan van de Ven wrote:
> Hi,
> 
> Below is a patch ported to 2.6.8 by John A. Hull (john_hull@dell.com) to add
> support for the aic79xx hostraid family support. Please consider.


since AHD_HOSTRAID_BOARD wasn't actually used... a more minimal patch:

diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_osm_pci.c linux/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
--- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_osm_pci.c	2004-08-14 07:36:33.000000000 +0200
+++ linux/drivers/scsi/aic7xxx/aic79xx_osm_pci.c	2004-08-21 15:27:18.902556250 +0200
@@ -64,6 +64,10 @@ static struct pci_device_id ahd_linux_pc
 		0x9005, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
 		PCI_CLASS_STORAGE_SCSI << 8, 0xFFFF00, 0
 	},
+	{
+		0x9005, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
+		PCI_CLASS_STORAGE_RAID << 8, 0xFFFF00, 0
+	},
 	{ 0 }
 };
 
diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_pci.c linux/drivers/scsi/aic7xxx/aic79xx_pci.c
--- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_pci.c	2004-08-14 07:36:57.000000000 +0200
+++ linux/drivers/scsi/aic7xxx/aic79xx_pci.c	2004-08-21 15:27:18.903556130 +0200
@@ -65,10 +65,10 @@ ahd_compose_id(u_int device, u_int vendo
 }
 
 #define ID_ALL_MASK			0xFFFFFFFFFFFFFFFFull
-#define ID_ALL_IROC_MASK		0xFFFFFF7FFFFFFFFFull
+#define ID_ALL_IROC_MASK		0xFF7FFFFFFFFFFFFFull
 #define ID_DEV_VENDOR_MASK		0xFFFFFFFF00000000ull
 #define ID_9005_GENERIC_MASK		0xFFF0FFFF00000000ull
-#define ID_9005_GENERIC_IROC_MASK	0xFFF0FF7F00000000ull
+#define ID_9005_GENERIC_IROC_MASK	0xFF70FFFF00000000ull
 
 #define ID_AIC7901			0x800F9005FFFF9005ull
 #define ID_AHA_29320A			0x8000900500609005ull
@@ -92,6 +92,8 @@ ahd_compose_id(u_int device, u_int vendo
 #define ID_AIC7902_PCI_REV_B0		0x10
 #define SUBID_HP			0x0E11
 
+#define DEVID_9005_HOSTRAID(id) ((id) & 0x80)
+
 #define DEVID_9005_TYPE(id) ((id) & 0xF)
 #define		DEVID_9005_TYPE_HBA		0x0	/* Standard Card */
 #define		DEVID_9005_TYPE_HBA_2EXT	0x1	/* 2 External Ports */
@@ -134,18 +136,18 @@ struct ahd_pci_identity ahd_pci_ident_ta
 		"Adaptec 29320ALP Ultra320 SCSI adapter",
 		ahd_aic7901_setup
 	},
-	/* aic7901A based controllers */
+	/* aic7902 based controllers */	
 	{
 		ID_AHA_29320,
 		ID_ALL_MASK,
 		"Adaptec 29320 Ultra320 SCSI adapter",
-		ahd_aic7901A_setup
+		ahd_aic7902_setup
 	},
 	{
 		ID_AHA_29320B,
 		ID_ALL_MASK,
 		"Adaptec 29320B Ultra320 SCSI adapter",
-		ahd_aic7901A_setup
+		ahd_aic7902_setup
 	},
 	{
 		ID_AHA_29320LP,
@@ -153,7 +155,6 @@ struct ahd_pci_identity ahd_pci_ident_ta
 		"Adaptec 29320LP Ultra320 SCSI adapter",
 		ahd_aic7901A_setup
 	},
-	/* aic7902 based controllers */	
 	{
 		ID_AHA_39320,
 		ID_ALL_MASK,
@@ -196,22 +197,10 @@ struct ahd_pci_identity ahd_pci_ident_ta
 		"Adaptec (HP OEM) 39320D Ultra320 SCSI adapter",
 		ahd_aic7902_setup
 	},
-	{
-		ID_AHA_29320,
-		ID_ALL_MASK,
-		"Adaptec 29320 Ultra320 SCSI adapter",
-		ahd_aic7902_setup
-	},
-	{
-		ID_AHA_29320B,
-		ID_ALL_MASK,
-		"Adaptec 29320B Ultra320 SCSI adapter",
-		ahd_aic7902_setup
-	},
 	/* Generic chip probes for devices we don't know 'exactly' */
 	{
-		ID_AIC7901 & ID_DEV_VENDOR_MASK,
-		ID_DEV_VENDOR_MASK,
+		ID_AIC7901 & ID_9005_GENERIC_MASK,
+		ID_9005_GENERIC_MASK,
 		"Adaptec AIC7901 Ultra320 SCSI adapter",
 		ahd_aic7901_setup
 	},
@@ -294,6 +283,12 @@ ahd_find_pci_device(ahd_dev_softc_t pci)
 				 subdevice,
 				 subvendor);
 
+	/*
+	 * Controllers, mask out the IROC/HostRAID bit
+	 */
+	
+	full_id &= ID_ALL_IROC_MASK;
+
 	for (i = 0; i < ahd_num_pci_devs; i++) {
 		entry = &ahd_pci_ident_table[i];
 		if (entry->full_id == (full_id & entry->id_mask)) {

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [patch] aic79xx hostraid support
  2004-08-21 14:22 ` Arjan van de Ven
@ 2004-08-23 13:45   ` Luben Tuikov
  2004-08-23 13:51     ` Arjan van de Ven
  0 siblings, 1 reply; 7+ messages in thread
From: Luben Tuikov @ 2004-08-23 13:45 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: linux-scsi

Those patches are from the more recent versions of the AIC drivers,
from Justin's website, verbatim but somehow reworked to compile for older
versions of the drivers.

Please don't claim that you wrote it or as your own work.

I've been sent _several_ such patches privately for review.
This seems to be a version of one.

Also, those patches do no allow one to disable probing for
HostRAID drivers, which some users of Linux prefer.

		Luben

Arjan van de Ven wrote:
> On Sat, Aug 21, 2004 at 04:22:06PM +0200, Arjan van de Ven wrote:
>  > Hi,
>  >
>  > Below is a patch ported to 2.6.8 by John A. Hull (john_hull@dell.com) 
> to add
>  > support for the aic79xx hostraid family support. Please consider.
> 
> 
> since AHD_HOSTRAID_BOARD wasn't actually used... a more minimal patch:
> 
> diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_osm_pci.c 
> linux/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
> --- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_osm_pci.c  2004-08-14 
> 07:36:33.000000000 +0200
> +++ linux/drivers/scsi/aic7xxx/aic79xx_osm_pci.c        2004-08-21 
> 15:27:18.902556250 +0200
> @@ -64,6 +64,10 @@ static struct pci_device_id ahd_linux_pc
>                 0x9005, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
>                 PCI_CLASS_STORAGE_SCSI << 8, 0xFFFF00, 0
>         },
> +       {
> +               0x9005, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
> +               PCI_CLASS_STORAGE_RAID << 8, 0xFFFF00, 0
> +       },
>         { 0 }
>  };
>  
> diff -purN linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_pci.c 
> linux/drivers/scsi/aic7xxx/aic79xx_pci.c
> --- linux-2.6.8/drivers/scsi/aic7xxx/aic79xx_pci.c      2004-08-14 
> 07:36:57.000000000 +0200
> +++ linux/drivers/scsi/aic7xxx/aic79xx_pci.c    2004-08-21 
> 15:27:18.903556130 +0200
> @@ -65,10 +65,10 @@ ahd_compose_id(u_int device, u_int vendo
>  }
>  
>  #define ID_ALL_MASK                    0xFFFFFFFFFFFFFFFFull
> -#define ID_ALL_IROC_MASK               0xFFFFFF7FFFFFFFFFull
> +#define ID_ALL_IROC_MASK               0xFF7FFFFFFFFFFFFFull
>  #define ID_DEV_VENDOR_MASK             0xFFFFFFFF00000000ull
>  #define ID_9005_GENERIC_MASK           0xFFF0FFFF00000000ull
> -#define ID_9005_GENERIC_IROC_MASK      0xFFF0FF7F00000000ull
> +#define ID_9005_GENERIC_IROC_MASK      0xFF70FFFF00000000ull
>  
>  #define ID_AIC7901                     0x800F9005FFFF9005ull
>  #define ID_AHA_29320A                  0x8000900500609005ull
> @@ -92,6 +92,8 @@ ahd_compose_id(u_int device, u_int vendo
>  #define ID_AIC7902_PCI_REV_B0          0x10
>  #define SUBID_HP                       0x0E11
>  
> +#define DEVID_9005_HOSTRAID(id) ((id) & 0x80)
> +
>  #define DEVID_9005_TYPE(id) ((id) & 0xF)
>  #define                DEVID_9005_TYPE_HBA             0x0     /* 
> Standard Card */
>  #define                DEVID_9005_TYPE_HBA_2EXT        0x1     /* 2 
> External Ports */
> @@ -134,18 +136,18 @@ struct ahd_pci_identity ahd_pci_ident_ta
>                 "Adaptec 29320ALP Ultra320 SCSI adapter",
>                 ahd_aic7901_setup
>         },
> -       /* aic7901A based controllers */
> +       /* aic7902 based controllers */
>         {
>                 ID_AHA_29320,
>                 ID_ALL_MASK,
>                 "Adaptec 29320 Ultra320 SCSI adapter",
> -               ahd_aic7901A_setup
> +               ahd_aic7902_setup
>         },
>         {
>                 ID_AHA_29320B,
>                 ID_ALL_MASK,
>                 "Adaptec 29320B Ultra320 SCSI adapter",
> -               ahd_aic7901A_setup
> +               ahd_aic7902_setup
>         },
>         {
>                 ID_AHA_29320LP,
> @@ -153,7 +155,6 @@ struct ahd_pci_identity ahd_pci_ident_ta
>                 "Adaptec 29320LP Ultra320 SCSI adapter",
>                 ahd_aic7901A_setup
>         },
> -       /* aic7902 based controllers */
>         {
>                 ID_AHA_39320,
>                 ID_ALL_MASK,
> @@ -196,22 +197,10 @@ struct ahd_pci_identity ahd_pci_ident_ta
>                 "Adaptec (HP OEM) 39320D Ultra320 SCSI adapter",
>                 ahd_aic7902_setup
>         },
> -       {
> -               ID_AHA_29320,
> -               ID_ALL_MASK,
> -               "Adaptec 29320 Ultra320 SCSI adapter",
> -               ahd_aic7902_setup
> -       },
> -       {
> -               ID_AHA_29320B,
> -               ID_ALL_MASK,
> -               "Adaptec 29320B Ultra320 SCSI adapter",
> -               ahd_aic7902_setup
> -       },
>         /* Generic chip probes for devices we don't know 'exactly' */
>         {
> -               ID_AIC7901 & ID_DEV_VENDOR_MASK,
> -               ID_DEV_VENDOR_MASK,
> +               ID_AIC7901 & ID_9005_GENERIC_MASK,
> +               ID_9005_GENERIC_MASK,
>                 "Adaptec AIC7901 Ultra320 SCSI adapter",
>                 ahd_aic7901_setup
>         },
> @@ -294,6 +283,12 @@ ahd_find_pci_device(ahd_dev_softc_t pci)
>                                  subdevice,
>                                  subvendor);
>  
> +       /*
> +        * Controllers, mask out the IROC/HostRAID bit
> +        */
> +      
> +       full_id &= ID_ALL_IROC_MASK;
> +
>         for (i = 0; i < ahd_num_pci_devs; i++) {
>                 entry = &ahd_pci_ident_table[i];
>                 if (entry->full_id == (full_id & entry->id_mask)) {
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [patch] aic79xx hostraid support
  2004-08-23 13:45   ` Luben Tuikov
@ 2004-08-23 13:51     ` Arjan van de Ven
  2004-08-23 14:02       ` Luben Tuikov
  0 siblings, 1 reply; 7+ messages in thread
From: Arjan van de Ven @ 2004-08-23 13:51 UTC (permalink / raw)
  To: Luben Tuikov; +Cc: linux-scsi

[-- Attachment #1: Type: text/plain, Size: 576 bytes --]

On Mon, Aug 23, 2004 at 09:45:06AM -0400, Luben Tuikov wrote:
> Those patches are from the more recent versions of the AIC drivers,
> from Justin's website, verbatim but somehow reworked to compile for older
> versions of the drivers.
> 
> Please don't claim that you wrote it or as your own work.

I do not and never have claimed it as my own work.

Do you agree that this patch is a useful addition for now, until you start
sending similar small improvements to linux-scsi for the drivers?

(I for sure think it would be a bad thing if aic7xxx entirely halted pending
that)

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [patch] aic79xx hostraid support
  2004-08-23 13:51     ` Arjan van de Ven
@ 2004-08-23 14:02       ` Luben Tuikov
  2004-08-23 14:05         ` Arjan van de Ven
  0 siblings, 1 reply; 7+ messages in thread
From: Luben Tuikov @ 2004-08-23 14:02 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: linux-scsi

Arjan van de Ven wrote:
> On Mon, Aug 23, 2004 at 09:45:06AM -0400, Luben Tuikov wrote:
> 
>>Those patches are from the more recent versions of the AIC drivers,
>>from Justin's website, verbatim but somehow reworked to compile for older
>>versions of the drivers.
>>
>>Please don't claim that you wrote it or as your own work.
> 
> 
> I do not and never have claimed it as my own work.
> 
> Do you agree that this patch is a useful addition for now, until you start
> sending similar small improvements to linux-scsi for the drivers?

Yes, sure, if it works for people _and_ they realize that it is:
a) temporary,
b) taken from the later version of the driver on Justin's website, and
c) it will be scrapped completely since the later versions which I'll
   be sending *contain and absolve* this or any other similar patches.
 
> (I for sure think it would be a bad thing if aic7xxx entirely halted pending
> that)

Yes, I agree.

		Luben



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [patch] aic79xx hostraid support
  2004-08-23 14:02       ` Luben Tuikov
@ 2004-08-23 14:05         ` Arjan van de Ven
  2004-08-23 16:56           ` James Bottomley
  0 siblings, 1 reply; 7+ messages in thread
From: Arjan van de Ven @ 2004-08-23 14:05 UTC (permalink / raw)
  To: Luben Tuikov; +Cc: linux-scsi

[-- Attachment #1: Type: text/plain, Size: 1321 bytes --]

On Mon, Aug 23, 2004 at 10:02:30AM -0400, Luben Tuikov wrote:
> Arjan van de Ven wrote:
> >On Mon, Aug 23, 2004 at 09:45:06AM -0400, Luben Tuikov wrote:
> >
> >>Those patches are from the more recent versions of the AIC drivers,
> >>from Justin's website, verbatim but somehow reworked to compile for older
> >>versions of the drivers.
> >>
> >>Please don't claim that you wrote it or as your own work.
> >
> >
> >I do not and never have claimed it as my own work.
> >
> >Do you agree that this patch is a useful addition for now, until you start
> >sending similar small improvements to linux-scsi for the drivers?
> 
> Yes, sure, if it works for people _and_ they realize that it is:
> a) temporary,
> b) taken from the later version of the driver on Justin's website, and
> c) it will be scrapped completely since the later versions which I'll
>   be sending *contain and absolve* this or any other similar patches.

sure since I assume your later drop will also just support the hardware I
dont see a user regression in this... I mean, when you replace the code the
hardware keeps working, end user remains happy. Nobody is married to this
patch; I just care about getting this new hardware working without having to
go to a whopping gigantic patch immediately.

So it seems we are basically in agreement ;)

Thanks.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [patch] aic79xx hostraid support
  2004-08-23 14:05         ` Arjan van de Ven
@ 2004-08-23 16:56           ` James Bottomley
  0 siblings, 0 replies; 7+ messages in thread
From: James Bottomley @ 2004-08-23 16:56 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: Luben Tuikov, SCSI Mailing List

On Mon, 2004-08-23 at 10:05, Arjan van de Ven wrote:
> So it seems we are basically in agreement ;)

OK, I'll put it in, then.

James



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2004-08-23 17:01 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-08-21 13:32 [patch] aic79xx hostraid support Arjan van de Ven
2004-08-21 14:22 ` Arjan van de Ven
2004-08-23 13:45   ` Luben Tuikov
2004-08-23 13:51     ` Arjan van de Ven
2004-08-23 14:02       ` Luben Tuikov
2004-08-23 14:05         ` Arjan van de Ven
2004-08-23 16:56           ` James Bottomley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox