Re: [PATCH 1/2] [ide] mmio ide support

2007-07-10 Thread Linas Vepstas
On Sun, Jul 08, 2007 at 03:15:41PM +0200, Bartlomiej Zolnierkiewicz wrote: on the argument that drivers/ide/ is going away soon. Most current distros have already moved over to using libata exclusively. The in-kernel default for PATA systems is still IDE subsystem. In part because

Re: [PATCH pata-2.6 fix] hpt366: use correct enablebits for HPT36x

2007-07-03 Thread Linas Vepstas
Hi, On Sat, Jun 30, 2007 at 12:04:22AM +0400, Sergei Shtylyov wrote: The HPT36x chips finally turned out to have the channel enable bits -- however, badly implemented. Make use of them despite it's probably only going to burden the driver's code -- assuming both channels are always

Re: [PATCH] hpt366: blacklist MAXTOR STM3320620A for UltraDMA/66

2007-07-03 Thread Linas Vepstas
On Fri, Jun 29, 2007 at 11:57:17PM +0400, Sergei Shtylyov wrote: Add the MAXTOR STM3320620A drive into the UltraDMA/66 mode blacklist for the HPT36x chips. Signed-off-by: Sergei Shtylyov [EMAIL PROTECTED] Acked-by: Linas Vepstas [EMAIL PROTECTED] Applied tested it worked for me. --linas

Re: [RFT] hpt366: reset DMA state machine on timeouts

2007-06-25 Thread Linas Vepstas
On Sat, Jun 23, 2007 at 10:10:34PM +0400, Sergei Shtylyov wrote: Now I'm confused too -- did you get any DMA timeouts this time? No. And yes, this is confusing, as the initial hang that I was seeing was preceeded by DMA timeout messages on the screen (as posted in the initial email). Now,

Re: [RFT] hpt366: reset DMA state machine on timeouts

2007-06-22 Thread Linas Vepstas
On Thu, Jun 21, 2007 at 09:54:47PM +0400, Sergei Shtylyov wrote: Reset HPT36x's DMA state machine on a DMA timeout the way it's done for HPT370. Signed-off-by: Sergei Shtylyov [EMAIL PROTECTED] --- Linas, here's what I've come up with -- this should apply against 2.6.21.y.

Re: [RFT] hpt366: reset DMA state machine on timeouts

2007-06-22 Thread Linas Vepstas
On Fri, Jun 22, 2007 at 07:32:44PM +0400, Sergei Shtylyov wrote: Reset HPT36x's DMA state machine on a DMA timeout the way it's done for HPT370. drivers/ide/pci/hpt366.c | 24 +++- This worked great! I hope you meant those messages were preceeded by DMA

Re: [RFT] hpt366: reset DMA state machine on timeouts

2007-06-21 Thread Linas Vepstas
On Thu, Jun 21, 2007 at 09:54:47PM +0400, Sergei Shtylyov wrote: --- Linas, here's what I've come up with -- this should apply against 2.6.21.y. Compile-tested only, not for merging. Thanks, I'll test tonight. Meanwhile, under spoearate cover, I'll post the libata debug info. --linas - To

Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-21 Thread Linas Vepstas
On Wed, Jun 20, 2007 at 06:01:23PM +0100, Alan Cox wrote: Its unlikely the command got lost. The IRQ could have done but the error path tries to spot that case by reading the status register - which hangs. So in theory it could be a lost IRQ and if the reset works we'll find that out. OK,

Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-20 Thread Linas Vepstas
On Wed, Jun 20, 2007 at 12:07:19AM +0400, Sergei Shtylyov wrote: Bartlomiej Zolnierkiewicz wrote: [...frmware...] Google seems to show that there is no publically available firmware updates for Maxtor disks. It would be useful to see hdparm --Istdout output for *both* disks. Lets do one

bug in libata [was Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-19 Thread Linas Vepstas
On Mon, Jun 18, 2007 at 04:22:38PM -0500, linas wrote: On Mon, Jun 18, 2007 at 10:04:41PM +0100, Alan Cox wrote: please try using the libata driver. Its worse. I get a hard hang (sysrq doesn't work) during boot, just when the system goes to read the partition table. Recap: this is an

Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-19 Thread Linas Vepstas
Hi Sergei, On Tue, Jun 19, 2007 at 06:07:07PM +0400, Sergei Shtylyov wrote: [EMAIL PROTECTED] wrote: I think reading the IDE status register clears the interrupt in the IDE device, which might be causing the drive to think it's OK to generate another interrupt. This is not how IDE

Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-19 Thread Linas Vepstas
On Tue, Jun 19, 2007 at 08:10:25PM +0400, Sergei Shtylyov wrote: I'm thinking that trying to debug libata is a better idea, rather than investing time in ide, right? Although at the moment, libata works even less; see other email. Which makes me think this really is some *hardware*

[BUG] ide dma_timer_expiry, then hard lockup

2007-06-18 Thread Linas Vepstas
I've got a hard lockup in the ide subsystem, probably due to some irq spew or something like that. I've just bought a brand new Maxtor 320GB disk driver for the insane price of $70 US to replace another failing drive. It works well under light load; I was able to copy about 60GB to it.

Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-18 Thread Linas Vepstas
On Mon, Jun 18, 2007 at 09:27:04PM +0100, Alan Cox wrote: ide_dma_timeout_retry() in ide-io.c prints the hdc: DMA Timeout error then calls HWIF(drive)-ide_dma_end(drive); which returns, and then calls hwif-INB(IDE_STATUS_REG) which is needed as an argument to ide_error()

Re: [BUG] ide dma_timer_expiry, then hard lockup

2007-06-18 Thread Linas Vepstas
On Mon, Jun 18, 2007 at 10:04:41PM +0100, Alan Cox wrote: If you've got a nice repeatable problem Very highly repeatable :-( please try using the libata driver. That handles the error paths differently and doesn't try a FIFO drain which might matter in this case I guess. Dohh, yes, of

Re: [PATCH pata-2.6 fix queue] hpt366: don't check enablebits for HPT36x

2007-05-25 Thread Linas Vepstas
On Fri, May 25, 2007 at 01:47:04AM +0400, Sergei Shtylyov wrote: Hello, I wrote: Linas, Andries, Michal, cound you try this instead: d-enablebits[0].mask = d-enablebits[0].val = 0x10; It probably won't work the way it should anyway -- the secondary channel (and controller in

Re: [PATCH pata-2.6 fix queue] hpt366: don't check enablebits for HPT36x

2007-05-24 Thread Linas Vepstas
Hi, On Fri, May 25, 2007 at 12:23:25AM +0400, Sergei Shtylyov wrote: Hello, I wrote: HPT36x chip don't seem to have the channel enable bits, so prevent the IDE core from checking them... Signed-off-by: Sergei Shtylyov [EMAIL PROTECTED] d-channels = 1; +d-enablebits[0].reg = 0;

Re: [PATCH pata-2.6 fix queue] hpt366: don't check enablebits for HPT36x

2007-05-24 Thread Linas Vepstas
On Fri, May 25, 2007 at 12:53:19AM +0400, Sergei Shtylyov wrote: knows how to make broken hardware. :-) You mean, value-added hardware ;- --linas - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message to [EMAIL PROTECTED] More majordomo info at

[BUG] 2.6.21.1: ide-pci is not finding ide controller.

2007-05-23 Thread Linas Vepstas
I've got a rather old x86 box that I'm booting 2.6.21.1 on; this kernel is not finding an ide controller on it. The motherboard has 4 ide controllers total; two olde-fashioned ones (PIIX4, using the original 40-pin IDE ribbon cable) and two HighPoint HPT366 controllers, taking the 80-pin

Re: [BUG] 2.6.21.1: ide-pci is not finding ide controller.

2007-05-23 Thread Linas Vepstas
On Thu, May 24, 2007 at 12:26:29AM +0400, Sergei Shtylyov wrote: Hello. Linas Vepstas wrote: I've got a rather old x86 box that I'm booting 2.6.21.1 on; this kernel is not finding an ide controller on it. The motherboard has 4 ide controllers total; two olde-fashioned ones (PIIX4, using

Re: [PATCHSET] libata: PATA driver for Celleb

2007-02-15 Thread Linas Vepstas
On Thu, Feb 15, 2007 at 07:09:25PM +0100, Arnd Bergmann wrote: On Thursday 15 February 2007 18:14, Linas Vepstas wrote: Linas has done a pretty good job on improving the driver in the last half year or so and he also has access to all the necessary hardware so I think he would

[PATCHSET] spidernet, sungem_phy: consolidated patch series

2007-02-15 Thread Linas Vepstas
On Fri, Feb 16, 2007 at 07:46:32AM +1100, Benjamin Herrenschmidt wrote: I can send a tested patch series (merging all three sources) Just send the blast to our list first so I can verify that the sungem_phy doesn't adversely affect sungem and everybody By our list, I assume you mean

Re: [PATCH] drivers/ata: PATA driver for Celleb

2006-12-12 Thread Linas Vepstas
On Tue, Dec 12, 2006 at 11:08:24AM +0900, Akira Iguchi wrote: I use some static functions in libata-core.c because it makes You will want to split this patch into several pars, and submit the changes to libata-core.c separately. --- linux-2.6.19/drivers/ata/libata-core.cWed Dec 6