]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/dev.c
This commit was manufactured by cvs2svn to create tag
[bacula/bacula] / bacula / src / stored / dev.c
index 194890794729b603ea5a4ae6dfe9bd0d391d86be..b01c6703832aa9026432f799a4c7e6b40f39ad93 100644 (file)
@@ -33,7 +33,7 @@
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License
-   version 2 as ammended with additional clauses defined in the
+   version 2 as amended with additional clauses defined in the
    file LICENSE in the main source directory.
 
    This program is distributed in the hope that it will be useful,
 #define O_NONBLOCK 0
 #endif
 
-/* Functions in dvd.c */ 
-void update_free_space_dev(DEVICE* dev);
-
-
 /* Forward referenced functions */
 void set_os_device_parameters(DEVICE *dev);
 static bool dev_get_os_pos(DEVICE *dev, struct mtget *mt_stat);
-static void open_tape_device(DCR *dcr, int mode);
-static void open_file_device(DCR *dcr, int mode);
-static void open_dvd_device(DCR *dcr, int mode);
 static char *mode_to_str(int mode);
 
 /*
@@ -265,31 +258,33 @@ init_dev(JCR *jcr, DEVRES *device)
  *    (archive_name) with the VolName concatenated.
  */
 int
-DEVICE::open(DCR *dcr, int mode)
+DEVICE::open(DCR *dcr, int omode)
 {
    if (is_open()) {
-      if (openmode == mode) {
+      if (openmode == omode) {
          return fd;
       } else {
         ::close(fd); /* use system close so correct mode will be used on open */
+        fd = -1;
+        Dmsg0(100, "Close fd for mode change.\n");
       }
    }
-  if (dcr) {
-     bstrncpy(VolCatInfo.VolCatName, dcr->VolumeName, sizeof(VolCatInfo.VolCatName));
-  }
+   if (dcr) {
+      bstrncpy(VolCatInfo.VolCatName, dcr->VolumeName, sizeof(VolCatInfo.VolCatName));
+   }
 
    Dmsg4(29, "open dev: tape=%d dev_name=%s vol=%s mode=%s\n", is_tape(),
-         dev_name, VolCatInfo.VolCatName, mode_to_str(mode));
+         print_name(), VolCatInfo.VolCatName, mode_to_str(omode));
    state &= ~(ST_LABEL|ST_APPEND|ST_READ|ST_EOT|ST_WEOT|ST_EOF);
    label_type = B_BACULA_LABEL;
    if (is_tape() || is_fifo()) {
-      open_tape_device(dcr, mode);
+      open_tape_device(dcr, omode);
    } else if (is_dvd()) {
-      Dmsg1(100, "call open_dvd_device mode=%s\n", mode_to_str(mode));
-      open_dvd_device(dcr, mode);
+      Dmsg1(100, "call open_dvd_device mode=%s\n", mode_to_str(omode));
+      open_dvd_device(dcr, omode);
    } else {
-      Dmsg1(100, "call open_file_device mode=%d\n", mode_to_str(mode));
-      open_file_device(dcr, mode);
+      Dmsg1(100, "call open_file_device mode=%s\n", mode_to_str(omode));
+      open_file_device(omode);
    }
    return fd;
 }
@@ -318,236 +313,323 @@ void DEVICE::set_mode(int new_mode)
    }
 }
 
-static void open_tape_device(DCR *dcr, int mode) 
+/*
+ * If the flage open_nowait is set, which is the case
+ *   when the daemon is initially trying to open the device,
+ *   we open it with O_NONBLOCK set and O_RONLY, which will
+ *   allow us to open normal Linux tape drives with no tape
+ *   in the drive without blocking.  We then immediately
+ *   set blocking status so that if we read from the device they
+ *   will be normal blocking reads.
+ *
+ * If later, we want to write on the device, it will be freed and
+ *   reopened, but hopefully there will be a tape in the drive so
+ *   we will not block.
+ */
+void DEVICE::open_tape_device(DCR *dcr, int omode) 
 {
-   DEVICE *dev = dcr->dev;
-   int nonblocking = 0;;
-   dev->file_size = 0;
+   int nonblocking = 0;
+   file_size = 0;
    int timeout;
    int ioerrcnt = 10;
    Dmsg0(29, "open dev: device is tape\n");
 
-   dev->set_mode(mode);
-   timeout = dev->max_open_wait;
+   if (is_tape() && is_autochanger()) {
+      get_autochanger_loaded_slot(dcr);
+   }
+
+   set_mode(omode);
+   timeout = max_open_wait;
    errno = 0;
-   if (dev->open_nowait) {
+#ifdef HAVE_LINUX_OS
+   if (open_nowait) {
        /* Set wait counters to zero for no wait */
        timeout = ioerrcnt = 0;
        /* Open drive in non-block mode */
        nonblocking = O_NONBLOCK;
    }
-   if (dev->is_fifo() && timeout) {
+#endif
+   if (is_fifo() && timeout) {
       /* Set open timer */
-      dev->tid = start_thread_timer(pthread_self(), timeout);
+      tid = start_thread_timer(pthread_self(), timeout);
    }
    /* If busy retry each second for max_open_wait seconds */
-open_again:
-   Dmsg1(500, "Try open %s\n", dev->dev_name);
-   while ((dev->fd = open(dev->dev_name, dev->mode, MODE_RW+nonblocking)) < 0) {
+   Dmsg3(100, "Try open %s mode=%s nonblocking=%d\n", print_name(),
+      mode_to_str(omode), nonblocking);
+   /* Use system open() */
+   while ((fd = ::open(dev_name, mode+nonblocking, MODE_RW)) < 0) {
       berrno be;
-      Dmsg2(500, "Open error errno=%d ERR=%s\n", errno, be.strerror());
+      Dmsg2(100, "Open error errno=%d ERR=%s\n", errno, be.strerror());
       if (errno == EINTR || errno == EAGAIN) {
-         Dmsg0(500, "Continue open\n");
+         Dmsg0(100, "Continue open\n");
          continue;
       }
       /* Busy wait for specified time (default = 5 mins) */
       if (errno == EBUSY && timeout-- > 0) {
-         Dmsg2(100, "Device %s busy. ERR=%s\n", dev->print_name(), be.strerror());
+         Dmsg2(100, "Device %s busy. ERR=%s\n", print_name(), be.strerror());
          bmicrosleep(1, 0);
          continue;
       }
       /* IO error (no volume) try 10 times every 6 seconds */
       if (errno == EIO && ioerrcnt-- > 0) {
          bmicrosleep(5, 0);
-         Dmsg0(500, "Continue open\n");
+         Dmsg0(100, "Continue open\n");
          continue;
       }
-      dev->dev_errno = errno;
-      Mmsg2(dev->errmsg, _("Unable to open device %s: ERR=%s\n"),
-            dev->print_name(), be.strerror(dev->dev_errno));
+      dev_errno = errno;
+      Mmsg2(errmsg, _("Unable to open device %s: ERR=%s\n"),
+            print_name(), be.strerror(dev_errno));
       /* Stop any open timer we set */
-      if (dev->tid) {
-         stop_thread_timer(dev->tid);
-         dev->tid = 0;
+      if (tid) {
+         stop_thread_timer(tid);
+         tid = 0;
       }
-      Emsg0(M_FATAL, 0, dev->errmsg);
+      Jmsg0(dcr->jcr, M_FATAL, 0, errmsg);
       break;
    }
-   if (dev->fd >= 0) {
-      /* If opened in non-block mode, close it an open it normally */
-      if (nonblocking) {
-         nonblocking = 0;
-         close(dev->fd);
-         goto open_again;
-      }
-      dev->openmode = mode;               /* save open mode */
-      dev->dev_errno = 0;
-      dev->state |= ST_OPENED;
-      dev->use_count = 1;
-      update_pos_dev(dev);                /* update position */
-      set_os_device_parameters(dev);      /* do system dependent stuff */
-      Dmsg0(500, "Open OK\n");
+
+   if (nonblocking) {
+      set_blocking();
    }
+
+   if (fd >= 0) {
+      openmode = omode;              /* save open mode */
+      Dmsg2(100, "openmode=%d %s\n", openmode, mode_to_str(openmode));
+      dev_errno = 0;
+      set_opened();
+      use_count = 1;
+      update_pos_dev(this);                /* update position */
+      set_os_device_parameters(this);      /* do system dependent stuff */
+   }
+
    /* Stop any open() timer we started */
-   if (dev->tid) {
-      stop_thread_timer(dev->tid);
-      dev->tid = 0;
+   if (tid) {
+      stop_thread_timer(tid);
+      tid = 0;
+   }
+   Dmsg1(29, "open dev: tape %d opened\n", fd);
+}
+
+void DEVICE::set_blocking()
+{
+   int oflags;
+   /* Try to reset blocking */
+   if ((oflags = fcntl(fd, F_GETFL, 0)) < 0 ||
+       fcntl(fd, F_SETFL, oflags & ~O_NONBLOCK) < 0) {
+      berrno be;
+      ::close(fd);                   /* use system close() */
+      fd = ::open(dev_name, mode, MODE_RW);       
+      Dmsg2(100, "fcntl error. ERR=%s. Close-reopen fd=%d\n", be.strerror(), fd);
    }
-   Dmsg1(29, "open dev: tape %d opened\n", dev->fd);
 }
 
 /*
  * Open a file device
  */
-static void open_file_device(DCR *dcr, int mode) 
+void DEVICE::open_file_device(int omode) 
 {
-   DEVICE *dev = dcr->dev;
    POOL_MEM archive_name(PM_FNAME);
 
    /*
     * Handle opening of File Archive (not a tape)
     */     
-   Dmsg3(29, "Enter: open_file_dev: %s dev=%s mode=%s\n", dev->is_dvd()?"DVD":"disk",
-         archive_name.c_str(), mode_to_str(mode));
+   Dmsg3(29, "Enter: open_file_dev: %s dev=%s mode=%s\n", is_dvd()?"DVD":"disk",
+         archive_name.c_str(), mode_to_str(omode));
 
-   if (dev->VolCatInfo.VolCatName[0] == 0) {
-      Mmsg(dev->errmsg, _("Could not open file device %s. No Volume name given.\n"),
-         dev->print_name());
-      dev->fd = -1;
+   if (VolCatInfo.VolCatName[0] == 0) {
+      Mmsg(errmsg, _("Could not open file device %s. No Volume name given.\n"),
+         print_name());
+      fd = -1;
       return;
    }
 
-   pm_strcpy(archive_name, dev->dev_name);
+   pm_strcpy(archive_name, dev_name);
    if (archive_name.c_str()[strlen(archive_name.c_str())-1] != '/') {
       pm_strcat(archive_name, "/");
    }
-   pm_strcat(archive_name, dev->VolCatInfo.VolCatName);
+   pm_strcat(archive_name, VolCatInfo.VolCatName);
          
-   Dmsg3(29, "open dev: %s dev=%s mode=%s\n", dev->is_dvd()?"DVD":"disk",
-         archive_name.c_str(), mode_to_str(mode));
-   dev->openmode = mode;
+   Dmsg3(29, "open dev: %s dev=%s mode=%s\n", is_dvd()?"DVD":"disk",
+         archive_name.c_str(), mode_to_str(omode));
+   openmode = omode;
+   Dmsg2(100, "openmode=%d %s\n", openmode, mode_to_str(openmode));
    
-   dev->set_mode(mode);
+   set_mode(omode);
    /* If creating file, give 0640 permissions */
-   Dmsg3(29, "mode=%s open(%s, 0x%x, 0640)\n", mode, archive_name.c_str(), 
-         mode_to_str(dev->mode));
-   if ((dev->fd = open(archive_name.c_str(), dev->mode, 0640)) < 0) {
+   Dmsg3(29, "mode=%s open(%s, 0x%x, 0640)\n", mode_to_str(omode), 
+         archive_name.c_str(), mode);
+   /* Use system open() */
+   if ((fd = ::open(archive_name.c_str(), mode, 0640)) < 0) {
       berrno be;
-      dev->dev_errno = errno;
-      Mmsg2(dev->errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), 
+      dev_errno = errno;
+      Mmsg2(errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), 
             be.strerror());
-      Dmsg1(29, "open failed: %s", dev->errmsg);
-      Emsg0(M_FATAL, 0, dev->errmsg);
+      Dmsg1(29, "open failed: %s", errmsg);
+      Emsg0(M_FATAL, 0, errmsg);
    } else {
-      dev->dev_errno = 0;
-      dev->state |= ST_OPENED;
-      dev->use_count = 1;
-      update_pos_dev(dev);                /* update position */
+      dev_errno = 0;
+      set_opened();
+      use_count = 1;
+      update_pos_dev(this);                /* update position */
    }
    Dmsg5(29, "open dev: %s fd=%d opened, part=%d/%d, part_size=%u\n", 
-      dev->is_dvd()?"DVD":"disk", dev->fd, dev->part, dev->num_parts, 
-      dev->part_size);
+      is_dvd()?"DVD":"disk", fd, part, num_parts, 
+      part_size);
 }
 
 /*
- * Open a DVD device. N.B. at this point, dcr->VolCatInfo.VolCatName
+ * Open a DVD device. N.B. at this point, dcr->VolCatInfo.VolCatName (NB:??? I think it's VolCatInfo.VolCatName that is right)
  *  has the desired Volume name, but there is NO assurance that
  *  any other field of VolCatInfo is correct.
  */
-static void open_dvd_device(DCR *dcr, int mode) 
+void DEVICE::open_dvd_device(DCR *dcr, int omode) 
 {
-   DEVICE *dev = dcr->dev;
    POOL_MEM archive_name(PM_FNAME);
    struct stat filestat;
 
    /*
     * Handle opening of DVD Volume
     */     
-   Dmsg3(29, "Enter: open_dvd_dev: %s dev=%s mode=%s\n", dev->is_dvd()?"DVD":"disk",
-         archive_name.c_str(), mode_to_str(mode));
-
-   if (dev->VolCatInfo.VolCatName[0] == 0) {
-      Mmsg(dev->errmsg, _("Could not open file device %s. No Volume name given.\n"),
-         dev->print_name());
-      dev->fd = -1;
+   Dmsg3(29, "Enter: open_dvd_dev: %s dev=%s mode=%s\n", is_dvd()?"DVD":"disk",
+         archive_name.c_str(), mode_to_str(omode));
+
+   if (VolCatInfo.VolCatName[0] == 0) {
+      Dmsg1(10,  "Could not open file device %s. No Volume name given.\n",
+         print_name());
+      Mmsg(errmsg, _("Could not open file device %s. No Volume name given.\n"),
+         print_name());
+      fd = -1;
       return;
    }
 
-   if (dev->part == 0) {
-      dev->file_size = 0;
+   if (part == 0) {
+      file_size = 0;
    }
-   dev->part_size = 0;
-   
+   part_size = 0;
 
-   if (mount_dev(dev, 1) < 0) {
-      Mmsg(dev->errmsg, _("Could not mount device %s.\n"),
-           dev->print_name());
-      Emsg0(M_FATAL, 0, dev->errmsg);
-      dev->fd = -1;
-      return;
+   Dmsg2(99, "open_dvd_device: num_parts=%d, VolCatInfo.VolCatParts=%d\n",
+      dcr->dev->num_parts, dcr->VolCatInfo.VolCatParts);
+   if (dcr->dev->num_parts < dcr->VolCatInfo.VolCatParts) {
+      Dmsg2(99, "open_dvd_device: num_parts updated to %d (was %d)\n",
+         dcr->VolCatInfo.VolCatParts, dcr->dev->num_parts);
+      dcr->dev->num_parts = dcr->VolCatInfo.VolCatParts;
+   }
+
+   if (mount_dev(this, 1)) {
+      if ((num_parts == 0) && (!truncating)) {
+         /* If we can mount the device, and we are not truncating the DVD, we usually want to abort. */
+         /* There is one exception, if there is only one 0-sized file on the DVD, with the right volume name,
+          * we continue (it's the method used by truncate_dvd_dev to truncate a volume). */
+         if (!check_can_write_on_non_blank_dvd(dcr)) {
+            Mmsg(errmsg, _("The media in the device %s is not empty, please blank it before writing anything to it.\n"), print_name());
+            Emsg0(M_FATAL, 0, errmsg);
+            unmount_dev(this, 1); /* Unmount the device, so the operator can change it. */
+            fd = -1;
+            return;
+         }
+      }
+   }
+   else {
+      /* We cannot mount the device */
+      if (num_parts == 0) {
+         /* Run free space, check there is a media. */
+         update_free_space_dev(this);
+         if (have_media()) {
+            Dmsg1(29, "Could not mount device %s, this is not a problem (num_parts == 0), and have media.\n", print_name());
+         }
+         else {
+            Mmsg(errmsg, _("There is no valid media in the device %s.\n"), print_name());
+            Emsg0(M_FATAL, 0, errmsg);
+            fd = -1;
+            return;
+         }
+      }
+      else {
+         Mmsg(errmsg, _("Could not mount device %s.\n"), print_name());
+         Emsg0(M_FATAL, 0, errmsg);
+         fd = -1;
+         return;
+      }
    }
-         
-   Dmsg3(29, "open dev: %s dev=%s mode=%s\n", dev->is_dvd()?"DVD":"disk",
-         archive_name.c_str(), mode_to_str(mode));
-   dev->openmode = mode;
+   
+   Dmsg6(29, "open dev: %s dev=%s mode=%s part=%d npart=%d volcatnparts=%d\n", 
+      is_dvd()?"DVD":"disk", archive_name.c_str(), mode_to_str(omode),
+      part, num_parts, dcr->VolCatInfo.VolCatParts);
+   openmode = omode;
+   Dmsg2(100, "openmode=%d %s\n", openmode, mode_to_str(openmode));
    
    /*
     * If we are not trying to access the last part, set mode to 
     *   OPEN_READ_ONLY as writing would be an error.
     */
-   if (dev->part < dev->num_parts) {
-      mode = OPEN_READ_ONLY;
+   if (part < num_parts) {
+      omode = OPEN_READ_ONLY;
+      make_mounted_dvd_filename(this, archive_name);
    }
-   dev->set_mode(mode);
-
-   /* 
-    * If we are opening it read-only, it is *probably* on the
-    *   DVD, so try the DVD first, otherwise look in the spool dir.
-    */
-   if (mode == OPEN_READ_ONLY) {
-      make_mounted_dvd_filename(dev, archive_name);
-   } else {
-      make_spooled_dvd_filename(dev, archive_name);
+   else {
+      make_spooled_dvd_filename(this, archive_name);
    }
+   set_mode(omode);
 
    /* If creating file, give 0640 permissions */
-   Dmsg3(29, "mode=%s open(%s, 0x%x, 0640)\n", mode_to_str(mode), 
-         archive_name.c_str(), dev->mode);
-   if ((dev->fd = open(archive_name.c_str(), dev->mode, 0640)) < 0) {
+   Dmsg3(29, "mode=%s open(%s, 0x%x, 0640)\n", mode_to_str(omode), 
+         archive_name.c_str(), mode);
+   /* Use system open() */
+   if ((fd = ::open(archive_name.c_str(), mode, 0640)) < 0) {
       berrno be;
-      dev->dev_errno = errno;
-      Mmsg2(dev->errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), 
+      Mmsg2(errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), 
             be.strerror());
-      Dmsg1(29, "open failed: %s", dev->errmsg);
-      if (mode == OPEN_READ_ONLY) {
-         make_spooled_dvd_filename(dev, archive_name);
-         dev->fd = open(archive_name.c_str(), dev->mode, 0640);  /* try on spool */
+      dev_errno = EIO; /* Interpreted as no device present by acquire.c:acquire_device_for_read(). */
+      Dmsg1(29, "open failed: %s", errmsg);
+      
+      if ((omode == OPEN_READ_ONLY) && (part == num_parts)) {
+         /* If the last part (on spool), doesn't exists when reading, create it and read from it
+          * (it will report immediately an EOF):
+          * Sometimes it is better to finish with an EOF than with an error. */
+         set_mode(OPEN_READ_WRITE);
+         fd = ::open(archive_name.c_str(), mode, 0640);
+         set_mode(OPEN_READ_ONLY);
       }
-   }
-   if (dev->fd >= 0) {
+      
+      /* We don't need it. Only the last part is on spool */
+      /*if (omode == OPEN_READ_ONLY) {
+         make_spooled_dvd_filename(this, archive_name);
+         fd = ::open(archive_name.c_str(), mode, 0640);  // try on spool
+      }*/
+   }
+   Dmsg1(100, "after open fd=%d\n", fd);
+   if (fd >= 0) {
       /* Get size of file */
-      if (fstat(dev->fd, &filestat) < 0) {
+      if (fstat(fd, &filestat) < 0) {
          berrno be;
-         dev->dev_errno = errno;
-         Mmsg2(dev->errmsg, _("Could not fstat: %s, ERR=%s\n"), archive_name.c_str(), 
+         dev_errno = errno;
+         Mmsg2(errmsg, _("Could not fstat: %s, ERR=%s\n"), archive_name.c_str(), 
                be.strerror());
-         Dmsg1(29, "open failed: %s", dev->errmsg);
-         close(dev->fd);
-         dev->fd = -1;
+         Dmsg1(29, "open failed: %s", errmsg);
+         /* Use system close() */
+         ::close(fd);
+         fd = -1;
       } else {
-         dev->part_size = filestat.st_size;
-         dev->dev_errno = 0;
-         dev->state |= ST_OPENED;
-         dev->use_count = 1;
-         update_pos_dev(dev);                /* update position */
+         part_size = filestat.st_size;
+         dev_errno = 0;
+         set_opened();
+         use_count = 1;
+         update_pos_dev(this);                /* update position */
+         
+         /* NB: It seems this code is wrong... part number is incremented in open_next_part, not here */
+         
+         /* Check if just created Volume  part */
+/*         if (omode == OPEN_READ_WRITE && (part == 0 || part_size == 0)) {
+            part++;
+            num_parts = part;
+            VolCatInfo.VolCatParts = num_parts;
+         } else {
+            if (part == 0) {             // we must have opened the first part
+               part++;
+            }
+         }*/
       }
    }
-   Dmsg4(29, "open dev: DVD fd=%d opened, part=%d/%d, part_size=%u\n", 
-      dev->fd, dev->part, dev->num_parts, dev->part_size);
-   if (dev->is_open() && dev->is_dvd() && (mode != OPEN_READ_ONLY) && 
-       (dev->free_space_errno == 0 || dev->num_parts == dev->part)) {
-      update_free_space_dev(dev);
-   }
 }
 
 
@@ -555,7 +637,7 @@ static void open_dvd_device(DCR *dcr, int mode)
 #undef rewind_dev
 bool _rewind_dev(char *file, int line, DEVICE *dev)
 {
-   Dmsg2(100, "rewind_dev called from %s:%d\n", file, line);
+   Dmsg3(100, "rewind_dev fd=%d called from %s:%d\n", dev->fd, file, line);
    return rewind_dev(dev);
 }
 #endif
@@ -570,12 +652,14 @@ bool rewind_dev(DEVICE *dev)
    struct mtop mt_com;
    unsigned int i;
 
-   Dmsg1(29, "rewind_dev %s\n", dev->print_name());
+   Dmsg2(29, "rewind_dev fd=%d %s\n", dev->fd, dev->print_name());
    if (dev->fd < 0) {
-      dev->dev_errno = EBADF;
-      Mmsg1(dev->errmsg, _("Bad call to rewind_dev. Device %s not open\n"),
+      if (!dev->is_dvd()) { /* In case of major error, the fd is not open on DVD, so we don't want to abort. */
+         dev->dev_errno = EBADF;
+         Mmsg1(dev->errmsg, _("Bad call to rewind_dev. Device %s not open\n"),
             dev->print_name());
-      Emsg0(M_ABORT, 0, dev->errmsg);
+         Emsg0(M_ABORT, 0, dev->errmsg);
+      }
       return false;
    }
    dev->state &= ~(ST_EOT|ST_EOF|ST_WEOT);  /* remove EOF/EOT flags */
@@ -651,7 +735,7 @@ const char *DEVICE::print_blocked() const
    case BST_MOUNT:
       return "BST_MOUNT";
    default:
-      return "unknown blocked code";
+      return _("unknown blocked code");
    }
 }
 
@@ -662,7 +746,9 @@ const char *DEVICE::print_blocked() const
 void DEVICE::set_ateof() 
 { 
    set_eof();
-   file++;
+   if (is_tape()) {
+      file++;
+   }
    file_addr = 0;
    file_size = 0;
    block_num = 0;
@@ -796,18 +882,17 @@ eod_dev(DEVICE *dev)
             return false;
          }
          /*
-          * Avoid infinite loop. ***FIXME*** possibly add code
-          *   to set EOD or to turn off CAP_FASTFSF if on.
+          * Avoid infinite loop by ensuring we advance.
           */
          if (file_num == (int)dev->file) {
             struct mtget mt_stat;
             Dmsg1(100, "fsf did not advance from file %d\n", file_num);
+            dev->set_ateof();
             if (dev_get_os_pos(dev, &mt_stat)) {
                Dmsg2(100, "Adjust file from %d to %d\n", dev->file , mt_stat.mt_fileno);
-               dev->set_ateof();
                dev->file = mt_stat.mt_fileno;
-            }
-            return false;
+            }       
+            break;
          }
       }
    }
@@ -835,7 +920,7 @@ eod_dev(DEVICE *dev)
 }
 
 /*
- * Set the position of the device -- only for files
+ * Set the position of the device -- only for files and DVD
  *   For other devices, there is no generic way to do it.
  *  Returns: true  on succes
  *           false on error
@@ -860,7 +945,7 @@ bool update_pos_dev(DEVICE *dev)
       if (pos < 0) {
          berrno be;
          dev->dev_errno = errno;
-         Pmsg1(000, "Seek error: ERR=%s\n", be.strerror());
+         Pmsg1(000, _("Seek error: ERR=%s\n"), be.strerror());
          Mmsg2(dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
             dev->print_name(), be.strerror());
          ok = false;
@@ -896,8 +981,8 @@ uint32_t status_dev(DEVICE *dev)
    }
    if (dev->is_tape()) {
       stat |= BMT_TAPE;
-      Pmsg0(-20," Bacula status:");
-      Pmsg2(-20," file=%d block=%d\n", dev->file, dev->block_num);
+      Pmsg0(-20,_(" Bacula status:"));
+      Pmsg2(-20,_(" file=%d block=%d\n"), dev->file, dev->block_num);
       if (ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) < 0) {
          berrno be;
          dev->dev_errno = errno;
@@ -905,7 +990,7 @@ uint32_t status_dev(DEVICE *dev)
             dev->print_name(), be.strerror());
          return 0;
       }
-      Pmsg0(-20, " Device status:");
+      Pmsg0(-20, _(" Device status:"));
 
 #if defined(HAVE_LINUX_OS)
       if (GMT_EOF(mt_stat.mt_gstat)) {
@@ -946,9 +1031,9 @@ uint32_t status_dev(DEVICE *dev)
       }
 #endif /* !SunOS && !OSF */
       if (dev_cap(dev, CAP_MTIOCGET)) {
-         Pmsg2(-20, " file=%d block=%d\n", mt_stat.mt_fileno, mt_stat.mt_blkno);
+         Pmsg2(-20, _(" file=%d block=%d\n"), mt_stat.mt_fileno, mt_stat.mt_blkno);
       } else {
-         Pmsg2(-20, " file=%d block=%d\n", -1, -1);
+         Pmsg2(-20, _(" file=%d block=%d\n"), -1, -1);
       }
    } else {
       stat |= BMT_ONLINE | BMT_BOT;
@@ -1011,14 +1096,8 @@ bool offline_dev(DEVICE *dev)
 {
    struct mtop mt_com;
 
-   if (dev->fd < 0) {
-      dev->dev_errno = EBADF;
-      Mmsg0(dev->errmsg, _("Bad call to offline_dev. Device not open\n"));
-      Emsg0(M_FATAL, 0, dev->errmsg);
-      return false;
-   }
-   if (!(dev->is_tape())) {
-      return true;
+   if (!dev || dev->fd < 0 || !dev->is_tape()) {
+      return true;                    /* device not open */
    }
 
    dev->state &= ~(ST_APPEND|ST_READ|ST_EOT|ST_EOF|ST_WEOT);  /* remove EOF/EOT flags */
@@ -1146,6 +1225,12 @@ bool DEVICE::fsf(int num)
          if ((stat = read(fd, (char *)rbuf, rbuf_len)) < 0) {
             if (errno == ENOMEM) {     /* tape record exceeds buf len */
                stat = rbuf_len;        /* This is OK */
+            /*
+             * On IBM drives, they return ENOSPC at EOM
+             *  instead of EOF status
+             */
+            } else if (at_eof() && errno == ENOSPC) {
+               stat = 0;
             } else {
                berrno be;
                set_eot();
@@ -1375,7 +1460,7 @@ reposition_dev(DEVICE *dev, uint32_t file, uint32_t block)
    }
 
    if (!dev->is_tape()) {
-      off_t pos = (((off_t)file)<<32) + block;
+      off_t pos = (((off_t)file)<<32) + (off_t)block;
       Dmsg1(100, "===== lseek_dev to %d\n", (int)pos);
       if (lseek_dev(dev, pos, SEEK_SET) == (off_t)-1) {
          berrno be;
@@ -1506,7 +1591,7 @@ clrerror_dev(DEVICE *dev, int func)
    if (errno == ENOTTY || errno == ENOSYS) { /* Function not implemented */
       switch (func) {
       case -1:
-         Emsg0(M_ABORT, 0, "Got ENOTTY on read/write!\n");
+         Emsg0(M_ABORT, 0, _("Got ENOTTY on read/write!\n"));
          break;
       case MTWEOF:
          msg = "WTWEOF";
@@ -1553,7 +1638,7 @@ clrerror_dev(DEVICE *dev, int func)
          break;
 #endif
       default:
-         bsnprintf(buf, sizeof(buf), "unknown func code %d", func);
+         bsnprintf(buf, sizeof(buf), _("unknown func code %d"), func);
          msg = buf;
          break;
       }
@@ -1619,12 +1704,12 @@ static void do_close(DEVICE *dev)
       close(dev->fd);
    }
 
-   if (unmount_dev(dev, 1) < 0) {
+   if (!unmount_dev(dev, 1)) {
       Dmsg1(0, "Cannot unmount device %s.\n", dev->print_name());
    }
    
    /* Remove the last part file if it is empty */
-   if (dev->can_append() && (dev->num_parts > 0)) {
+   if (dev->num_parts > 0) {
       struct stat statp;
       POOL_MEM archive_name(PM_FNAME);
       dev->part = dev->num_parts;
@@ -1645,10 +1730,12 @@ static void do_close(DEVICE *dev)
    dev->file_size = 0;
    dev->file_addr = 0;
    dev->part = 0;
+   dev->num_parts = 0;
    dev->part_size = 0;
    dev->part_start = 0;
    dev->EndFile = dev->EndBlock = 0;
    memset(&dev->VolCatInfo, 0, sizeof(dev->VolCatInfo));
+   free_volume(dev);
    memset(&dev->VolHdr, 0, sizeof(dev->VolHdr));
    if (dev->tid) {
       stop_thread_timer(dev->tid);
@@ -1679,8 +1766,10 @@ void DEVICE::close()
 }
 
 
-bool truncate_dev(DEVICE *dev)
+bool truncate_dev(DCR *dcr) /* We need the DCR for DVD-writing */
 {
+   DEVICE *dev = dcr->dev;
+
    Dmsg1(100, "truncate_dev %s\n", dev->print_name());
    if (dev->is_tape()) {
       return true;                    /* we don't really truncate tapes */
@@ -1688,8 +1777,7 @@ bool truncate_dev(DEVICE *dev)
    }
    
    if (dev->is_dvd()) {
-      Mmsg1(dev->errmsg, _("Truncate DVD %s not supported.\n"), dev->print_name());
-      return false;   /* we cannot truncate DVDs */
+      return truncate_dvd_dev(dcr);
    }
    
    if (ftruncate(dev->fd, 0) != 0) {