Author: avg
Date: Wed Jun 21 18:19:27 2017
New Revision: 320195
URL: https://svnweb.freebsd.org/changeset/base/320195

Log:
  bhyveload: correctly query size of disks
  
  On FreeBSD fstat(2) works fine for querying sizes of plain files,
  but not so much for character devices.
  So, use DIOCGMEDIASIZE to try to get the correct size for disks
  and disk-like devices (e.g. zvols).
  
  PR:           220186
  Reviewed by:  tsoome, grehan
  MFC after:    1 week

Modified:
  head/usr.sbin/bhyveload/bhyveload.c

Modified: head/usr.sbin/bhyveload/bhyveload.c
==============================================================================
--- head/usr.sbin/bhyveload/bhyveload.c Wed Jun 21 18:17:32 2017        
(r320194)
+++ head/usr.sbin/bhyveload/bhyveload.c Wed Jun 21 18:19:27 2017        
(r320195)
@@ -311,10 +311,12 @@ cb_diskioctl(void *arg, int unit, u_long cmd, void *da
                *(u_int *)data = 512;
                break;
        case DIOCGMEDIASIZE:
-               if (fstat(disk_fd[unit], &sb) == 0)
-                       *(off_t *)data = sb.st_size;
-               else
+               if (fstat(disk_fd[unit], &sb) != 0)
                        return (ENOTTY);
+               if (S_ISCHR(sb.st_mode) &&
+                   ioctl(disk_fd[unit], DIOCGMEDIASIZE, &sb.st_size) != 0)
+                               return (ENOTTY);
+               *(off_t *)data = sb.st_size;
                break;
        default:
                return (ENOTTY);
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to