linux-kernel - Re: [PATCH] ide-floppy fix

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-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 15 Jul 2008 22:39:13 +0200
From:	Borislav Petkov <petkovbb@...glemail.com>
To:	Bartlomiej Zolnierkiewicz <bzolnier@...il.com>
Cc:	linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org,
	stable@...nel.org
Subject: Re: [PATCH] ide-floppy fix

On Wed, Jul 16, 2008 at 05:59:16PM +0200, Bartlomiej Zolnierkiewicz wrote:
> 
> Hi,
> 
> On Tuesday 15 July 2008, Borislav Petkov wrote:
> 
> [...]
> 
> > On a different note, the current pata tree on top of v2.6.25-2125-g50515af blows
> > up here with the following error:
> > 
> > [    4.296729] Uniform Multi-Platform E-IDE driver
> > [    4.297905] ICH4: IDE controller (0x8086:0x24cb rev 0x02) at  PCI slot 0000:00:1f.1
> > [    4.297986] ACPI: PCI Interrupt 0000:00:1f.1[A] -> GSI 18 (level, low) -> IRQ 18
> > [    4.298153] ICH4: not 100% native mode: will probe irqs later
> > [    4.298213]     ide0: BM-DMA at 0xfc00-0xfc07
> > [    4.298282]     ide1: BM-DMA at 0xfc08-0xfc0f
> > [    4.561768] hda: QUANTUM FIREBALLlct10 20, ATA DISK drive
> > [    4.816724] hdb: SAMSUNG SP2014N, ATA DISK drive
> > [    4.867959] hda: drive side 80-wire cable detection failed, limiting max speed to UDMA33
> > [    4.868027] hda: UDMA/33 mode selected
> > [    4.868441] hdb: UDMA/100 mode selected
> > [    5.540683] hdc: IOMEGA ZIP 100 ATAPI, ATAPI FLOPPY drive
> > [    5.795564] hdd: IC35L120AVV207-0, ATA DISK drive
> > [    5.847295] hdd: host side 80-wire cable detection failed, limiting max speed to UDMA33
> > [    5.847362] hdd: UDMA/33 mode selected
> > [    5.847715] ide0 at 0x1f0-0x1f7,0x3f6 on irq 14
> > [    5.855487] ide1 at 0x170-0x177,0x376 on irq 15
> > [    5.875927] ide_generic: please use "probe_mask=0x3f" module parameter for probing all legacy ISA IDE ports
> > [    5.876012] ide_generic: I/O resource 0x1F0-0x1F7 not free.
> > [    5.876074] ide_generic: I/O resource 0x170-0x177 not free.
> > [   11.342504] hde: no response (status = 0xa1), resetting drive
> > [   17.206535] hdf: no response (status = 0xa1), resetting drive
> 
> hde? hdf?
> 
> [...]

yep, looks strange to me too. Isn't that the MAX_HWIFS upper bound of a loop.. ?

> 
> > I tracked the error down to the call to ide_register_port(hwif) in
> > ide-probe.c:ide_host_register() which does device_register(&hwif->gendev) and
> > the hwif->gendev->kobj seems unitialized thus the WARN_ON on its refcount in
> > kref_get(). Will look into it more when i get some free time.
> 
> Unfortunately I couldn't reproduce this problem here (2.6.26 + pata tree)
> so please try to debug it and/or narrow it down to the guilty change.

Will do.

> > --
> > From: Borislav Petkov <petkovbb@...il.com>
> > 
> > Check the correct flags-location for set features.
> > 
> > Signed-off-by: Borislav Petkov <petkovbb@...il.com>
> 
> Thanks, I folded the fix into original patch (->dev_flags is not yet
> upstream so -stable fix shouldn't be necessary).
> 
> While on it: I later noticed that there will be also need for common
> ATA/ATAPI ->dev_flags in the future so I wonder whether current
> ->dev_flags should be renamed to ->atapi_flags (& s/*DFLAG*/*AFLAG*/).
> 
> If there is agreement on this I'll fix it in pata tree.

... or if there's room, use a single ->dev_flags for all possible flag settings?

-- 
Regards/Gruß,
    Boris.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/