Re: [PATCH 06/13] libata-acpi: clean up parameters and misc stuff

2007-04-28 Thread Jeff Garzik
Tejun Heo wrote: This patch cleans up libata-acpi such that it looks similar to other libata files. This patch doesn't introuce any behavior changes. * make libata-acpi functions take ata_device instead of ata_port + device index * s/atadev/adev/ * de-indent local variable declarations I

Re: [PATCH 09/13] libata-acpi: clean up ata_acpi_exec_tfs()

2007-04-28 Thread Jeff Garzik
Tejun Heo wrote: This patch cleans up ata_acpi_exec_tfs() and its friends. * Rename taskfile_array to ata_acpi_gtf and make it __packed as it's used as argument to ACPI method, and use pointer to ata_acpi_gtf and number of taskfiles to represent _GTF taskfiles instead of a pointer casted

Re: [PATCH 10/13] libata-acpi: miscellaneous cleanups

2007-04-28 Thread Jeff Garzik
Tejun Heo wrote: * Add missing LOCKING: and RETURNS: to function comment. * Don't conditionalize warning messages with ata_msg_probe(). Print directly with KERN_WARNING. * Drop duplicate debug messages. Signed-off-by: Tejun Heo [EMAIL PROTECTED] --- drivers/ata/libata-acpi.c | 51

Re: [patch 02/10] sata_nv: add back some verbosity into ADMA error_handler

2007-04-28 Thread Jeff Garzik
) are active and only prints the CPBs for those commands. That way it won't spew at inappropriate times like suspend/resume. [EMAIL PROTECTED]: cleanup] Signed-off-by: Robert Hancock [EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Cc: Tejun Heo [EMAIL PROTECTED] Signed-off-by: Andrew Morton

Re: [patch 07/10] pata_hpt3x2n: Add HPT371N support and other bits

2007-04-28 Thread Jeff Garzik
From: Alan Cox [EMAIL PROTECTED] Yes its no longer 3x2n but 3xxn, I can rename it if you want Jeff Choose whatever you feel is the best long term name; I have no preference here. If you do rename, please follow this slightly special procedure: * add a MODULE_ALIAS() for the old name * send

Re: [patch 01/10] ata: printk warning fixes

2007-04-28 Thread Jeff Garzik
/libata-core.c:1003: warning: format '%lld' expects type 'long long int', but argument 4 has type 'u64' Also fix various 80-col bustage. Cc: Jeff Garzik [EMAIL PROTECTED] Cc: Tejun Heo [EMAIL PROTECTED] Signed-off-by: Andrew Morton [EMAIL PROTECTED] applied - To unsubscribe from this list: send

Re: [patch 06/10] pata_hpt37x: Further small fixes

2007-04-28 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Alan Cox [EMAIL PROTECTED] Further HPT37x changes - No 66MHz 370/370A - Remove dead special case check now we use the DPLL (as per the IDE driver) Pointed out by Sergei Signed-off-by: Alan Cox [EMAIL PROTECTED] Signed-off-by: Andrew Morton [EMAIL PROTECTED] ---

Re: [patch 08/10] drivers/ata/pata_cmd640.c: fix build with CONFIG_PM=n

2007-04-28 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Andrew Morton [EMAIL PROTECTED] This is grubby, but all the ata drivers do it this way. Would it not be better to do #define ata_scsi_device_resume NULL in libata.h, remove all those ifdefs? (updated version, ug, ug) Cc: Jeff Garzik [EMAIL PROTECTED] Cc: Alan

Re: libata fails to recover from HSM violation involving DRQ status

2007-04-28 Thread Jeff Garzik
Mark Lord wrote: Tejun, While working on the new hdparm (version 7.0, released today), I ran into trouble when a buggy SG_IO/ATA_16 packet caused the libata EH to get confused. I triggered this by accident, issuing an IDENTIFY command which incorrectly specified ATA_PROT_NODATA. My error, for

Re: libata fails to recover from HSM violation involving DRQ status

2007-04-28 Thread Jeff Garzik
Mark Lord wrote: Actually, I'm not so sure that this problem hasn't *already* been posted to this very mailing list. http://lkml.org/lkml/2006/10/1/264 http://www.mail-archive.com/linux-ide@vger.kernel.org/msg05078.html ... What Tejun said at the end of that thread :) That one is a phy-level

Re: libata fails to recover from HSM violation involving DRQ status

2007-04-28 Thread Jeff Garzik
Alan Cox wrote: I am reluctant to do anything about this. This one does need dealing with. It happens in the real world and the old IDE paths for this do get triggered and used now and then (we know this because bugs in them were found). All it takes is a device and a controller disagreeing

Re: [PATCH 06/13] libata-acpi: clean up parameters and misc stuff

2007-04-28 Thread Jeff Garzik
Tejun Heo wrote: Jeff Garzik wrote: Tejun Heo wrote: This patch cleans up libata-acpi such that it looks similar to other libata files. This patch doesn't introuce any behavior changes. * make libata-acpi functions take ata_device instead of ata_port + device index * s/atadev/adev/ * de

Re: [PATCH] libata-acpi: fix _GTF command protocol for ATAPI devices

2007-04-28 Thread Jeff Garzik
Tejun Heo wrote: ATA_PROT_ATAPI_NODATA is used for PACKET command without CDB. ACPI _GTF never contains PACKET command. It's always ATA commands. Ah yes. Thanks for the reminder about the code I wrote :) Jeff - To unsubscribe from this list: send the line unsubscribe linux-ide in

Re: libata fails to recover from HSM violation involving DRQ status

2007-04-28 Thread Jeff Garzik
Tejun Heo wrote: and thus clear DRQ, right? Stuck DRQ after SRST seems odd to me. Unfortunately not odd on ata_piix, which can get stuck DRQ-on somewhere deep inside its IDE emulation engine. And neither draining the FIFO nor SRST nor a couple other tricks ever helped. The only thing that

[git patches] libata updates

2007-04-29 Thread Jeff Garzik
: printk warning fixes drivers/ata/pata_cmd640.c: fix build with CONFIG_PM=n Conke Hu (1): ahci.c: remove non-existing SB600 raid id (re-send) Dmitriy Monakhov (1): libata: handle ata_pci_device_do_resume() failure while resuming Jeff Garzik (8): [libata] sata_mv: remove

Re: [git patches] libata updates

2007-04-30 Thread Jeff Garzik
Chuck Ebbert wrote: Jeff Garzik wrote: Jeff Garzik (8): libata/IDE: remove combined mode quirk You can't just remove the combined_mode= kernel parameter or every Linux user who uses that will get an unbootable kernel with no good way of diagnosing the problem. It should still

Re: [git patches] libata updates

2007-04-30 Thread Jeff Garzik
Stephen Clark wrote: Yeah the kernel will boot but the hd performance is sh*t on my laptop. I am running FC6 with kernel 2.6.21 and without the combined_mode setting my disk performance goes down to a whopping 1.25mb/sec from 44mb/sec when I boot with combined_mode=libata. It make my system

Re: [git patches] libata updates

2007-04-30 Thread Jeff Garzik
Chuck Ebbert wrote: Jeff Garzik wrote: Chuck Ebbert wrote: Jeff Garzik wrote: Jeff Garzik (8): libata/IDE: remove combined mode quirk You can't just remove the combined_mode= kernel parameter or every Linux user who uses that will get an unbootable kernel with no good way of diagnosing

Re: 2.6.20 libata cdrom

2007-05-01 Thread Jeff Garzik
Mark Lord wrote: Gee... Two crazy screwed up devices today. What's going on? libata is finally seeing use by non-kernel developers. Amusing but factually incorrect. libata has been shipping in major distros for years, with bazoodles of active users. Jeff - To unsubscribe from

Re: [Fwd: Re: PATCH: libata: add support for ATA_16 on ATAPI]

2007-05-03 Thread Jeff Garzik
Mark Lord wrote: Jeff? This has been pending for a very long time now. This patch brings libata up to similar standards as Linux IDE with support for things like hdparm -I (identify) and -C (check power) among others on ATAPI drives. Andrew, can you pick this up for -mm ?? ??? My reluctance

Re: regressions

2007-05-06 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: 2.6.21 breaks HPT366 - back to 2.6.20 I thought Bart/Sergei had fixed this. Are you using libata or old-IDE driver? 2.6.20 breaks ZIP - back to 2.6.19 What does this mean? How is it broken? Jeff - To unsubscribe from this list: send the line

[git patches] libata reset sequence update

2007-05-08 Thread Jeff Garzik
+++- 32 files changed, 321 insertions(+), 212 deletions(-) Jeff Garzik (1): [libata reset-seq] build and merge fixes Tejun Heo (4): libata: add deadline support to prereset and reset methods libata: improve 0xff status handling libata: improve ata_std_prereset

Re: [PATCH] libata: disable_irq() during polling IDENTIFY (take 2)

2007-05-08 Thread Jeff Garzik
Tejun Heo wrote: Albert, if Jeff agrees with pushing HSM or data transfer to workqueue, are you interested in doing that? The HSM is your baby after all. :-) It's always been my desire to do PIO from a workqueue if possible, since it is so slow. Jeff - To unsubscribe from this

Re: [patch] Add the device IDs for AMD/ATI SB700

2007-05-09 Thread Jeff Garzik
Henry Su wrote: From: [EMAIL PROTECTED] Adding the device ID for AMD/ATI SB700. Signed-off-by:henry su [EMAIL PROTECTED] Time to train new people... You need to split up your patches: * send I2C and PCI quirk patches to GregKH * send drivers/ide/* stuff to Bart * send drivers/ata/* patches to

Re: [PATCH] pci-quirks: disable MSI on RS400-200 and RS480, take #2

2007-05-09 Thread Jeff Garzik
Chuck Ebbert wrote: Tejun Heo wrote: FWIW several distros have turned off MSI by default and added a pci=msi option to enable it. Yeah, it seem to cause a lot of problems on certain chips but I think the correct path is to add PCI quirks for those. Most MSI problems I've seen are on these ATI

Re: [git patches] IDE update

2007-05-09 Thread Jeff Garzik
Bartlomiej Zolnierkiewicz wrote: Bartlomiej Zolnierkiewicz (11): ide: fix UDMA/MWDMA/SWDMA masks (v3) ide: rework the code for selecting the best DMA transfer mode (v3) ide: add ide_tune_dma() helper ide: make /proc/ide/ optional ide: split off ioctl handling from

Re: [git patches] IDE update

2007-05-09 Thread Jeff Garzik
Bartlomiej Zolnierkiewicz wrote: * the code for selecting and programming the best DMA transfer mode has been reworked to be cleaner, more generic and more libata-like, ( 500 LOCs gone and this change allows the change described below) Bartlomiej Zolnierkiewicz (11): ide: rework the

Re: [git patches] IDE update

2007-05-09 Thread Jeff Garzik
Andrew Morton wrote: Jeff Garzik [EMAIL PROTECTED] wrote: Has this seen testing/exposure in -mm tree? argh. If this was in a file called ide-rework-the-code-for-selecting-the-best-DMA-transfer-mode.patch then it would be so easy. logs into hera greps ah, it's hidden in ide-max-dma-mode

Re: [git patches] IDE update

2007-05-09 Thread Jeff Garzik
Bartlomiej Zolnierkiewicz wrote: On Thursday 10 May 2007, Jeff Garzik wrote: The limit was raised to 400K IIRC. That's (good) news to me, here goes the actual 150K patch: Thanks. I did in fact receive copies from vger, so it went through. Jeff - To unsubscribe from this list

Re: [PATCH] libata: fallback to the other IDENTIFY on device error

2007-05-09 Thread Jeff Garzik
Tejun Heo wrote: It seems the world isn't as frank as we thought and some devices lie about who they are. Fallback to the other IDENTIFY if IDENTIFY fails with device error. As this is the strategy used by IDE for a long time, it shouldn't cause too much problem. Signed-off-by: Tejun Heo

Re: [PATCH 2.6.21-mm1 1/2] sata_promise: fix another error decode regression

2007-05-09 Thread Jeff Garzik
Mikael Pettersson wrote: The sata_promise error decode update changed pdc_host_intr() to return and not complete the qc after detecting an error. Unfortunately not completing the qc:s causes them to always time out on error, which is wrong and has nasty side-effects. This patch updates

Re: [PATCH] pata_pcmcia.c: add card ident for jvc cdrom

2007-05-09 Thread Jeff Garzik
Richard Kennedy wrote: update pata_pcmcia to add card ident for JVC MP-CDX1 cdrom drive card info: PRODID_1=KME PRODID_2=KXLC005 PRODID_3=00 MANFID=0032,2904 Signed-off-by: Richard Kennedy [EMAIL PROTECTED] --- patch against 2.6.21.1 applied - To unsubscribe from this list: send the line

Re: [PATCH] sata_via: add missing PM hooks

2007-05-09 Thread Jeff Garzik
Tejun Heo wrote: For some reason, sata_via is missing PM hooks. Add them. Spotted by Jeroen Janssen [EMAIL PROTECTED]. Signed-off-by: Tejun Heo [EMAIL PROTECTED] Cc: Jeroen Janssen [EMAIL PROTECTED] --- I have no idea how we missed this. This is for both libata-dev -stable. Thanks.

Re: [PATCH] pata_scc: fix compilation

2007-05-09 Thread Jeff Garzik
Alexey Dobriyan wrote: Signed-off-by: Alexey Dobriyan [EMAIL PROTECTED] --- drivers/ata/pata_scc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message to [EMAIL PROTECTED] More

Re: [PATCH] libata: fix kernel-doc parameters

2007-05-09 Thread Jeff Garzik
Randy Dunlap wrote: From: Randy Dunlap [EMAIL PROTECTED] Warning(linux-2.6.21-git4//drivers/ata/libata-core.c:904): No description found for parameter 'new_sectors' Warning(linux-2.6.21-git4//drivers/ata/libata-core.c:941): No description found for parameter 'new_sectors' Signed-off-by:

Re: [git patches] libata updates

2007-05-09 Thread Jeff Garzik
Jesse Barnes wrote: On Sunday, April 29, 2007 9:15 am Jeff Garzik wrote: Noteworthy changes: * remove combined mode PCI quirk. IDE driver selection (libata or old-IDE) is now determined purely by module load order. * new driver API, that is far more like other kernel APIs: alloc...register

[git patches] libata fixes, minor updates

2007-05-09 Thread Jeff Garzik
Please pull from 'upstream-linus' branch of master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/libata-dev.git upstream-linus to receive the following updates: Documentation/kernel-parameters.txt |8 --- drivers/ata/libata-acpi.c |3 +- drivers/ata/libata-core.c |

Re: [PATCH] libata: add human-readable error value decoding

2007-05-10 Thread Jeff Garzik
Mark Lord wrote: Same here, but I would like to see it in there under a CONFIG_DEBUG_LIBATA kernel build option or something. Kind of like the FANCY_STATUS_DUMPS flag that drivers/ide used to have for this kind of stuff. The long term goal is to enable verbose output with a module option

Re: [PATCH] libata: add human-readable error value decoding

2007-05-10 Thread Jeff Garzik
Mark Lord wrote: If we're compiling the messages into the kernel regardless, then it doesn't really make much sense to NOT show all of them on the error paths. Not true. Uncontrolled message spewage inevitably results in critical information scrolling off the screen, before a user can take

Re: [PATCH] libata: add human-readable error value decoding

2007-05-10 Thread Jeff Garzik
Robert Hancock wrote: I don't think this is as big of a deal here as in other cases, like oops output. With libata errors, if they're at the console (which they'd have to be to see these messages), unless something has actually caused a panic the scrollback buffer should still be functional

Re: sata_mv hotplug status?

2007-05-11 Thread Jeff Garzik
Tomasz Chmielewski wrote: I'm a little confused about sata_mv hotplug status. According to http://linux-ata.org/driver-status.html#marvell, the driver has full SATA control including hotplug. A bit down on that same page, the matrix http://linux-ata.org/driver-status.html#matrix says

Re: [PATCH 1/2] libata: reimplement suspend/resume support using sdev-manage_start_stop

2007-05-11 Thread Jeff Garzik
Tejun Heo wrote: Jeff Garzik wrote: Tejun Heo wrote: Reimplement suspend/resume support using sdev-manage_start_stop. * Device suspend/resume is now SCSI layer's responsibility and the code is simplified a lot. * DPM is dropped. This also simplifies code a lot. Suspend/resume status

Re: [PATCH] libata: ignore EH scheduling during initialization

2007-05-11 Thread Jeff Garzik
Tejun Heo wrote: libata enables SCSI host during ATA host activation which happens after IRQ handler is registered and IRQ is enabled. All ATA ports are in frozen state when IRQ is enabled but frozen ports may raise limited number of IRQs after being frozen - IOW, -freeze() is not responsible

Re: [PATCH] libata: fallback to the other IDENTIFY on device error, take#2

2007-05-11 Thread Jeff Garzik
Tejun Heo wrote: It seems the world isn't as frank as we thought and some devices lie about who they are. Fallback to the other IDENTIFY if IDENTIFY is aborted by the device. As this is the strategy used by IDE for a long time, it shouldn't cause too much problem. Signed-off-by: Tejun Heo

Re: [PATCH] libata: fallback to the other IDENTIFY on device error, take#2

2007-05-11 Thread Jeff Garzik
Tejun Heo wrote: + if (class == ATA_DEV_ATA) + class = ATA_DEV_ATAPI; + else + class = ATA_DEV_ATA; the 'else' branch is obviously redundant - To unsubscribe from this list: send the line

Re: [PATCH 03/11] libata-acpi: s/CONFIG_SATA_ACPI/CONFIG_ATA_ACPI/

2007-05-11 Thread Jeff Garzik
Tejun Heo wrote: ACPI applies to both SATA and PATA. Drop the 'S' from the config variable. Signed-off-by: Tejun Heo [EMAIL PROTECTED] --- drivers/ata/Kconfig| 26 +- drivers/ata/Makefile |2 +- drivers/ata/libata.h |2 +- include/linux/libata.h |

Re: [patch 12/20] libata-core: convert to use cancel_rearming_delayed_work()

2007-05-11 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Oleg Nesterov [EMAIL PROTECTED] We should not use cancel_work_sync(delayed_work-work). This works, but not good. We can use cancel_rearming_delayed_work(), this also simplifies the code. Signed-off-by: Oleg Nesterov [EMAIL PROTECTED] Signed-off-by: Andrew Morton

Re: [patch 01/20] git-libata-all: sata_via build fix

2007-05-11 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Andrew Morton [EMAIL PROTECTED] Cc: Tejun Heo [EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Signed-off-by: Andrew Morton [EMAIL PROTECTED] --- drivers/ata/sata_via.c |4 1 file changed, 4 deletions(-) applied - To unsubscribe from this list

Re: [patch 07/20] pata_pcmcia: recognize 2GB CompactFlash from Transcend

2007-05-11 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Aeschbacher, Fabrice [EMAIL PROTECTED] Allow the pata_pcmcia driver to automatically detect 2GB CompactFlash cards from Transcend. Signed-off-by: Fabrice Aeschbacher [EMAIL PROTECTED] Cc: Peter Stuge [EMAIL PROTECTED] Acked-by: Alan Cox [EMAIL PROTECTED] Cc:

Re: [patch 09/20] Add the combined mode for ATI SB700

2007-05-11 Thread Jeff Garzik
for Linux since the kernel has supported AHCI. Signed-off-by: Luugi Marsan [EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Cc: Tejun Heo [EMAIL PROTECTED] Signed-off-by: Andrew Morton [EMAIL PROTECTED] --- drivers/pci/quirks.c|1 + include/linux/pci_ids.h |1 + 2 files changed, 2

Re: [patch 10/20] add the ATI SB700 SATA controller device id to AHCI pci table

2007-05-11 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Henry Su [EMAIL PROTECTED] Add the device ID to AHCI pci table for ATI SB700 SATA controller, the subsequent chipset of SB600. Signed-off-by: henry su[EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Cc: Tejun Heo [EMAIL PROTECTED] Signed-off-by: Andrew Morton

Re: [patch 13/20] Use menuconfig objects: libata

2007-05-11 Thread Jeff Garzik
[EMAIL PROTECTED] wrote: From: Jan Engelhardt [EMAIL PROTECTED] Use menuconfigs instead of menus, so the whole menu can be disabled at once instead of going through all options. Signed-off-by: Jan Engelhardt [EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Signed-off-by: Andrew Morton

Re: [PATCH] pata_platform: don't use generic ata_port_start

2007-05-11 Thread Jeff Garzik
Olof Johansson wrote: Hi, I have a system where I have a simple IDE controller that sits on a local bus without bus master dma capability, and thus no dma_mapping ops defined for the device/bus. pata_platform works great for me, with the exception of using the generic ata_port_start which

Re: [PATCH] libata: add human-readable error value decoding

2007-05-11 Thread Jeff Garzik
Robert Hancock wrote: The ATA ones are more of a pain in that regard than SCSI though - SCSI has all distinct error codes for different errors, whereas ATA has bitmasks for everything.. That should not affect implementation. Either way, a table-driven approach can easily work. I favor

Re: [PATCH] drivers/ata: correct a wrong free function for sata_nv driver

2007-05-15 Thread Jeff Garzik
Peer Chen wrote: For sata_nv driver in kernel 2.6.21 onward, Inside nv_init_one(),use 'hpriv = devm_kzalloc(pdev-dev, sizeof(*hpriv), GFP_KERNEL);' but using the kfree(hpriv) to free that data struction in nv_remove_one(), which will cause system hang when removing the sata_nv module. Change

Re: [PATCH] libata: fix shutdown warning message printing

2007-05-15 Thread Jeff Garzik
Tejun Heo wrote: Unlocking ap-lock and ssleeping don't work because SCSI commands can be issued from completion path without context. Reimplement delayed completion by allowing translation functions to override qc-scsidone(), storing the original completion function to scmd-scsi_done() and

Re: [PATCH] libata: track spindown status and skip spindown_compat if possible

2007-05-15 Thread Jeff Garzik
Tejun Heo wrote: Our assumption that most distros issue STANDBYNOW seems wrong. The upstream sysvinit and thus many distros including gentoo and opensuse don't take any action for libata disks on spindown. We can skip compat handling for these distros so that they don't need to update anything

[git patches] libata fixes

2007-05-15 Thread Jeff Garzik
All bug fixes. The two things that do not seem like bugfixes (separate out... and add ATA_FLAG_ACPI_SATA) are not as they seem. The former is a prep patch for a fix, and the latter fixes what ACPI considers a legacy IDE interface. Please pull from 'upstream-linus' branch of

Re: [patch 08/20] drivers/ata: remove the wildcard from sata_nv driver

2007-05-17 Thread Jeff Garzik
[EMAIL PROTECTED] Cc: Tejun Heo [EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Signed-off-by: Andrew Morton [EMAIL PROTECTED] --- drivers/ata/sata_nv.c |6 -- 1 file changed, 6 deletions(-) applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body

Re: [PATCH] sata_nv: fix fallout of devres conversion

2007-05-17 Thread Jeff Garzik
Tejun Heo wrote: As with all other drivers, sata_nv's hpriv is allocated with devm_kzalloc() and there's no need to free it explicitly. Kill nv_remove_one() which incorrectly used kfree() instead of devm_kfree() and use ata_pci_remove_one() directly. Original fix is from Peer Chen.

Re: [PATCH] libata: remove libata.spindown_compat

2007-05-17 Thread Jeff Garzik
Tejun Heo wrote: With STANDBYDOWN tracking added, libata.spindown_compat isn't necessary anymore. If userspace shutdown(8) issues STANDBYNOW, libata warns. If userspace shutdown(8) doesn't issue STANDBYNOW, libata does the right thing. Userspace can tell whether kernel supports spindown by

Re: [PATCH] sata_via: pcim_iomap_regions() conversion missed BAR5

2007-05-17 Thread Jeff Garzik
Tejun Heo wrote: pcim_iomap_regions() conversion missed BAR5. Fix it. Signed-off-by: Tejun Heo [EMAIL PROTECTED] --- Sorry, dunno how I missed that during testing. :-( Please verify this fixes your problem. diff --git a/drivers/ata/sata_via.c b/drivers/ata/sata_via.c index d105d2c..ac4f43c

Re: [PATCH] libata: Add MMIO support to pata_sil680

2007-05-17 Thread Jeff Garzik
Benjamin Herrenschmidt wrote: This patch adds MMIO support to the pata_sil680 for taskfile IOs, based on what the old siimage does. I haven't bothered changing the chip setup stuff from PCI config cycles to MMIO though (siimage does it), I don't think it matters, I've only adapted it to use

[git patches] libata fixes

2007-05-17 Thread Jeff Garzik
); -MODULE_PARM_DESC(spindown_compat, Enable backward compatible spindown -behavior. Will be removed. More info can be found in -Documentation/feature-removal-schedule.txt\n); - MODULE_AUTHOR(Jeff Garzik); MODULE_DESCRIPTION(Library module for ATA devices); MODULE_LICENSE(GPL

[PATCH] sata_sil: Greatly improve DMA support

2007-05-18 Thread Jeff Garzik
b2a4d29570403f611689e786372e8a4afbdf8891 Author: Jeff Garzik [EMAIL PROTECTED] Date: Fri Dec 1 22:58:31 2006 -0500 [libata sata_sil] Greatly improve DMA handling 311x hardware includes a vendor-specific scatter/gather format which eliminates the traditional 64k boundary limit found in PCI IDE DMA

[PATCH] sata_mv: new exception handling (hotplug, NCQ framework)

2007-05-18 Thread Jeff Garzik
Below is a refresh of my on-going effort to convert sata_mv to the new exception handling framework. sata_mv is one of the last hold-outs, and its old-EH implementation blocks new features like hotplug and NCQ. It works for me on the one 50xx and one 60xx card I tested it on, but other testers

[PATCH] ahci: add Marvell support (WIP)

2007-05-18 Thread Jeff Garzik
This is the work-in-progress for supporting the newer Marvell PATA/SATA chips like the 6145. SATA support works, but PATA support is not implemented at all. There is some temporary code that makes sure the PATA port is disabled, if enabled. This can be found in the 'mv-ahci' branch of

Re: [PATCH] sata_sil: Greatly improve DMA support

2007-05-19 Thread Jeff Garzik
Robert Hancock wrote: Jeff Garzik wrote: Since Alan expressed a desire to see Large Block Transfer (LBT) support in pata_sil680, I though I would re-post my patch for adding LBT support to sata_sil. Silicon Image's Large Block Transfer (LBT) support is a vendor-specific DMA scatter/gather

Re: [PATCH] sata_sil: Greatly improve DMA support

2007-05-19 Thread Jeff Garzik
Jeff Garzik wrote: rotfl. Boy am I dumb. I -thought- my patch, written months ago, included that bit. But obviously it does not. Let's add that... Or not. It just gets rid of the 64k boundary -and- allows for larger scatter/gather entries. LBT does not add 64-bit DMA support

Re: [PATCH] sata_sil: Greatly improve DMA support

2007-05-19 Thread Jeff Garzik
Indan Zupancic wrote: This patch seems to work with my SiI 3512, though I don't notice any difference, neither a speedup, nor a slowdown. Hdparm gives the same speeds (-tT), and cp -a'ing kernel sources is abysmal slow in both cases, (need to look into that one) so I didn't really test it that

Re: something strange in libata-core.c for kernel 2.6.22-rc3

2007-05-21 Thread Jeff Garzik
Stupid BIOS crap like that is largely the reason why I am reluctant to merge pata_acpi. There is a lot of knowledge like this yet to be discovered. I would feel far more comfortable disassembling the AML like you are doing... Jeff - To unsubscribe from this list: send the line

Re: use_clustering (sht) bit set to 0 in AHCI ?

2007-05-21 Thread Jeff Garzik
Jens Axboe wrote: ahci has always had clustering disabled, perhaps Jeff can expand on why? Just historical reasons. libata had clustering disabled by default in the beginning, for all drivers. Then we enabled it globally by changing the value of ATA_SHT_USE_CLUSTERING... but apparently

Re: [PATCH] ahci: disable 64bit dma on sb600

2007-05-21 Thread Jeff Garzik
Tejun Heo wrote: SB600 claims it can do 64bit DMA but it can't. Disable it. Signed-off-by: Tejun Heo [EMAIL PROTECTED] --- Jeff, this one is for #upstream-fixes. ahci.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) applied - To unsubscribe from this list: send the

Re: [PATCH] pata_sil680: compile fix

2007-05-21 Thread Jeff Garzik
Tejun Heo wrote: Recent mmio change broke compilation if PM is turned on. Fix it. Signed-off-by: Tejun Heo [EMAIL PROTECTED] Cc: Benjamin Herrenschmidt [EMAIL PROTECTED] --- pata_sil680.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) applied - To unsubscribe from this list:

Re: Add Seagate STT20000A to DMA blacklist.

2007-05-21 Thread Jeff Garzik
Dave Jones wrote: http://bugzilla.kernel.org/show_bug.cgi?id=1044 points out an additional hard disk that doesn't handle DMA transfers correctly. This patch is the libata variant of the earlier patch to drivers/ide/ Signed-off-by: Dave Jones [EMAIL PROTECTED] applied - To unsubscribe from

Re: [PATCH] pata_hpt366: Enable bits are unreliable so don't use them

2007-05-21 Thread Jeff Garzik
Alan Cox wrote: Various people had problems with both old and new IDE when hpt366 enable bits started getting honoured. It turns out they are not reliable so don't rely on them Signed-off-by: Alan Cox [EMAIL PROTECTED] applied - To unsubscribe from this list: send the line unsubscribe

Re: [PATCH] ata_piix: clean up

2007-05-21 Thread Jeff Garzik
Alan Cox wrote: With cable methods in place we don't need a custom error handler for SATA so get rid of it Signed-off-by: Alan Cox [EMAIL PROTECTED] applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message to [EMAIL PROTECTED] More majordomo info

Re: [PATCH] libata: Kiss post_set_mode goodbye

2007-05-21 Thread Jeff Garzik
Alan Cox wrote: As of the -mm tree we don't have post_set_mode users any more. Signed-off-by: Alan Cox [EMAIL PROTECTED] applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message to [EMAIL PROTECTED] More majordomo info at

[git patches] libata fixes and administrivia

2007-05-21 Thread Jeff Garzik
): libata: Add Seagate STT2A to DMA blacklist. Jeff Garzik (2): libata: Trim trailing whitespace libata: bump versions Tejun Heo (1): ahci: disable 64bit dma on sb600 diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig index ad1f59c..b4a8d60 100644 --- a/drivers/ata

Re: JMicron JMB361 sporadically failing to initialize from at least 2.6.18.4 to 2.6.22-rc2

2007-05-21 Thread Jeff Garzik
Andrew Paprocki wrote: I have a Gigabyte GA-965P-DQ6 motherboard which has onboard Intel ICH8 raid as well as a Gigabyte (rebranded JMicron) chipset for 2 separate SATA ports. When I boot the machine, it completely sporadically fails to initialize the JMB361 chipset which it detects, claiming

Re: [PATCH] ahci: add Marvell support (WIP)

2007-05-21 Thread Jeff Garzik
George T. Joseph (development) wrote: Hi Jeff, Two issues with the patch... msi has to be disabled for the Marvell or the driver load will throw a nobody cared message and eventually hang before discovering all the drives. Light I/O works fine but heavy I/O generates exception Emask 0x0 Sact

Re: sata_mv driver

2007-05-22 Thread Jeff Garzik
Wakko Warner wrote: In kernel 2.6.20, this driver is marked as experimental. How well does this driver work? I was concidering a supermicro 8 port sata controller and it appears to use this driver. I really do not wish to use anything that is experimental. It's still beta. It needs error

Re: [PATCH] libata: Add MMIO support to pata_sil680

2007-05-24 Thread Jeff Garzik
MMIO has always been like this (libata-core.c): /* software reset. causes dev0 to be selected */ iowrite8(ap-ctl, ioaddr-ctl_addr); udelay(20); /* FIXME: flush */ iowrite8(ap-ctl | ATA_SRST, ioaddr-ctl_addr); udelay(20); /* FIXME: flush */

Re: [PATCH 2.6.22-rc2] libata: sata_sis fixes

2007-05-24 Thread Jeff Garzik
Uwe Koziolek wrote: The sata_sis driver supports SATA and PATA ports. The broken support of both types in one controller is fixed. the pata133 sis controllers does not support a drive present status in the pci configspace like the older sis controllers, check removed. Signed-off-by: Uwe

Re: [PATCH] libata: Add MMIO support to pata_sil680

2007-05-24 Thread Jeff Garzik
Alan Cox wrote: Currently libata does the 400ns thingy in the inline ata_pause() which unconditionally does ata_altstatus(); ndelay(400); (and thus does it on PIO, MMIO, anything ...). It might actually improve performance for PATA if we did as we'll get rid of an excess (expensive) PCI read

Re: [PATCH] Fix pio/mwdma programming on ata_piix.c

2007-05-24 Thread Jeff Garzik
Art Haas wrote: On Tue, May 01, 2007 at 05:02:22AM +0200, Tejun Heo wrote: Hello, Art Haas. Art Haas wrote: Hi. Back in the beginning of February I was sent a patch to test during the time the CD-ROM identification bug was affecting my computer. The first posting of the patch appeared on

Re: [PATCH] libata: Add MMIO support to pata_sil680

2007-05-24 Thread Jeff Garzik
Alan Cox wrote: On Thu, 24 May 2007 06:52:13 -0400 Jeff Garzik [EMAIL PROTECTED] wrote: Alan Cox wrote: Currently libata does the 400ns thingy in the inline ata_pause() which unconditionally does ata_altstatus(); ndelay(400); (and thus does it on PIO, MMIO, anything ...). It might actually

[git patches] libata fixes

2007-05-24 Thread Jeff Garzik
Please pull from 'upstream-linus' branch of master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/libata-dev.git upstream-linus to receive the following updates: drivers/ata/ata_piix.c |1 + drivers/ata/libata-core.c |4 +- drivers/ata/pata_hpt3x2n.c |4 +-

Re: [PATCH] libata: Add MMIO support to pata_sil680

2007-05-24 Thread Jeff Garzik
Alan Cox wrote: #define DELAY400NS { pio_inbyte( CB_ASTAT ); pio_inbyte( CB_ASTAT ); \ pio_inbyte( CB_ASTAT ); pio_inbyte( CB_ASTAT ); } Totally unrelated. Hal is using the cycle time of the four I/O reads to do the 400nS delay. Its a neat way to do the delay on

Re: [PATCH 4/9] libata-acpi: implement ata_acpi_associate()

2007-05-24 Thread Jeff Garzik
Tejun Heo wrote: * Add acpi_handle to ata_host and ata_port. Rename ata_device-obj_handle to -acpi_handle and move it above such that it doesn't get cleared on reconfiguration. * Replace ACPI node association which ata_acpi_associate() which is called once during host initialization.

Re: [PATCH] libata: replace ap-cbl tests with ATA_FLAG_SATA tests

2007-05-24 Thread Jeff Garzik
Tejun Heo wrote: ap-cbl == ATA_CBL_SATA indicates SATA cable while ap-flags ATA_FLAG_SATA indicates SATA host port. Till now they always gave the same result but SATA/PATA bridge handling will change that. Switch to ATA_FLAG_SATA test if we're testing for host port type. Signed-off-by: Tejun

Re: use_clustering (sht) bit set to 0 in AHCI ?

2007-05-24 Thread Jeff Garzik
Jens Axboe wrote: On Mon, May 21 2007, Jeff Garzik wrote: Jens Axboe wrote: ahci has always had clustering disabled, perhaps Jeff can expand on why? Just historical reasons. libata had clustering disabled by default in the beginning, for all drivers. Then we enabled it globally

Re: [patch 0/7] Asynchronous Notification for ATAPI devices (resend)

2007-05-24 Thread Jeff Garzik
Kristen Carlson Accardi wrote: Hi Jeff, Here are the AN patches again, they have not changed with the exception of patch #1, which does set the host flag in board_ahci and board_ahci_pi now (thanks Tejun). This patch series implements Asynchronous Notification (AN) for SATA ATAPI devices as

Re: [patch 1/7] libata: check for AN support

2007-05-24 Thread Jeff Garzik
Kristen Carlson Accardi wrote: Check to see if an ATAPI device supports Asynchronous Notification. If so, enable it. Signed-off-by: Kristen Carlson Accardi [EMAIL PROTECTED] --- Andrew, I cleaned up the function header to properly comply with kernel doc requirements. Other than that, this

Re: [patch] NCQ blacklist Maxtor 6B200M0/BANC1B10

2007-05-24 Thread Jeff Garzik
Simon Griph wrote: With NCQ enabled, this drive always hangs when under load and remains hung until the next cold boot. Very similar to: http://thread.gmane.org/gmane.linux.ide/17640 diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index 0abd72d..17e27e9 100644 ---

Re: [PATCH] Re: 2.6.19.1, sata_sil: sata dvd writer doesn't work

2007-05-24 Thread Jeff Garzik
Daniel Beichl wrote: +static int sil_check_atapi_dma(struct ata_queued_cmd *qc) +{ + /* limit atapi commands less than 60bytes to pio as certain sil */ + /* chips seem to have trouble to perform dma for these commands */ + if ( (qc-scsicmd != NULL) +

Re: [patch] Add the device IDs for AMD/ATI SB700

2007-05-24 Thread Jeff Garzik
Henry Su wrote: --- linux-2.6.21.1.orig/drivers/ata/pata_atiixp.c 2007-05-10 06:30:14.0 폍 linux-2.6.21.1/drivers/ata/pata_atiixp.c 2007-05-10 07:17:07.0 폍 @@ -283,6 ,7 @@ static const struct pci_device_id atiixp { PCI_VDEVICE(ATI, PCI_DEVICE_ID_ATI_IXP300_IDE),

Re: [patch 03/11] Optional LED trigger for libata

2007-05-24 Thread Jeff Garzik
) controller. Now respun as a single patch, and the function name shortened as requested. Signed-off-by: Tony Vroon [EMAIL PROTECTED] Acked-by: Tejun Heo [EMAIL PROTECTED] Cc: Jeff Garzik [EMAIL PROTECTED] Signed-off-by: Andrew Morton [EMAIL PROTECTED] --- drivers/ata/libata-core.c | 21

sil680 MMIO changes moved to branch

2007-05-24 Thread Jeff Garzik
I moved the sil680 MMIO changes from 'upstream' (queued for 2.6.23) to a side branch 'sil680-mmio'. To be clear, I am still quite interested in the patch (w/ the reset fix), but I did not want to delay other unrelated stuff that was about to be queued into #upstream. The patch will still

Re: sil680 MMIO changes moved to branch

2007-05-24 Thread Jeff Garzik
Benjamin Herrenschmidt wrote: On Thu, 2007-05-24 at 23:33 -0400, Jeff Garzik wrote: I moved the sil680 MMIO changes from 'upstream' (queued for 2.6.23) to a side branch 'sil680-mmio'. To be clear, I am still quite interested in the patch (w/ the reset fix), but I did not want to delay other

<    2   3   4   5   6   7   8   9   10   11   >