lkml.org 
[lkml]   [2004]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: ide taskfile and cdrom hang
Date
On Monday 02 of February 2004 19:46, Mark Haverkamp wrote:
> On Sun, 2004-02-01 at 12:48, Bartlomiej Zolnierkiewicz wrote:
> > Hi,
> >
> > On Friday 30 of January 2004 23:36, Mark Haverkamp wrote:
> > > We have some test machines here at OSDL that have a problem with the
> > > ide cdrom driver hanging when we cat the /proc/ide/hda/identify file.
> > > After 30 seconds the console displays: "hda: lost interrupt" which
> > > reoccurs every 30 seconds forever. We noticed it on 2.6.2-rc2-mm1 but
> > > It looks like this has been a problem for a while. Our test machines
> > > just changed their configuration to use make defconfig. I found that
> > > if
> > > CONFIG_IDE_TASKFILE_IO is N then the hang doesn't occur. Is this a
> > > common problem or are there just certain drives that won't work with
> > > taskfile i/o enabled? I've included my .config, lspci as attachments.
> > > The cdrom model is CD-224E.
> >
> > Does 'hdparm -I /dev/hda" work? If so can you try this patch?
> > It reverts taskfile code to use old (non-taskfile) order of accessing
> > port registers for ATAPI identify command. This is a good starting
> > point.
> >
> > If CONFIG_IDE_TASKFILE_IO is N code just ignores BUSY status of a drive
> > and reads some random data. I would like to have this problem solved
> > at least instead of adding the same workaround to CONFIG_IDE_TASKFILE_IO.
> >
> > Cheers,
> > --bart
>
> Thanks,
>
> hdparm -I /dev/hda didn't hang. I got this on the console:
>
> hda: drive_cmd: status=0x51 { DriveReady SeekComplete Error }
> hda: drive_cmd: error=0x04Aborted Command

That's okay.
hdparm first tries WIN_IDENTIFY which can fail on ATAPI devices.

> I added the patch that you provided and I still get the hang when I cat
> /proc/ide/hda/identify. I put a printk In the code to make sure that it
> was going through the added code, and it is.

So it is something different. Can you give this patch a go?
We will know more about what's going on.

Thanks,
--bart

linux-2.6.2-rc2-root/drivers/ide/ide-taskfile.c | 3 +++
1 files changed, 3 insertions(+)

diff -puN drivers/ide/ide-taskfile.c~ide_tf_identify_debug1 drivers/ide/ide-taskfile.c
--- linux-2.6.2-rc2/drivers/ide/ide-taskfile.c~ide_tf_identify_debug1 2004-02-02 20:30:16.435636072 +0100
+++ linux-2.6.2-rc2-root/drivers/ide/ide-taskfile.c 2004-02-02 20:40:25.604028400 +0100
@@ -794,6 +794,7 @@ static u8 wait_drive_not_busy(ide_drive_
*/
while (((stat = hwif->INB(IDE_STATUS_REG)) & BUSY_STAT) && retries--)
udelay(10);
+ printk("%s: (WAIT_NOT_BUSY) status=0x%02x\n", drive->name, stat);
return stat;
}

@@ -811,6 +812,7 @@ check_status:
if (!OK_STAT(stat, good_stat, BAD_R_STAT)) {
if (stat & (ERR_STAT | DRQ_STAT))
return DRIVER(drive)->error(drive, __FUNCTION__, stat);
+ printk("%s: (BUSY) status=0x%02x\n", drive->name, stat);
/* BUSY_STAT: No data yet, so wait for another IRQ. */
ide_set_handler(drive, &task_in_intr, WAIT_WORSTCASE, NULL);
return ide_started;
@@ -836,6 +838,7 @@ check_status:
if (!rq->nr_sectors) {
good_stat = 0;
stat = wait_drive_not_busy(drive);
+ printk("%s: (CHECK_STATUS) status=0x%02x\n", drive->name, stat);
goto check_status;
}

_

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

\
 
 \ /
  Last update: 2005-03-22 14:00    [W:0.110 / U:0.164 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site