linux-kernel - libata regression in current -git

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <480D6A25.2040206@trash.net>
Date:	Tue, 22 Apr 2008 06:31:33 +0200
From:	Patrick McHardy <kaber@...sh.net>
To:	Tejun Heo <htejun@...il.com>
CC:	Linux Kernel Mailinglist <linux-kernel@...r.kernel.org>
Subject: libata regression in current -git

I bisected a boot-failure with sata-sis back to this libata-change:

---
libata: clear SError after link resume

SError used to be cleared in ->postreset.  This has small hotplug race
condition.  If a device is plugged in after reset is complete but
postreset hasn't run yet, its hotplug event gets lost when SError is
cleared.  This patch makes sata_link_resume() clear SError.  This
kills the race condition and makes a lot of sense as some PMP and host
PHYs don't work properly without SError cleared.

This change makes sata_pmp_std_{pre|post}_reset()'s unnecessary as
they become identical to ata_std counterparts.  It also simplifies
sata_pmp_hardreset() and ahci_vt8251_hardreset().
---

Whats happening is that sata_scr_read() in sata_link_resume()
returns -1 and the reset never completes. Ignoring the return
value as before this patch makes it work for me again.

This is lspci-output of the controller, dmesg of a working and
non-working boot and .config are attached.

If you need further information or want me to test something,
please let me know.

00:05.0 RAID bus controller: Silicon Integrated Systems [SiS] RAID bus 
controller 180 SATA/PATA  [SiS] (rev 01) (prog-if 85)
         Subsystem: ASUSTeK Computer Inc. Unknown device 810e
         Control: I/O+ Mem- BusMaster+ SpecCycle- MemWINV- VGASnoop- 
ParErr- Stepping- SERR- FastB2B- DisINTx-
         Status: Cap- 66MHz+ UDF- FastB2B- ParErr- DEVSEL=medium 
 >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
         Latency: 128
         Interrupt: pin A routed to IRQ 17
         Region 0: I/O ports at eff0 [size=8]
         Region 1: I/O ports at efe4 [size=4]
         Region 2: I/O ports at efa8 [size=8]
         Region 3: I/O ports at efe0 [size=4]
         Region 4: I/O ports at ef90 [size=16]
         Region 5: I/O ports at <unassigned>
         Kernel driver in use: sata_sis

View attachment "config" of type "text/plain" (48456 bytes)

View attachment "log.broken" of type "text/plain" (9588 bytes)

View attachment "log.ok" of type "text/plain" (20675 bytes)