On Sat, Jan 25 2003, Gregoire Favre wrote:
> On Fri, Jan 24, 2003 at 10:26:16AM +0100, Jens Axboe wrote:
>
> > The interesting part is whether failed has a sense attached, and if so
> > what length.
> > >
> > > > cdrom_analyze_sense_data(drive, failed, sense);
> >
> > To avoid confusion, I made the patch. Would have been easier to do in
> > the first place it seems :)
> >
> > ===== drivers/ide/ide-cd.c 1.35 vs edited =====
> > --- 1.35/drivers/ide/ide-cd.c Thu Nov 21 22:56:59 2002
> > +++ edited/drivers/ide/ide-cd.c Fri Jan 24 10:25:53 2003
> > @@ -649,6 +649,8 @@
> > struct cdrom_info *info = drive->driver_data;
> > void *sense = &info->sense_data;
> >
> > + if (failed && blk_pc_request(failed))
> > + printk("%s: failed, sense %p, len=%d\n", __FUNCTION__, failed->sense, failed->sense_len);
> > if (failed && failed->sense)
> > sense = failed->sense;
>
> Hello,
>
> sorry I didn't understood that I was supposed to do this...
> I have recompiled with your patch, and here the result of dmesg:
>
> cdrom_end_request: failed, sense ce68de20, len=0
> cdrom_end_request: failed, sense ce68de20, len=0
Thanks, this is the info I wanted. The sense info has not been read from
the drive, it seems.
-- Jens Axboe- 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/
This archive was generated by hypermail 2b29 : Fri Jan 31 2003 - 22:00:15 EST