Re: 2.6.25-rc2 + smartd = hang

2008-02-24 Thread Anders Eriksson
[EMAIL PROTECTED] said: Could you try bisecting it down to the guilty commit using git-bisect? [ the old stuff got few hundred commits in 2.6.25 ] Thanks, Bart Will do. It'll take a while though. Not a fast machine and used by the household... /A - To unsubscribe from this list: send the

Re: JMicron - hard resetting link

2008-02-24 Thread Gabor FUNK
Since it is not a switch on and see problem, I'm not in too good position, so unless someone have a really great idea or observation, I seriously have to consider to replace the MB and probably add some extra sata controllers. If you can still do some testing, what happens if you unplug power

Re: [Fixed PATCH] hpt366: fix section mismatch warnings

2008-02-24 Thread Bartlomiej Zolnierkiewicz
On Saturday 23 February 2008, Sam Ravnborg wrote: hpt366: fix section mismatch warnings Fix following warnings: WARNING: o-sparc64/vmlinux.o(.data+0x195a38): Section mismatch in reference from the variable hpt37x_info.0 to the variable .devinit.data:hpt370 WARNING:

Re: [PATCH 00/10] IDE: Coding Style Fixes

2008-02-24 Thread Bartlomiej Zolnierkiewicz
On Saturday 23 February 2008, Paolo Ciarrocchi wrote: Hi Bartlomiej, The following 10 patches remove a huge number of errors and warnings reported by checkpatch.pl from the IDE subsystem. [...] Thanks! I applied everything (patch #8 with fixes based on comments from Adrian Cyrill + patches

Re: [PATCH 0/6] IDE: new series of Coding Style Fixes

2008-02-24 Thread Bartlomiej Zolnierkiewicz
On Saturday 23 February 2008, Paolo Ciarrocchi wrote: Bart, here is a new series of patches that remove some errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches have been compile tested and are against this morning (CET time :-) linux-next Paolo

Re: [git patches] libata fixes

2008-02-24 Thread Bartlomiej Zolnierkiewicz
On Sunday 24 February 2008, Jeff Garzik wrote: [...] Alan Cox (1): pata_atiixp: Use 255 sector limit AHCI needs sorting too but this deals with the old interface Signed-off-by: Alan Cox [EMAIL PROTECTED] Signed-off-by: Jeff Garzik [EMAIL PROTECTED] [...] diff --git

Re: [git patches] libata fixes

2008-02-24 Thread Alan Cox
From the patch description it can't be told whether the patch itself is correct and only the patch description is bogus... zero length PRD misparsing. If I remember rightly old IDE never generates 64K PRD slots because other hardware can't handle it either (CS5520/30 etc) - To unsubscribe from

Re: [git patches] libata fixes

2008-02-24 Thread Bartlomiej Zolnierkiewicz
On Sunday 24 February 2008, Alan Cox wrote: From the patch description it can't be told whether the patch itself is correct and only the patch description is bogus... zero length PRD misparsing. If I remember rightly old IDE never generates 64K PRD slots because other hardware can't handle

Re: IDE cdrom problem with PLEXTOR DVDR PX-608AL

2008-02-24 Thread Borislav Petkov
On Sun, Feb 24, 2008 at 03:38:34PM +1000, Brad Rosser wrote: [ Added Bart to CC: ] Hi Brad, Borislav Petkov wrote: Feb 14 00:18:18 kernel: hde: cdrom_pc_intr: The drive appears confused (ireason = 0x01). Trying to recover by ending request. Feb 14 00:27:27 kernel: hdc:

Re: [Fixed PATCH] hpt366: fix section mismatch warnings

2008-02-24 Thread Sam Ravnborg
On Sun, Feb 24, 2008 at 03:33:51PM +0100, Bartlomiej Zolnierkiewicz wrote: On Saturday 23 February 2008, Sam Ravnborg wrote: hpt366: fix section mismatch warnings Fix following warnings: WARNING: o-sparc64/vmlinux.o(.data+0x195a38): Section mismatch in reference from the variable

Re: IDE cdrom problem with PLEXTOR DVDR PX-608AL

2008-02-24 Thread Bartlomiej Zolnierkiewicz
On Sunday 24 February 2008, Borislav Petkov wrote: On Sun, Feb 24, 2008 at 03:38:34PM +1000, Brad Rosser wrote: [...] kernel: hda: cdrom_pc_intr: The drive appears confused (ireason = 0x01). Trying to recover by ending request. last message repeated 3 times kernel: ide: failed opcode

Re: [PATCH 0/6] IDE: new series of Coding Style Fixes

2008-02-24 Thread Paolo Ciarrocchi
On Sun, Feb 24, 2008 at 5:38 PM, Bartlomiej Zolnierkiewicz [EMAIL PROTECTED] wrote: On Saturday 23 February 2008, Paolo Ciarrocchi wrote: Bart, here is a new series of patches that remove some errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches

[PATCH 4/6] IDE: Coding Style fixes to drivers/ide/pci/it8213.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free, only a few WARNING: line over 80 characters are left. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/it8213.c |9 - 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/ide/pci/it8213.c

[PATCH 1/3] IDE: Coding Style fixes to drivers/ide/ide-cd.c

2008-02-24 Thread Paolo Ciarrocchi
Before: total: 43 errors, 66 warnings, 2183 lines checked After: total: 0 errors, 36 warnings, 2192 lines checked I didn't (and I don't plan to) fix the warnings: WARNING: line over 80 characters Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/ide-cd.c | 245

[PATCH 3/3] IDE: Coding Style fixes to drivers/ide/pci/cy82c693.c

2008-02-24 Thread Paolo Ciarrocchi
Before: total: 34 errors, 14 warnings, 456 lines checked After: total: 0 errors, 8 warnings, 456 lines checked Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/cy82c693.c | 74 ++-- 1 files changed, 37 insertions(+), 37

[PATCH 0/3] Various IDE coding style fixes

2008-02-24 Thread Paolo Ciarrocchi
Hi Bart, three more patches, all are compile tested and against yesterday linux-next IDE: Replace __FUNCTION__ with __func__ depends on IDE: Coding Style fixes to drivers/ide/ide-cd.c Paolo Ciarrocchi (3): IDE: Coding Style fixes to drivers/ide/ide-cd.c IDE: Replace __FUNCTION__

Need help with libata error handling in libsas

2008-02-24 Thread James Bottomley
I keep hearing that we need to convert libsas to use libata's new error handling. Unfortunately, I have very little conception of what that means. Right at the moment, libsas doesn't use any error handling functions of libata at all. I've looked through the libata-eh functions, and I find them

Problem with pata_amd/sata_nv on ASRock Penryn1600SLI-110dB board

2008-02-24 Thread Andreas Bierfert
Hi there, I got the above mentioned board [1] two weeks ago with a Q6600 cpu. I am running Fedora 8 x86_64. While using 2.6.23, the attached cdrom/dvd drives went all crazy together with a nice error message of the hdd from time to time. After switching to 2.6.24 the cd drives were useful again

Re: [PATCH 2/3] IDE: Replace __FUNCTION__ with __func__

2008-02-24 Thread Joe Perches
On Sun, 2008-02-24 at 22:11 +0100, Paolo Ciarrocchi wrote: - driver.\n, __FUNCTION__, drive-name); + driver.\n, __func__, drive-name); Is there consensus on this style conversion? Right now, there's 629 uses of __FUNCTION__ and 439 uses

[2.6 patch] make atapi_dmadir static

2008-02-24 Thread Adrian Bunk
atapi_dmadir can now become static. Signed-off-by: Adrian Bunk [EMAIL PROTECTED] --- drivers/ata/libata-core.c |2 +- drivers/ata/libata.h |1 - 2 files changed, 1 insertion(+), 2 deletions(-) ad621a905275c744307cdb9bb4bab47a187a477b diff --git a/drivers/ata/libata-core.c

Re: IDE cdrom problem with PLEXTOR DVDR PX-608AL

2008-02-24 Thread Brad Rosser
On Mon, Feb 25, 2008 at 3:34 AM, Borislav Petkov [EMAIL PROTECTED] wrote: On Sun, Feb 24, 2008 at 03:38:34PM +1000, Brad Rosser wrote: There's nothing in the bios that allows me to move one away from the other. I can 'reserve' or block IRQs, but that only shifts them both to another

Re: new ata_port_operations for .pmp_{read,write} ?

2008-02-24 Thread Mark Lord
Tejun Heo wrote: Hello, Mark. Mark Lord wrote: Mark Lord wrote: An alternative to all this, might be to expose the select_pmp() function shown in the sample code, and have libata-pmp.c call that, instead of having the new new .pmp_{read,write} functions. .. I wonder if this might be more

Re: new ata_port_operations for .pmp_{read,write} ?

2008-02-24 Thread Mark Lord
Jeff Garzik wrote: Tejun Heo wrote: Yeah, exactly. I think what needs to be done is to separate out SFF assumptions from core layer, factor out SFF-proper helpers and use them to implement LLDs for quasi-SFF controllers. Thinking long term, I continue to hope that SFF support can eventually

Re: new ata_port_operations for .pmp_{read,write} ?

2008-02-24 Thread Jeff Garzik
Mark Lord wrote: Jeff Garzik wrote: Tejun Heo wrote: Yeah, exactly. I think what needs to be done is to separate out SFF assumptions from core layer, factor out SFF-proper helpers and use them to implement LLDs for quasi-SFF controllers. Thinking long term, I continue to hope that SFF

Re: new ata_port_operations for .pmp_{read,write} ?

2008-02-24 Thread Mark Lord
Tejun Heo wrote: Hello, Mark. Mark Lord wrote: Mark Lord wrote: An alternative to all this, might be to expose the select_pmp() function shown in the sample code, and have libata-pmp.c call that, instead of having the new new .pmp_{read,write} functions. .. I wonder if this might be more

Re: new ata_port_operations for .pmp_{read,write} ?

2008-02-24 Thread Jeff Garzik
To repeat myself from threads past... In particular with the Marvell 6440 (SATA/SAS, drivers/scsi/mvsas.c) when SATA PMP support is completed, we will want to look at passing things other than normal ATA commands via -qc_issue. Although selection isn't necessarily, talking to a PMP is

Re: new ata_port_operations for .pmp_{read,write} ?

2008-02-24 Thread Tejun Heo
Mark Lord wrote: No, the quickest solution for sata_mv, the one I apparently will now be using, is to just clone about 250 lines of reset/debouce/probe code from libata-core and change perhaps five lines of it to work around this issue plus some chipset errata. What I was thinking of,

Re: IDE cdrom problem with PLEXTOR DVDR PX-608AL

2008-02-24 Thread Brad Rosser
Hi Boris, Well, this sounds strange. Are you sure you're entering the boot options correctly on the kernel command line? Which is your boot loader? I just booted my machine with 'hdc=noprobe' (hdc is my cdrom drive) and here's what i get: ... [0.304774] Probing IDE interface

Re: [PATCH 2/3] IDE: Replace __FUNCTION__ with __func__

2008-02-24 Thread Paolo Ciarrocchi
On 2/25/08, Joe Perches [EMAIL PROTECTED] wrote: On Sun, 2008-02-24 at 22:11 +0100, Paolo Ciarrocchi wrote: - driver.\n, __FUNCTION__, drive-name); + driver.\n, __func__, drive-name); Is there consensus on this style conversion? I did that in a separate patch because i'm not sure about

Re: 2.6.26-git0: IDE oops during boot

2008-02-24 Thread Yinghai Lu
On Fri, Feb 15, 2008 at 3:15 AM, Kamalesh Babulal [EMAIL PROTECTED] wrote: Thanks for pointing the patch, I do not have the SES config option enabled, then too i tried your patch, but that does not solve the panic. The kernel panic's with the same panic message as before. I have attached

Re: [Bugme-new] [Bug 10091] New: GGW-H10N SATA DVD drive write errors using AHCI driver with nForce 630a chipset

2008-02-24 Thread Andrew Morton
On Sun, 24 Feb 2008 15:23:24 -0800 (PST) [EMAIL PROTECTED] wrote: http://bugzilla.kernel.org/show_bug.cgi?id=10091 Summary: GGW-H10N SATA DVD drive write errors using AHCI driver with nForce 630a chipset Product: IO/Storage Version: 2.5

Re: 2.6.26-git0: IDE oops during boot

2008-02-24 Thread Yinghai Lu
On Sun, Feb 24, 2008 at 11:05 PM, Yinghai Lu [EMAIL PROTECTED] wrote: On Fri, Feb 15, 2008 at 3:15 AM, Kamalesh Babulal [EMAIL PROTECTED] wrote: Thanks for pointing the patch, I do not have the SES config option enabled, then too i tried your patch, but that does not solve the

Re: [PATCH 3/3] IDE: Coding Style fixes to drivers/ide/pci/cy82c693.c

2008-02-24 Thread Ingo Molnar
* Paolo Ciarrocchi [EMAIL PROTECTED] wrote: Before: total: 34 errors, 14 warnings, 456 lines checked After: total: 0 errors, 8 warnings, 456 lines checked sidenote: please also indicate to maintainers that the cleanup causes no change in generated code. Find below of how one of your

[Bug 10086] New: 2.6.25-rc2 + smartd = hang

2008-02-24 Thread bugme-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=10086 Summary: 2.6.25-rc2 + smartd = hang Product: IO/Storage Version: 2.5 KernelVersion: 2.6.25-rc2 Platform: All OS/Version: Linux Tree: Mainline Status: NEW