Re: [Cbe-oss-dev] [regression/bisected] corrupt CD data after media change and delay

[James Bottomley <James.Bottomley@HansenPartnership.com>]

On Tue, 2008-06-10 at 17:11 +0200, Geert Uytterhoeven wrote:
> 	Hi James,
> 
> On Mon, 9 Jun 2008, James Bottomley wrote:
> > My best guess is that it's something to do with eaten UNIT_ATTENTION
> > keys.  I theorise that in the successful case, they get processed in
> > scsi_io_completion which sets the changed bit.  In the delay case, the
> > unit attention is eaten by sr_test_unit_ready so the changed bit is
> > always kept at zero.  If this is the case, then the patch below should
> > fix it.  If not, we're back to more debugging ...
> 
> Yep, that fixed it. Thanks!

OK, will add to rc-fixes and ask for stable backport; thanks.

James


> Inserting first CD, mounting:
> 
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | Sense Key : 0x2 [current] 
> | +5+ CDS_DISC_OK
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | Sense Key : 0x2 [current] 
> | +5+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +11+ Return forcing update is 1
>                                 ^
> 				OK
> 
> | ISO 9660 Extensions: RRIP_1991A
> 
> Unmounting first CD:
> 
> | +0+ the_result = 0x0 (sdev->changed = 0)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 0)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 0)
> | Sense Key : 0x0 [current] [descriptor]
> | +11+ Return forcing update is 0
> 
> Inserting second CD, mounting after 30 seconds:
> 
> | +0+ the_result = 0x8000002 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +1+ CDS_DISC_OK
> | +0+ the_result = 0x0 (sdev->changed = 1)
> | Sense Key : 0x0 [current] [descriptor]
> | +11+ Return forcing update is 1
>                                 ^
> 				OK!
> 
> | ISO 9660 Extensions: Microsoft Joliet Level 3
> | ISO 9660 Extensions: RRIP_1991A
> 
> => success!
> 
> 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@sonycom.com
> 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

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