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

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

2007-08-08 Thread MikeW
Alan Cox 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 specif

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 repo

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

2007-08-08 Thread Michael Sedkowski
Dnia 07-08-2007, Wt o godzinie 23:28 +0200, Maciej Rutecki napisał(a): > > > Well, on my box (nx6325) with the appended (experimental) patch applied > > > on top of 2.6.23-rc1 with the patchset from > > > > s/2.6.23-rc1/2.6.23-rc2/ > > > > HP nx 6310, _old_ shutdown 2.6.23-rc2+above patches: > >

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 spe

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

2007-08-08 Thread MikeW
Alan Cox lxorguk.ukuu.org.uk> writes: > >>> 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 thi

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

2007-08-08 Thread Alan Cox
> Thanks for clarifying - as you say, the spec is specific to PCI. > Our silicon has its own host interface which was confused by a length of zero > ! You aren't the first. If you look at the current tree you'll see we now have ata_dumb_qc_prep() for interfaces that think 0x is zero

Re: libata-core.c: ata_fill_sg() ... SATA AHCI

2007-08-08 Thread MikeW
Alan Cox lxorguk.ukuu.org.uk> writes: > > SFF 8038i covers the standard taskfile IDE class PCI device interface for > PIO and DMA. It also covers the alternate ADMA interface. > >> Since DMA controller details may vary according to hardware, I >> would have expected that ata_fill_sg() would be on

Re: libata-core.c: ata_fill_sg() ... SATA AHCI

2007-08-08 Thread Alan Cox
> I note also that Intel's Serial ATA AHCI spec has a 22-bit DMA byte count > in the PRDT, so 0 == 64k cannot be assumed here either. Different standard and our AHCI driver doesn't use ata_fill_sg either 8) - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a mes

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

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 0x40

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 o

[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 linux-2.6.23rc1-mm1

[PATCH] libata-core: Document some limits/assumptions about ID_ATA

2007-08-08 Thread Alan Cox
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 --- linux.vanilla-2.6.23rc1-mm1/drivers/ata/libata-core.c 2007-07-26 15:02:57.

[PATCH] libata-core: Allow translation setting to fail

2007-08-08 Thread Alan Cox
On some early drives (pre ATA1) this feature is not supported. If it fails then we know the drive geometry is the hardware geometry and the one we tried to set anyway so just carry on. Signed-off-by: Alan Cox <[EMAIL PROTECTED]> diff -u --new-file --recursive --exclude-from /usr/src/exclude linu

[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 --- linux.vanilla-

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 r

[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 linux-2.6.23r

[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 linux.vanilla-2.6.23r

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 i

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 tes

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 do

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 in

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 i

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 is

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 ## suspen

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().

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

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 +++ a/drivers/ata/libata-core.

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

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 sd_su

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.

[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 cha

[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 === --- 2.6-git.orig/dr

[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 = (

[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 b

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 that

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 f

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

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

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 http://bugzilla.kernel.org/show_bug.cgi?i

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 > in

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 v

Re: SATA open bugs

2007-08-08 Thread Jens Axboe
On Thu, Aug 09 2007, Tejun Heo wrote: > > http://bugzilla.kernel.org/show_bug.cgi?id=7805 tejun is pinging > > pktcdvd maintainer (?) > > Not a libata bug. This is caused by pktcdvd holding onto the device > thus preventing it from being revalidated. Didn't get pong for my ping > yet. Jens, any