]> 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 ab671bdcfd972f6e909ae64089abfdf98aa187fe..294c17d9c33cda68bc667f92c6506eb5768a683a 100644 (file)
@@ -29,7 +29,7 @@
  *   Version $Id$
  */
 /*
-   Copyright (C) 2000-2005 Kern Sibbald
+   Copyright (C) 2000-2006 Kern Sibbald
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License
@@ -99,33 +99,34 @@ DEVICE *
 init_dev(JCR *jcr, DEVRES *device)
 {
    struct stat statp;
-   bool tape, fifo;
    int errstat;
    DCR *dcr = NULL;
    DEVICE *dev;
 
-   /* Check that device is available */
-   if (stat(device->device_name, &statp) < 0) {
-      berrno be;
-      Jmsg2(jcr, M_ERROR, 0, _("Unable to stat device %s: ERR=%s\n"), 
-         device->device_name, be.strerror());
-      return NULL;
-   }
-   
-   
-   tape = false;
-   fifo = false;
-   if (S_ISDIR(statp.st_mode)) {
-      tape = false;
-   } else if (S_ISCHR(statp.st_mode)) {
-      tape = true;
-   } else if (S_ISFIFO(statp.st_mode)) {
-      fifo = true;
-   } else if (!(device->cap_bits & CAP_REQMOUNT)) {
-      Jmsg2(jcr, M_ERROR, 0, _("%s is an unknown device type. Must be tape or directory\n"
-            " or have RequiresMount=yes for DVD. st_mode=%x\n"),
-         device->device_name, statp.st_mode);
-      return NULL;
+
+   /* If no device type specified, try to guess */
+   if (!device->dev_type) {
+      /* Check that device is available */
+      if (stat(device->device_name, &statp) < 0) {
+         berrno be;
+         Jmsg2(jcr, M_ERROR, 0, _("Unable to stat device %s: ERR=%s\n"), 
+            device->device_name, be.strerror());
+         return NULL;
+      }
+      if (S_ISDIR(statp.st_mode)) {
+         device->dev_type = B_FILE_DEV;
+      } else if (S_ISCHR(statp.st_mode)) {
+         device->dev_type = B_TAPE_DEV;
+      } else if (S_ISFIFO(statp.st_mode)) {
+         device->dev_type = B_FILE_DEV;
+      } else if (!(device->cap_bits & CAP_REQMOUNT)) {
+         Jmsg2(jcr, M_ERROR, 0, _("%s is an unknown device type. Must be tape or directory\n"
+               " or have RequiresMount=yes for DVD. st_mode=%x\n"),
+            device->device_name, statp.st_mode);
+         return NULL;
+      } else {
+         device->dev_type = B_DVD_DEV;
+      }
    }
 
    dev = (DEVICE *)get_memory(sizeof(DEVICE));
@@ -151,10 +152,10 @@ init_dev(JCR *jcr, DEVRES *device)
    dev->max_spool_size = device->max_spool_size;
    dev->drive_index = device->drive_index;
    dev->autoselect = device->autoselect;
-   if (tape) { /* No parts on tapes */
-      dev->max_part_size = 0;
-   }
-   else {
+   dev->dev_type = device->dev_type;
+   if (dev->is_tape()) { /* No parts on tapes */
+       dev->max_part_size = 0;
+   else {
       dev->max_part_size = device->max_part_size;
    }
    /* Sanity check */
@@ -163,34 +164,30 @@ init_dev(JCR *jcr, DEVRES *device)
    }
    dev->device = device;
 
-   if (tape) {
-      dev->state |= ST_TAPE;
-   } else if (fifo) {
-      dev->state |= ST_FIFO;
+   if (dev->is_fifo()) {
       dev->capabilities |= CAP_STREAM; /* set stream device */
-   } else {
-      dev->state |= ST_FILE;
    }
 
    /* If the device requires mount :
     * - Check that the mount point is available 
     * - Check that (un)mount commands are defined
     */
-   if (dev->is_file() && dev->requires_mount()) {
-      if (stat(device->mount_point, &statp) < 0) {
+   if ((dev->is_file() || dev->is_dvd()) && dev->requires_mount()) {
+      if (!device->mount_point || stat(device->mount_point, &statp) < 0) {
          berrno be;
          dev->dev_errno = errno;
          Jmsg2(jcr, M_ERROR, 0, _("Unable to stat mount point %s: ERR=%s\n"), 
             device->mount_point, be.strerror());
          return NULL;
       }
+   }
+   if (dev->is_dvd()) {
       if (!device->mount_command || !device->unmount_command) {
          Jmsg0(jcr, M_ERROR_TERM, 0, _("Mount and unmount commands must defined for a device which requires mount.\n"));
       }
       if (!device->write_part_command) {
          Jmsg0(jcr, M_ERROR_TERM, 0, _("Write part command must be defined for a device which requires mount.\n"));
       }
-      dev->state |= ST_DVD;
    }
 
    if (dev->max_block_size > 1000000) {
@@ -237,7 +234,7 @@ init_dev(JCR *jcr, DEVRES *device)
       Jmsg0(jcr, M_ERROR_TERM, 0, dev->errmsg);
    }
 
-   dev->fd = -1;
+   dev->clear_opened();
    dev->attached_dcrs = New(dlist(dcr, &dcr->dev_link));
    Dmsg2(29, "init_dev: tape=%d dev_name=%s\n", dev->is_tape(), dev->dev_name);
    
@@ -260,23 +257,27 @@ init_dev(JCR *jcr, DEVRES *device)
 int
 DEVICE::open(DCR *dcr, int omode)
 {
+   int preserve = 0;
    if (is_open()) {
       if (openmode == omode) {
          return fd;
       } else {
         ::close(fd); /* use system close so correct mode will be used on open */
+        clear_opened();
+        Dmsg0(100, "Close fd for mode change.\n");
+        preserve = state & (ST_LABEL|ST_APPEND|ST_READ);
       }
    }
-  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(),
          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(omode);
+      open_tape_device(dcr, omode);
    } else if (is_dvd()) {
       Dmsg1(100, "call open_dvd_device mode=%s\n", mode_to_str(omode));
       open_dvd_device(dcr, omode);
@@ -284,6 +285,7 @@ DEVICE::open(DCR *dcr, int omode)
       Dmsg1(100, "call open_file_device mode=%s\n", mode_to_str(omode));
       open_file_device(omode);
    }
+   state |= preserve;                 /* reset any important state info */
    return fd;
 }
 
@@ -311,51 +313,45 @@ void DEVICE::set_mode(int new_mode)
    }
 }
 
-void DEVICE::open_tape_device(int omode) 
+/*
+ */
+void DEVICE::open_tape_device(DCR *dcr, int omode) 
 {
-   int nonblocking = 0;;
    file_size = 0;
    int timeout;
-   int ioerrcnt = 10;
+   int nonblocking = O_NONBLOCK;
    Dmsg0(29, "open dev: device is tape\n");
 
+   if (is_autochanger()) {
+      get_autochanger_loaded_slot(dcr);
+   }
+
    set_mode(omode);
    timeout = max_open_wait;
    errno = 0;
-   if (open_nowait) {
-       /* Set wait counters to zero for no wait */
-       timeout = ioerrcnt = 0;
-       /* Open drive in non-block mode */
-       nonblocking = O_NONBLOCK;
-   }
    if (is_fifo() && timeout) {
       /* Set open timer */
       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", print_name());
+   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, MODE_RW+nonblocking)) < 0) {
+   while ((fd = ::open(dev_name, mode+nonblocking)) < 0) {
       berrno be;
-      Dmsg2(500, "Open error errno=%d ERR=%s\n", errno, be.strerror());
-      if (errno == EINTR || errno == EAGAIN) {
-         Dmsg0(500, "Continue open\n");
+      dev_errno = errno;
+      Dmsg5(050, "Open omode=%d mode=%x nonblock=%d error errno=%d ERR=%s\n", 
+           omode, mode, nonblocking, errno, be.strerror());
+      if (dev_errno == EINTR || dev_errno == EAGAIN) {
+         Dmsg0(100, "Continue open\n");
          continue;
       }
       /* Busy wait for specified time (default = 5 mins) */
-      if (errno == EBUSY && timeout-- > 0) {
+      if (dev_errno == EBUSY && timeout-- > 0) {
          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");
-         continue;
-      }
-      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 */
@@ -363,25 +359,21 @@ open_again:
          stop_thread_timer(tid);
          tid = 0;
       }
-      Emsg0(M_FATAL, 0, errmsg);
+      Jmsg0(dcr->jcr, M_FATAL, 0, errmsg);
       break;
    }
+
    if (fd >= 0) {
-      /* If opened in non-block mode, close it an open it normally */
-      if (nonblocking) {
-         nonblocking = 0;
-         ::close(fd);                /* use system close() */
-         goto open_again;
-      }
       openmode = omode;              /* save open mode */
+      set_blocking();   
       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 */
-      Dmsg0(500, "Open OK\n");
+   } else {
+      clear_opened();
    }
+
    /* Stop any open() timer we started */
    if (tid) {
       stop_thread_timer(tid);
@@ -390,6 +382,25 @@ open_again:
    Dmsg1(29, "open dev: tape %d opened\n", fd);
 }
 
+void DEVICE::set_blocking()
+{
+   int oflags;
+   /* Try to reset blocking */
+#ifdef xxx
+   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);       
+      Dmsg2(100, "fcntl error. ERR=%s. Close-reopen fd=%d\n", be.strerror(), fd);
+   }
+#endif
+   oflags = fcntl(fd, F_GETFL, 0);       
+   if (oflags > 0 && (oflags & O_NONBLOCK)) {
+      fcntl(fd, F_SETFL, oflags & ~O_NONBLOCK);
+   }
+}
+
 /*
  * Open a file device
  */
@@ -400,21 +411,27 @@ void DEVICE::open_file_device(int omode)
    /*
     * Handle opening of File Archive (not a tape)
     */     
-   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 (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_name);
-   if (archive_name.c_str()[strlen(archive_name.c_str())-1] != '/') {
-      pm_strcat(archive_name, "/");
+   /*  
+    * If this is a virtual autochanger (i.e. changer_res != NULL)
+    *  we simply use the deviced name, assuming it has been
+    *  appropriately setup by the "autochanger".
+    */
+   if (!device->changer_res) {
+      if (VolCatInfo.VolCatName[0] == 0) {
+         Mmsg(errmsg, _("Could not open file device %s. No Volume name given.\n"),
+            print_name());
+         clear_opened();
+         return;
+      }
+
+      if (archive_name.c_str()[strlen(archive_name.c_str())-1] != '/') {
+         pm_strcat(archive_name, "/");
+      }
+      pm_strcat(archive_name, VolCatInfo.VolCatName);
    }
-   pm_strcat(archive_name, VolCatInfo.VolCatName);
+
          
    Dmsg3(29, "open dev: %s dev=%s mode=%s\n", is_dvd()?"DVD":"disk",
          archive_name.c_str(), mode_to_str(omode));
@@ -435,8 +452,6 @@ void DEVICE::open_file_device(int omode)
       Emsg0(M_FATAL, 0, errmsg);
    } else {
       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", 
@@ -445,7 +460,7 @@ void DEVICE::open_file_device(int omode)
 }
 
 /*
- * 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.
  */
@@ -461,9 +476,11 @@ void DEVICE::open_dvd_device(DCR *dcr, int omode)
          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;
+      clear_opened();
       return;
    }
 
@@ -471,18 +488,55 @@ void DEVICE::open_dvd_device(DCR *dcr, int omode)
       file_size = 0;
    }
    part_size = 0;
-   
 
-   if (!mount_dev(this, 1)) {
-      Mmsg(errmsg, _("Could not mount device %s.\n"), print_name());
-      Emsg0(M_FATAL, 0, errmsg);
-      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. */
+            clear_opened();
+            return;
+         }
+      }
    }
-         
-   Dmsg5(29, "open dev: %s dev=%s mode=%s part=%d npart=%d\n", 
+   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);
+            clear_opened();
+            return;
+         }
+      }
+      else {
+         Mmsg(errmsg, _("Could not mount device %s.\n"), print_name());
+         Emsg0(M_FATAL, 0, errmsg);
+         clear_opened();
+         return;
+      }
+   }
+   
+   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);
+      part, num_parts, dcr->VolCatInfo.VolCatParts);
    openmode = omode;
    Dmsg2(100, "openmode=%d %s\n", openmode, mode_to_str(openmode));
    
@@ -492,18 +546,12 @@ void DEVICE::open_dvd_device(DCR *dcr, int omode)
     */
    if (part < num_parts) {
       omode = OPEN_READ_ONLY;
-   }
-   set_mode(omode);
-
-   /* 
-    * 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 (omode == OPEN_READ_ONLY) {
       make_mounted_dvd_filename(this, archive_name);
-   } else {
+   }
+   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(omode), 
@@ -511,15 +559,25 @@ void DEVICE::open_dvd_device(DCR *dcr, int omode)
    /* Use system open() */
    if ((fd = ::open(archive_name.c_str(), mode, 0640)) < 0) {
       berrno be;
-      dev_errno = errno;
       Mmsg2(errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), 
             be.strerror());
+      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) {
-         make_spooled_dvd_filename(this, archive_name);
-        /* Use system open() */
-         fd = ::open(archive_name.c_str(), mode, 0640);  /* try on spool */
+      
+      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);
       }
+      
+      /* 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) {
@@ -532,90 +590,102 @@ void DEVICE::open_dvd_device(DCR *dcr, int omode)
          Dmsg1(29, "open failed: %s", errmsg);
          /* Use system close() */
          ::close(fd);
-         fd = -1;
+         clear_opened();
       } else {
          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)) {
+/*         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 */
+            if (part == 0) {             // we must have opened the first part
                part++;
             }
-         }
+         }*/
       }
    }
 }
 
 
-#ifdef debug_tracing
-#undef rewind_dev
-bool _rewind_dev(char *file, int line, DEVICE *dev)
-{
-   Dmsg3(100, "rewind_dev fd=%d called from %s:%d\n", dev->fd, file, line);
-   return rewind_dev(dev);
-}
-#endif
-
 /*
  * Rewind the device.
  *  Returns: true  on success
  *           false on failure
  */
-bool rewind_dev(DEVICE *dev)
+bool DEVICE::rewind(DCR *dcr)
 {
    struct mtop mt_com;
    unsigned int i;
+   bool first = true;
 
-   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"),
-            dev->print_name());
-      Emsg0(M_ABORT, 0, dev->errmsg);
+   Dmsg3(29, "rewind res=%d fd=%d %s\n", reserved_device, fd, print_name());
+   if (fd < 0) {
+      if (!is_dvd()) { /* In case of major error, the fd is not open on DVD, so we don't want to abort. */
+         dev_errno = EBADF;
+         Mmsg1(errmsg, _("Bad call to rewind. Device %s not open\n"),
+            print_name());
+         Emsg0(M_ABORT, 0, errmsg);
+      }
       return false;
    }
-   dev->state &= ~(ST_EOT|ST_EOF|ST_WEOT);  /* remove EOF/EOT flags */
-   dev->block_num = dev->file = 0;
-   dev->file_size = 0;
-   dev->file_addr = 0;
-   if (dev->is_tape()) {
+   state &= ~(ST_EOT|ST_EOF|ST_WEOT);  /* remove EOF/EOT flags */
+   block_num = file = 0;
+   file_size = 0;
+   file_addr = 0;
+   if (is_tape()) {
       mt_com.mt_op = MTREW;
       mt_com.mt_count = 1;
       /* If we get an I/O error on rewind, it is probably because
        * the drive is actually busy. We loop for (about 5 minutes)
        * retrying every 5 seconds.
        */
-      for (i=dev->max_rewind_wait; ; i -= 5) {
-         if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
+      for (i=max_rewind_wait; ; i -= 5) {
+         if (ioctl(fd, MTIOCTOP, (char *)&mt_com) < 0) {
             berrno be;
-            clrerror_dev(dev, MTREW);
-            if (i == dev->max_rewind_wait) {
+            clrerror_dev(this, MTREW);
+            if (i == max_rewind_wait) {
                Dmsg1(200, "Rewind error, %s. retrying ...\n", be.strerror());
             }
-            if (dev->dev_errno == EIO && i > 0) {
+            /*
+             * This is a gross hack, because if the user has the
+             *   device mounted (i.e. open), then uses mtx to load
+             *   a tape, the current open file descriptor is invalid.
+             *   So, we close the drive and re-open it.
+             */
+            if (first && dcr) {
+               int open_mode = openmode;
+               ::close(fd);
+               clear_opened();
+               open(dcr, open_mode);
+               if (fd < 0) {
+                  return false;
+               }
+               first = false;
+               continue;
+            }
+            if (dev_errno == EIO && i > 0) {
                Dmsg0(200, "Sleeping 5 seconds.\n");
                bmicrosleep(5, 0);
                continue;
             }
-            Mmsg2(dev->errmsg, _("Rewind error on %s. ERR=%s.\n"),
-               dev->print_name(), be.strerror());
+            Mmsg2(errmsg, _("Rewind error on %s. ERR=%s.\n"),
+               print_name(), be.strerror());
             return false;
          }
          break;
       }
-   } else if (dev->is_file()) {      
-      if (lseek_dev(dev, (off_t)0, SEEK_SET) < 0) {
+   } else if (is_file()) {      
+      if (lseek_dev(this, (off_t)0, SEEK_SET) < 0) {
          berrno be;
-         dev->dev_errno = errno;
-         Mmsg2(dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
-            dev->print_name(), be.strerror());
+         dev_errno = errno;
+         Mmsg2(errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
+            print_name(), be.strerror());
          return false;
       }
    }
@@ -654,7 +724,7 @@ const char *DEVICE::print_blocked() const
    case BST_MOUNT:
       return "BST_MOUNT";
    default:
-      return "unknown blocked code";
+      return _("unknown blocked code");
    }
 }
 
@@ -665,7 +735,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;
@@ -714,7 +786,7 @@ eod_dev(DEVICE *dev)
    dev->block_num = dev->file = 0;
    dev->file_size = 0;
    dev->file_addr = 0;
-   if (dev->state & (ST_FIFO | ST_PROG)) {
+   if (dev->is_fifo() || dev->is_prog()) {
       return true;
    }
    if (!dev->is_tape()) {
@@ -736,7 +808,7 @@ eod_dev(DEVICE *dev)
       Dmsg0(100,"Using FAST FSF for EOM\n");
       /* If unknown position, rewind */
       if (!dev_get_os_pos(dev, &mt_stat)) {
-        if (!rewind_dev(dev)) {
+        if (!dev->rewind(NULL)) {
           return false;
         }
       }
@@ -785,7 +857,7 @@ eod_dev(DEVICE *dev)
       /*
        * Rewind then use FSF until EOT reached
        */
-      if (!rewind_dev(dev)) {
+      if (!dev->rewind(NULL)) {
          return false;
       }
       /*
@@ -799,18 +871,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;
          }
       }
    }
@@ -838,7 +909,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
@@ -863,7 +934,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;
@@ -899,8 +970,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;
@@ -908,7 +979,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)) {
@@ -949,9 +1020,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;
@@ -1014,14 +1085,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 */
@@ -1063,7 +1128,7 @@ bool offline_or_rewind_dev(DEVICE *dev)
     *  done, all future references to the drive get and I/O error.
     */
       clrerror_dev(dev, MTREW);
-      return rewind_dev(dev);
+      return dev->rewind(NULL);
    }
 }
 
@@ -1384,7 +1449,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;
@@ -1401,8 +1466,8 @@ reposition_dev(DEVICE *dev, uint32_t file, uint32_t block)
    Dmsg4(100, "reposition_dev from %u:%u to %u:%u\n",
       dev->file, dev->block_num, file, block);
    if (file < dev->file) {
-      Dmsg0(100, "Rewind_dev\n");
-      if (!rewind_dev(dev)) {
+      Dmsg0(100, "Rewind\n");
+      if (!dev->rewind(NULL)) {
          return false;
       }
    }
@@ -1515,7 +1580,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";
@@ -1562,7 +1627,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;
       }
@@ -1623,9 +1688,9 @@ int flush_dev(DEVICE *dev)
 static void do_close(DEVICE *dev)
 {
 
-   Dmsg1(29, "really close_dev %s\n", dev->print_name());
+   Dmsg1(100, "really close_dev %s\n", dev->print_name());
    if (dev->fd >= 0) {
-      close(dev->fd);
+      ::close(dev->fd);
    }
 
    if (!unmount_dev(dev, 1)) {
@@ -1633,7 +1698,7 @@ static void do_close(DEVICE *dev)
    }
    
    /* 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;
@@ -1647,8 +1712,8 @@ static void do_close(DEVICE *dev)
    }
    
    /* Clean up device packet so it can be reused */
-   dev->fd = -1;
-   dev->state &= ~(ST_OPENED|ST_LABEL|ST_READ|ST_APPEND|ST_EOT|ST_WEOT|ST_EOF);
+   dev->clear_opened();
+   dev->state &= ~(ST_LABEL|ST_READ|ST_APPEND|ST_EOT|ST_WEOT|ST_EOF);
    dev->label_type = B_BACULA_LABEL;
    dev->file = dev->block_num = 0;
    dev->file_size = 0;
@@ -1658,13 +1723,13 @@ static void do_close(DEVICE *dev)
    dev->part_size = 0;
    dev->part_start = 0;
    dev->EndFile = dev->EndBlock = 0;
+   free_volume(dev);
    memset(&dev->VolCatInfo, 0, sizeof(dev->VolCatInfo));
    memset(&dev->VolHdr, 0, sizeof(dev->VolHdr));
    if (dev->tid) {
       stop_thread_timer(dev->tid);
       dev->tid = 0;
    }
-   dev->use_count = 0;
    dev->openmode = 0;
 }
 
@@ -1673,24 +1738,14 @@ static void do_close(DEVICE *dev)
  */
 void DEVICE::close()
 {
-   /*if (fd >= 0 && use_count == 1) {*/
-   /* No need to check if fd >= 0: it is checked again
-    * in do_close, and do_close MUST be called for volumes
-    * splitted in parts, even if fd == -1. */
-   if (use_count == 1) {
-      do_close(this);
-   } else if (use_count > 0) {
-      use_count--;
-   }
-
-#ifdef FULL_DEBUG
-   ASSERT(use_count >= 0);
-#endif
+   do_close(this);
 }
 
 
-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 */
@@ -1698,8 +1753,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) {