Getting prepared for 5.30 release.

git-svn-id: https://smartmontools.svn.sourceforge.net/svnroot/smartmontools/trunk@1567 4ea69e1a-61f1-4043-bf83-b5c94c648137

sm5/AUTHORS

-$Id: AUTHORS,v 1.10 2004/03/06 03:13:37 pervalidus Exp $
+$Id: AUTHORS,v 1.11 2004/03/06 19:43:16 ballen4705 Exp $
 
 This code was originally developed as a Senior Thesis by Michael
 Cornwell at the Concurrent Systems Laboratory (now part of the Storage
@@ -19,8 +19,10 @@ Christian Franke	<franke@computer.org>
 Guilhem Frzou		<guilhem.frezou@catii.fr>
 Douglas Gilbert		<dougg@torque.net>
 Guido Guenther		<agx@sigxcpu.org>
+Dr. David Kirkby	<drkirkby@ntlworld.com>
 Kai Mkisara		<kai.makisara@kolumbus.fi>
 Eduard Martinescu	<martines@rochester.rr.com>
 Frdric L. W. Meunier	<http://www.pervalidus.net/contact.html>
+Keiji Sawada		<card_captor@users.sourceforge.net>
+Sergey Svishchev	<svs@ropnet.ru>
 Phil Williams		<phil@subbacultcha.demon.co.uk>
-Sawada Keiji		<card_captor@users.sourceforge.net>

sm5/CHANGELOG

 CHANGELOG for smartmontools
 
-$Id: CHANGELOG,v 1.360 2004/03/05 23:38:46 pjwilliams Exp $
+$Id: CHANGELOG,v 1.361 2004/03/06 19:43:17 ballen4705 Exp $
 
 The most recent version of this file is:
 http://cvs.sourceforge.net/viewcvs.py/smartmontools/sm5/CHANGELOG?sortby=date&view=markup
@@ -20,13 +20,15 @@ Maintainers / Developers Key:
 [EM] Eduard Martinescu
 [FM] Frédéric L. W. Meunier
 [KS] Keiji Sawada
-[PW] Phil Williams
 [SS] Sergey Svishchev
+[PW] Phil Williams
 
 NOTES FOR FUTURE RELEASES: see TODO file.
 
 <ADDITIONS TO THE CHANGE LOG SHOULD BE ADDED JUST BELOW HERE, PLEASE>
 
+smartmontools 5.30
+
   [PW] Added QUANTUM FIREBALLlct15 30, QUANTUM FIREBALLlct20 40, and
        Maxtor 6Y060P0 (DiamondMax Plus 9 60GB) to knowndrives table.
 

sm5/NEWS

 smartmontools NEWS
 ------------------
-CVS ID: $Id: NEWS,v 1.18 2004/02/24 09:57:27 ballen4705 Exp $
+CVS ID: $Id: NEWS,v 1.19 2004/03/06 19:43:17 ballen4705 Exp $
 
 The most up-to-date version of this file is:
 http://cvs.sourceforge.net/viewcvs.py/smartmontools/sm5/NEWS?sortby=date&view=markup
 
+Date: 2003-3-6
+Summary: smartmontools release 5.30 (STABLE)
+--------------------------------------------
+This is a stable release of smartmontools: the first stable release
+since 5.26.
+
+
 Date: 2003-2-24
 Summary: smartmontools release 5.29 (Experimental, not STABLE)
 --------------------------------------------------------------

sm5/knowndrives.c

@@ -27,7 +27,7 @@
 #include "utility.h"
 #include "config.h"
 
-const char *knowndrives_c_cvsid="$Id: knowndrives.c,v 1.95 2004/03/05 23:38:46 pjwilliams Exp $"
+const char *knowndrives_c_cvsid="$Id: knowndrives.c,v 1.96 2004/03/06 19:43:17 ballen4705 Exp $"
                                 ATACMDS_H_CVSID ATAPRINT_H_CVSID CONFIG_H_CVSID EXTERN_H_CVSID KNOWNDRIVES_H_CVSID UTILITY_H_CVSID;
 
 #define MODEL_STRING_LENGTH                         40
@@ -119,10 +119,12 @@ const char may_need_minus_F2_disabled[]="May need -F samsung2 disabled; see manu
 /* Special-purpose functions for use in knowndrives[]. */
 void specialpurpose_reverse_samsung(smartmonctrl *con)
 {
+  if (con->fixfirmwarebug==FIX_NOTSPECIFIED)
     con->fixfirmwarebug = FIX_SAMSUNG;
 }
 void specialpurpose_reverse_samsung2(smartmonctrl *con)
 {
+  if (con->fixfirmwarebug==FIX_NOTSPECIFIED)
     con->fixfirmwarebug = FIX_SAMSUNG2;
 }
 
@@ -820,10 +822,10 @@ int applypresets(const struct ata_identify_device *drive, unsigned char **optspt
     }
     
     // If a special-purpose function is defined for this drive then
-    // call it.  We might call a special purpose function even if now
-    // Attribute interpretation presets are set.  We ONLY call this
-    // function if the user has NOT already specified a '-F' option.
-    if (knowndrives[i].specialpurpose && con->fixfirmwarebug==FIX_NOTSPECIFIED)
+    // call it. Note that if command line arguments or Directives
+    // over-ride this choice, then the specialpurpose function that is
+    // called must deal with this.
+    if (knowndrives[i].specialpurpose)
       (*knowndrives[i].specialpurpose)(con);
   }
   

sm5/knowndrives.cpp

@@ -27,7 +27,7 @@
 #include "utility.h"
 #include "config.h"
 
-const char *knowndrives_c_cvsid="$Id: knowndrives.cpp,v 1.95 2004/03/05 23:38:46 pjwilliams Exp $"
+const char *knowndrives_c_cvsid="$Id: knowndrives.cpp,v 1.96 2004/03/06 19:43:17 ballen4705 Exp $"
                                 ATACMDS_H_CVSID ATAPRINT_H_CVSID CONFIG_H_CVSID EXTERN_H_CVSID KNOWNDRIVES_H_CVSID UTILITY_H_CVSID;
 
 #define MODEL_STRING_LENGTH                         40
@@ -119,10 +119,12 @@ const char may_need_minus_F2_disabled[]="May need -F samsung2 disabled; see manu
 /* Special-purpose functions for use in knowndrives[]. */
 void specialpurpose_reverse_samsung(smartmonctrl *con)
 {
+  if (con->fixfirmwarebug==FIX_NOTSPECIFIED)
     con->fixfirmwarebug = FIX_SAMSUNG;
 }
 void specialpurpose_reverse_samsung2(smartmonctrl *con)
 {
+  if (con->fixfirmwarebug==FIX_NOTSPECIFIED)
     con->fixfirmwarebug = FIX_SAMSUNG2;
 }
 
@@ -820,10 +822,10 @@ int applypresets(const struct ata_identify_device *drive, unsigned char **optspt
     }
     
     // If a special-purpose function is defined for this drive then
-    // call it.  We might call a special purpose function even if now
-    // Attribute interpretation presets are set.  We ONLY call this
-    // function if the user has NOT already specified a '-F' option.
-    if (knowndrives[i].specialpurpose && con->fixfirmwarebug==FIX_NOTSPECIFIED)
+    // call it. Note that if command line arguments or Directives
+    // over-ride this choice, then the specialpurpose function that is
+    // called must deal with this.
+    if (knowndrives[i].specialpurpose)
       (*knowndrives[i].specialpurpose)(con);
   }
   

sm5/smartctl.8.in

 .ig
  Copyright (C) 2002-4 Bruce Allen <smartmontools-support@lists.sourceforge.net>
 
- $Id: smartctl.8.in,v 1.34 2004/03/05 23:06:42 ballen4705 Exp $
+ $Id: smartctl.8.in,v 1.35 2004/03/06 19:43:18 ballen4705 Exp $
  
  This program is free software; you can redistribute it and/or modify it
  under the terms of the GNU General Public License as published by the Free
@@ -734,8 +734,9 @@ value for Attribute 123 in this form.
 .TP
 .B \-F TYPE, \-\-firmwarebug=TYPE
 Modifies the behavior of \fBsmartctl\fP to compensate for some known
-and understood device firmware bug.  The valid arguments to this
-option are:
+and understood device firmware bug.  The arguments to this option are
+exclusive, so that only the final option given is used.  The valid
+values are:
 
 .I none
 \- Assume that the device firmware obeys the ATA specifications.  This
@@ -1075,4 +1076,4 @@ these documents may be found in the References section of the
 
 .SH
 CVS ID OF THIS PAGE:
-$Id: smartctl.8.in,v 1.34 2004/03/05 23:06:42 ballen4705 Exp $
+$Id: smartctl.8.in,v 1.35 2004/03/06 19:43:18 ballen4705 Exp $

sm5/smartd.8.in

 .ig
 Copyright (C) 2002-4 Bruce Allen <smartmontools-support@lists.sourceforge.net>
  
-$Id: smartd.8.in,v 1.45 2004/03/05 18:00:00 ballen4705 Exp $
+$Id: smartd.8.in,v 1.46 2004/03/06 19:43:18 ballen4705 Exp $
 
 This program is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
@@ -1058,11 +1058,11 @@ different types of system behavior affects the values of certain
 Attributes.
 
 .TP
-.B \-F TYPE, \-\-firmwarebug=TYPE
-Modifies the behavior of
-.B smartctl
-to compensate for some known and understood device firmware bug.  The
-valid arguments to this option are:
+.B \-F TYPE
+Modifies the behavior of \fBsmartd\fP to compensate for some known and
+understood device firmware bug.  The arguments to this Directive are
+exclusive, so that only the final Directive given is used.  The valid
+values are:
 
 .I none
 \- Assume that the device firmware obeys the ATA specifications.  This is
@@ -1073,7 +1073,7 @@ database.
 \- In some Samsung disks (example: model SV4012H Firmware Version:
 RM100-08) some of the two- and four-byte quantities in the SMART data
 structures are byte-swapped (relative to the ATA specification).
-Enabling this option tells \fBsmartctl\fP to evaluate these quantities
+Enabling this option tells \fBsmartd\fP to evaluate these quantities
 in byte-reversed order.  Some signs that your disk needs this option
 are (1) no self-test log printed, even though you have run self-tests;
 (2) very large numbers of ATA errors reported in the ATA error log;
@@ -1082,7 +1082,7 @@ are (1) no self-test log printed, even though you have run self-tests;
 .I samsung2
 \- In more recent Samsung disks (firmware revisions ending in "\-23") the
 number of ATA errors reported is byte swapped.  Enabling this option
-tells \fBsmartctl\fP to evaluate this quantity in byte-reversed order.
+tells \fBsmartd\fP to evaluate this quantity in byte-reversed order.
 
 Note that an explicit \'\-F\' Directive will over-ride any preset
 values for \'\-F\' (see the \'\-P\' option below).
@@ -1524,4 +1524,4 @@ smartmontools home page at \fBhttp://smartmontools.sourceforge.net/\fP .
 
 .SH
 CVS ID OF THIS PAGE:
-$Id: smartd.8.in,v 1.45 2004/03/05 18:00:00 ballen4705 Exp $
+$Id: smartd.8.in,v 1.46 2004/03/06 19:43:18 ballen4705 Exp $

sm5/smartd.conf.5.in

 .ig
 Copyright (C) 2002-4 Bruce Allen <smartmontools-support@lists.sourceforge.net>
 
-$Id: smartd.conf.5.in,v 1.35 2004/03/05 18:00:00 ballen4705 Exp $
+$Id: smartd.conf.5.in,v 1.36 2004/03/06 19:43:19 ballen4705 Exp $
 
 This program is free software; you can redistribute it and/or modify it
 under the terms of the GNU General Public License as published by the Free
@@ -779,11 +779,11 @@ different types of system behavior affects the values of certain
 Attributes.
 
 .TP
-.B \-F TYPE, \-\-firmwarebug=TYPE
-Modifies the behavior of
-.B smartctl
-to compensate for some known and understood device firmware bug.  The
-valid arguments to this option are:
+.B \-F TYPE
+Modifies the behavior of \fBsmartd\fP to compensate for some known and
+understood device firmware bug.  The arguments to this Directive are
+exclusive, so that only the final Directive given is used.  The valid
+values are:
 
 .I none
 \- Assume that the device firmware obeys the ATA specifications.  This is
@@ -794,7 +794,7 @@ database.
 \- In some Samsung disks (example: model SV4012H Firmware Version:
 RM100-08) some of the two- and four-byte quantities in the SMART data
 structures are byte-swapped (relative to the ATA specification).
-Enabling this option tells \fBsmartctl\fP to evaluate these quantities
+Enabling this option tells \fBsmartd\fP to evaluate these quantities
 in byte-reversed order.  Some signs that your disk needs this option
 are (1) no self-test log printed, even though you have run self-tests;
 (2) very large numbers of ATA errors reported in the ATA error log;
@@ -803,7 +803,7 @@ are (1) no self-test log printed, even though you have run self-tests;
 .I samsung2
 \- In more recent Samsung disks (firmware revisions ending in "\-23") the
 number of ATA errors reported is byte swapped.  Enabling this option
-tells \fBsmartctl\fP to evaluate this quantity in byte-reversed order.
+tells \fBsmartd\fP to evaluate this quantity in byte-reversed order.
 
 Note that an explicit \'\-F\' Directive will over-ride any preset
 values for \'\-F\' (see the \'\-P\' option below).
@@ -1099,4 +1099,4 @@ SEE ALSO:
 
 .SH
 CVS ID OF THIS PAGE:
-$Id: smartd.conf.5.in,v 1.35 2004/03/05 18:00:00 ballen4705 Exp $
+$Id: smartd.conf.5.in,v 1.36 2004/03/06 19:43:19 ballen4705 Exp $

sm5/smartmontools.spec

@@ -35,7 +35,7 @@ Packager:       Bruce Allen <smartmontools-support@lists.sourceforge.net>
 # http://ftp1.sourceforge.net/smartmontools/smartmontools-%{version}-%{release}.tar.gz
 
 # CVS ID of this file is:
-# $Id: smartmontools.spec,v 1.152 2004/03/05 14:09:53 ballen4705 Exp $
+# $Id: smartmontools.spec,v 1.153 2004/03/06 19:43:22 ballen4705 Exp $
 
 # Copyright (C) 2002-4 Bruce Allen <smartmontools-support@lists.sourceforge.net>
 # Home page: http://smartmontools.sourceforge.net/
@@ -311,14 +311,54 @@ fi
 # [SB] Stanislav Brabec
 # [PC] Peter Cassidy
 # [CD] Capser Dik
-# [DK] David Kirkby
+# [CF] Christian Franke
+# [GF] Guilhem Frzou
 # [DG] Douglas Gilbert
 # [GG] Guido Guenther
+# [DK] David Kirkby
 # [KM] Kai Mkisarai
+# [EM] Eduard Martinescu
 # [FM] Frdric L. W. Meunier
+# [KS] Keiji Sawada
+# [SS] Sergey Svishchev
 # [PW] Phil Williams
 
 %changelog
+
+* Sat Mar 6 2004 Bruce Allen  <smartmontools-support@lists.sourceforge.net>
+  [PW] Added QUANTUM FIREBALLlct15 30, QUANTUM FIREBALLlct20 40, and
+       Maxtor 6Y060P0 (DiamondMax Plus 9 60GB) to knowndrives table.
+  [PW] Added Maxtor MaXLine II family to knowndrives table (thanks to
+       Brett Russ for submitting the patch).
+  [BA] Added remaining read/write commands to detailed list of
+       error log commands that have text descriptions of problem
+       printed.  For commands that support it, print number of failed
+       sectors at problem LBA.
+  [BA] Made SuSE section of smartd init script more SuSE 9 compatible.
+       Thanks to Hans-Peter Jansen.
+  [CF] Windows smartd: Added IDE/ATA device scan
+       Added windows device names to smartctl.8.in, smartd.8.in
+  [BA] smartctl/smartd: user-provided '-F samsung' and '-F samsung2'
+       command line options/Directives did NOT over-ride preset values
+       unless user specified '-P ignore'.  Now they will always over-ride
+       preset values from the database.
+  [BA] Added error decoding for a few more READ and WRITE commands.
+  [PW] Added Maxtor MaXLine Plus II, Western Digital Caviar SE (Serial ATA)
+       series, Hitachi Deskstar 7K250 series, and Ultra ATA 66 models of
+       the Maxtor DiamondMax Plus 40 series to knowndrives table.
+  [BA] Added Maxtor Diamondmax 250 GB drives to database.  Note that
+       these model numbers are not listed in Maxtor documentation, but
+       they exist.
+  [BA] Removed the 'contact developers' phrase from the Samsung disk
+       warning messages.
+  [PW] Added TOSHIBA MK2017GAP, IBM Deskstar 14GXP and 16GP series,
+       Fujitsu MPC series, Seagate Barracuda ATA III family, and missing
+       Seagate Barracuda U Series drives to knowndrives table
+  [BA] smartd: wrong loglevel for message: Configuration file
+       /etc/smartd.conf parsed.  Changed to LOG_INFO from LOG_CRIT.
+       Thanks to  Emmanuel CHANTREAU for the report.
+  [CF] Checked in development version of windows code base.
+
 * Tue Feb 24 2004 Bruce Allen  <smartmontools-support@lists.sourceforge.net>
   [BA] smartd: configure script did not set correct directory to search for
        smartd.conf based on --prefix argument to ./configure.  Thanks to