Bug 89716

Summary: Optical Scsi disk drives upset disk numbering
Product: [Retired] Red Hat Linux Reporter: c.m.western
Component: kudzuAssignee: Bill Nottingham <notting>
Status: CLOSED RAWHIDE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: 9CC: rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2003-04-28 15:46:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description c.m.western 2003-04-26 18:05:30 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20030225

Description of problem:
The disk numbering generated by updfstab is wrong in the presence of optical
disks, as it seems these are counted in the same way as hard disks by the kernel
but ignored by kudzu

Version-Release number of selected component (if applicable):
kudzu-0.99.69-1

How reproducible:
Always

Steps to Reproduce:
1.Install optical scsi disk drive
2.Install removable USB hard-disk
3.updfstab
    

Additional info:

The device that causes the problem is:
Host: scsi0 Channel: 00 Id: 02 Lun: 00
  Vendor: MATSHITA Model: PD-1 LF-1000     Rev: A105
  Type:   Optical Device                   ANSI SCSI revision: 02
Host: scsi0 Channel: 00 Id: 02 Lun: 01
  Vendor: MATSHITA Model: PD-1 LF-1000     Rev: A105
  Type:   CD-ROM                           ANSI SCSI revision: 02

The following patch works for me:

--- ./scsi.c.orig       2003-02-13 21:48:53.000000000 +0000
+++ ./scsi.c    2003-04-26 18:15:44.000000000 +0100
@@ -493,7 +493,7 @@
                }

                newdev = scsiNewDevice(NULL);
-               if (strstr(start, "Direct-Access")) {
+               if (strstr(start, "Direct-Access") || strstr(start,"Optical
Device")) {
                    newdev->class = CLASS_HD;
                } else if (strstr(start, "Sequential-Access")) {
                    newdev->class = CLASS_TAPE;

Comment 1 Bill Nottingham 2003-04-28 15:46:15 UTC
Added, will be in 1.1.4-1.