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

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 you

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 no

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:

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 atta

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

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 interfa

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

2008-02-24 Thread Borislav Petkov
On Mon, Feb 25, 2008 at 11:58:57AM +1000, Brad Rosser wrote: > 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.

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

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 fundam

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 suppo

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 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: 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 an

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

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

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 b

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

2008-02-24 Thread Adrian Bunk
On Sun, Feb 24, 2008 at 10:10:33PM +0100, Paolo Ciarrocchi wrote: > 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-

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 f

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

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

2008-02-24 Thread Paolo Ciarrocchi
Applies on top of the previous patch. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/ide-cd.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c index 3181e71..76317ab 100644 --- a/drivers/ide/ide

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

[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 b/drivers/ide/p

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

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 o

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 var

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: h

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

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, 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 a/drivers/

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 + patc

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

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: o-sparc64/vmlinux.o

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