Bug 8587 - disk on promise pdc20378 disabled at boot
Summary: disk on promise pdc20378 disabled at boot
Status: RESOLVED CODE_FIX
Alias: None
Product: Drivers
Classification: Unclassified
Component: PCI (show other bugs)
Hardware: i386 Linux
: P2 normal
Assignee: Tejun Heo
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-05 06:34 UTC by Jean-Luc Coulon
Modified: 2007-06-06 08:17 UTC (History)
0 users

See Also:
Kernel Version: 2.6.22-rc4
Subsystem:
Regression: ---
Bisected commit-id:


Attachments
kernel config file (11.76 KB, application/x-gzip)
2007-06-05 06:36 UTC, Jean-Luc Coulon
Details
syslog (959 bytes, application/x-gzip)
2007-06-05 06:41 UTC, Jean-Luc Coulon
Details
sata_promise-use-TF-issue-for-polling-NODATA (1.24 KB, patch)
2007-06-06 03:50 UTC, Tejun Heo
Details | Diff

Description Jean-Luc Coulon 2007-06-05 06:34:33 UTC
Most recent kernel where this bug did *NOT* occur: 2.6.22-rc3-git7
Distribution: Debian sid X86_64
Hardware Environment: PDC20378 in ASUS A8V Deluxe, Athlon64 3600+
Software Environment: sata_promise
Problem Description: disk is disabled

Steps to reproduce: boot the system
Comment 1 Jean-Luc Coulon 2007-06-05 06:36:43 UTC
Created attachment 11685 [details]
kernel config file
Comment 2 Jean-Luc Coulon 2007-06-05 06:41:10 UTC
Created attachment 11686 [details]
syslog

Related part of the syslog
Comment 3 Anonymous Emailer 2007-06-05 10:16:25 UTC
Reply-To: akpm@linux-foundation.org

On Tue, 5 Jun 2007 06:30:52 -0700
bugme-daemon@bugzilla.kernel.org wrote:

> http://bugzilla.kernel.org/show_bug.cgi?id=8587
> 
>            Summary: disk on promise pdc20378 disabled at boot
>     Kernel Version: 2.6.22-rc4
>             Status: NEW
>           Severity: normal
>              Owner: greg@kroah.com
>          Submitter: jean.luc.coulon@gmail.com
> 
> 
> Most recent kernel where this bug did *NOT* occur: 2.6.22-rc3-git7
> Distribution: Debian sid X86_64
> Hardware Environment: PDC20378 in ASUS A8V Deluxe, Athlon64 3600+
> Software Environment: sata_promise
> Problem Description: disk is disabled
> 
> Steps to reproduce: boot the system
> 

If Jean really meant 2.6.22-rc3-git7 then this is a very recent regression.

Comment 4 Jean-Luc Coulon 2007-06-05 10:51:18 UTC
Yes, really 2.6.22-rc3-git7


[jean-luc@tangerine] % uname -a
Linux tangerine 2.6.22-rc3-git7-k8-1 #1 PREEMPT Mon Jun 4 13:08:28 CEST 2007
x86_64 GNU/Linux
Comment 5 Michal Piotrowski 2007-06-05 11:17:50 UTC
# CONFIG_BLK_DEV_PDC202XX_OLD is not set
# CONFIG_BLK_DEV_PDC202XX_NEW is not set
CONFIG_BLK_DEV_VIA82CXXX=m


# CONFIG_PATA_PDC_OLD is not set
# CONFIG_PATA_PDC2027X is not set

Hmmm... Which PDC driver do you use?
Comment 6 Jean-Luc Coulon 2007-06-05 11:26:28 UTC
It is a sata drive.
I use sata_promise

CONFIG_SATA_PROMISE=m


Regards

Jean-Luc
Comment 7 Michal Piotrowski 2007-06-05 11:40:46 UTC
Please try
wget http://www.stardust.webpages.pl/files/patches/libata_revert/libata-1.patch
cat /patch/to/libata-1.patch | patch -p1 -R
make && make test_kernel
Comment 8 Jean-Luc Coulon 2007-06-05 12:17:13 UTC
Ok, it works that way

J-L
Comment 9 Michal Piotrowski 2007-06-05 12:43:37 UTC
Tejun, it is your patch

commit 464cf177df7727efcc5506322fc5d0c8b896f545
Author: Tejun Heo <htejun@gmail.com>
Date:   Sun May 27 15:10:40 2007 +0200

    libata: always use polling SETXFER
Comment 10 Mikael Pettersson 2007-06-05 14:26:43 UTC

    
Comment 11 Tejun Heo 2007-06-06 02:13:20 UTC
This is because sata_promise doesn't like polling SETXFER.  Under investigation.
Comment 12 Tejun Heo 2007-06-06 03:50:22 UTC
Created attachment 11694 [details]
sata_promise-use-TF-issue-for-polling-NODATA

Please test this patch.  Thanks.
Comment 13 Jean-Luc Coulon 2007-06-06 05:38:45 UTC
With the patch applied, it works fine.
Comment 14 Tejun Heo 2007-06-06 08:17:13 UTC
Thanks.  Resolving the bug.

Note You need to log in before you can comment on or make changes to this bug.