Bug 196243 - [bhyve] vmrun.sh spurious warning using zvol diskdev
Summary: [bhyve] vmrun.sh spurious warning using zvol diskdev
Status: New
Alias: None
Product: Base System
Classification: Unclassified
Component: misc (show other bugs)
Version: 10.1-RELEASE
Hardware: amd64 Any
: Normal Affects Some People
Assignee: Rodney W. Grimes
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-24 01:24 UTC by PaulC
Modified: 2018-02-02 18:11 UTC (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description PaulC 2014-12-24 01:24:19 UTC
Hello,

I noticed that there is a minor problem in the vmrun.sh script if you are using a zvol as the virtio_diskdev. The 'make_and_check_diskdev' check will fail if virtio_diskdev is a zvol as it only checks for files (vs char devices). This is harmless as the truncate doesn't actually do anything if the device is a zvol however the attached small patch fixes the problem and removes the spurious error message.

Paul


--- vmrun.sh.orig       2014-12-24 01:56:58.468954569 +0100
+++ vmrun.sh    2014-12-24 02:03:23.543145433 +0100
@@ -152,7 +152,7 @@
 {
     local virtio_diskdev="$1"
     # Create the virtio diskdev file if needed
-    if [ ! -f ${virtio_diskdev} ]; then
+    if [ ! -f ${virtio_diskdev} -a ! -c ${virtio_diskdev} ]; then
            echo "virtio disk device file \"${virtio_diskdev}\" does not exist."
            echo "Creating it ..."
            truncate -s 8G ${virtio_diskdev} > /dev/null
Comment 1 Jimmy Olgeni freebsd_committer freebsd_triage 2015-05-27 09:19:25 UTC
CURRENT already has "-e ${virtio_diskdev}", so maybe vmrun.sh can just be merged?
Comment 2 Neel Natu freebsd_committer freebsd_triage 2015-05-27 16:25:43 UTC
(In reply to Jimmy Olgeni from comment #1)

Yes, there are a number of changes that need to be merged before the stable/10 slush. I'll do this one along with the others.
Comment 3 amvandemore 2015-10-28 21:18:45 UTC
Ping
Comment 4 Rodney W. Grimes freebsd_committer freebsd_triage 2018-02-02 18:11:08 UTC
I am taking any input on this anyone has.
I plan to test the state on ^head, and if we need to patch vmrun.sh that shall be done shortly.