Re: [PATCH 2/2 #upstream] libata: assume ATA_DEV_ATA on diagnostic failure

2007-08-08 Thread Alan Cox
On Wed, 8 Aug 2007 14:40:08 +0900 Tejun Heo [EMAIL PROTECTED] wrote: Certain device which reports diagnostic failure also reports invalid device signature. Assume ATA_DEV_ATA on diagnostic failure if reset indicates device presence. This is fix for bugzilla bug 8784. Acked-by: Alan Cox

Re: libata-core.c: ata_fill_sg() ... sanity check on arithmentic ?

2007-08-08 Thread MikeW
Alan Cox alan at lxorguk.ukuu.org.uk writes: So then ap-prd[idx].flags_len gets set to (0x1 0x) = 0 ! So this sg element ends up with a zero length, even though the transfer size should be 64k. Is this correct behaviour, if not, should it be corrected ? The specification

Re: [PATCH] libata: implement BROKEN_HPA horkage and apply it to HDS724040KLSA80

2007-08-08 Thread Alan Cox
On Wed, 08 Aug 2007 11:21:08 +0900 Tejun Heo [EMAIL PROTECTED] wrote: Alan Cox wrote: I'd rather know what is going on here. A drive can legitimately support LBA48 and HPA and refuse READ_NATIVE_MAX_EXT. READ_NATIVE_MAX_EXT is mandatory if HPA LBA48, no Ok the report in that thread

Re: libata-core.c: ata_fill_sg() ... sanity check on arithmentic ?

2007-08-08 Thread Alan Cox
The specification says that 0x means 64K. A couple of controllers do get this wrong and we did recently add a second sg list builder for those. Perhaps a comment or two is in order ? - Thanks - but which specification is this ? As far as I am aware there is no spec for DMA

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Rafael J. Wysocki
On Wednesday, 8 August 2007 04:56, Tejun Heo wrote: Rafael J. Wysocki wrote: Well, on my box (nx6325) with the appended (experimental) patch applied on top of 2.6.23-rc1 with the patchset from http://www.sisk.pl/kernel/hibernation_and_suspend/2.6.23-rc2/patches/ , the double spin down

sata_nv + ADMA + Samsung disk problem

2007-08-08 Thread Gabor Gombas
Hi, Since I have upgraded to 2.6.22.1 from 2.6.20 I have problems with Samsung disks. Sometimes the disks stall for about half a minute and then I have these messages in the logs: Aug 6 20:10:11 twister kernel: ata7: EH in ADMA mode, notifier 0x0 notifier_error 0x0 gen_ctl 0x1501000 status

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Rafael J. Wysocki
On Wednesday, 8 August 2007 02:23, Robert Hancock wrote: Henrique de Moraes Holschuh wrote: On Tue, 07 Aug 2007, Tejun Heo wrote: Henrique de Moraes Holschuh wrote: On Tue, 07 Aug 2007, Tejun Heo wrote: Michael Sedkowski wrote: Hmmm... If the problem only shows up on nx6325, it might be

[PATCH] ST340823A, HPA and libata

2007-08-08 Thread Alan Cox
Ok this is a different approach to handling it - knowing it is an off by one so we can handle the drive if clipped Signed-off-by: Alan Cox [EMAIL PROTECTED] diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.23rc1-mm1/drivers/ata/libata-core.c

[PATCH] libata: Note that our cache flush code needs fixing up

2007-08-08 Thread Alan Cox
Remembered this while doing auditing and code review versus the specs Signed-off-by: Alan Cox [EMAIL PROTECTED] diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.23rc1-mm1/drivers/ata/libata-core.c linux-2.6.23rc1-mm1/drivers/ata/libata-core.c ---

Re: [PATCH] libata: implement BROKEN_HPA horkage and apply it to HDS724040KLSA80

2007-08-08 Thread Luiz Fernando N. Capitulino
Em Wed, 08 Aug 2007 11:21:08 +0900 Tejun Heo [EMAIL PROTECTED] escreveu: | Alan Cox wrote: | I'd rather know what is going on here. A drive can legitimately | support LBA48 and HPA and refuse READ_NATIVE_MAX_EXT. | READ_NATIVE_MAX_EXT is mandatory if HPA LBA48, no | | Ok the report in that

[PATCH] pata_cmd64x: Set up MWDMA modes properly

2007-08-08 Thread Alan Cox
Set the MWDMA timing by updating the correct registers. Split the PIO path as this is mostly shared code. Wants testing. Signed-off-by: Alan Cox [EMAIL PROTECTED] diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.23rc1-mm1/drivers/ata/pata_cmd64x.c

[PATCH] libata: First cut at verifying and validating all the id checks.

2007-08-08 Thread Alan Cox
- Fix those which require a word is checked for validity - Note limits on version information with ancient ATA - Correct EIDE dword_io check ready for adding 32bit PIO Signed-off-by: Alan Cox [EMAIL PROTECTED] diff -u --new-file --recursive --exclude-from /usr/src/exclude

Re: [PATCH] libata: implement BROKEN_HPA horkage and apply it to HDS724040KLSA80

2007-08-08 Thread Tejun Heo
Luiz Fernando N. Capitulino wrote: Em Wed, 08 Aug 2007 11:21:08 +0900 Tejun Heo [EMAIL PROTECTED] escreveu: | Alan Cox wrote: | I'd rather know what is going on here. A drive can legitimately | support LBA48 and HPA and refuse READ_NATIVE_MAX_EXT. | READ_NATIVE_MAX_EXT is mandatory if

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Mark Lord
Tejun Heo wrote: Michael Sedkowski wrote: Dnia 07-08-2007, Wt o godzinie 03:43 +0900, Tejun Heo napisał(a): Does emergency unload count increase after each power down? I think I got it. Using smartctl I've done a test and shut down, then repeted the test. The only values that where diffrent

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Tejun Heo
Mark Lord wrote: Tejun Heo wrote: Michael Sedkowski wrote: Dnia 07-08-2007, Wt o godzinie 03:43 +0900, Tejun Heo napisał(a): Does emergency unload count increase after each power down? I think I got it. Using smartctl I've done a test and shut down, then repeted the test. The only values

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Mark Lord
Tejun Heo wrote: Mark Lord wrote: Tejun Heo wrote: Michael Sedkowski wrote: Dnia 07-08-2007, Wt o godzinie 03:43 +0900, Tejun Heo napisał(a): Does emergency unload count increase after each power down? I think I got it. Using smartctl I've done a test and shut down, then repeted

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Tejun Heo
Mark Lord wrote: Heh.. I haven't instrumented it yet, but I did discover a bit more about it: The Power-Off_Retract_Count incrmenents *only* when there's data in the on-drive write-cache. So if I haven't written anything significantly large before suspending, then it often does NOT

Re: libata git tree, mbox queue status and contents

2007-08-08 Thread Mark Lord
Jeff, This patch (below) is needed by WD drives that powerup-in-standby. Cheers Mark Original Message Subject: + libata-adjust-libata-to-ignore-errors-after.patch added to -mm tree Date: Wed, 18 Jul 2007 17:03:29 -0700 From: [EMAIL PROTECTED] To: [EMAIL PROTECTED] CC: [EMAIL

Re: [PATCH 1/1] AVR32 PATA driver

2007-08-08 Thread Kristoffer Nyborg Gregertsen
Updated and simplified driver. Use only register transfer timing for both data and register transfers. This gives poorer performance in PIO1 and 2, but should not be a problem in PIO3 and 4, correct me if I'm wrong :) The driver works very we'll but I still wonder about the interrupts. I have an

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Mark Lord
Mark Lord wrote: Tejun Heo wrote: Mark Lord wrote: .. FWIW, Tejun, with 2.6.22, my new Seagate 160GB SATA drive (notebook) increments the Power-Off_Retract_Count on each suspend-to-RAM operation. It does not do any double spin-up/spin-down things though. Hmmm.. It shouldn't. libata now

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Mark Lord
Mark Lord wrote: My suspend script now has this little chunk of code at the point where it actually does the suspend-to-RAM: sync; sync hdparm -F /dev/sda ## flush drive write cache sleep 1 ## allow time for the flush to complete echo mem /sys/power/state ##

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Tejun Heo
Mark Lord wrote: Further to this, if I have an active-writer running at the time of suspend, then even my scripted sleep 1 is not good enough, as additional writes are still happening before/after the flush. Now I'll reboot and try it with the sleep 1 hardcoded inside sd_suspend(). Hmmm...

problems while mounting /boot partition

2007-08-08 Thread Michal Piotrowski
Hi Brian, Brian J. Murrell pisze: I am using Ubuntu Gutsy, which is the in-development branch heading for their next stable release. You forgot about message subject, so no one has read this report. I have noticed that since some kernel release post-2.6.20 I have been unable to mount my

Re: libata git tree, mbox queue status and contents

2007-08-08 Thread Jeff Garzik
Mark Lord wrote: Jeff, This patch (below) is needed by WD drives that powerup-in-standby. drivers/ata/libata-core.c~libata-adjust-libata-to-ignore-errors-after drivers/ata/libata-core.c --- a/drivers/ata/libata-core.c~libata-adjust-libata-to-ignore-errors-after +++

Re: [PATCH] libata: implement BROKEN_HPA horkage and apply it to HDS724040KLSA80

2007-08-08 Thread Quel Qun
-- Original message -- From: Tejun Heo [EMAIL PROTECTED] Luiz Fernando N. Capitulino wrote: Em Wed, 08 Aug 2007 11:21:08 +0900 Tejun Heo [EMAIL PROTECTED] escreveu: | Alan Cox wrote: | I'd rather know what is going on here. A drive can legitimately |

Re: [PATCH] libata: implement BROKEN_HPA horkage and apply it to HDS724040KLSA80

2007-08-08 Thread Luiz Fernando N. Capitulino
Em Wed, 08 Aug 2007 16:50:39 + [EMAIL PROTECTED] (Quel Qun) escreveu: | -- Original message -- | From: Tejun Heo [EMAIL PROTECTED] | Luiz Fernando N. Capitulino wrote: | Em Wed, 08 Aug 2007 11:21:08 +0900 | Tejun Heo [EMAIL PROTECTED] escreveu: | | |

Re: Disk spin down issue on shut down/suspend to disk

2007-08-08 Thread Mark Lord
Tejun Heo wrote: Mark Lord wrote: Further to this, if I have an active-writer running at the time of suspend, then even my scripted sleep 1 is not good enough, as additional writes are still happening before/after the flush. Now I'll reboot and try it with the sleep 1 hardcoded inside

Re: [SOLVED] problems while mounting /boot partition

2007-08-08 Thread Brian J. Murrell
On Wed, 2007-08-08 at 18:28 +0200, Michal Piotrowski wrote: Hi Brian, Hi Michal, Brian J. Murrell pisze: I am using Ubuntu Gutsy, which is the in-development branch heading for their next stable release. You forgot about message subject, so no one has read this report. Indeed. Very

[patch 0/4] Updated AN patches, now without gendisk

2007-08-08 Thread Kristen Carlson Accardi
Hello, Here is an updated set of patches that implement Asynchronous Notification support for ATAPI devices. In this version I no longer export the AN capability through genhd, and the uevent is sent by the scsi_device instead of gendisk. I added a generic SCSI device event mechanism so that it

[patch 1/4] libata: check for AN support

2007-08-08 Thread Kristen Carlson Accardi
Check to see if an ATAPI device supports Asynchronous Notification. If so, enable it. Signed-off-by: Kristen Carlson Accardi [EMAIL PROTECTED] Index: 2.6-git/drivers/ata/libata-core.c === --- 2.6-git.orig/drivers/ata/libata-core.c

[patch 2/4] scsi: expose AN support to user space

2007-08-08 Thread Kristen Carlson Accardi
If a scsi_device supports async notification for media change, then let user space know this capability exists by creating a new sysfs entry media_change_notify, which will be 1 if it is supported, and 0 if not supported. Create a routine which allows scsi devices to send a uevent when media

[patch 3/4] libata: expose AN to user space

2007-08-08 Thread Kristen Carlson Accardi
If Asynchronous Notification of media change events is supported, pass that information up to the SCSI layer. Signed-off-by: Kristen Carlson Accardi [EMAIL PROTECTED] Index: 2.6-git/drivers/ata/libata-scsi.c === ---

[patch 4/4] libata: send event when AN received

2007-08-08 Thread Kristen Carlson Accardi
When we get an SDB FIS with the 'N' bit set, we should send an event to user space to indicate that there has been a media change. This will be done via the scsi device. Signed-off-by: Kristen Carlson Accardi [EMAIL PROTECTED] Index: 2.6-git/drivers/ata/ahci.c

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

2007-08-08 Thread Andrew Morton
On Wed, 8 Aug 2007 12:08:10 -0700 Kristen Carlson Accardi [EMAIL PROTECTED] wrote: === --- 2.6-git.orig/include/linux/libata.h +++ 2.6-git/include/linux/libata.h @@ -139,7 +139,8 @@ enum { ATA_DFLAG_FLUSH_EXT = (1 4),

[PATCH] libata-core: support wildcard matching in ata_blacklist_entry

2007-08-08 Thread David Milburn
Support the use of '*' in model_num and model_rev entries in ata_device_blacklist[]. CC: [EMAIL PROTECTED] Signed-off-by: David Milburn [EMAIL PROTECTED] --- drivers/ata/libata-core.c | 11 +-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/ata/libata-core.c

SATA open bugs

2007-08-08 Thread Natalie Protasevich
Hi Tejun, Alan, Jeff, This is open bug list for SATA subsystem. It has more items than other subsystems, but mainly because this is such a hot technology now and so much work is being done in this area. Needless to say It is very well maintained and exemplary for some other areas that are not

Re: [PATCH 1/2] hpt366: fix PCI clock detection for HPT374

2007-08-08 Thread Bartlomiej Zolnierkiewicz
On Sunday 05 August 2007, Sergei Shtylyov wrote: HPT374 BIOS seems to only save f_CNT register value for the function #0 before re-tuning DPLL causing the driver to report obviously distorted f_CNT for the function #1 -- fix this by always reading the saved f_CNT register value from in the

Re: [PATCH 2/2] hpt366: UltraDMA filtering for SATA cards

2007-08-08 Thread Bartlomiej Zolnierkiewicz
On Sunday 05 August 2007, Sergei Shtylyov wrote: The Marvell bridge chips used on HighPoint SATA cards do not seem to support the UltraDMA modes 1, 2, and 3 (as well as any MWDMA modes), so the driver needs to account for this in the udma_filter() method. In order to achieve that, do the

Re: [PATCH 3/11] au1xxx-ide: use ide_tune_dma()

2007-08-08 Thread Bartlomiej Zolnierkiewicz
On Monday 06 August 2007, Sergei Shtylyov wrote: Bartlomiej Zolnierkiewicz wrote: Good, that's what I lacked for hpt366.c! Were you planning to push it to Linus soon? Not really but if needed I will extract MWDMA filter part and push it sooner. * Add -mdma_filter to ide_hwif_t and

Re: [PATCH] libata-core: support wildcard matching in ata_blacklist_entry

2007-08-08 Thread Alan Cox
On Wed, 8 Aug 2007 15:20:57 -0500 David Milburn [EMAIL PROTECTED] wrote: Support the use of '*' in model_num and model_rev entries in ata_device_blacklist[]. CC: [EMAIL PROTECTED] Signed-off-by: David Milburn [EMAIL PROTECTED] Suggestion: Pull the match function out of line so you don't

Re: SATA open bugs

2007-08-08 Thread Mark Lord
Natalie Protasevich wrote: Hi Tejun, Alan, Jeff, This is open bug list for SATA subsystem. It has more items than other ... http://bugzilla.kernel.org/show_bug.cgi?id=7693 http://bugzilla.kernel.org/show_bug.cgi?id=8738 adaptec and sata bad interaction

Re: SATA open bugs

2007-08-08 Thread Tejun Heo
Hello, Natalie. Natalie Protasevich wrote: http://bugzilla.kernel.org/show_bug.cgi?id=7693 This bug isn't related to libata in anyway. It's Can't use radeonfb with dvi port on a Samsung SyncMaster 204B (1600x1200). http://bugzilla.kernel.org/show_bug.cgi?id=8738 adaptec and sata bad

Re: SATA open bugs

2007-08-08 Thread James Bottomley
On Wed, 2007-08-08 at 15:31 -0700, Natalie Protasevich wrote: Hi Tejun, Alan, Jeff, This is open bug list for SATA subsystem. It has more items than other subsystems, but mainly because this is such a hot technology now and so much work is being done in this area. Needless to say It is very