linux-kernel - Re: 2.6.19-rc5: known regressions

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]

Message-ID: <87ejsd3gcr.fsf@sycorax.lbl.gov>
Date:	Wed, 08 Nov 2006 11:09:40 -0800
From:	Alex Romosan <romosan@...orax.lbl.gov>
To:	Jens Axboe <jens.axboe@...cle.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: 2.6.19-rc5: known regressions

Jens Axboe <jens.axboe@...cle.com> writes:

> On Wed, Nov 08 2006, Adrian Bunk wrote:
>> Subject    : unable to rip cd
>> References : http://lkml.org/lkml/2006/10/13/100
>> Submitter  : Alex Romosan <romosan@...orax.lbl.gov>
>> Status     : unknown
>
> Alex, was/is this repeatable? If so I'd like you to repeat with this
> debug patch applied, I cannot reproduce it locally.
>
> diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c
> index bddfebd..ad03e19 100644
> --- a/drivers/ide/ide-cd.c
> +++ b/drivers/ide/ide-cd.c
> @@ -1726,8 +1726,10 @@ static ide_startstop_t cdrom_newpc_intr(
>  		/*
>  		 * write to drive
>  		 */
> -		if (cdrom_write_check_ireason(drive, len, ireason))
> +		if (cdrom_write_check_ireason(drive, len, ireason)) {
> +			blk_dump_rq_flags(rq, "cdrom_newpc");
>  			return ide_stopped;
> +		}
>  
>  		xferfunc = HWIF(drive)->atapi_output_bytes;
>  	} else  {
> @@ -1859,8 +1861,10 @@ static ide_startstop_t cdrom_write_intr(
>  	}
>  
>  	/* Check that the drive is expecting to do the same thing we are. */
> -	if (cdrom_write_check_ireason(drive, len, ireason))
> +	if (cdrom_write_check_ireason(drive, len, ireason)) {
> +		blk_dump_rq_flags(rq, "cdrom_pc");
>  		return ide_stopped;
> +	}
>  
>  	sectors_to_transfer = len / SECTOR_SIZE;
>  

i've tried it again with the above patch applied and when i start
cdparanoia i get:

kernel: hdc: write_intr: wrong transfer direction!
kernel: cdrom_newpc: dev hdc: type=2, flags=114c9
kernel: 
kernel: sector 59534648, nr/cnr 0/0
kernel: bio 00000000, biotail c14b2800, buffer 00000000, data 00000000, len 56
kernel: cdb: 12 00 00 00 38 00 00 00 00 00 00 00 00 00 00 00 

as for the lock up, the ripping process never completes, it starts and
then it hangs somewhere in the middle of the track. it could be that
the disk has some problems. anyway, abort execution doesn't work until
i physically eject the cd from the drive (which seems to be an
improvement from a couple of rc's ago). hope this helps.

--alex--

-- 
| I believe the moment is at hand when, by a paranoiac and active |
|  advance of the mind, it will be possible (simultaneously with  |
|  automatism and other passive states) to systematize confusion  |
|  and thus to help to discredit completely the world of reality. |
-
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/