From mboxrd@z Thu Jan  1 00:00:00 1970
From: Harald Hoyer <harald@redhat.com>
Date: Wed, 30 Jun 2010 10:09:50 +0000
Subject: Re: problem with commit "cdrom_id: Do not ignore errors from scsi_cmd_run()"
Message-Id: <4C2B17EE.20001@redhat.com>
List-Id: <linux-hotplug.vger.kernel.org>
References: <4C293677.3030308@yahoo.com.ar>
In-Reply-To: <4C293677.3030308@yahoo.com.ar>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: linux-hotplug@vger.kernel.org

On 06/29/2010 12:38 PM, Harald Hoyer wrote:
> On 06/29/2010 01:55 AM, Gerardo Exequiel Pozzi wrote:
>> Hello
>>
>> I have a problem with this commit [#1] "cdrom_id: Do not ignore errors
>> from scsi_cmd_run()" and qemu-kvm-0.12.4.
>>
>> running /lib/udev/cdrom_id /dev/sr0 does not return
>> ID_CDROM_MEDIA_TRACK_COUNT_DATA (there is a iso image specified on
>> qemu-kvm command line, so there is a media present)
>> Running with --debug show that error is produced (add below)
>>
>> For working again I done:
>> 1) Only reverting this commit [#1] "cdrom_id: Do not ignore errors from
>> scsi_cmd_run()"
>> 2) Only revert revert the commit [#2] "cdrom_id: Swap media state and
>> TOC info probing"
>>
>> In case (1) no error is displayed and ID_CDROM_MEDIA_TRACK_COUNT_DATA is
>> printed.
>> In case (2) error is displayed and ID_CDROM_MEDIA_TRACK_COUNT_DATA is
>> printed.
>>
>> As result udev rule that uses this info will fail.
>>
>> Tested on Linux 2.6.33.5 and 2.6.34, under 32 bits and 64 bits with the
>> same results.
>>
>> If need more info, please let me know.
>>
>> Thanks.
>>
>> # /lib/udev/cdrom_id --debug /dev/sr0
>> main: probing: '/dev/sr0'
>> cd_inquiry: INQUIRY: [QEMU ][QEMU DVD-ROM ][0.12]
>> cd_profiles: GET CONFIGURATION: size of features buffer 0x0010
>> cd_profiles: GET CONFIGURATION: feature 'profiles', with 2 entries
>> feature_profiles: profile 0x10 dvd_rom
>> feature_profiles: profile 0x08 cd_rom
>> cd_profiles: current profile 0x08
>> cd_profiles: profile 0x08 media_cd_rom
>> info_scsi_cmd_err: READ DISC INFORMATION failed with
>> SK=3D5h/ASC h/ACQ=00h
>> ID_CDROM=3D1
>> ID_CDROM_CD=3D1
>> ID_CDROM_DVD=3D1
>> ID_CDROM_MRW=3D1
>> ID_CDROM_MRW_W=3D1
>> ID_CDROM_MEDIA=3D1
>> ID_CDROM_MEDIA_CD=3D1
>>
>>
>> [#1]
>> http://git.kernel.org/?p=3Dlinux/hotplug/udev.git;a=3Dcommitdiff;hXe1788=
94bfc040834e1270c6fe9b9fdef513550#patch1
>>
>>
>> [#2]
>> http://git.kernel.org/?p=3Dlinux/hotplug/udev.git;a=3Dcommitdiff;h0e3b1a=
0d3a3ec76f16736470dc656744848d941
>>
>>
>>
>
> Apparently qemu does not implement the scsi "READ DISC INFORMATION"
> command for cdroms.
>
> Filed on the Red Hat bugzilla
> https://bugzilla.redhat.com/show_bug.cgi?id`9049


Comment of Gerardo Exequiel Pozzi

Additional note: This only fail with IDE cd-rom, SCSI cd-rom works OK

With "-drive file=3Dxyz.iso,if=3Dscsi,media=CDrom" works fine, but with "-d=
rive
file=3Dxyz.iso,if=3Dide,media=CDrom" (or directly -cdrom) fails.