Re: highpoint driver problem, 2.4.21-ac4

[Wilfried Weissmann <Wilfried.Weissmann@gmx.at>]

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

Wil Reichert wrote:
> Thats the kernel I'm using.

could you try the attachted patch, and report if this changes something?

bye,
wilfried

> 
> Wil
> 
> On Thu, 03 Jul 2003 21:21:30 +0200
> Wilfried Weissmann <Wilfried.Weissmann@gmx.at> wrote:
> 
> 
>>Wil Reichert wrote:
>> >>> The on-board Highpoint controller (HPT372A) on my DFI NF2 is
>> >>> having issue.  Loading the hptraid module results in a 'No such
>> >>> device' message while the hpt366 module segfaults and leaves an
>> >>> oops in my logs.  These errors occur regardless of the disk/raid
>> >>> configuration in the hpt BIOS.   Following are the oops trace, an
>> >>> lsmod, the .config and a lspci -vvv.
>> >>
>> >> The crash occurs in the hpt366 module. Loading hptraid will not
>> >> work because it depends on the kernel to claim the disks of the
>> >> raid volume (that is what hpt366 would do). I will add autoloading
>> >> of the ide-controller module in the next raid-driver release.
>> >> However, I do not know why the kernel oopses. You might want to try
>> >> to build the hpt366 code into the kernel instead of a module. If it
>> >> works it would probably mean that "ide_hwif_t *hwif" was not
>> >> properly initalized.
>> >
>> > I initially had all the hpt modules built into the kernel, but that
>> > would also produce an oops and die immediately after ID'ing the two
>> > drives I have on attached.  Would any more information be of use to
>> > you?
>>
>>2.4.21-ac4 contains several fixes for the hpt37x. Please try this patch out.
>>
>>bye,
>>Wilfried

[-- Attachment #2: linux-2.4.21-ac4-idedmafix.patch --]
[-- Type: text/plain, Size: 489 bytes --]

--- linux/drivers/ide/setup-pci.c.orig	2003-07-06 00:04:06.000000000 +0200
+++ linux/drivers/ide/setup-pci.c	2003-07-06 00:04:12.000000000 +0200
@@ -172,7 +172,7 @@ static int ide_setup_pci_baseregs (struc
  *	is already in DMA mode we check and enforce IDE simplex rules.
  */
 
-static unsigned long __init ide_get_or_set_dma_base (ide_hwif_t *hwif)
+static unsigned long ide_get_or_set_dma_base (ide_hwif_t *hwif)
 {
 	unsigned long	dma_base = 0;
 	struct pci_dev	*dev = hwif->pci_dev;