]> 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 1bdd4b9f8589626a7dc40540f1008c662aafb643..294c17d9c33cda68bc667f92c6506eb5768a683a 100644 (file)
@@ -2,18 +2,18 @@
  *
  *   dev.c  -- low level operations on device (storage device)
  *
- *             Kern Sibbald, MM
+ *              Kern Sibbald, MM
  *
  *     NOTE!!!! None of these routines are reentrant. You must
- *       use lock_device() and unlock_device() at a higher level,
- *       or use the xxx_device() equivalents.  By moving the
- *       thread synchronization to a higher level, we permit
+ *        use lock_device() and unlock_device() at a higher level,
+ *        or use the xxx_device() equivalents.  By moving the
+ *        thread synchronization to a higher level, we permit
  *        the higher level routines to "seize" the device and
- *       to carry out operations without worrying about who
- *       set what lock (i.e. race conditions).
+ *        to carry out operations without worrying about who
+ *        set what lock (i.e. race conditions).
  *
  *     Note, this is the device dependent code, and my have
- *          to be modified for each system, but is meant to
+ *           to be modified for each system, but is meant to
  *           be as "generic" as possible.
  *
  *     The purpose of this code is to develop a SIMPLE Storage
  *   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 as
-   published by the Free Software Foundation; either version 2 of
-   the License, or (at your option) any later version.
+   modify it under the terms of the GNU General Public License
+   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,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-   General Public License for more details.
-
-   You should have received a copy of the GNU General Public
-   License along with this program; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
-   MA 02111-1307, USA.
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 
+   the file LICENSE for additional details.
 
  */
 
 #include "bacula.h"
 #include "stored.h"
 
+#ifndef O_NONBLOCK 
+#define O_NONBLOCK 0
+#endif
+
 /* Forward referenced functions */
 void set_os_device_parameters(DEVICE *dev);
-int mount_dev(DEVICE* dev, int timeout);
-int unmount_dev(DEVICE* dev, int timeout);
-int write_part(DEVICE *dev);
-char *edit_device_codes_dev(DEVICE* dev, char *omsg, const char *imsg);
-static void get_filename(DEVICE *dev, char *VolName, POOL_MEM& archive_name);
-static void update_free_space_dev(DEVICE* dev);
+static bool dev_get_os_pos(DEVICE *dev, struct mtget *mt_stat);
+static char *mode_to_str(int mode);
 
 /*
  * Allocate and initialize the DEVICE structure
@@ -101,52 +96,49 @@ static void update_free_space_dev(DEVICE* dev);
  *
  */
 DEVICE *
-init_dev(DEVICE *dev, DEVRES *device)
+init_dev(JCR *jcr, DEVRES *device)
 {
    struct stat statp;
-   bool tape, fifo;
    int errstat;
    DCR *dcr = NULL;
-
-   /* Check that device is available */
-   if (stat(device->device_name, &statp) < 0) {
-      berrno be;
-      if (dev) {
-        dev->dev_errno = errno;
-      }
-      Jmsg2(NULL, M_FATAL, 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 (dev) {
-        dev->dev_errno = ENODEV;
+   DEVICE *dev;
+
+
+   /* 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;
       }
-      Emsg2(M_FATAL, 0, _("%s is an unknown device type. Must be tape or directory. st_mode=%x\n"),
-        device->device_name, statp.st_mode);
-      return NULL;
-   }
-   if (!dev) {
-      dev = (DEVICE *)get_memory(sizeof(DEVICE));
-      memset(dev, 0, sizeof(DEVICE));
-      dev->state = ST_MALLOC;
-   } else {
-      memset(dev, 0, sizeof(DEVICE));
    }
 
+   dev = (DEVICE *)get_memory(sizeof(DEVICE));
+   memset(dev, 0, sizeof(DEVICE));
+   dev->state = ST_MALLOC;
+
    /* Copy user supplied device parameters from Resource */
    dev->dev_name = get_memory(strlen(device->device_name)+1);
    pm_strcpy(dev->dev_name, device->device_name);
+   dev->prt_name = get_memory(strlen(device->device_name) + strlen(device->hdr.name) + 20);
+   /* We edit "Resource-name" (physical-name) */
+   Mmsg(dev->prt_name, "\"%s\" (%s)", device->hdr.name, device->device_name);
    dev->capabilities = device->cap_bits;
    dev->min_block_size = device->min_block_size;
    dev->max_block_size = device->max_block_size;
@@ -159,10 +151,11 @@ init_dev(DEVICE *dev, DEVRES *device)
    dev->vol_poll_interval = device->vol_poll_interval;
    dev->max_spool_size = device->max_spool_size;
    dev->drive_index = device->drive_index;
-   if (tape) { /* No parts on tapes */
-      dev->max_part_size = 0;
-   }
-   else {
+   dev->autoselect = device->autoselect;
+   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 */
@@ -171,44 +164,40 @@ init_dev(DEVICE *dev, 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() && device->cap_bits & CAP_REQMOUNT) {
-      if (stat(device->mount_point, &statp) < 0) {
-        berrno be;
-        dev->dev_errno = errno;
-         Jmsg2(NULL, M_FATAL, 0, _("Unable to stat mount point %s: ERR=%s\n"), 
-           device->mount_point, be.strerror());
-        return NULL;
+   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(NULL, M_ERROR_TERM, 0, _("Mount and unmount commands must defined for a device which requires mount.\n"));
+         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(NULL, M_ERROR_TERM, 0, _("Write part command must be defined for a device which requires mount.\n"));
+         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) {
-      Emsg3(M_ERROR, 0, _("Block size %u on device %s is too large, using default %u\n"),
-        dev->max_block_size, dev->dev_name, DEFAULT_BLOCK_SIZE);
+      Jmsg3(jcr, M_ERROR, 0, _("Block size %u on device %s is too large, using default %u\n"),
+         dev->max_block_size, dev->print_name(), DEFAULT_BLOCK_SIZE);
       dev->max_block_size = 0;
    }
    if (dev->max_block_size % TAPE_BSIZE != 0) {
-      Emsg2(M_WARNING, 0, _("Max block size %u not multiple of device %s block size.\n"),
-        dev->max_block_size, dev->dev_name);
+      Jmsg2(jcr, M_WARNING, 0, _("Max block size %u not multiple of device %s block size.\n"),
+         dev->max_block_size, dev->print_name());
    }
 
    dev->errmsg = get_pool_memory(PM_EMSG);
@@ -218,82 +207,46 @@ init_dev(DEVICE *dev, DEVRES *device)
       berrno be;
       dev->dev_errno = errstat;
       Mmsg1(dev->errmsg, _("Unable to init mutex: ERR=%s\n"), be.strerror(errstat));
-      Emsg0(M_FATAL, 0, dev->errmsg);
+      Jmsg0(jcr, M_ERROR_TERM, 0, dev->errmsg);
    }
    if ((errstat = pthread_cond_init(&dev->wait, NULL)) != 0) {
       berrno be;
       dev->dev_errno = errstat;
       Mmsg1(dev->errmsg, _("Unable to init cond variable: ERR=%s\n"), be.strerror(errstat));
-      Emsg0(M_ERROR_TERM, 0, dev->errmsg);
+      Jmsg0(jcr, M_ERROR_TERM, 0, dev->errmsg);
    }
    if ((errstat = pthread_cond_init(&dev->wait_next_vol, NULL)) != 0) {
       berrno be;
       dev->dev_errno = errstat;
       Mmsg1(dev->errmsg, _("Unable to init cond variable: ERR=%s\n"), be.strerror(errstat));
-      Emsg0(M_ERROR_TERM, 0, dev->errmsg);
+      Jmsg0(jcr, M_ERROR_TERM, 0, dev->errmsg);
    }
    if ((errstat = pthread_mutex_init(&dev->spool_mutex, NULL)) != 0) {
       berrno be;
       dev->dev_errno = errstat;
       Mmsg1(dev->errmsg, _("Unable to init mutex: ERR=%s\n"), be.strerror(errstat));
-      Emsg0(M_ERROR_TERM, 0, dev->errmsg);
+      Jmsg0(jcr, M_ERROR_TERM, 0, dev->errmsg);
    }
    if ((errstat = rwl_init(&dev->lock)) != 0) {
       berrno be;
       dev->dev_errno = errstat;
       Mmsg1(dev->errmsg, _("Unable to init mutex: ERR=%s\n"), be.strerror(errstat));
-      Emsg0(M_ERROR_TERM, 0, dev->errmsg);
+      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->dev_name);
+   Dmsg2(29, "init_dev: tape=%d dev_name=%s\n", dev->is_tape(), dev->dev_name);
    
    return dev;
 }
 
-/* 
- * Write the current volume/part filename to archive_name.
- */
-static void get_filename(DEVICE *dev, char *VolName, POOL_MEM& archive_name) 
-{
-   char partnumber[20];
-   
-   if (dev->is_dvd()) {
-        /* If we try to open the last part, just open it from disk, 
-        * otherwise, open it from the spooling directory */
-      if (dev->part < dev->num_parts) {
-        pm_strcpy(archive_name, dev->device->mount_point);
-      } else {
-        /* Use the working directory if spool directory is not defined */
-        if (dev->device->spool_directory) {
-           pm_strcpy(archive_name, dev->device->spool_directory);
-        } else {
-           pm_strcpy(archive_name, working_directory);
-        }
-      }
-   } else {
-      pm_strcpy(archive_name, dev->dev_name);
-   }
-      
-   if (archive_name.c_str()[strlen(archive_name.c_str())-1] != '/') {
-      pm_strcat(archive_name, "/");
-   }
-   pm_strcat(archive_name, VolName);
-   /* if part != 0, append .# to the filename (where # is the part number) */
-   if (dev->is_dvd() && dev->part != 0) {
-      pm_strcat(archive_name, ".");
-      bsnprintf(partnumber, sizeof(partnumber), "%d", dev->part);
-      pm_strcat(archive_name, partnumber);
-   }
-}  
-
 /*
  * Open the device with the operating system and
  * initialize buffer pointers.
  *
  * Returns:  -1  on error
- *          fd  on success
+ *           fd  on success
  *
  * Note, for a tape, the VolName is the name we give to the
  *    volume (not really used here), but for a file, the
@@ -302,839 +255,562 @@ static void get_filename(DEVICE *dev, char *VolName, POOL_MEM& archive_name)
  *    (archive_name) with the VolName concatenated.
  */
 int
-open_dev(DEVICE *dev, char *VolName, int mode)
+DEVICE::open(DCR *dcr, int omode)
 {
-   if (dev->is_open()) {
-      /*
-       *  *****FIXME***** how to handle two threads wanting
-       *  different volumes mounted???? E.g. one is waiting
-       *  for the next volume to be mounted, and a new job
-       *  starts and snatches up the device.
-       */
-      if (VolName && strcmp(dev->VolCatInfo.VolCatName, VolName) != 0) {
-        return -1;
-      }
-      dev->use_count++;
-      Mmsg2(&dev->errmsg, _("WARNING!!!! device %s opened %d times!!!\n"),
-           dev->dev_name, dev->use_count);
-      Emsg1(M_WARNING, 0, "%s", dev->errmsg);
-      return dev->fd;
-   }
-   if (VolName) {
-      bstrncpy(dev->VolCatInfo.VolCatName, VolName, sizeof(dev->VolCatInfo.VolCatName));
-   }
-
-   Dmsg3(29, "open_dev: tape=%d dev_name=%s vol=%s\n", dev_is_tape(dev),
-        dev->dev_name, dev->VolCatInfo.VolCatName);
-   dev->state &= ~(ST_LABEL|ST_APPEND|ST_READ|ST_EOT|ST_WEOT|ST_EOF);
-   if (dev->is_tape() || dev->is_fifo()) {
-      dev->file_size = 0;
-      int timeout;
-      Dmsg0(29, "open_dev: device is tape\n");
-      if (mode == OPEN_READ_WRITE) {
-        dev->mode = O_RDWR | O_BINARY;
-      } else if (mode == OPEN_READ_ONLY) {
-        dev->mode = O_RDONLY | O_BINARY;
-      } else if (mode == OPEN_WRITE_ONLY) {
-        dev->mode = O_WRONLY | O_BINARY;
+   int preserve = 0;
+   if (is_open()) {
+      if (openmode == omode) {
+         return fd;
       } else {
-         Emsg0(M_ABORT, 0, _("Illegal mode given to open_dev.\n"));
-      }
-      timeout = dev->max_open_wait;
-      errno = 0;
-      if (dev->is_fifo() && timeout) {
-        /* Set open timer */
-        dev->tid = start_thread_timer(pthread_self(), timeout);
-      }
-      /* If busy retry each second for max_open_wait seconds */
-      while ((dev->fd = open(dev->dev_name, dev->mode, MODE_RW)) < 0) {
-        berrno be;
-        if (errno == EINTR || errno == EAGAIN) {
-           continue;
-        }
-        if (errno == EBUSY && timeout-- > 0) {
-            Dmsg2(100, "Device %s busy. ERR=%s\n", dev->dev_name, be.strerror());
-           bmicrosleep(1, 0);
-           continue;
-        }
-        dev->dev_errno = errno;
-         Mmsg2(&dev->errmsg, _("stored: unable to open device %s: ERR=%s\n"),
-              dev->dev_name, be.strerror());
-        /* Stop any open timer we set */
-        if (dev->tid) {
-           stop_thread_timer(dev->tid);
-           dev->tid = 0;
-        }
-        Emsg0(M_FATAL, 0, dev->errmsg);
-        break;
-      }
-      if (dev->fd >= 0) {
-        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 */
-      }
-      /* Stop any open() timer we started */
-      if (dev->tid) {
-        stop_thread_timer(dev->tid);
-        dev->tid = 0;
-      }
-      Dmsg1(29, "open_dev: tape %d opened\n", dev->fd);
-   } else {
-      POOL_MEM archive_name(PM_FNAME);
-      struct stat filestat;
-      /*
-       * Handle opening of File Archive (not a tape)
-       */     
-      if (dev->part == 0) {
-        dev->file_size = 0;
-      }
-      dev->part_size = 0;
-      
-      /* if num_parts has not been set, but VolCatInfo is available, copy
-       * it from the VolCatInfo.VolCatParts */
-      if (dev->num_parts < dev->VolCatInfo.VolCatParts) {
-        dev->num_parts = dev->VolCatInfo.VolCatParts;
-      }
-      
-      if (VolName == NULL || *VolName == 0) {
-         Mmsg(dev->errmsg, _("Could not open file device %s. No Volume name given.\n"),
-           dev->dev_name);
-        return -1;
-      }
-      get_filename(dev, VolName, archive_name);
-
-      if (dev->is_dvd()) {
-        if (mount_dev(dev, 1) < 0) {
-            Mmsg(dev->errmsg, _("Could not mount archive device %s.\n"),
-                dev->dev_name);
-           Emsg0(M_FATAL, 0, dev->errmsg);
-           dev->fd = -1;
-           return dev->fd;
-        }
-      }
-           
-      Dmsg2(29, "open_dev: device is disk %s (mode:%d)\n", archive_name.c_str(), mode);
-      dev->openmode = mode;
-      
-      /*
-       * 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 (mode == OPEN_READ_WRITE) {
-        dev->mode = O_CREAT | O_RDWR | O_BINARY;
-      } else if (mode == OPEN_READ_ONLY) {
-        dev->mode = O_RDONLY | O_BINARY;
-      } else if (mode == OPEN_WRITE_ONLY) {
-        dev->mode = O_WRONLY | O_BINARY;
-      } else {
-         Emsg0(M_ABORT, 0, _("Illegal mode given to open_dev.\n"));
-      }
-      /* If creating file, give 0640 permissions */
-      if ((dev->fd = open(archive_name.c_str(), dev->mode, 0640)) < 0) {
-        berrno be;
-        dev->dev_errno = errno;
-         Mmsg2(&dev->errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), be.strerror());
-        Emsg0(M_FATAL, 0, dev->errmsg);
-      } else {
-        dev->dev_errno = 0;
-        dev->state |= ST_OPENED;
-        dev->use_count = 1;
-        update_pos_dev(dev);                /* update position */
-        if (fstat(dev->fd, &filestat) < 0) {
-           berrno be;
-           dev->dev_errno = errno;
-            Mmsg2(&dev->errmsg, _("Could not fstat: %s, ERR=%s\n"), archive_name.c_str(), be.strerror());
-           Emsg0(M_FATAL, 0, dev->errmsg);
-        } else {
-           dev->part_size = filestat.st_size;
-        }
-      }
-      Dmsg4(29, "open_dev: disk fd=%d opened, part=%d/%d, part_size=%u\n", dev->fd, dev->part, dev->num_parts, dev->part_size);
-      if (dev->is_dvd() && (dev->mode != OPEN_READ_ONLY) && 
-         ((dev->free_space_errno == 0) || (dev->num_parts == dev->part))) {
-        update_free_space_dev(dev);
+        ::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);
       }
    }
-   return dev->fd;
-}
-
-/* (Un)mount the device */
-int do_mount_dev(DEVICE* dev, int mount, int dotimeout) {
-   POOL_MEM ocmd(PM_FNAME);
-   POOLMEM* results;
-   results = get_pool_memory(PM_MESSAGE);
-   char* icmd;
-   int status, timeout;
-   
-   if (mount) {
-      icmd = dev->device->mount_command;
+   if (dcr) {
+      bstrncpy(VolCatInfo.VolCatName, dcr->VolumeName, sizeof(VolCatInfo.VolCatName));
    }
-   else {
-      icmd = dev->device->unmount_command;
-   }
-   
-   edit_device_codes_dev(dev, ocmd.c_str(), icmd);
-   
-   Dmsg2(29, "do_mount_dev: cmd=%s state=%d\n", ocmd.c_str(), dev->state & ST_MOUNTED);
 
-   if (dotimeout) {
-      /* Try at most 5 times to (un)mount the device. This should perhaps be configurable. */
-      timeout = 5;
+   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(dcr, omode);
+   } else if (is_dvd()) {
+      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=%s\n", mode_to_str(omode));
+      open_file_device(omode);
    }
-   else {
-      timeout = 0;
-   }
-   /* If busy retry each second */
-   while ((status = run_program_full_output(ocmd.c_str(), dev->max_open_wait/2, results)) != 0) {
-      if (--timeout > 0) {
-         Dmsg2(40, "Device %s cannot be (un)mounted. Retrying... ERR=%s\n", dev->dev_name, results);
-        /* Sometimes the device cannot be mounted because it is already mounted.
-         * Try to unmount it, then remount it */
-        if (mount) {
-            Dmsg1(40, "Trying to unmount the device %s...\n", dev->dev_name);
-           do_mount_dev(dev, 0, 0);
-        }
-        bmicrosleep(1, 0);
-        continue;
+   state |= preserve;                 /* reset any important state info */
+   return fd;
+}
+
+void DEVICE::set_mode(int new_mode) 
+{
+   switch (new_mode) {
+   case CREATE_READ_WRITE:
+      mode = O_CREAT | O_RDWR | O_BINARY;
+      break;
+   case OPEN_READ_WRITE:
+      if (is_dvd() || is_file()) {
+         mode = O_CREAT | O_RDWR | O_BINARY;
+      } else {
+         mode = O_RDWR | O_BINARY;
       }
-      free_pool_memory(results);
-      Dmsg2(40, "Device %s cannot be mounted. ERR=%s\n", dev->dev_name, results);
-      return -1;
-   }
-   
-   if (mount) {
-     dev->state |= ST_MOUNTED;
-   }
-   else {
-     dev->state &= ~ST_MOUNTED;
+      break;
+   case OPEN_READ_ONLY:
+      mode = O_RDONLY | O_BINARY;
+      break;
+   case OPEN_WRITE_ONLY:
+      mode = O_WRONLY | O_BINARY;
+      break;
+   default:
+      Emsg0(M_ABORT, 0, _("Illegal mode given to open dev.\n"));
    }
-   free_pool_memory(results);
-   
-   Dmsg1(29, "do_mount_dev: end_state=%d\n", dev->state & ST_MOUNTED);
-   return 0;
 }
 
-/* Only for devices that requires mount.
- * Try to find the volume name of the loaded device, and open the
- * first part of this volume. 
- *
- * Returns 0 if read_dev_volume_label can now read the label,
- * -1 if an error occured, and read_dev_volume_label_guess must abort with an IO_ERROR.
- *
- * To guess the device name, it lists all the files on the DVD, and searches for a 
- * file which has a minimum size (500 bytes). If this file has a numeric extension,
- * like part files, try to open the file which has no extension (e.g. the first
- * part file).
- * So, if the DVD does not contains a Bacula volume, a random file is opened,
- * and no valid label could be read from this file.
- *
- * It is useful, so the operator can be told that a wrong volume is mounted, with
- * the label name of the current volume. We can also check that the currently
- * mounted disk is writable. (See also read_dev_volume_label_guess in label.c).
- *
- * Note that if the right volume is mounted, open_guess_name_dev returns the same
- * result as an usual open_dev.
+/*
  */
-int open_guess_name_dev(DEVICE *dev
+void DEVICE::open_tape_device(DCR *dcr, int omode
 {
-   Dmsg1(29, "open_guess_name_dev: dev=%s\n", dev->dev_name);
-   POOL_MEM guessedname(PM_FNAME);
-   DIR* dp;
-   struct dirent *entry, *result;
-   struct stat statp;
-   int index;
-   int name_max;
-   
-   if (!dev->is_dvd()) {
-      Dmsg1(100, "open_guess_name_dev: device does not require mount, returning 0. dev=%s\n", dev->dev_name);
-      return 0;
-   }
-
-#ifndef HAVE_DIRENT_H
-   Dmsg0(29, "open_guess_name_dev: readdir not available, cannot guess volume name\n");
-   return 0; 
-#endif
-   
-   update_free_space_dev(dev);
+   file_size = 0;
+   int timeout;
+   int nonblocking = O_NONBLOCK;
+   Dmsg0(29, "open dev: device is tape\n");
 
-   if (mount_dev(dev, 1) < 0) {
-      /* If the device cannot be mounted, check if it is writable */
-      if (dev->free_space_errno >= 0) {
-         Dmsg1(100, "open_guess_name_dev: device cannot be mounted, but it seems to be writable, returning 0. dev=%s\n", dev->dev_name);
-        return 0;
-      } else {
-         Dmsg1(100, "open_guess_name_dev: device cannot be mounted, and is not writable, returning 0. dev=%s\n", dev->dev_name);
-        /* read_dev_volume_label_guess must now check dev->free_space_errno to understand that the media is not writable. */
-        return 0;
-      }
+   if (is_autochanger()) {
+      get_autochanger_loaded_slot(dcr);
    }
-      
-   name_max = pathconf(".", _PC_NAME_MAX);
-   if (name_max < 1024) {
-      name_max = 1024;
+
+   set_mode(omode);
+   timeout = max_open_wait;
+   errno = 0;
+   if (is_fifo() && timeout) {
+      /* Set open timer */
+      tid = start_thread_timer(pthread_self(), timeout);
    }
-      
-   if (!(dp = opendir(dev->device->mount_point))) {
+   /* If busy retry each second for max_open_wait seconds */
+   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)) < 0) {
       berrno be;
-      dev->dev_errno = errno;
-      Dmsg3(29, "open_guess_name_dev: failed to open dir %s (dev=%s), ERR=%s\n", dev->device->mount_point, dev->dev_name, be.strerror());
-      return -1;
-   }
-   
-   entry = (struct dirent *)malloc(sizeof(struct dirent) + name_max + 100);
-   while (1) {
-      if ((readdir_r(dp, entry, &result) != 0) || (result == NULL)) {
-        dev->dev_errno = ENOENT;
-         Dmsg2(29, "open_guess_name_dev: failed to find suitable file in dir %s (dev=%s)\n", dev->device->mount_point, dev->dev_name);
-        closedir(dp);
-        return -1;
-      }
-      
-      ASSERT(name_max+1 > (int)sizeof(struct dirent) + (int)NAMELEN(entry));
-      
-      if (strcmp(entry->d_name, ".") == 0 || strcmp(entry->d_name, "..") == 0) {
-        continue;
-      }
-      
-      pm_strcpy(guessedname, dev->device->mount_point);
-      if (guessedname.c_str()[strlen(guessedname.c_str())-1] != '/') {
-         pm_strcat(guessedname, "/");
-      }
-      pm_strcat(guessedname, entry->d_name);
-      
-      if (stat(guessedname.c_str(), &statp) < 0) {
-        berrno be;
-         Dmsg3(29, "open_guess_name_dev: failed to stat %s (dev=%s), ERR=%s\n",
-              guessedname.c_str(), dev->dev_name, be.strerror());
-        continue;
-      }
-      
-      if (!S_ISREG(statp.st_mode) || (statp.st_size < 500)) {
-         Dmsg2(100, "open_guess_name_dev: %s is not a regular file, or less than 500 bytes (dev=%s)\n", 
-              guessedname.c_str(), dev->dev_name);
-        continue;
-      }
-      
-      /* Ok, we found a good file, remove the part extension if possible. */
-      for (index = strlen(guessedname.c_str())-1; index >= 0; index--) {
-         if ((guessedname.c_str()[index] == '/') || 
-             (guessedname.c_str()[index] < '0') || 
-             (guessedname.c_str()[index] > '9')) {
-           break;
-        }
-         if (guessedname.c_str()[index] == '.') {
-            guessedname.c_str()[index] = '\0';
-           break;
-        }
-      }
-      
-      if ((stat(guessedname.c_str(), &statp) < 0) || (statp.st_size < 500)) {
-        /* The file with extension truncated does not exists or is too small, so use it with its extension. */
-        berrno be;
-         Dmsg3(100, "open_guess_name_dev: failed to stat %s (dev=%s), using the file with its extension, ERR=%s\n", 
-              guessedname.c_str(), dev->dev_name, be.strerror());
-        pm_strcpy(guessedname, dev->device->mount_point);
-         if (guessedname.c_str()[strlen(guessedname.c_str())-1] != '/') {
-            pm_strcat(guessedname, "/");
-        }
-        pm_strcat(guessedname, entry->d_name);
-        continue;
-      }
+      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 (dev_errno == EBUSY && timeout-- > 0) {
+         Dmsg2(100, "Device %s busy. ERR=%s\n", print_name(), be.strerror());
+         bmicrosleep(1, 0);
+         continue;
+      }
+      Mmsg2(errmsg, _("Unable to open device %s: ERR=%s\n"),
+            print_name(), be.strerror(dev_errno));
+      /* Stop any open timer we set */
+      if (tid) {
+         stop_thread_timer(tid);
+         tid = 0;
+      }
+      Jmsg0(dcr->jcr, M_FATAL, 0, errmsg);
       break;
    }
-   
-   closedir(dp);
-   
-   if (dev->fd >= 0) {
-      close(dev->fd);
-   }
-     
-   if ((dev->fd = open(guessedname.c_str(), O_RDONLY | O_BINARY)) < 0) {
-      berrno be;
-      dev->dev_errno = errno;
-      Dmsg3(29, "open_guess_name_dev: failed to open %s (dev=%s), ERR=%s\n", 
-           guessedname.c_str(), dev->dev_name, be.strerror());
-      if (open_first_part(dev) < 0) {
-        berrno be;
-        dev->dev_errno = errno;
-         Mmsg1(&dev->errmsg, _("Could not open_first_part, ERR=%s\n"), be.strerror());
-        Emsg0(M_FATAL, 0, dev->errmsg);         
-      }
-      return -1;
-   }
-   dev->part_start = 0;
-   dev->part_size = statp.st_size;
-   dev->part = 0;
-   dev->state |= ST_OPENED;
-   dev->use_count = 1;
-   
-   Dmsg2(29, "open_guess_name_dev: %s opened (dev=%s)\n", guessedname.c_str(), dev->dev_name);
-   
-   return 0;
-}
 
-/* Mount the device.
- * If timeout, wait until the mount command returns 0.
- * If !timeout, try to mount the device only once.
- */
-int mount_dev(DEVICE* dev, int timeout) 
-{
-   if (dev->state & ST_MOUNTED) {
-      Dmsg0(100, "mount_dev: Device already mounted\n");
-      return 0;
+   if (fd >= 0) {
+      openmode = omode;              /* save open mode */
+      set_blocking();   
+      Dmsg2(100, "openmode=%d %s\n", openmode, mode_to_str(openmode));
+      dev_errno = 0;
+      update_pos_dev(this);                /* update position */
+      set_os_device_parameters(this);      /* do system dependent stuff */
    } else {
-      return do_mount_dev(dev, 1, timeout);
+      clear_opened();
    }
+
+   /* Stop any open() timer we started */
+   if (tid) {
+      stop_thread_timer(tid);
+      tid = 0;
+   }
+   Dmsg1(29, "open dev: tape %d opened\n", fd);
 }
 
-/* Unmount the device
- * If timeout, wait until the unmount command returns 0.
- * If !timeout, try to unmount the device only once.
- */
-int unmount_dev(DEVICE* dev, int timeout) 
+void DEVICE::set_blocking()
 {
-   if (dev->state & ST_MOUNTED) {
-      return do_mount_dev(dev, 0, timeout);
-   } else {
-      Dmsg0(100, "mount_dev: Device already unmounted\n");
-      return 0;
+   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);
    }
 }
 
-/* Update the free space on the device */
-static void update_free_space_dev(DEVICE* dev) 
+/*
+ * Open a file device
+ */
+void DEVICE::open_file_device(int omode) 
 {
-   POOL_MEM ocmd(PM_FNAME);
-   POOLMEM* results;
-   char* icmd;
-   int timeout;
-   long long int free;
-   
-   icmd = dev->device->free_space_command;
-   
-   if (!icmd) {
-      dev->free_space = 0;
-      dev->free_space_errno = 0;
-      Dmsg2(29, "update_free_space_dev: free_space=%d, free_space_errno=%d (!icmd)\n", dev->free_space, dev->free_space_errno);
-      return;
-   }
-   
-   edit_device_codes_dev(dev, ocmd.c_str(), icmd);
-   
-   Dmsg1(29, "update_free_space_dev: cmd=%s\n", ocmd.c_str());
+   POOL_MEM archive_name(PM_FNAME);
 
-   results = get_pool_memory(PM_MESSAGE);
-   
-   /* Try at most 3 times to get the free space on the device. This should perhaps be configurable. */
-   timeout = 3;
-   
-   while (1) {
-      char ed1[50];
-      if (run_program_full_output(ocmd.c_str(), dev->max_open_wait/2, results) == 0) {
-         Dmsg1(100, "Free space program run : %s\n", results);
-        free = str_to_int64(results);
-        if (free >= 0) {
-           dev->free_space = free;
-           dev->free_space_errno = 1;
-            Mmsg0(dev->errmsg, "");
-           break;
-        }
-      }
-      dev->free_space = 0;
-      dev->free_space_errno = -EPIPE;
-      Mmsg1(dev->errmsg, "Cannot run free space command (%s)\n", results);
-      
-      if (--timeout > 0) {
-         Dmsg4(40, "Cannot get free space on device %s. free_space=%s, "
-            "free_space_errno=%d ERR=%s\n", dev->dev_name, 
-              edit_uint64(dev->free_space, ed1), dev->free_space_errno, 
-              dev->errmsg);
-        bmicrosleep(1, 0);
-        continue;
+   /*
+    * Handle opening of File Archive (not a tape)
+    */     
+
+   pm_strcpy(archive_name, dev_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;
       }
 
-      dev->dev_errno = -dev->free_space_errno;
-      Dmsg4(40, "Cannot get free space on device %s. free_space=%s, "
-         "free_space_errno=%d ERR=%s\n",
-           dev->dev_name, edit_uint64(dev->free_space, ed1),
-           dev->free_space_errno, dev->errmsg);
-      break;
+      if (archive_name.c_str()[strlen(archive_name.c_str())-1] != '/') {
+         pm_strcat(archive_name, "/");
+      }
+      pm_strcat(archive_name, VolCatInfo.VolCatName);
    }
-   
-   free_pool_memory(results);
-   Dmsg2(29, "update_free_space_dev: free_space=%lld, free_space_errno=%d\n", dev->free_space, dev->free_space_errno);
-   return;
-}
 
-int write_part(DEVICE *dev) 
-{
-   Dmsg1(29, "write_part: device is %s\n", dev->dev_name);
-   
-   if (unmount_dev(dev, 1) < 0) {
-      Dmsg0(29, "write_part: unable to unmount the device\n");
-   }
-   
-   POOL_MEM ocmd(PM_FNAME);
-   POOLMEM *results;
-   results = get_pool_memory(PM_MESSAGE);
-   char* icmd;
-   int status;
-   int timeout;
+         
+   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));
    
-   icmd = dev->device->write_part_command;
-   
-   edit_device_codes_dev(dev, ocmd.c_str(), icmd);
-      
-   /* Wait at most the time a maximum size part is written in DVD 0.5x speed
-    * FIXME: Minimum speed should be in device configuration 
-    */
-   timeout = dev->max_open_wait + (dev->max_part_size/(1350*1024/2));
-   
-   Dmsg2(29, "write_part: cmd=%s timeout=%d\n", ocmd.c_str(), timeout);
-      
-   status = run_program_full_output(ocmd.c_str(), timeout, results);
-   if (status != 0) {
-      Mmsg1(dev->errmsg, "Error while writing current part to the DVD: %s", results);
-      dev->dev_errno = EIO;
-      free_pool_memory(results);
-      return -1;
-   }
-   else {
-      Dmsg1(29, "write_part: command output=%s\n", results);
-      POOL_MEM archive_name(PM_FNAME);
-      get_filename(dev, dev->VolCatInfo.VolCatName, archive_name);
-      unlink(archive_name.c_str());
-      free_pool_memory(results);
-      return 0;
+   set_mode(omode);
+   /* If creating file, give 0640 permissions */
+   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_errno = errno;
+      Mmsg2(errmsg, _("Could not open: %s, ERR=%s\n"), archive_name.c_str(), 
+            be.strerror());
+      Dmsg1(29, "open failed: %s", errmsg);
+      Emsg0(M_FATAL, 0, errmsg);
+   } else {
+      dev_errno = 0;
+      update_pos_dev(this);                /* update position */
    }
+   Dmsg5(29, "open dev: %s fd=%d opened, part=%d/%d, part_size=%u\n", 
+      is_dvd()?"DVD":"disk", fd, part, num_parts, 
+      part_size);
 }
 
-/* Open the next part file.
- *  - Close the fd
- *  - Increment part number 
- *  - Reopen the device
+/*
+ * 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.
  */
-int open_next_part(DEVICE *dev) {
-   int state;
-      
-   Dmsg3(29, "open_next_part %s %s %d\n", dev->dev_name, dev->VolCatInfo.VolCatName, dev->openmode);
-   /* When appending, do not open a new part if the current is empty */
-   if (dev->can_append() && (dev->part == dev->num_parts) && 
-       (dev->part_size == 0)) {
-      Dmsg0(29, "open_next_part exited immediately (dev->part_size == 0).\n");
-      return dev->fd;
+void DEVICE::open_dvd_device(DCR *dcr, int omode) 
+{
+   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", 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());
+      clear_opened();
+      return;
    }
-   
-   if (dev->fd >= 0) {
-      close(dev->fd);
+
+   if (part == 0) {
+      file_size = 0;
    }
-   
-   dev->fd = -1;
-   
-   state = dev->state;
-   dev->state &= ~ST_OPENED;
-   
-   if (dev->is_dvd() && (dev->part == dev->num_parts) && dev->can_append()) {
-      if (write_part(dev) < 0) {
-        return -1;
-      }
+   part_size = 0;
+
+   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;
    }
-     
-   dev->part_start += dev->part_size;
-   dev->part++;
-   
-   if ((dev->num_parts < dev->part) && (dev->state & ST_APPEND)) {
-      dev->num_parts = dev->part;
-      
-      /* Check that the next part file does not exists.
-       * If it does, move it away... */
-      POOL_MEM archive_name(PM_FNAME);
-      POOL_MEM archive_bkp_name(PM_FNAME);
-      struct stat buf;
-      
-      get_filename(dev, dev->VolCatInfo.VolCatName, archive_name);
-      
-      /* Check if the next part exists. */
-      if ((stat(archive_name.c_str(), &buf) == 0) || (errno != ENOENT)) {
-         Dmsg1(29, "open_next_part %s is in the way, moving it away...\n", archive_name.c_str());
-        pm_strcpy(archive_bkp_name, archive_name.c_str());
-         pm_strcat(archive_bkp_name, ".bak");
-        unlink(archive_bkp_name.c_str()); 
-        
-        /* First try to rename it */
-        if (rename(archive_name.c_str(), archive_bkp_name.c_str()) < 0) {
-           berrno be;
-            Dmsg3(29, "open_next_part can't rename %s to %s, ERR=%s\n", 
-                 archive_name.c_str(), archive_bkp_name.c_str(), be.strerror());
-           /* Then try to unlink it */
-           if (unlink(archive_name.c_str()) < 0) {
-              berrno be;
-              dev->dev_errno = errno;
-               Mmsg2(&dev->errmsg, _("open_next_part can't unlink existing part %s, ERR=%s\n"), 
-                     archive_name.c_str(), be.strerror());
-              Emsg0(M_FATAL, 0, dev->errmsg);
-              return -1;
-           }
-        }
+
+   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;
+         }
       }
    }
-   
-   if (open_dev(dev, dev->VolCatInfo.VolCatName, dev->openmode) < 0) {
-      return -1;
-   } else {
-      dev->state = state;
-      return dev->fd;
-   }
-}
-
-/* Open the first part file.
- *  - Close the fd
- *  - Reopen the device
- */
-int open_first_part(DEVICE *dev) {
-   int state;
-      
-   Dmsg3(29, "open_first_part %s %s %d\n", dev->dev_name, dev->VolCatInfo.VolCatName, dev->openmode);
-   if (dev->fd >= 0) {
-      close(dev->fd);
+   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;
+      }
    }
    
-   dev->fd = -1;
-   state = dev->state;
-   dev->state &= ~ST_OPENED;
-   
-   dev->part_start = 0;
-   dev->part = 0;
+   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 (open_dev(dev, dev->VolCatInfo.VolCatName, dev->openmode)) {
-      dev->state = state;
-      return dev->fd;
-   } else {
-      return 0;
+   /*
+    * If we are not trying to access the last part, set mode to 
+    *   OPEN_READ_ONLY as writing would be an error.
+    */
+   if (part < num_parts) {
+      omode = OPEN_READ_ONLY;
+      make_mounted_dvd_filename(this, archive_name);
    }
-}
-
-#ifdef debug_tracing
-#undef rewind_dev
-bool _rewind_dev(char *file, int line, DEVICE *dev)
-{
-   Dmsg2(100, "rewind_dev called from %s:%d\n", file, line);
-   return rewind_dev(dev);
-}
-#endif
-
-/* Protected version of lseek, which opens the right part if necessary */
-off_t lseek_dev(DEVICE *dev, off_t offset, int whence)
-{
-   int pos, openmode;
-   
-   if (dev->num_parts == 0) { /* If there is only one part, simply call lseek. */
-      return lseek(dev->fd, offset, whence);
+   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), 
+         archive_name.c_str(), mode);
+   /* Use system open() */
+   if ((fd = ::open(archive_name.c_str(), mode, 0640)) < 0) {
+      berrno be;
+      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);
       
-   switch(whence) {
-   case SEEK_SET:
-      Dmsg1(100, "lseek_dev SEEK_SET called %d\n", offset);
-      if ((uint64_t)offset >= dev->part_start) {
-        if ((uint64_t)(offset - dev->part_start) < dev->part_size) {
-           /* We are staying in the current part, just seek */
-           if ((pos = lseek(dev->fd, (off_t)(offset-dev->part_start), SEEK_SET)) < 0) {
-              return pos;   
-           } else {
-              return pos + dev->part_start;
-           }
-        } else {
-           /* Load next part, and start again */
-           if (open_next_part(dev) < 0) {
-               Dmsg0(100, "lseek_dev failed while trying to open the next part\n");
-              return -1;
-           }
-           return lseek_dev(dev, offset, SEEK_SET);
-        }
-      } else {
-        /* pos < dev->part_start :
-         * We need to access a previous part, 
-         * so just load the first one, and seek again
-         * until the right one is loaded */
-        if (open_first_part(dev) < 0) {
-            Dmsg0(100, "lseek_dev failed while trying to open the first part\n");
-           return -1;
-        }
-        return lseek_dev(dev, offset, SEEK_SET);
-      }
-      break;
-   case SEEK_CUR:
-      Dmsg1(100, "lseek_dev SEEK_CUR called %d\n", offset);
-      if ((pos = lseek(dev->fd, (off_t)0, SEEK_CUR)) < 0) {
-        return pos;   
-      }
-      pos += dev->part_start;
-      if (offset == 0) {
-        return pos;
-      }
-      else { /* Not used in Bacula, but should work */
-        return lseek_dev(dev, pos, SEEK_SET);
-      }
-      break;
-   case SEEK_END:
-      Dmsg1(100, "lseek_dev SEEK_END called %d\n", offset);
-      if (offset > 0) { /* Not used by bacula */
-         Dmsg1(100, "lseek_dev SEEK_END called with an invalid offset %d\n", offset);
-        errno = EINVAL;
-        return -1;
+      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->part == dev->num_parts) { /* The right part is already loaded */
-        if ((pos = lseek(dev->fd, (off_t)0, SEEK_END)) < 0) {
-           return pos;   
-        } else {
-           return pos + dev->part_start;
-        }
+      /* 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(fd, &filestat) < 0) {
+         berrno be;
+         dev_errno = errno;
+         Mmsg2(errmsg, _("Could not fstat: %s, ERR=%s\n"), archive_name.c_str(), 
+               be.strerror());
+         Dmsg1(29, "open failed: %s", errmsg);
+         /* Use system close() */
+         ::close(fd);
+         clear_opened();
       } else {
-        /* Load the first part, then load the next until we reach the last one.
-         * This is the only way to be sure we compute the right file address. */
-        /* Save previous openmode, and open all but last part read-only (useful for DVDs) */
-        openmode = dev->openmode;
-        dev->openmode = OPEN_READ_ONLY;
-        
-        /* Works because num_parts > 0. */
-        if (open_first_part(dev) < 0) {
-            Dmsg0(100, "lseek_dev failed while trying to open the first part\n");
-           return -1;
-        }
-        while (dev->part < (dev->num_parts-1)) {
-           if (open_next_part(dev) < 0) {
-               Dmsg0(100, "lseek_dev failed while trying to open the next part\n");
-              return -1;
-           }
-        }
-        dev->openmode = openmode;
-        if (open_next_part(dev) < 0) {
-            Dmsg0(100, "lseek_dev failed while trying to open the next part\n");
-           return -1;
-        }
-        return lseek_dev(dev, 0, SEEK_END);
+         part_size = filestat.st_size;
+         dev_errno = 0;
+         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++;
+            }
+         }*/
       }
-      break;
-   default:
-      errno = EINVAL;
-      return -1;
    }
 }
 
+
 /*
  * Rewind the device.
  *  Returns: true  on success
- *          false on failure
+ *           false on failure
  */
-bool rewind_dev(DEVICE *dev)
+bool DEVICE::rewind(DCR *dcr)
 {
    struct mtop mt_com;
    unsigned int i;
+   bool first = true;
 
-   Dmsg1(29, "rewind_dev %s\n", dev->dev_name);
-   if (dev->fd < 0) {
-      dev->dev_errno = EBADF;
-      Mmsg1(&dev->errmsg, _("Bad call to rewind_dev. Device %s not open\n"),
-           dev->dev_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) {
-           berrno be;
-           clrerror_dev(dev, MTREW);
-           if (i == dev->max_rewind_wait) {
+      for (i=max_rewind_wait; ; i -= 5) {
+         if (ioctl(fd, MTIOCTOP, (char *)&mt_com) < 0) {
+            berrno be;
+            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->dev_name, be.strerror());
-           return false;
-        }
-        break;
-      }
-   } else if (dev->is_file()) {      
-      if (lseek_dev(dev, (off_t)0, SEEK_SET) < 0) {
-        berrno be;
-        dev->dev_errno = errno;
-         Mmsg2(&dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-        return false;
+               bmicrosleep(5, 0);
+               continue;
+            }
+            Mmsg2(errmsg, _("Rewind error on %s. ERR=%s.\n"),
+               print_name(), be.strerror());
+            return false;
+         }
+         break;
+      }
+   } else if (is_file()) {      
+      if (lseek_dev(this, (off_t)0, SEEK_SET) < 0) {
+         berrno be;
+         dev_errno = errno;
+         Mmsg2(errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
+            print_name(), be.strerror());
+         return false;
       }
    }
    return true;
 }
 
+void DEVICE::block(int why)
+{
+   lock_device(this);
+   block_device(this, why);
+   V(mutex);
+}
+
+void DEVICE::unblock()
+{  
+   P(mutex);
+   unblock_device(this);
+   V(mutex);
+}
+
+const char *DEVICE::print_blocked() const 
+{
+   switch (dev_blocked) {
+   case BST_NOT_BLOCKED:
+      return "BST_NOT_BLOCKED";
+   case BST_UNMOUNTED:
+      return "BST_UNMOUNTED";
+   case BST_WAITING_FOR_SYSOP:
+      return "BST_WAITING_FOR_SYSOP";
+   case BST_DOING_ACQUIRE:
+      return "BST_DOING_ACQUIRE";
+   case BST_WRITING_LABEL:
+      return "BST_WRITING_LABEL";
+   case BST_UNMOUNTED_WAITING_FOR_SYSOP:
+      return "BST_UNMOUNTED_WAITING_FOR_SYSOP";
+   case BST_MOUNT:
+      return "BST_MOUNT";
+   default:
+      return _("unknown blocked code");
+   }
+}
+
+/*
+ * Called to indicate that we have just read an
+ *  EOF from the device.
+ */
+void DEVICE::set_ateof() 
+{ 
+   set_eof();
+   if (is_tape()) {
+      file++;
+   }
+   file_addr = 0;
+   file_size = 0;
+   block_num = 0;
+}
+
+/*
+ * Called to indicate we are now at the end of the tape, and
+ *   writing is not possible.
+ */
+void DEVICE::set_ateot() 
+{
+   /* Make tape effectively read-only */
+   state |= (ST_EOF|ST_EOT|ST_WEOT);
+   clear_append();
+}
+
 /*
  * Position device to end of medium (end of data)
- *  Returns: 1 on succes
- *          0 on error
+ *  Returns: true  on succes
+ *           false on error
  */
-int
+bool
 eod_dev(DEVICE *dev)
 {
    struct mtop mt_com;
    struct mtget mt_stat;
-   int stat = 0;
+   bool ok = true;
    off_t pos;
 
+   if (dev->fd < 0) {
+      dev->dev_errno = EBADF;
+      Mmsg1(dev->errmsg, _("Bad call to eod_dev. Device %s not open\n"),
+            dev->print_name());
+      return false;
+   }
+
+#if defined (__digital__) && defined (__unix__)
+   return dev->fsf(dev->VolCatInfo.VolCatFiles);
+#endif
+
    Dmsg0(29, "eod_dev\n");
-   if (dev->state & ST_EOT) {
-      return 1;
+   if (dev->at_eot()) {
+      return true;
    }
    dev->state &= ~(ST_EOF);  /* remove EOF flags */
    dev->block_num = dev->file = 0;
    dev->file_size = 0;
    dev->file_addr = 0;
-   if (dev->state & (ST_FIFO | ST_PROG)) {
-      return 1;
+   if (dev->is_fifo() || dev->is_prog()) {
+      return true;
    }
-   if (!(dev->is_tape())) {
+   if (!dev->is_tape()) {
       pos = lseek_dev(dev, (off_t)0, SEEK_END);
 //    Dmsg1(100, "====== Seek to %lld\n", pos);
       if (pos >= 0) {
-        update_pos_dev(dev);
-        dev->state |= ST_EOT;
-        return 1;
+         update_pos_dev(dev);
+         dev->state |= ST_EOT;
+         return true;
       }
       dev->dev_errno = errno;
       berrno be;
-      Mmsg2(&dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
-            dev->dev_name, be.strerror());
-      return 0;
+      Mmsg2(dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
+             dev->print_name(), be.strerror());
+      return false;
    }
 #ifdef MTEOM
-
-   if (dev_cap(dev, CAP_MTIOCGET) && dev_cap(dev, CAP_FASTFSF) && 
-      !dev_cap(dev, CAP_EOM)) {
-      struct mtget mt_stat;
+   if (dev_cap(dev, CAP_FASTFSF) && !dev_cap(dev, CAP_EOM)) {
       Dmsg0(100,"Using FAST FSF for EOM\n");
-      if (ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) == 0 && mt_stat.mt_fileno <= 0) {
-       if (!rewind_dev(dev)) {
-         return 0;
-       }
+      /* If unknown position, rewind */
+      if (!dev_get_os_pos(dev, &mt_stat)) {
+        if (!dev->rewind(NULL)) {
+          return false;
+        }
       }
       mt_com.mt_op = MTFSF;
       /*
@@ -1143,72 +819,70 @@ eod_dev(DEVICE *dev)
        */
       mt_com.mt_count = INT16_MAX;    /* use big positive number */
       if (mt_com.mt_count < 0) {
-        mt_com.mt_count = INT16_MAX; /* brain damaged system */
+         mt_com.mt_count = INT16_MAX; /* brain damaged system */
       }
    }
 
    if (dev_cap(dev, CAP_MTIOCGET) && (dev_cap(dev, CAP_FASTFSF) || dev_cap(dev, CAP_EOM))) {
       if (dev_cap(dev, CAP_EOM)) {
          Dmsg0(100,"Using EOM for EOM\n");
-        mt_com.mt_op = MTEOM;
-        mt_com.mt_count = 1;
+         mt_com.mt_op = MTEOM;
+         mt_com.mt_count = 1;
       }
 
-      if ((stat=ioctl(dev->fd, MTIOCTOP, (char *)&mt_com)) < 0) {
-        berrno be;
-        clrerror_dev(dev, mt_com.mt_op);
+      if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
+         berrno be;
+         clrerror_dev(dev, mt_com.mt_op);
          Dmsg1(50, "ioctl error: %s\n", be.strerror());
-        update_pos_dev(dev);
-         Mmsg2(&dev->errmsg, _("ioctl MTEOM error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-        return 0;
+         update_pos_dev(dev);
+         Mmsg2(dev->errmsg, _("ioctl MTEOM error on %s. ERR=%s.\n"),
+            dev->print_name(), be.strerror());
+         return false;
       }
 
-      if (ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) < 0) {
-        berrno be;
-        clrerror_dev(dev, -1);
-         Mmsg2(&dev->errmsg, _("ioctl MTIOCGET error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-        return 0;
+      if (!dev_get_os_pos(dev, &mt_stat)) {
+         berrno be;
+         clrerror_dev(dev, -1);
+         Mmsg2(dev->errmsg, _("ioctl MTIOCGET error on %s. ERR=%s.\n"),
+            dev->print_name(), be.strerror());
+         return false;
       }
       Dmsg2(100, "EOD file=%d block=%d\n", mt_stat.mt_fileno, mt_stat.mt_blkno);
+      dev->set_ateof();
       dev->file = mt_stat.mt_fileno;
-
-   /*
-    * Rewind then use FSF until EOT reached
-    */
    } else {
 #else
    {
 #endif
-      if (!rewind_dev(dev)) {
-        return 0;
+      /*
+       * Rewind then use FSF until EOT reached
+       */
+      if (!dev->rewind(NULL)) {
+         return false;
       }
       /*
        * Move file by file to the end of the tape
        */
       int file_num;
-      for (file_num=dev->file; !(dev->state & ST_EOT); file_num++) {
+      for (file_num=dev->file; !dev->at_eot(); file_num++) {
          Dmsg0(200, "eod_dev: doing fsf 1\n");
-        if (!fsf_dev(dev, 1)) {
-            Dmsg0(200, "fsf_dev error.\n");
-           return 0;
-        }
-        /*
-         * Avoid infinite loop. ***FIXME*** possibly add code
-         *   to set EOD or to turn off CAP_FASTFSF if on.
-         */
-        if (file_num == (int)dev->file) {
-           struct mtget mt_stat;
-            Dmsg1(100, "fsf_dev did not advance from file %d\n", file_num);
-           if (dev_cap(dev, CAP_MTIOCGET) && ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) == 0 &&
-                     mt_stat.mt_fileno >= 0) {
+         if (!dev->fsf(1)) {
+            Dmsg0(200, "fsf error.\n");
+            return false;
+         }
+         /*
+          * 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->file = mt_stat.mt_fileno;
-           }
-           stat = 0;
-           break;                    /* we are not progressing, bail out */
-        }
+               dev->file = mt_stat.mt_fileno;
+            }       
+            break;
+         }
       }
    }
    /*
@@ -1219,27 +893,26 @@ eod_dev(DEVICE *dev)
    if (dev_cap(dev, CAP_BSFATEOM)) {
       struct mtget mt_stat;
       /* Backup over EOF */
-      stat = bsf_dev(dev, 1);
+      ok = bsf_dev(dev, 1);
       /* If BSF worked and fileno is known (not -1), set file */
-      if (dev_cap(dev, CAP_MTIOCGET) && ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) == 0 && mt_stat.mt_fileno >= 0) {
+      if (dev_get_os_pos(dev, &mt_stat)) {
          Dmsg2(100, "BSFATEOF adjust file from %d to %d\n", dev->file , mt_stat.mt_fileno);
-        dev->file = mt_stat.mt_fileno;
+         dev->file = mt_stat.mt_fileno;
       } else {
-        dev->file++;                 /* wing it -- not correct on all OSes */
+         dev->file++;                 /* wing it -- not correct on all OSes */
       }
    } else {
-      update_pos_dev(dev);                  /* update position */
-      stat = 1;
+      update_pos_dev(dev);                   /* update position */
    }
    Dmsg1(200, "EOD dev->file=%d\n", dev->file);
-   return stat;
+   return ok;
 }
 
 /*
- * 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
+ *           false on error
  */
 bool update_pos_dev(DEVICE *dev)
 {
@@ -1248,7 +921,7 @@ bool update_pos_dev(DEVICE *dev)
 
    if (dev->fd < 0) {
       dev->dev_errno = EBADF;
-      Mmsg0(&dev->errmsg, _("Bad device call. Archive not open\n"));
+      Mmsg0(dev->errmsg, _("Bad device call. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return false;
    }
@@ -1259,14 +932,14 @@ bool update_pos_dev(DEVICE *dev)
       dev->file_addr = 0;
       pos = lseek_dev(dev, (off_t)0, SEEK_CUR);
       if (pos < 0) {
-        berrno be;
-        dev->dev_errno = errno;
-         Pmsg1(000, "Seek error: ERR=%s\n", be.strerror());
-         Mmsg2(&dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-        ok = false;
+         berrno be;
+         dev->dev_errno = errno;
+         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;
       } else {
-        dev->file_addr = pos;
+         dev->file_addr = pos;
       }
    }
    return ok;
@@ -1289,67 +962,67 @@ uint32_t status_dev(DEVICE *dev)
 
    if (dev->state & (ST_EOT | ST_WEOT)) {
       stat |= BMT_EOD;
-      Dmsg0(-20, " EOD");
+      Pmsg0(-20, " EOD");
    }
    if (dev->state & ST_EOF) {
       stat |= BMT_EOF;
-      Dmsg0(-20, " EOF");
+      Pmsg0(-20, " EOF");
    }
    if (dev->is_tape()) {
       stat |= BMT_TAPE;
-      Dmsg0(-20," Bacula status:");
-      Dmsg2(-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;
-         Mmsg2(&dev->errmsg, _("ioctl MTIOCGET error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-        return 0;
+         berrno be;
+         dev->dev_errno = errno;
+         Mmsg2(dev->errmsg, _("ioctl MTIOCGET error on %s. ERR=%s.\n"),
+            dev->print_name(), be.strerror());
+         return 0;
       }
-      Dmsg0(-20, " Device status:");
+      Pmsg0(-20, _(" Device status:"));
 
 #if defined(HAVE_LINUX_OS)
       if (GMT_EOF(mt_stat.mt_gstat)) {
-        stat |= BMT_EOF;
-         Dmsg0(-20, " EOF");
+         stat |= BMT_EOF;
+         Pmsg0(-20, " EOF");
       }
       if (GMT_BOT(mt_stat.mt_gstat)) {
-        stat |= BMT_BOT;
-         Dmsg0(-20, " BOT");
+         stat |= BMT_BOT;
+         Pmsg0(-20, " BOT");
       }
       if (GMT_EOT(mt_stat.mt_gstat)) {
-        stat |= BMT_EOT;
-         Dmsg0(-20, " EOT");
+         stat |= BMT_EOT;
+         Pmsg0(-20, " EOT");
       }
       if (GMT_SM(mt_stat.mt_gstat)) {
-        stat |= BMT_SM;
-         Dmsg0(-20, " SM");
+         stat |= BMT_SM;
+         Pmsg0(-20, " SM");
       }
       if (GMT_EOD(mt_stat.mt_gstat)) {
-        stat |= BMT_EOD;
-         Dmsg0(-20, " EOD");
+         stat |= BMT_EOD;
+         Pmsg0(-20, " EOD");
       }
       if (GMT_WR_PROT(mt_stat.mt_gstat)) {
-        stat |= BMT_WR_PROT;
-         Dmsg0(-20, " WR_PROT");
+         stat |= BMT_WR_PROT;
+         Pmsg0(-20, " WR_PROT");
       }
       if (GMT_ONLINE(mt_stat.mt_gstat)) {
-        stat |= BMT_ONLINE;
-         Dmsg0(-20, " ONLINE");
+         stat |= BMT_ONLINE;
+         Pmsg0(-20, " ONLINE");
       }
       if (GMT_DR_OPEN(mt_stat.mt_gstat)) {
-        stat |= BMT_DR_OPEN;
-         Dmsg0(-20, " DR_OPEN");
+         stat |= BMT_DR_OPEN;
+         Pmsg0(-20, " DR_OPEN");
       }
       if (GMT_IM_REP_EN(mt_stat.mt_gstat)) {
-        stat |= BMT_IM_REP_EN;
-         Dmsg0(-20, " IM_REP_EN");
+         stat |= BMT_IM_REP_EN;
+         Pmsg0(-20, " IM_REP_EN");
       }
 #endif /* !SunOS && !OSF */
       if (dev_cap(dev, CAP_MTIOCGET)) {
-         Dmsg2(-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 {
-         Dmsg2(-20, " file=%d block=%d\n", -1, -1);
+         Pmsg2(-20, _(" file=%d block=%d\n"), -1, -1);
       }
    } else {
       stat |= BMT_ONLINE | BMT_BOT;
@@ -1361,7 +1034,7 @@ uint32_t status_dev(DEVICE *dev)
 /*
  * Load medium in device
  *  Returns: true  on success
- *          false on failure
+ *           false on failure
  */
 bool load_dev(DEVICE *dev)
 {
@@ -1371,7 +1044,7 @@ bool load_dev(DEVICE *dev)
 
    if (dev->fd < 0) {
       dev->dev_errno = EBADF;
-      Mmsg0(&dev->errmsg, _("Bad call to load_dev. Archive not open\n"));
+      Mmsg0(dev->errmsg, _("Bad call to load_dev. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return false;
    }
@@ -1381,9 +1054,9 @@ bool load_dev(DEVICE *dev)
 #ifndef MTLOAD
    Dmsg0(200, "stored: MTLOAD command not available\n");
    berrno be;
-   dev->dev_errno = ENOTTY;          /* function not available */
-   Mmsg2(&dev->errmsg, _("ioctl MTLOAD error on %s. ERR=%s.\n"),
-        dev->dev_name, be.strerror());
+   dev->dev_errno = ENOTTY;           /* function not available */
+   Mmsg2(dev->errmsg, _("ioctl MTLOAD error on %s. ERR=%s.\n"),
+         dev->print_name(), be.strerror());
    return false;
 #else
 
@@ -1395,8 +1068,8 @@ bool load_dev(DEVICE *dev)
    if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
       berrno be;
       dev->dev_errno = errno;
-      Mmsg2(&dev->errmsg, _("ioctl MTLOAD error on %s. ERR=%s.\n"),
-        dev->dev_name, be.strerror());
+      Mmsg2(dev->errmsg, _("ioctl MTLOAD error on %s. ERR=%s.\n"),
+         dev->print_name(), be.strerror());
       return false;
    }
    return true;
@@ -1406,20 +1079,14 @@ bool load_dev(DEVICE *dev)
 /*
  * Rewind device and put it offline
  *  Returns: true  on success
- *          false on failure
+ *           false on failure
  */
 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. Archive 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 */
@@ -1437,11 +1104,11 @@ bool offline_dev(DEVICE *dev)
    if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
       berrno be;
       dev->dev_errno = errno;
-      Mmsg2(&dev->errmsg, _("ioctl MTOFFL error on %s. ERR=%s.\n"),
-        dev->dev_name, be.strerror());
+      Mmsg2(dev->errmsg, _("ioctl MTOFFL error on %s. ERR=%s.\n"),
+         dev->print_name(), be.strerror());
       return false;
    }
-   Dmsg1(100, "Offlined device %s\n", dev->dev_name);
+   Dmsg1(100, "Offlined device %s\n", dev->print_name());
    return true;
 }
 
@@ -1461,43 +1128,42 @@ 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);
    }
 }
 
 /*
  * Foward space a file
  *   Returns: true  on success
- *           false on failure
+ *            false on failure
  */
-bool
-fsf_dev(DEVICE *dev, int num)
+bool DEVICE::fsf(int num)
 {
    struct mtget mt_stat;
    struct mtop mt_com;
    int stat = 0;
 
-   if (dev->fd < 0) {
-      dev->dev_errno = EBADF;
-      Mmsg0(&dev->errmsg, _("Bad call to fsf_dev. Archive not open\n"));
-      Emsg0(M_FATAL, 0, dev->errmsg);
+   if (fd < 0) {
+      dev_errno = EBADF;
+      Mmsg0(errmsg, _("Bad call to fsf_dev. Device not open\n"));
+      Emsg0(M_FATAL, 0, errmsg);
       return false;
    }
 
-   if (!dev->is_tape()) {
+   if (!is_tape()) {
       return true;
    }
-   if (dev->state & ST_EOT) {
-      dev->dev_errno = 0;
-      Mmsg1(dev->errmsg, _("Device %s at End of Tape.\n"), dev->dev_name);
+   if (at_eot()) {
+      dev_errno = 0;
+      Mmsg1(errmsg, _("Device %s at End of Tape.\n"), print_name());
       return false;
    }
-   if (dev->state & ST_EOF) {
+   if (at_eof()) {
       Dmsg0(200, "ST_EOF set on entry to FSF\n");
    }
 
-   Dmsg0(100, "fsf_dev\n");
-   dev->block_num = 0;
+   Dmsg0(100, "fsf\n");
+   block_num = 0;
    /*
     * If Fast forward space file is set, then we
     *  use MTFSF to forward space and MTIOCGET
@@ -1505,25 +1171,23 @@ fsf_dev(DEVICE *dev, int num)
     *  the SCSI driver will ensure that we do not
     *  forward space past the end of the medium.
     */
-   if (dev_cap(dev, CAP_FSF) && dev_cap(dev, CAP_MTIOCGET) && dev_cap(dev, CAP_FASTFSF)) {
+   if (dev_cap(this, CAP_FSF) && dev_cap(this, CAP_MTIOCGET) && dev_cap(this, CAP_FASTFSF)) {
       mt_com.mt_op = MTFSF;
       mt_com.mt_count = num;
-      stat = ioctl(dev->fd, MTIOCTOP, (char *)&mt_com);
-      if (stat < 0 || ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) < 0) {
-        berrno be;
-        dev->state |= ST_EOT;
+      stat = ioctl(fd, MTIOCTOP, (char *)&mt_com);
+      if (stat < 0 || !dev_get_os_pos(this, &mt_stat)) {
+         berrno be;
+         set_eot();
          Dmsg0(200, "Set ST_EOT\n");
-        clrerror_dev(dev, MTFSF);
-         Mmsg2(dev->errmsg, _("ioctl MTFSF error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-         Dmsg1(200, "%s", dev->errmsg);
-        return false;
+         clrerror_dev(this, MTFSF);
+         Mmsg2(errmsg, _("ioctl MTFSF error on %s. ERR=%s.\n"),
+            print_name(), be.strerror());
+         Dmsg1(200, "%s", errmsg);
+         return false;
       }
       Dmsg2(200, "fsf file=%d block=%d\n", mt_stat.mt_fileno, mt_stat.mt_blkno);
-      dev->file = mt_stat.mt_fileno;
-      dev->state |= ST_EOF;    /* just read EOF */
-      dev->file_addr = 0;
-      dev->file_size = 0;
+      set_ateof();
+      file = mt_stat.mt_fileno;
       return true;
 
    /*
@@ -1533,71 +1197,72 @@ fsf_dev(DEVICE *dev, int num)
     *  is the only way we can be sure that we don't read
     *  two consecutive EOF marks, which means End of Data.
     */
-   } else if (dev_cap(dev, CAP_FSF)) {
+   } else if (dev_cap(this, CAP_FSF)) {
       POOLMEM *rbuf;
       int rbuf_len;
       Dmsg0(200, "FSF has cap_fsf\n");
-      if (dev->max_block_size == 0) {
-        rbuf_len = DEFAULT_BLOCK_SIZE;
+      if (max_block_size == 0) {
+         rbuf_len = DEFAULT_BLOCK_SIZE;
       } else {
-        rbuf_len = dev->max_block_size;
+         rbuf_len = max_block_size;
       }
       rbuf = get_memory(rbuf_len);
       mt_com.mt_op = MTFSF;
       mt_com.mt_count = 1;
-      while (num-- && !(dev->state & ST_EOT)) {
+      while (num-- && !at_eot()) {
          Dmsg0(100, "Doing read before fsf\n");
-        if ((stat = read(dev->fd, (char *)rbuf, rbuf_len)) < 0) {
-           if (errno == ENOMEM) {     /* tape record exceeds buf len */
-              stat = rbuf_len;        /* This is OK */
-           } else {
-              berrno be;
-              dev->state |= ST_EOT;
-              clrerror_dev(dev, -1);
-               Dmsg2(100, "Set ST_EOT read errno=%d. ERR=%s\n", dev->dev_errno,
-                 be.strerror());
-               Mmsg2(dev->errmsg, _("read error on %s. ERR=%s.\n"),
-                 dev->dev_name, be.strerror());
-               Dmsg1(100, "%s", dev->errmsg);
-              break;
-           }
-        }
-        if (stat == 0) {                /* EOF */
-           update_pos_dev(dev);
-            Dmsg1(100, "End of File mark from read. File=%d\n", dev->file+1);
-           /* Two reads of zero means end of tape */
-           if (dev->state & ST_EOF) {
-              dev->state |= ST_EOT;
+         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();
+               clrerror_dev(this, -1);
+               Dmsg2(100, "Set ST_EOT read errno=%d. ERR=%s\n", dev_errno,
+                  be.strerror());
+               Mmsg2(errmsg, _("read error on %s. ERR=%s.\n"),
+                  print_name(), be.strerror());
+               Dmsg1(100, "%s", errmsg);
+               break;
+            }
+         }
+         if (stat == 0) {                /* EOF */
+            update_pos_dev(this);
+            Dmsg1(100, "End of File mark from read. File=%d\n", file+1);
+            /* Two reads of zero means end of tape */
+            if (at_eof()) {
+               set_eot();
                Dmsg0(100, "Set ST_EOT\n");
-              break;
-           } else {
-              dev->state |= ST_EOF;
-              dev->file++;
-              dev->file_addr = 0;
-              dev->file_size = 0;
-              continue;
-           }
-        } else {                        /* Got data */
-           dev->state &= ~(ST_EOF|ST_EOT);
-        }
+               break;
+            } else {
+               set_ateof();
+               continue;
+            }
+         } else {                        /* Got data */
+            clear_eot();
+            clear_eof();
+         }
 
          Dmsg0(100, "Doing MTFSF\n");
-        stat = ioctl(dev->fd, MTIOCTOP, (char *)&mt_com);
-        if (stat < 0) {                 /* error => EOT */
-           berrno be;
-           dev->state |= ST_EOT;
+         stat = ioctl(fd, MTIOCTOP, (char *)&mt_com);
+         if (stat < 0) {                 /* error => EOT */
+            berrno be;
+            set_eot();
             Dmsg0(100, "Set ST_EOT\n");
-           clrerror_dev(dev, MTFSF);
-            Mmsg2(&dev->errmsg, _("ioctl MTFSF error on %s. ERR=%s.\n"),
-              dev->dev_name, be.strerror());
+            clrerror_dev(this, MTFSF);
+            Mmsg2(errmsg, _("ioctl MTFSF error on %s. ERR=%s.\n"),
+               print_name(), be.strerror());
             Dmsg0(100, "Got < 0 for MTFSF\n");
-            Dmsg1(100, "%s", dev->errmsg);
-        } else {
-           dev->state |= ST_EOF;     /* just read EOF */
-           dev->file++;
-           dev->file_addr = 0;
-           dev->file_size = 0;
-        }
+            Dmsg1(100, "%s", errmsg);
+         } else {
+            set_ateof();
+         }
       }
       free_memory(rbuf);
 
@@ -1606,31 +1271,31 @@ fsf_dev(DEVICE *dev, int num)
     */
    } else {
       Dmsg0(200, "Doing FSR for FSF\n");
-      while (num-- && !(dev->state & ST_EOT)) {
-        fsr_dev(dev, INT32_MAX);    /* returns -1 on EOF or EOT */
+      while (num-- && !at_eot()) {
+         fsr(INT32_MAX);    /* returns -1 on EOF or EOT */
       }
-      if (dev->state & ST_EOT) {
-        dev->dev_errno = 0;
-         Mmsg1(dev->errmsg, _("Device %s at End of Tape.\n"), dev->dev_name);
-        stat = -1;
+      if (at_eot()) {
+         dev_errno = 0;
+         Mmsg1(errmsg, _("Device %s at End of Tape.\n"), print_name());
+         stat = -1;
       } else {
-        stat = 0;
+         stat = 0;
       }
    }
-   update_pos_dev(dev);
+   update_pos_dev(this);
    Dmsg1(200, "Return %d from FSF\n", stat);
-   if (dev->state & ST_EOF)
+   if (at_eof())
       Dmsg0(200, "ST_EOF set on exit FSF\n");
-   if (dev->state & ST_EOT)
+   if (at_eot())
       Dmsg0(200, "ST_EOT set on exit FSF\n");
-   Dmsg1(200, "Return from FSF file=%d\n", dev->file);
+   Dmsg1(200, "Return from FSF file=%d\n", file);
    return stat == 0;
 }
 
 /*
  * Backward space a file
  *  Returns: false on failure
- *          true  on success
+ *           true  on success
  */
 bool
 bsf_dev(DEVICE *dev, int num)
@@ -1640,14 +1305,14 @@ bsf_dev(DEVICE *dev, int num)
 
    if (dev->fd < 0) {
       dev->dev_errno = EBADF;
-      Mmsg0(dev->errmsg, _("Bad call to bsf_dev. Archive device not open\n"));
+      Mmsg0(dev->errmsg, _("Bad call to bsf_dev. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return false;
    }
 
    if (!dev->is_tape()) {
       Mmsg1(dev->errmsg, _("Device %s cannot BSF because it is not a tape.\n"),
-        dev->dev_name);
+         dev->print_name());
       return false;
    }
    Dmsg0(29, "bsf_dev\n");
@@ -1662,7 +1327,7 @@ bsf_dev(DEVICE *dev, int num)
       berrno be;
       clrerror_dev(dev, MTBSF);
       Mmsg2(dev->errmsg, _("ioctl MTBSF error on %s. ERR=%s.\n"),
-        dev->dev_name, be.strerror());
+         dev->print_name(), be.strerror());
    }
    update_pos_dev(dev);
    return stat == 0;
@@ -1670,70 +1335,65 @@ bsf_dev(DEVICE *dev, int num)
 
 
 /*
- * Foward space a record
+ * Foward space num records
  *  Returns: false on failure
- *          true  on success
+ *           true  on success
  */
-bool
-fsr_dev(DEVICE *dev, int num)
+bool DEVICE::fsr(int num)
 {
    struct mtop mt_com;
    int stat;
 
-   if (dev->fd < 0) {
-      dev->dev_errno = EBADF;
-      Mmsg0(dev->errmsg, _("Bad call to fsr_dev. Archive not open\n"));
-      Emsg0(M_FATAL, 0, dev->errmsg);
+   if (fd < 0) {
+      dev_errno = EBADF;
+      Mmsg0(errmsg, _("Bad call to fsr. Device not open\n"));
+      Emsg0(M_FATAL, 0, errmsg);
       return false;
    }
 
-   if (!dev->is_tape()) {
+   if (!is_tape()) {
       return false;
    }
-   if (!dev_cap(dev, CAP_FSR)) {
-      Mmsg1(dev->errmsg, _("ioctl MTFSR not permitted on %s.\n"), dev->dev_name);
+   if (!dev_cap(this, CAP_FSR)) {
+      Mmsg1(errmsg, _("ioctl MTFSR not permitted on %s.\n"), print_name());
       return false;
    }
 
-   Dmsg1(29, "fsr_dev %d\n", num);
+   Dmsg1(29, "fsr %d\n", num);
    mt_com.mt_op = MTFSR;
    mt_com.mt_count = num;
-   stat = ioctl(dev->fd, MTIOCTOP, (char *)&mt_com);
+   stat = ioctl(fd, MTIOCTOP, (char *)&mt_com);
    if (stat == 0) {
-      dev->state &= ~ST_EOF;
-      dev->block_num += num;
+      clear_eof();
+      block_num += num;
    } else {
       berrno be;
       struct mtget mt_stat;
-      clrerror_dev(dev, MTFSR);
+      clrerror_dev(this, MTFSR);
       Dmsg1(100, "FSF fail: ERR=%s\n", be.strerror());
-      if (dev_cap(dev, CAP_MTIOCGET) && ioctl(dev->fd, MTIOCGET, (char *)&mt_stat) == 0 && mt_stat.mt_fileno >= 0) {
-         Dmsg4(100, "Adjust from %d:%d to %d:%d\n", dev->file,
-           dev->block_num, mt_stat.mt_fileno, mt_stat.mt_blkno);
-        dev->file = mt_stat.mt_fileno;
-        dev->block_num = mt_stat.mt_blkno;
+      if (dev_get_os_pos(this, &mt_stat)) {
+         Dmsg4(100, "Adjust from %d:%d to %d:%d\n", file,
+            block_num, mt_stat.mt_fileno, mt_stat.mt_blkno);
+         file = mt_stat.mt_fileno;
+         block_num = mt_stat.mt_blkno;
       } else {
-        if (dev->state & ST_EOF) {
-           dev->state |= ST_EOT;
-        } else {
-           dev->state |= ST_EOF;           /* assume EOF */
-           dev->file++;
-           dev->block_num = 0;
-           dev->file_addr = 0;
-           dev->file_size = 0;
-        }
+         if (at_eof()) {
+            set_eot();
+         } else {
+            set_ateof();
+         }
       }
-      Mmsg2(dev->errmsg, _("ioctl MTFSR error on %s. ERR=%s.\n"),
-        dev->dev_name, be.strerror());
+      Mmsg3(errmsg, _("ioctl MTFSR %d error on %s. ERR=%s.\n"),
+         num, print_name(), be.strerror());
    }
-   update_pos_dev(dev);
+   update_pos_dev(this);
    return stat == 0;
 }
 
 /*
  * Backward space a record
  *   Returns:  false on failure
- *            true  on success
+ *             true  on success
  */
 bool
 bsr_dev(DEVICE *dev, int num)
@@ -1743,7 +1403,7 @@ bsr_dev(DEVICE *dev, int num)
 
    if (dev->fd < 0) {
       dev->dev_errno = EBADF;
-      Mmsg0(dev->errmsg, _("Bad call to bsr_dev. Archive not open\n"));
+      Mmsg0(dev->errmsg, _("Bad call to bsr_dev. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return false;
    }
@@ -1753,7 +1413,7 @@ bsr_dev(DEVICE *dev, int num)
    }
 
    if (!dev_cap(dev, CAP_BSR)) {
-      Mmsg1(dev->errmsg, _("ioctl MTBSR not permitted on %s.\n"), dev->dev_name);
+      Mmsg1(dev->errmsg, _("ioctl MTBSR not permitted on %s.\n"), dev->print_name());
       return false;
    }
 
@@ -1767,7 +1427,7 @@ bsr_dev(DEVICE *dev, int num)
       berrno be;
       clrerror_dev(dev, MTBSR);
       Mmsg2(dev->errmsg, _("ioctl MTBSR error on %s. ERR=%s.\n"),
-        dev->dev_name, be.strerror());
+         dev->print_name(), be.strerror());
    }
    update_pos_dev(dev);
    return stat == 0;
@@ -1776,27 +1436,27 @@ bsr_dev(DEVICE *dev, int num)
 /*
  * Reposition the device to file, block
  * Returns: false on failure
- *         true  on success
+ *          true  on success
  */
 bool
 reposition_dev(DEVICE *dev, uint32_t file, uint32_t block)
 {
    if (dev->fd < 0) {
       dev->dev_errno = EBADF;
-      Mmsg0(dev->errmsg, _("Bad call to reposition_dev. Archive not open\n"));
+      Mmsg0(dev->errmsg, _("Bad call to reposition_dev. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return false;
    }
 
    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;
-        dev->dev_errno = errno;
+         berrno be;
+         dev->dev_errno = errno;
          Mmsg2(dev->errmsg, _("lseek_dev error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
-        return false;
+            dev->print_name(), be.strerror());
+         return false;
       }
       dev->file = file;
       dev->block_num = block;
@@ -1806,16 +1466,16 @@ 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)) {
-        return false;
+      Dmsg0(100, "Rewind\n");
+      if (!dev->rewind(NULL)) {
+         return false;
       }
    }
    if (file > dev->file) {
       Dmsg1(100, "fsf %d\n", file-dev->file);
-      if (!fsf_dev(dev, file-dev->file)) {
+      if (!dev->fsf(file-dev->file)) {
          Dmsg1(100, "fsf failed! ERR=%s\n", strerror_dev(dev));
-        return false;
+         return false;
       }
       Dmsg2(100, "wanted_file=%d at_file=%d\n", file, dev->file);
    }
@@ -1824,13 +1484,13 @@ reposition_dev(DEVICE *dev, uint32_t file, uint32_t block)
       Dmsg0(100, "bsf_dev 1\n");
       bsf_dev(dev, 1);
       Dmsg0(100, "fsf_dev 1\n");
-      fsf_dev(dev, 1);
+      dev->fsf(1);
       Dmsg2(100, "wanted_blk=%d at_blk=%d\n", block, dev->block_num);
    }
    if (dev_cap(dev, CAP_POSITIONBLOCKS) && block > dev->block_num) {
       /* Ignore errors as Bacula can read to the correct block */
       Dmsg1(100, "fsr %d\n", block-dev->block_num);
-      return fsr_dev(dev, block-dev->block_num);
+      return dev->fsr(block-dev->block_num);
    }
    return true;
 }
@@ -1840,7 +1500,7 @@ reposition_dev(DEVICE *dev, uint32_t file, uint32_t block)
 /*
  * Write an end of file on the device
  *   Returns: 0 on success
- *           non-zero on failure
+ *            non-zero on failure
  */
 int
 weof_dev(DEVICE *dev, int num)
@@ -1851,7 +1511,7 @@ weof_dev(DEVICE *dev, int num)
    
    if (dev->fd < 0) {
       dev->dev_errno = EBADF;
-      Mmsg0(dev->errmsg, _("Bad call to weof_dev. Archive drive not open\n"));
+      Mmsg0(dev->errmsg, _("Bad call to weof_dev. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return -1;
    }
@@ -1860,6 +1520,12 @@ weof_dev(DEVICE *dev, int num)
    if (!dev->is_tape()) {
       return 0;
    }
+   if (!dev->can_append()) {
+      Mmsg0(dev->errmsg, _("Attempt to WEOF on non-appendable Volume\n"));
+      Emsg0(M_FATAL, 0, dev->errmsg);
+      return -1;
+   }
+      
    dev->state &= ~(ST_EOT | ST_EOF);  /* remove EOF/EOT flags */
    mt_com.mt_op = MTWEOF;
    mt_com.mt_count = num;
@@ -1873,7 +1539,7 @@ weof_dev(DEVICE *dev, int num)
       clrerror_dev(dev, MTWEOF);
       if (stat == -1) {
          Mmsg2(dev->errmsg, _("ioctl MTWEOF error on %s. ERR=%s.\n"),
-           dev->dev_name, be.strerror());
+            dev->print_name(), be.strerror());
        }
    }
    return stat;
@@ -1903,7 +1569,7 @@ clrerror_dev(DEVICE *dev, int func)
    struct mtget mt_stat;
    char buf[100];
 
-   dev->dev_errno = errno;        /* save errno */
+   dev->dev_errno = errno;         /* save errno */
    if (errno == EIO) {
       dev->VolCatInfo.VolCatErrors++;
    }
@@ -1914,61 +1580,61 @@ 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");
-        break;
+         Emsg0(M_ABORT, 0, _("Got ENOTTY on read/write!\n"));
+         break;
       case MTWEOF:
          msg = "WTWEOF";
-        dev->capabilities &= ~CAP_EOF; /* turn off feature */
-        break;
+         dev->capabilities &= ~CAP_EOF; /* turn off feature */
+         break;
 #ifdef MTEOM
       case MTEOM:
          msg = "WTEOM";
-        dev->capabilities &= ~CAP_EOM; /* turn off feature */
-        break;
+         dev->capabilities &= ~CAP_EOM; /* turn off feature */
+         break;
 #endif
       case MTFSF:
          msg = "MTFSF";
-        dev->capabilities &= ~CAP_FSF; /* turn off feature */
-        break;
+         dev->capabilities &= ~CAP_FSF; /* turn off feature */
+         break;
       case MTBSF:
          msg = "MTBSF";
-        dev->capabilities &= ~CAP_BSF; /* turn off feature */
-        break;
+         dev->capabilities &= ~CAP_BSF; /* turn off feature */
+         break;
       case MTFSR:
          msg = "MTFSR";
-        dev->capabilities &= ~CAP_FSR; /* turn off feature */
-        break;
+         dev->capabilities &= ~CAP_FSR; /* turn off feature */
+         break;
       case MTBSR:
          msg = "MTBSR";
-        dev->capabilities &= ~CAP_BSR; /* turn off feature */
-        break;
+         dev->capabilities &= ~CAP_BSR; /* turn off feature */
+         break;
       case MTREW:
          msg = "MTREW";
-        break;
+         break;
 #ifdef MTSETBLK
       case MTSETBLK:
          msg = "MTSETBLK";
-        break;
+         break;
 #endif
 #ifdef MTSETBSIZ 
       case MTSETBSIZ:
          msg = "MTSETBSIZ";
-        break;
+         break;
 #endif
 #ifdef MTSRSZ
       case MTSRSZ:
          msg = "MTSRSZ";
-        break;
+         break;
 #endif
       default:
-         bsnprintf(buf, sizeof(buf), "unknown func code %d", func);
-        msg = buf;
-        break;
+         bsnprintf(buf, sizeof(buf), _("unknown func code %d"), func);
+         msg = buf;
+         break;
       }
       if (msg != NULL) {
-        dev->dev_errno = ENOSYS;
+         dev->dev_errno = ENOSYS;
          Mmsg1(dev->errmsg, _("I/O function \"%s\" not supported on this device.\n"), msg);
-        Emsg0(M_ERROR, 0, dev->errmsg);
+         Emsg0(M_ERROR, 0, dev->errmsg);
       }
    }
    /* On some systems such as NetBSD, this clears all errors */
@@ -2022,145 +1688,87 @@ int flush_dev(DEVICE *dev)
 static void do_close(DEVICE *dev)
 {
 
-   Dmsg1(29, "really close_dev %s\n", dev->dev_name);
+   Dmsg1(100, "really close_dev %s\n", dev->print_name());
    if (dev->fd >= 0) {
-      close(dev->fd);
+      ::close(dev->fd);
    }
-   if (dev_cap(dev, CAP_REQMOUNT)) {
-      if (unmount_dev(dev, 1) < 0) {
-         Dmsg1(0, "Cannot unmount device %s.\n", dev->dev_name);
-      }
+
+   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;
-      get_filename(dev, dev->VolCatInfo.VolCatName, archive_name);
+      Dmsg1(100, "Call make_dvd_filename. Vol=%s\n", dev->VolCatInfo.VolCatName);
+      make_spooled_dvd_filename(dev, archive_name);
       /* Check that the part file is empty */
       if ((stat(archive_name.c_str(), &statp) == 0) && (statp.st_size == 0)) {
-        unlink(archive_name.c_str());
+         Dmsg1(100, "unlink(%s)\n", archive_name.c_str());
+         unlink(archive_name.c_str());
       }
    }
    
    /* 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;
    dev->file_addr = 0;
    dev->part = 0;
+   dev->num_parts = 0;
    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;
 }
 
 /*
  * Close the device
  */
-void
-close_dev(DEVICE *dev)
+void DEVICE::close()
 {
-   if (!dev) {
-      Mmsg0(&dev->errmsg, _("Bad call to close_dev. Archive not open\n"));
-      Emsg0(M_FATAL, 0, dev->errmsg);
-      return;
-   }
-   /*if (dev->fd >= 0 && dev->use_count == 1) {*/
-   /* No need to check if dev->fd >= 0: it is checked again
-    * in do_close, and do_close MUST be called for volumes
-    * splitted in parts, even if dev->fd == -1. */
-   if (dev->use_count == 1) {
-      do_close(dev);
-   } else if (dev->use_count > 0) {
-      dev->use_count--;
-   }
-
-#ifdef FULL_DEBUG
-   ASSERT(dev->use_count >= 0);
-#endif
+   do_close(this);
 }
 
-/*
- * Used when unmounting the device, ignore use_count
- */
-void force_close_dev(DEVICE *dev)
-{
-   if (!dev) {
-      Mmsg0(&dev->errmsg, _("Bad call to force_close_dev. Archive not open\n"));
-      Emsg0(M_FATAL, 0, dev->errmsg);
-      return;
-   }
-   Dmsg1(29, "Force close_dev %s\n", dev->dev_name);
-   do_close(dev);
-
-#ifdef FULL_DEBUG
-   ASSERT(dev->use_count >= 0);
-#endif
-}
 
-bool truncate_dev(DEVICE *dev)
+bool truncate_dev(DCR *dcr) /* We need the DCR for DVD-writing */
 {
-   Dmsg1(100, "truncate_dev %s\n", dev->dev_name);
+   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 */
       /* maybe we should rewind and write and eof ???? */
    }
    
-   /* If there is more than one part, open the first one, and then truncate it. */
-   if (dev->num_parts > 0) {
-      dev->num_parts = 0;
-      dev->VolCatInfo.VolCatParts = 0;
-      if (open_first_part(dev) < 0) {
-        berrno be;
-         Mmsg1(&dev->errmsg, "Unable to truncate device, because I'm unable to open the first part. ERR=%s\n", be.strerror());
-      }
+   if (dev->is_dvd()) {
+      return truncate_dvd_dev(dcr);
    }
    
    if (ftruncate(dev->fd, 0) != 0) {
       berrno be;
-      Mmsg1(&dev->errmsg, _("Unable to truncate device. ERR=%s\n"), be.strerror());
+      Mmsg2(dev->errmsg, _("Unable to truncate device %s. ERR=%s\n"), 
+            dev->print_name(), be.strerror());
       return false;
    }
    return true;
 }
 
-bool
-dev_is_tape(DEVICE *dev)
+/* Return the resource name for the device */
+const char *DEVICE::name() const
 {
-   return dev->is_tape() ? true : false;
-}
-
-
-/*
- * return 1 if the device is read for write, and 0 otherwise
- *   This is meant for checking at the end of a job to see
- *   if we still have a tape (perhaps not if at end of tape
- *   and the job is canceled).
- */
-bool
-dev_can_write(DEVICE *dev)
-{
-   if (dev->is_open() &&  dev->can_append() &&
-       dev->is_labeled()  && !(dev->state & ST_WEOT)) {
-      return true;
-   } else {
-      return false;
-   }
-}
-
-char *
-dev_name(DEVICE *dev)
-{
-   return dev->dev_name;
+   return device->hdr.name;
 }
 
 char *
@@ -2189,16 +1797,20 @@ term_dev(DEVICE *dev)
 {
    if (!dev) {
       dev->dev_errno = EBADF;
-      Mmsg0(&dev->errmsg, _("Bad call to term_dev. Archive not open\n"));
+      Mmsg0(dev->errmsg, _("Bad call to term_dev. Device not open\n"));
       Emsg0(M_FATAL, 0, dev->errmsg);
       return;
    }
+   Dmsg1(29, "term_dev: %s\n", dev->print_name());
    do_close(dev);
-   Dmsg0(29, "term_dev\n");
    if (dev->dev_name) {
       free_memory(dev->dev_name);
       dev->dev_name = NULL;
    }
+   if (dev->prt_name) {
+      free_memory(dev->prt_name);
+      dev->prt_name = NULL;
+   }
    if (dev->errmsg) {
       free_pool_memory(dev->errmsg);
       dev->errmsg = NULL;
@@ -2220,26 +1832,51 @@ term_dev(DEVICE *dev)
 /*
  * This routine initializes the device wait timers
  */
-void init_dev_wait_timers(DEVICE *dev)
+void init_device_wait_timers(DCR *dcr)
 {
+   DEVICE *dev = dcr->dev;
+   JCR *jcr = dcr->jcr;
+
    /* ******FIXME******* put these on config variables */
    dev->min_wait = 60 * 60;
    dev->max_wait = 24 * 60 * 60;
-   dev->max_num_wait = 9;             /* 5 waits =~ 1 day, then 1 day at a time */
+   dev->max_num_wait = 9;              /* 5 waits =~ 1 day, then 1 day at a time */
    dev->wait_sec = dev->min_wait;
    dev->rem_wait_sec = dev->wait_sec;
    dev->num_wait = 0;
    dev->poll = false;
    dev->BadVolName[0] = 0;
+
+   jcr->min_wait = 60 * 60;
+   jcr->max_wait = 24 * 60 * 60;
+   jcr->max_num_wait = 9;              /* 5 waits =~ 1 day, then 1 day at a time */
+   jcr->wait_sec = jcr->min_wait;
+   jcr->rem_wait_sec = jcr->wait_sec;
+   jcr->num_wait = 0;
+
+}
+
+void init_jcr_device_wait_timers(JCR *jcr)
+{
+   /* ******FIXME******* put these on config variables */
+   jcr->min_wait = 60 * 60;
+   jcr->max_wait = 24 * 60 * 60;
+   jcr->max_num_wait = 9;              /* 5 waits =~ 1 day, then 1 day at a time */
+   jcr->wait_sec = jcr->min_wait;
+   jcr->rem_wait_sec = jcr->wait_sec;
+   jcr->num_wait = 0;
 }
 
+
 /*
+ * The dev timers are used for waiting on a particular device 
+ *
  * Returns: true if time doubled
- *         false if max time expired
+ *          false if max time expired
  */
 bool double_dev_wait_time(DEVICE *dev)
 {
-   dev->wait_sec *= 2;              /* double wait time */
+   dev->wait_sec *= 2;               /* double wait time */
    if (dev->wait_sec > dev->max_wait) {   /* but not longer than maxtime */
       dev->wait_sec = dev->max_wait;
    }
@@ -2251,6 +1888,7 @@ bool double_dev_wait_time(DEVICE *dev)
    return true;
 }
 
+
 void set_os_device_parameters(DEVICE *dev)
 {
 #ifdef HAVE_LINUX_OS
@@ -2260,18 +1898,18 @@ void set_os_device_parameters(DEVICE *dev)
       mt_com.mt_op = MTSETBLK;
       mt_com.mt_count = 0;
       if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
-        clrerror_dev(dev, MTSETBLK);
+         clrerror_dev(dev, MTSETBLK);
       }
       mt_com.mt_op = MTSETDRVBUFFER;
       mt_com.mt_count = MT_ST_CLEARBOOLEANS;
       if (!dev_cap(dev, CAP_TWOEOF)) {
-        mt_com.mt_count |= MT_ST_TWO_FM;
+         mt_com.mt_count |= MT_ST_TWO_FM;
       }
       if (dev_cap(dev, CAP_EOM)) {
-        mt_com.mt_count |= MT_ST_FAST_MTEOM;
+         mt_com.mt_count |= MT_ST_FAST_MTEOM;
       }
       if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
-        clrerror_dev(dev, MTSETBLK);
+         clrerror_dev(dev, MTSETBLK);
       }
    }
    return;
@@ -2284,13 +1922,13 @@ void set_os_device_parameters(DEVICE *dev)
       mt_com.mt_op = MTSETBSIZ;
       mt_com.mt_count = 0;
       if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
-        clrerror_dev(dev, MTSETBSIZ);
+         clrerror_dev(dev, MTSETBSIZ);
       }
       /* Get notified at logical end of tape */
       mt_com.mt_op = MTEWARN;
       mt_com.mt_count = 1;
       if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
-        clrerror_dev(dev, MTEWARN);
+         clrerror_dev(dev, MTEWARN);
       }
    }
    return;
@@ -2303,7 +1941,7 @@ void set_os_device_parameters(DEVICE *dev)
       mt_com.mt_op = MTSETBSIZ;
       mt_com.mt_count = 0;
       if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
-        clrerror_dev(dev, MTSETBSIZ);
+         clrerror_dev(dev, MTSETBSIZ);
       }
    }
    return;
@@ -2316,69 +1954,29 @@ void set_os_device_parameters(DEVICE *dev)
       mt_com.mt_op = MTSRSZ;
       mt_com.mt_count = 0;
       if (ioctl(dev->fd, MTIOCTOP, (char *)&mt_com) < 0) {
-        clrerror_dev(dev, MTSRSZ);
+         clrerror_dev(dev, MTSRSZ);
       }
    }
    return;
 #endif
 }
 
-/*
- * Edit codes into (Un)MountCommand, Write(First)PartCommand
- *  %% = %
- *  %a = archive device name
- *  %m = mount point
- *  %v = last part name
- *
- *  omsg = edited output message
- *  imsg = input string containing edit codes (%x)
- *
- */
-char *edit_device_codes_dev(DEVICE* dev, char *omsg, const char *imsg)
+static bool dev_get_os_pos(DEVICE *dev, struct mtget *mt_stat)
 {
-   const char *p;
-   const char *str;
-   char add[20];
-   
-   POOL_MEM archive_name(PM_FNAME);
-   get_filename(dev, dev->VolCatInfo.VolCatName, archive_name);
-
-   *omsg = 0;
-   Dmsg1(800, "edit_device_codes: %s\n", imsg);
-   for (p=imsg; *p; p++) {
-      if (*p == '%') {
-        switch (*++p) {
-            case '%':
-               str = "%";
-              break;
-            case 'n':
-               bsnprintf(add, sizeof(add), "%d", dev->part);
-              str = add;
-              break;
-            case 'a':
-              str = dev->dev_name;
-              break;
-            case 'm':
-              str = dev->device->mount_point;
-              break;
-            case 'v':
-              str = archive_name.c_str();
-              break;
-           default:
-               add[0] = '%';
-              add[1] = *p;
-              add[2] = 0;
-              str = add;
-              break;
-        }
-      } else {
-        add[0] = *p;
-        add[1] = 0;
-        str = add;
-      }
-      Dmsg1(900, "add_str %s\n", str);
-      pm_strcat(&omsg, (char *)str);
-      Dmsg1(800, "omsg=%s\n", omsg);
-   }
-   return omsg;
+   return dev_cap(dev, CAP_MTIOCGET) && 
+          ioctl(dev->fd, MTIOCGET, (char *)mt_stat) == 0 &&
+          mt_stat->mt_fileno >= 0;
+}
+
+static char *modes[] = {
+   "CREATE_READ_WRITE",
+   "OPEN_READ_WRITE",
+   "OPEN_READ_ONLY",
+   "OPEN_WRITE_ONLY"
+};
+
+
+static char *mode_to_str(int mode)  
+{
+   return modes[mode-1];
 }