On Mon, 16 Jun 2008, James Bottomley wrote: > On Mon, 2008-06-16 at 17:05 +0200, Geert Uytterhoeven wrote: > > On Sun, 15 Jun 2008, James Bottomley wrote: > > > On Fri, 2008-06-13 at 13:57 +0200, Geert Uytterhoeven wrote: > > > > We've found another regression in 2.6.25 w.r.t. CD media change on PS3. > > > > > > > > It can easily be reproduced by: > > > > > > > > 1. Inserting an audio CD > > > > 2. Running the following command as soon as the blue CD/DVD/BD drive LED > > > > stops blinking and is lit continuously: > > > > > > > > cdparanoia -Z -q 1-1[:1] /dev/null || echo failed > > > > > > > > On 2.6.25 (and current mainline), you have to wait ca. 10 seconds after > > > > insertion, or it will fail. > > > > On 2.6.24 and older, it just works immediately. > > > > > > > > It does not matter whether > > > > http://git.kernel.org/?p=linux%2Fkernel%2Fgit%2Fjejb%2Fscsi-rc-fixes-2.6.git;a=commitdiff_plain;h=d1daeabf0da5bfa1943272ce508e2ba785730bf0 > > > > is applied or not. > > > > > > > > We haven't bisected it yet. > > > > > > There aren't that many commits affecting sr between 2.6.24 and 2.6.25, > > > so I'd bet on the previous culprits. > > > > > > This time, the taxonomy looks like NOT_READY isn't being waited for > > > properly. I'd still tend to blame > > > 210ba1d1724f5c4ed87a2ab1a21ca861a915f734 it's just that this time I > > > > Reverting that commit doesn't fix the problem. > > That's a bit of a problem. Particularly as: > > > > suspect this to be the problem line: > > > > > > if (sshdr.sense_key == NOT_READY && sshdr.asc == 0x04) > > > return CDS_DISC_OK; > > > > > > Previously the code would have returned CDS_NO_DISC for the not ready > > > case. The CD would have tried to close the door but (and this is the > > > key) it would have waited for the door to close before trying again. > > > > I tried the patch below. > > > > Now the kernel spits out messages every 2 seconds: > > > > | G: CDS_NO_DISC > > | G: CDS_NO_DISC > > > > Insert CD-DA media > > > > Reading from CD using cdparanioa fails > > > > | F: CDS_DISC_OK > > | F: CDS_DISC_OK > > | F: CDS_DISC_OK > > | F: CDS_DISC_OK > > | F: CDS_DISC_OK > > That's in the two lines I fingered as the source of the problem ... > however, if reversing the apparently bad commit (that introduces those > lines) doesn't work there's something else wrong. > > Could you try altering your F: case to return CDS_DRIVE_NOT_READY, but > if that doesn't work, it's going to take a bit of bisection, I'm > afraid ... Doesn't make any difference. git-bisect taught me it was introduced by commit 38582a62ecd337de4212004c7d4844899dc57890 Author: James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> Date: Wed Feb 6 13:01:58 2008 -0600 [SCSI] sr: fix test unit ready responses Commit 210ba1d1724f5c4ed87a2ab1a21ca861a915f734 updated sr.c to use the scsi_test_unit_ready() function. Unfortunately, this has the wrong characteristic of eating NOT_READY returns which sr.c relies on for tray status. Fix by rolling an internal sr_test_unit_ready() that doesn't do this. With kind regards, Geert Uytterhoeven Software Architect Sony Techsoft Centre The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone: +32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: Geert.Uytterhoeven@xxxxxxxxxxx Internet: http://www.sony-europe.com/ Sony Technology and Software Centre Europe A division of Sony Service Centre (Europe) N.V. Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium VAT BE 0413.825.160 · RPR Brussels Fortis 293-0376800-10 GEBA-BE-BB