]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/label.c
Massive SD calling sequence reorganization
[bacula/bacula] / bacula / src / stored / label.c
index db0ce2351854def1695e320f204689eaa4d329a5..aca6f901e65bcbed59dd3f3534895e428b055047 100644 (file)
@@ -8,7 +8,7 @@
  *   Version $Id$
  */
 /*
-   Copyright (C) 2000-2003 Kern Sibbald and John Walker
+   Copyright (C) 2000-2004 Kern Sibbald and John Walker
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License as
@@ -31,7 +31,7 @@
 #include "stored.h"                   /* pull in Storage Deamon headers */
 
 /* Forward referenced functions */
-static void create_volume_label_record(JCR *jcr, DEVICE *dev, DEV_RECORD *rec);
+static void create_volume_label_record(DCR *dcr, DEV_RECORD *rec);
 
 extern char my_name[];
 extern int debug_level;
@@ -39,9 +39,9 @@ extern int debug_level;
 /*
  * Read the volume label
  *
- *  If jcr->VolumeName == NULL, we accept any Bacula Volume
- *  If jcr->VolumeName[0] == 0, we accept any Bacula Volume
- *  otherwise jcr->VolumeName must match the Volume.
+ *  If dcr->VolumeName == NULL, we accept any Bacula Volume
+ *  If dcr->VolumeName[0] == 0, we accept any Bacula Volume
+ *  otherwise dcr->VolumeName must match the Volume.
  *
  *  If VolName given, ensure that it matches   
  *
@@ -56,11 +56,14 @@ extern int debug_level;
  *    VOL_LABEL_ERROR
  *    VOL_NO_MEDIA
  */  
-int read_dev_volume_label(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
+int read_dev_volume_label(DCR *dcr)
 {
-   char *VolName = jcr->VolumeName;
+   JCR *jcr = dcr->jcr;
+   DEVICE *dev = dcr->dev;
+   char *VolName = dcr->VolumeName;
    DEV_RECORD *record;
-   int ok = 0;
+   bool ok = false;
+   DEV_BLOCK *block = dcr->block;
 
    Dmsg3(100, "Enter read_volume_label device=%s vol=%s dev_Vol=%s\n", 
       dev_name(dev), VolName, dev->VolHdr.VolName);
@@ -68,57 +71,64 @@ int read_dev_volume_label(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
    if (dev_state(dev, ST_LABEL)) {      /* did we already read label? */
       /* Compare Volume Names allow special wild card */
       if (VolName && *VolName && *VolName != '*' && strcmp(dev->VolHdr.VolName, VolName) != 0) {
-         Mmsg(&jcr->errmsg, _("Wrong Volume mounted on device %s: Wanted %s have %s\n"),
+         Mmsg(jcr->errmsg, _("Wrong Volume mounted on device %s: Wanted %s have %s\n"),
            dev_name(dev), VolName, dev->VolHdr.VolName);
         /*
          * Cancel Job if too many label errors
          *  => we are in a loop
          */
-        if (jcr->label_errors++ > 100) {
-            Jmsg(jcr, M_FATAL, 0, "%s", jcr->errmsg);
+        if (!dev->poll && jcr->label_errors++ > 100) {
+            Jmsg(jcr, M_FATAL, 0, "Too many tries: %s", jcr->errmsg);
         }
-        return jcr->label_status = VOL_NAME_ERROR;
+        return VOL_NAME_ERROR;
       }
       Dmsg0(30, "Leave read_volume_label() VOL_OK\n");
-      return jcr->label_status = VOL_OK;       /* label already read */
+      return VOL_OK;      /* label already read */
    }
 
    dev->state &= ~(ST_LABEL|ST_APPEND|ST_READ);  /* set no label, no append */
 
    if (!rewind_dev(dev)) {
-      Mmsg(&jcr->errmsg, _("Couldn't rewind device %s ERR=%s\n"), dev_name(dev),
+      Mmsg(jcr->errmsg, _("Couldn't rewind device %s ERR=%s\n"), dev_name(dev),
         strerror_dev(dev));
-      return jcr->label_status = VOL_NO_MEDIA;
+      Dmsg1(30, "%s", jcr->errmsg);
+      return VOL_NO_MEDIA;
    }
    bstrncpy(dev->VolHdr.Id, "**error**", sizeof(dev->VolHdr.Id));
 
    /* Read the Volume label block */
    record = new_record();
+   empty_block(block);
    Dmsg0(90, "Big if statement in read_volume_label\n");
-   if (!read_block_from_dev(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) { 
-      Mmsg(&jcr->errmsg, _("Volume on %s is not a Bacula labeled Volume, \
-because:\n   %s"), dev_name(dev), strerror_dev(dev));
+   if (!read_block_from_dev(dcr, NO_BLOCK_NUMBER_CHECK)) { 
+      Mmsg(jcr->errmsg, _("Requested Volume \"%s\" on %s is not a Bacula "
+           "labeled Volume, because: ERR=%s"), NPRT(VolName), dev_name(dev), 
+          strerror_dev(dev));
+      Dmsg1(30, "%s", jcr->errmsg);
    } else if (!read_record_from_block(block, record)) {
-      Mmsg(&jcr->errmsg, _("Could not read Volume label from block.\n"));
+      Mmsg(jcr->errmsg, _("Could not read Volume label from block.\n"));
+      Dmsg1(30, "%s", jcr->errmsg);
    } else if (!unser_volume_label(dev, record)) {
-      Mmsg(&jcr->errmsg, _("Could not unserialize Volume label: %s\n"),
+      Mmsg(jcr->errmsg, _("Could not unserialize Volume label: ERR=%s\n"),
         strerror_dev(dev));
+      Dmsg1(30, "%s", jcr->errmsg);
    } else if (strcmp(dev->VolHdr.Id, BaculaId) != 0 && 
              strcmp(dev->VolHdr.Id, OldBaculaId) != 0) {
-      Mmsg(&jcr->errmsg, _("Volume Header Id bad: %s\n"), dev->VolHdr.Id);
+      Mmsg(jcr->errmsg, _("Volume Header Id bad: %s\n"), dev->VolHdr.Id);
+      Dmsg1(30, "%s", jcr->errmsg);
    } else {
-      ok = 1;
+      ok = true;
    }
    if (!ok) {
       free_record(record);
       if (forge_on || jcr->ignore_label_errors) {
         dev->state |= ST_LABEL;      /* set has Bacula label */
          Jmsg(jcr, M_ERROR, 0, "%s", jcr->errmsg);
-        return jcr->label_status = VOL_OK;
+        return VOL_OK;
       }
       empty_block(block);
       rewind_dev(dev);
-      return jcr->label_status = VOL_NO_LABEL;
+      return VOL_NO_LABEL;
    }
 
    free_record(record);
@@ -131,18 +141,20 @@ because:\n   %s"), dev_name(dev), strerror_dev(dev));
    if (dev->VolHdr.VerNum != BaculaTapeVersion && 
        dev->VolHdr.VerNum != OldCompatibleBaculaTapeVersion1 &&  
        dev->VolHdr.VerNum != OldCompatibleBaculaTapeVersion2) {
-      Mmsg(&jcr->errmsg, _("Volume on %s has wrong Bacula version. Wanted %d got %d\n"),
+      Mmsg(jcr->errmsg, _("Volume on %s has wrong Bacula version. Wanted %d got %d\n"),
         dev_name(dev), BaculaTapeVersion, dev->VolHdr.VerNum);
-      return jcr->label_status = VOL_VERSION_ERROR;
+      Dmsg1(30, "%s", jcr->errmsg);
+      return VOL_VERSION_ERROR;
    }
 
    /* We are looking for either an unused Bacula tape (PRE_LABEL) or
     * a Bacula volume label (VOL_LABEL)
     */
    if (dev->VolHdr.LabelType != PRE_LABEL && dev->VolHdr.LabelType != VOL_LABEL) {
-      Mmsg(&jcr->errmsg, _("Volume on %s has bad Bacula label type: %x\n"), 
+      Mmsg(jcr->errmsg, _("Volume on %s has bad Bacula label type: %x\n"), 
          dev_name(dev), dev->VolHdr.LabelType);
-      return jcr->label_status = VOL_LABEL_ERROR;
+      Dmsg1(30, "%s", jcr->errmsg);
+      return VOL_LABEL_ERROR;
    }
 
    dev->state |= ST_LABEL;           /* set has Bacula label */
@@ -150,16 +162,17 @@ because:\n   %s"), dev_name(dev), strerror_dev(dev));
    /* Compare Volume Names */
    Dmsg2(30, "Compare Vol names: VolName=%s hdr=%s\n", VolName?VolName:"*", dev->VolHdr.VolName);
    if (VolName && *VolName && *VolName != '*' && strcmp(dev->VolHdr.VolName, VolName) != 0) {
-      Mmsg(&jcr->errmsg, _("Wrong Volume mounted on device %s: Wanted %s have %s\n"),
+      Mmsg(jcr->errmsg, _("Wrong Volume mounted on device %s: Wanted %s have %s\n"),
           dev_name(dev), VolName, dev->VolHdr.VolName);
+      Dmsg1(30, "%s", jcr->errmsg);
       /*
        * Cancel Job if too many label errors
        *  => we are in a loop
        */
-      if (jcr->label_errors++ > 100) {
-         Jmsg(jcr, M_FATAL, 0, "%s", jcr->errmsg);
+      if (!dev->poll && jcr->label_errors++ > 100) {
+         Jmsg(jcr, M_FATAL, 0, "Too many tries: %s", jcr->errmsg);
       }
-      return jcr->label_status = VOL_NAME_ERROR;
+      return VOL_NAME_ERROR;
    }
    Dmsg1(30, "Copy vol_name=%s\n", dev->VolHdr.VolName);
 
@@ -167,7 +180,7 @@ because:\n   %s"), dev_name(dev), strerror_dev(dev));
       dump_volume_label(dev);
    }
    Dmsg0(30, "Leave read_volume_label() VOL_OK\n");
-   return jcr->label_status = VOL_OK;
+   return VOL_OK;
 }
 
 /*  unser_volume_label 
@@ -177,21 +190,21 @@ because:\n   %s"), dev_name(dev), strerror_dev(dev));
  *
  * Assumes that the record is already read.
  *
- * Returns: 0 on error
- *         1 on success
+ * Returns: false on error
+ *         true  on success
 */
 
-int unser_volume_label(DEVICE *dev, DEV_RECORD *rec)
+bool unser_volume_label(DEVICE *dev, DEV_RECORD *rec)
 {
    ser_declare;
 
    if (rec->FileIndex != VOL_LABEL && rec->FileIndex != PRE_LABEL) {
-      Mmsg3(&dev->errmsg, _("Expecting Volume Label, got FI=%s Stream=%s len=%d\n"), 
+      Mmsg3(dev->errmsg, _("Expecting Volume Label, got FI=%s Stream=%s len=%d\n"), 
              FI_to_ascii(rec->FileIndex), 
              stream_to_ascii(rec->Stream, rec->FileIndex),
              rec->data_len);
       if (!forge_on) {
-        return 0;
+        return false;
       }
    }
 
@@ -231,37 +244,40 @@ int unser_volume_label(DEVICE *dev, DEV_RECORD *rec)
    if (debug_level >= 90) {
       dump_volume_label(dev);     
    }
-   return 1;
+   return true;
 }
 
 /*
  * Put a volume label into the block
  *
- *  Returns: 0 on failure
- *          1 on success
+ *  Returns: false on failure
+ *          true  on success
  */
-int write_volume_label_to_block(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
+bool write_volume_label_to_block(DCR *dcr)
 {
    DEV_RECORD rec;
+   DEVICE *dev = dcr->dev;
+   JCR *jcr = dcr->jcr;
+   DEV_BLOCK *block = dcr->block;
 
    Dmsg0(20, "write Label in write_volume_label_to_block()\n");
    memset(&rec, 0, sizeof(rec));
    rec.data = get_memory(SER_LENGTH_Volume_Label);
+   empty_block(block);               /* Volume label always at beginning */
 
-   create_volume_label_record(jcr, dev, &rec);
+   create_volume_label_record(dcr, &rec);
 
-   empty_block(block);               /* Volume label always at beginning */
    block->BlockNumber = 0;
    if (!write_record_to_block(block, &rec)) {
       free_pool_memory(rec.data);
       Jmsg1(jcr, M_FATAL, 0, _("Cannot write Volume label to block for device %s\n"),
         dev_name(dev));
-      return 0;
+      return false;
    } else {
       Dmsg1(90, "Wrote label of %d bytes to block\n", rec.data_len);
    }
    free_pool_memory(rec.data);
-   return 1;
+   return true;
 }
 
 /* 
@@ -270,10 +286,12 @@ int write_volume_label_to_block(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
  *   Assumes that the dev->VolHdr structure is properly 
  *   initialized.
 */
-static void create_volume_label_record(JCR *jcr, DEVICE *dev, DEV_RECORD *rec)
+static void create_volume_label_record(DCR *dcr, DEV_RECORD *rec)
 {
    ser_declare;
    struct date_time dt;
+   DEVICE *dev = dcr->dev;
+   JCR *jcr = dcr->jcr;
 
    /* Serialize the label into the device record. */
 
@@ -324,10 +342,8 @@ static void create_volume_label_record(JCR *jcr, DEVICE *dev, DEV_RECORD *rec)
 
 /*
  * Create a volume label in memory
- *  Returns: 0 on error
- *          1 on success
  */
-void create_volume_label(DEVICE *dev, char *VolName, char *PoolName)
+void create_volume_label(DEVICE *dev, const char *VolName, const char *PoolName)
 {
    DEVRES *device = (DEVRES *)dev->device;
 
@@ -373,59 +389,53 @@ void create_volume_label(DEVICE *dev, char *VolName, char *PoolName)
  *
  *  This routine should be used only when labeling a blank tape.
  */
-int write_volume_label_to_dev(JCR *jcr, DEVRES *device, char *VolName, char *PoolName)
+bool write_new_volume_label_to_dev(DCR *dcr, const char *VolName, const char *PoolName)
 {
-   DEVICE *dev = device->dev;
-   DEV_RECORD rec;   
-   DEV_BLOCK *block;
-   int stat = 1;
+   bool ok = false;
+   DEVICE *dev = dcr->dev;
 
 
    Dmsg0(99, "write_volume_label()\n");
+   empty_block(dcr->block);
    create_volume_label(dev, VolName, PoolName);
 
    if (!rewind_dev(dev)) {
       memset(&dev->VolHdr, 0, sizeof(dev->VolHdr));
       Dmsg2(30, "Bad status on %s from rewind. ERR=%s\n", dev_name(dev), strerror_dev(dev));
       if (!forge_on) {
-        return 0;
+        return false;
       }
    }
 
-   block = new_block(dev);
-   memset(&rec, 0, sizeof(rec));
-   rec.data = get_memory(SER_LENGTH_Volume_Label);
-   create_volume_label_record(jcr, dev, &rec);
-   rec.Stream = 0;
+   create_volume_label_record(dcr, dcr->rec);
+   dcr->rec->Stream = 0;
 
-   if (!write_record_to_block(block, &rec)) {
+   if (!write_record_to_block(dcr->block, dcr->rec)) {
       Dmsg2(30, "Bad Label write on %s. ERR=%s\n", dev_name(dev), strerror_dev(dev));
       memset(&dev->VolHdr, 0, sizeof(dev->VolHdr));
-      free_block(block);
-      free_pool_memory(rec.data);
-      return 0;
+      goto bail_out;
    } else {
-      Dmsg2(30, "Wrote label of %d bytes to %s\n", rec.data_len, dev_name(dev));
+      Dmsg2(30, "Wrote label of %d bytes to %s\n", dcr->rec->data_len, dev_name(dev));
    }
-   free_pool_memory(rec.data);
       
    Dmsg0(99, "Call write_block_to_dev()\n");
-   if (!write_block_to_dev(jcr->dcr, block)) {
+   if (!write_block_to_dev(dcr)) {
       memset(&dev->VolHdr, 0, sizeof(dev->VolHdr));
       Dmsg2(30, "Bad Label write on %s. ERR=%s\n", dev_name(dev), strerror_dev(dev));
-      stat = 9;
+      goto bail_out;
    }
    Dmsg0(99, " Wrote block to device\n");
      
-   flush_dev(dev);
    weof_dev(dev, 1);
    dev->state |= ST_LABEL;
+   ok = true;
 
    if (debug_level >= 20)  {
       dump_volume_label(dev);
    }
-   free_block(block);
-   return stat;
+
+bail_out:
+   return ok;
 }     
 
 
@@ -433,9 +443,9 @@ int write_volume_label_to_dev(JCR *jcr, DEVRES *device, char *VolName, char *Poo
  * Create session label
  *  The pool memory must be released by the calling program
  */
-void create_session_label(JCR *jcr, DEV_RECORD *rec, int label)
+void create_session_label(DCR *dcr, DEV_RECORD *rec, int label)
 {
-   DCR *dcr = jcr->dcr;
+   JCR *jcr = dcr->jcr;
    ser_declare;
 
    rec->VolSessionId   = jcr->VolSessionId;
@@ -453,8 +463,8 @@ void create_session_label(JCR *jcr, DEV_RECORD *rec, int label)
    ser_btime(get_current_btime());
    ser_float64(0);
 
-   ser_string(jcr->pool_name);
-   ser_string(jcr->pool_type);
+   ser_string(dcr->pool_name);
+   ser_string(dcr->pool_type);
    ser_string(jcr->job_name);        /* base Job name */
    ser_string(jcr->client_name);
 
@@ -483,14 +493,15 @@ void create_session_label(JCR *jcr, DEV_RECORD *rec, int label)
 }
 
 /* Write session label
- *  Returns: 0 on failure
- *          1 on success 
+ *  Returns: false on failure
+ *          true  on success 
  */
-int write_session_label(JCR *jcr, DEV_BLOCK *block, int label)
+bool write_session_label(DCR *dcr, int label)
 {
-   DEVICE *dev = jcr->device->dev;
-   DCR *dcr = jcr->dcr;
+   JCR *jcr = dcr->jcr;
+   DEVICE *dev = dcr->dev;
    DEV_RECORD *rec;
+   DEV_BLOCK *block = dcr->block;
 
    rec = new_record();
    Dmsg1(90, "session_label record=%x\n", rec);
@@ -517,7 +528,7 @@ int write_session_label(JCR *jcr, DEV_BLOCK *block, int label)
       Jmsg1(jcr, M_ABORT, 0, _("Bad session label = %d\n"), label);
       break;
    }
-   create_session_label(jcr, rec, label);
+   create_session_label(dcr, rec, label);
    rec->FileIndex = label;
 
    /* 
@@ -529,19 +540,20 @@ int write_session_label(JCR *jcr, DEV_BLOCK *block, int label)
     */
    if (!can_write_record_to_block(block, rec)) {
       Dmsg0(100, "Cannot write session label to block.\n");
-      if (!write_block_to_device(jcr->dcr, block)) {
+      if (!write_block_to_device(dcr)) {
          Dmsg0(90, "Got session label write_block_to_dev error.\n");
+        /* ****FIXME***** errno is not set here */
          Jmsg(jcr, M_FATAL, 0, _("Error writing Session label to %s: %s\n"), 
                           dev_vol_name(dev), strerror(errno));
         free_record(rec);
-        return 0;
+        return false;
       }
    }
    if (!write_record_to_block(block, rec)) {
       Jmsg(jcr, M_FATAL, 0, _("Error writing Session label to %s: %s\n"), 
                        dev_vol_name(dev), strerror(errno));
       free_record(rec);
-      return 0;
+      return false;
    }
 
    Dmsg6(20, "Write sesson_label record JobId=%d FI=%s SessId=%d Strm=%s len=%d\n\
@@ -553,14 +565,15 @@ remainder=%d\n", jcr->JobId,
    free_record(rec);
    Dmsg2(20, "Leave write_session_label Block=%d File=%d\n", 
       dev->block_num, dev->file);
-   return 1;
+   return true;
 }
 
 void dump_volume_label(DEVICE *dev)
 {
    int dbl = debug_level;
    uint32_t File;
-   char *LabelType, buf[30];
+   const char *LabelType;
+   char buf[30];
    struct tm tm;
    struct date_time dt;
 
@@ -627,7 +640,7 @@ bail_out:
    debug_level = dbl;
 }
 
-int unser_session_label(SESSION_LABEL *label, DEV_RECORD *rec) 
+bool unser_session_label(SESSION_LABEL *label, DEV_RECORD *rec) 
 {
    ser_declare;
 
@@ -671,11 +684,11 @@ int unser_session_label(SESSION_LABEL *label, DEV_RECORD *rec)
         label->JobStatus = JS_Terminated; /* kludge */
       }
    }     
-   return 1;
+   return true;
 }
 
 
-static void dump_session_label(DEV_RECORD *rec, char *type)
+static void dump_session_label(DEV_RECORD *rec, const char *type)
 {
    int dbl;
    struct date_time dt;
@@ -744,79 +757,88 @@ Date written      : %04d-%02d-%02d at %02d:%02d\n"),
 
 void dump_label_record(DEVICE *dev, DEV_RECORD *rec, int verbose)
 {
-   char *type;
+   const char *type;
    int dbl;
 
    dbl = debug_level;
    debug_level = 1;
    switch (rec->FileIndex) {
+   case PRE_LABEL:
+      type = _("Fresh Volume");   
+      break;
+   case VOL_LABEL:
+      type = _("Volume");
+      break;
+   case SOS_LABEL:
+      type = _("Begin Job Session");
+      break;
+   case EOS_LABEL:
+      type = _("End Job Session");
+      break;
+   case EOM_LABEL:
+      type = _("End of Media");
+      break;
+   case EOT_LABEL:
+      type = ("End of Tape");
+      break;
+   default:
+      type = _("Unknown");
+      break;
+   }
+   if (verbose) {
+      switch (rec->FileIndex) {
       case PRE_LABEL:
-         type = _("Fresh Volume");   
-        break;
       case VOL_LABEL:
-         type = _("Volume");
+        unser_volume_label(dev, rec);
+        dump_volume_label(dev);
         break;
       case SOS_LABEL:
-         type = _("Begin Session");
+        dump_session_label(rec, type);
         break;
       case EOS_LABEL:
-         type = _("End Session");
+        dump_session_label(rec, type);
         break;
       case EOM_LABEL:
-         type = _("End of Media");
+         Pmsg5(-1, "%s Record: SessId=%d SessTime=%d JobId=%d DataLen=%d\n",
+           type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, rec->data_len);
         break;
       case EOT_LABEL:
-         type = ("End of Tape");
+         Pmsg0(-1, _("End of physical tape.\n"));
         break;
       default:
-         type = _("Unknown");
+         Pmsg5(-1, "%s Record: SessId=%d SessTime=%d JobId=%d DataLen=%d\n",
+           type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, rec->data_len);
         break;
-   }
-   if (verbose) {
-      switch (rec->FileIndex) {
-        case PRE_LABEL:
-        case VOL_LABEL:
-           unser_volume_label(dev, rec);
-           dump_volume_label(dev);
-           break;
-        case SOS_LABEL:
-           dump_session_label(rec, type);
-           break;
-        case EOS_LABEL:
-           dump_session_label(rec, type);
-           break;
-        case EOM_LABEL:
-            Pmsg5(-1, "%s Record: SessId=%d SessTime=%d JobId=%d DataLen=%d\n",
-              type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, rec->data_len);
-           break;
-        case EOT_LABEL:
-            Pmsg0(-1, _("End of physical tape.\n"));
-           break;
-        default:
-            Pmsg5(-1, "%s Record: SessId=%d SessTime=%d JobId=%d DataLen=%d\n",
-              type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, rec->data_len);
-           break;
       }
    } else {
+      SESSION_LABEL label;
       switch (rec->FileIndex) {
-        case SOS_LABEL:
-        case EOS_LABEL:
-           SESSION_LABEL label;
-           unser_session_label(&label, rec);
-            Pmsg6(-1, "%s Record: SessId=%d SessTime=%d JobId=%d Level=%c \
-Type=%c\n",
-              type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, 
-              label.JobLevel, label.JobType);
-           break;
-        case EOM_LABEL:
-        case PRE_LABEL:
-        case VOL_LABEL:
-        default:
-            Pmsg5(-1, "%s Record: SessId=%d SessTime=%d JobId=%d DataLen=%d\n",
-        type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, rec->data_len);
-           break;
-        case EOT_LABEL:
-           break;
+      case SOS_LABEL:
+        unser_session_label(&label, rec);
+         Pmsg6(-1, "%s Record: SessId=%d SessTime=%d JobId=%d Level=%c Type=%c\n",
+           type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, 
+           label.JobLevel, label.JobType);
+        break;
+      case EOS_LABEL:
+        char ed1[30], ed2[30];
+        unser_session_label(&label, rec);
+         Pmsg6(-1, "%s Record: SessId=%d SessTime=%d JobId=%d Level=%c Type=%c\n",
+           type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, 
+           label.JobLevel, label.JobType);
+         Pmsg4(-1, "   Files=%s Bytes=%s Errors=%d Status=%c\n",
+           edit_uint64_with_commas(label.JobFiles, ed1),
+           edit_uint64_with_commas(label.JobBytes, ed2),
+           label.JobErrors, (char)label.JobStatus);
+        break;
+      case EOM_LABEL:
+      case PRE_LABEL:
+      case VOL_LABEL:
+      default:
+         Pmsg5(-1, "%s Record: SessId=%d SessTime=%d JobId=%d DataLen=%d\n",
+      type, rec->VolSessionId, rec->VolSessionTime, rec->Stream, rec->data_len);
+        break;
+      case EOT_LABEL:
+        break;
       }
    }
    debug_level = dbl;