scsi_mmc.c: more aggressive about getting drive capabilites. Try to

get length first. Also try CAPABILITIES_PAGEs as well as ALL_PAGEs.

lib/_cdio_linux.c

@@ -1,5 +1,5 @@
 /*
-    $Id: _cdio_linux.c,v 1.94 2004/08/26 10:44:38 rocky Exp $
+    $Id: _cdio_linux.c,v 1.95 2004/08/27 01:24:40 rocky Exp $
 
     Copyright (C) 2001 Herbert Valerio Riedel <hvr@gnu.org>
     Copyright (C) 2002, 2003, 2004 Rocky Bernstein <rocky@panix.com>
@@ -27,7 +27,7 @@
 # include "config.h"
 #endif
 
-static const char _rcsid[] = "$Id: _cdio_linux.c,v 1.94 2004/08/26 10:44:38 rocky Exp $";
+static const char _rcsid[] = "$Id: _cdio_linux.c,v 1.95 2004/08/27 01:24:40 rocky Exp $";
 
 #include <string.h>
 
@@ -218,6 +218,8 @@ get_arg_linux (void *env, const char key[])
   return NULL;
 }
 
+#define USE_LINUX_CAP 1
+#ifdef USE_LINUX_CAP
 /*!
   Return the the kind of drive capabilities of device.
 
@@ -283,6 +285,7 @@ get_drive_cap_linux (const void *p_user_data,
   if (i_drivetype & CDC_RESET) 
     *p_misc_cap  |= CDIO_DRIVE_CAP_MISC_RESET;
 }
+#endif
 
 /*!
   Return the media catalog number MCN.
@@ -1107,7 +1110,11 @@ cdio_open_am_linux (const char *psz_orig_source, const char *access_mode)
     .get_default_device = cdio_get_default_device_linux,
     .get_devices        = cdio_get_devices_linux,
     .get_discmode       = get_discmode_linux,
+#if USE_LINUX_CAP
     .get_drive_cap      = get_drive_cap_linux,
+#else
+    .get_drive_cap      = scsi_mmc_get_drive_cap_generic,
+#endif
     .get_first_track_num= get_first_track_num_generic,
     .get_mcn            = get_mcn_linux,
     .get_num_tracks     = get_num_tracks_generic,

lib/scsi_mmc.c

@@ -1,6 +1,6 @@
 /*  Common SCSI Multimedia Command (MMC) routines.
 
-    $Id: scsi_mmc.c,v 1.24 2004/08/07 22:58:51 rocky Exp $
+    $Id: scsi_mmc.c,v 1.25 2004/08/27 01:24:40 rocky Exp $
 
     Copyright (C) 2004 Rocky Bernstein <rocky@panix.com>
 
@@ -235,6 +235,7 @@ scsi_mmc_set_blocksize ( const CdIo *cdio, unsigned int bsize)
 				    bsize);
 }
 
+
 /*!
   Return the the kind of drive capabilities of device.
  */
@@ -245,10 +246,13 @@ scsi_mmc_get_drive_cap_private (const void *p_env,
 				/*out*/ cdio_drive_write_cap_t *p_write_cap,
 				/*out*/ cdio_drive_misc_cap_t  *p_misc_cap)
 {
+  /* Largest buffer size we use. */
+#define BUF_MAX 2048
+  uint8_t buf[BUF_MAX] = { 0, };
+
   scsi_mmc_cdb_t cdb = {{0, }};
   int i_status;
-
+  uint16_t i_data = BUF_MAX;
-  uint8_t buf[192] = { 0, };
   
   if ( ! p_env || ! run_scsi_mmc_cmd )
     return;
@@ -256,19 +260,40 @@ scsi_mmc_get_drive_cap_private (const void *p_env,
   CDIO_MMC_SET_COMMAND(cdb.field, CDIO_MMC_GPCMD_MODE_SENSE_10);
   cdb.field[1] = 0x0;  
   cdb.field[2] = CDIO_MMC_ALL_PAGES; 
-  cdb.field[7] = 0x01;
-  cdb.field[8] = 0x00;
 
+ retry:
+  CDIO_MMC_SET_READ_LENGTH16(cdb.field, 8);
+
+  /* In the first run we run MODE SENSE 10 we are trying to get the
+     length of the data features. */
   i_status = run_scsi_mmc_cmd (p_env, DEFAULT_TIMEOUT_MS,
 			       scsi_mmc_get_cmd_len(cdb.field[0]), 
 			       &cdb, SCSI_MMC_DATA_READ, 
 			       sizeof(buf), &buf);
+  if (0 == i_status) {
+    uint16_t i_data_try = (uint16_t) CDIO_MMC_GET_LEN16(buf);
+    if (i_data_try < BUF_MAX) i_data = i_data_try;
+  }
+
+  /* Now try getting all features with length set above, possibly
+     truncated or the default length if we couldn't get the proper
+     length. */
+  CDIO_MMC_SET_READ_LENGTH16(cdb.field, i_data);
+
+  i_status = run_scsi_mmc_cmd (p_env, DEFAULT_TIMEOUT_MS,
+			       scsi_mmc_get_cmd_len(cdb.field[0]), 
+			       &cdb, SCSI_MMC_DATA_READ, 
+			       sizeof(buf), &buf);
+
+  if (0 != i_status && CDIO_MMC_CAPABILITIES_PAGE != cdb.field[2]) {
+    cdb.field[2] =  CDIO_MMC_CAPABILITIES_PAGE; 
+    goto retry;
+  }
+
   if (0 == i_status) {
     uint8_t *p;
-    uint16_t i_data;
     uint8_t *p_max = buf + 256;
     
-    i_data = (unsigned int) CDIO_MMC_GET_LEN16(buf);
     *p_read_cap  = 0;
     *p_write_cap = 0;
     *p_misc_cap  = 0;