qemu: use 9p by default

Templates for the following distro are updated to continue using reverse-sshfs, as 9p is not available on them. - AlmaLinux - CentOS Stream - Debian - openSUSE - Oracle Linux - Rocky Linux Close issue 971 Signed-off-by: Akihiro Suda <[email protected]>
lima-vm · Oct 4, 2024 · 47565ec · 47565ec
1 parent fd26375
commit 47565ec
Show file tree

Hide file tree

Showing 18 changed files with 112 additions and 33 deletions.
diff --git a/examples/almalinux-8.yaml b/examples/almalinux-8.yaml
@@ -16,6 +16,7 @@ images:
   arch: "x86_64"
 - location: "https://repo.almalinux.org/almalinux/8/cloud/aarch64/images/AlmaLinux-8-GenericCloud-latest.aarch64.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/almalinux-9.yaml b/examples/almalinux-9.yaml
@@ -13,6 +13,7 @@ images:
   arch: "x86_64"
 - location: "https://repo.almalinux.org/almalinux/9/cloud/aarch64/images/AlmaLinux-9-GenericCloud-latest.aarch64.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/centos-stream-9.yaml b/examples/centos-stream-9.yaml
@@ -14,6 +14,7 @@ images:
   arch: "x86_64"
 - location: "https://cloud.centos.org/centos/9-stream/aarch64/images/CentOS-Stream-GenericCloud-9-latest.aarch64.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/debian-11.yaml b/examples/debian-11.yaml
@@ -13,7 +13,7 @@ images:
   arch: "x86_64"
 - location: "https://cloud.debian.org/images/cloud/bullseye/latest/debian-11-genericcloud-arm64.qcow2"
   arch: "aarch64"
-
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/debian-12.yaml b/examples/debian-12.yaml
@@ -13,7 +13,7 @@ images:
   arch: "x86_64"
 - location: "https://cloud.debian.org/images/cloud/bookworm/latest/debian-12-genericcloud-arm64.qcow2"
   arch: "aarch64"
-
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/default.yaml b/examples/default.yaml
@@ -111,9 +111,13 @@ mounts:
   # 🔵 This file: true (only for "/tmp/lima")
   writable: true
 
+# Deny list of mount types for resolving the "default" type.
+# 🟢 Builtin default: null
+mountTypesUnsupported: null
+
 # Mount type for above mounts, such as "reverse-sshfs" (from sshocker), "9p" (EXPERIMENTAL until Lima v1.0, from QEMU’s virtio-9p-pci, aka virtfs),
 # or "virtiofs" (EXPERIMENTAL, needs `vmType: vz`; will graduate from experimental in Lima v1.0)
-# 🟢 Builtin default: "reverse-sshfs" (for QEMU), "virtiofs" (for vz)
+# 🟢 Builtin default: "default" (resolved to be "9p" for QEMU since Lima v1.0, "virtiofs" for vz)
 mountType: null
 
 # Enable inotify support for mounted directories (EXPERIMENTAL)

diff --git a/examples/experimental/opensuse-tumbleweed.yaml b/examples/experimental/opensuse-tumbleweed.yaml
@@ -5,6 +5,7 @@ images:
   arch: "x86_64"
 - location: "https://download.opensuse.org/ports/aarch64/tumbleweed/appliances/openSUSE-Tumbleweed-Minimal-VM.aarch64-Cloud.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/opensuse-leap.yaml b/examples/opensuse-leap.yaml
@@ -5,6 +5,7 @@ images:
   arch: "x86_64"
 - location: "https://download.opensuse.org/distribution/leap/15.6/appliances/openSUSE-Leap-15.6-Minimal-VM.aarch64-Cloud.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/oraclelinux-8.yaml b/examples/oraclelinux-8.yaml
@@ -12,6 +12,7 @@ images:
 - location: "https://yum.oracle.com/templates/OracleLinux/OL8/u10/aarch64/OL8U10_aarch64-kvm-cloud-b100.qcow2"
   arch: "aarch64"
   digest: "sha256:def7b8055e275507a593f07dc6076a2adc5967af046c003072b479e69f25f407"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/oraclelinux-9.yaml b/examples/oraclelinux-9.yaml
@@ -9,6 +9,7 @@ images:
 - location: "https://yum.oracle.com/templates/OracleLinux/OL9/u4/aarch64/OL9U4_aarch64-kvm-cloud-b90.qcow2"
   arch: "aarch64"
   digest: "sha256:1f4e20190e87c76e8c3b4a9e15e660972386cbfa4f128e5cdcd8faa43a713d44"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/rocky-8.yaml b/examples/rocky-8.yaml
@@ -16,6 +16,7 @@ images:
   arch: "x86_64"
 - location: "https://dl.rockylinux.org/pub/rocky/8/images/aarch64/Rocky-8-GenericCloud.latest.aarch64.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/examples/rocky-9.yaml b/examples/rocky-9.yaml
@@ -14,6 +14,7 @@ images:
   arch: "x86_64"
 - location: "https://dl.rockylinux.org/pub/rocky/9/images/aarch64/Rocky-9-GenericCloud.latest.aarch64.qcow2"
   arch: "aarch64"
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
 - location: "/tmp/lima"

diff --git a/hack/test-templates/test-misc.yaml b/hack/test-templates/test-misc.yaml
@@ -1,5 +1,6 @@
 # The test template for testing misc configurations:
 # - disk
+# - snapshots
 # - (More to come)
 #
 # This template requires Lima v1.0.0-alpha.0 or later.
@@ -18,6 +19,8 @@ images:
 - location: "https://cloud-images.ubuntu.com/releases/24.04/release/ubuntu-24.04-server-cloudimg-arm64.img"
   arch: "aarch64"
 
+# 9p is not compatible with `limactl snapshot`
+mountTypesUnsupported: ["9p"]
 mounts:
 - location: "~"
   writable: true

diff --git a/pkg/limayaml/defaults.go b/pkg/limayaml/defaults.go
@@ -27,6 +27,7 @@ import (
 	"github.com/lima-vm/lima/pkg/ptr"
 	"github.com/lima-vm/lima/pkg/store/dirnames"
 	"github.com/lima-vm/lima/pkg/store/filenames"
+	"github.com/lima-vm/lima/pkg/version/versionutil"
 )
 
 const (
@@ -171,6 +172,15 @@ func defaultGuestInstallPrefix() string {
 //   - CACertificates Files and Certs are uniquely appended in d, y, o order
 func FillDefault(y, d, o *LimaYAML, filePath string) {
 	instDir := filepath.Dir(filePath)
+
+	var existingLimaVersion string // empty if the instance was created with Lima prior to v0.20
+	limaVersionFile := filepath.Join(instDir, filenames.LimaVersion)
+	if b, err := os.ReadFile(limaVersionFile); err == nil {
+		existingLimaVersion = strings.TrimSpace(string(b))
+	} else if !errors.Is(err, os.ErrNotExist) {
+		logrus.WithError(err).Warnf("Failed to read %q", limaVersionFile)
+	}
+
 	if y.VMType == nil {
 		y.VMType = d.VMType
 	}
@@ -547,21 +557,41 @@ func FillDefault(y, d, o *LimaYAML, filePath string) {
 		}
 	}
 
+	y.MountTypesUnsupported = append(append(o.MountTypesUnsupported, y.MountTypesUnsupported...), d.MountTypesUnsupported...)
+	mountTypesUnsupported := make(map[string]struct{})
+	for _, f := range y.MountTypesUnsupported {
+		mountTypesUnsupported[f] = struct{}{}
+	}
 	// MountType has to be resolved before resolving Mounts
 	if y.MountType == nil {
 		y.MountType = d.MountType
 	}
 	if o.MountType != nil {
 		y.MountType = o.MountType
 	}
-	if y.MountType == nil || *y.MountType == "" {
-		if *y.VMType == VZ {
+	if y.MountType == nil || *y.MountType == "" || *y.MountType == "default" {
+		switch *y.VMType {
+		case VZ:
 			y.MountType = ptr.Of(VIRTIOFS)
-		} else {
+		case QEMU:
+			y.MountType = ptr.Of(NINEP)
+			if _, ok := mountTypesUnsupported[NINEP]; ok {
+				// Use REVSSHFS if the instance does not support 9p
+				y.MountType = ptr.Of(REVSSHFS)
+			} else if isExistingInstanceDir(instDir) && !versionutil.GreaterEqual(existingLimaVersion, "1.0.0") {
+				// Use REVSSHFS if the instance was created with Lima prior to v1.0
+				y.MountType = ptr.Of(REVSSHFS)
+			}
+		default:
 			y.MountType = ptr.Of(REVSSHFS)
 		}
 	}
 
+	if _, ok := mountTypesUnsupported[*y.MountType]; ok {
+		// We cannot return an error here, but Validate() will return it.
+		logrus.Warnf("Unsupported mount type: %q", *y.MountType)
+	}
+
 	if y.MountInotify == nil {
 		y.MountInotify = d.MountInotify
 	}

diff --git a/pkg/limayaml/limayaml.go b/pkg/limayaml/limayaml.go
@@ -7,33 +7,34 @@ import (
 )
 
 type LimaYAML struct {
-	MinimumLimaVersion *string       `yaml:"minimumLimaVersion,omitempty" json:"minimumLimaVersion,omitempty"`
-	VMType             *VMType       `yaml:"vmType,omitempty" json:"vmType,omitempty"`
-	VMOpts             VMOpts        `yaml:"vmOpts,omitempty" json:"vmOpts,omitempty"`
-	OS                 *OS           `yaml:"os,omitempty" json:"os,omitempty"`
-	Arch               *Arch         `yaml:"arch,omitempty" json:"arch,omitempty"`
-	Images             []Image       `yaml:"images" json:"images"` // REQUIRED
-	CPUType            CPUType       `yaml:"cpuType,omitempty" json:"cpuType,omitempty"`
-	CPUs               *int          `yaml:"cpus,omitempty" json:"cpus,omitempty"`
-	Memory             *string       `yaml:"memory,omitempty" json:"memory,omitempty"` // go-units.RAMInBytes
-	Disk               *string       `yaml:"disk,omitempty" json:"disk,omitempty"`     // go-units.RAMInBytes
-	AdditionalDisks    []Disk        `yaml:"additionalDisks,omitempty" json:"additionalDisks,omitempty"`
-	Mounts             []Mount       `yaml:"mounts,omitempty" json:"mounts,omitempty"`
-	MountType          *MountType    `yaml:"mountType,omitempty" json:"mountType,omitempty"`
-	MountInotify       *bool         `yaml:"mountInotify,omitempty" json:"mountInotify,omitempty"`
-	SSH                SSH           `yaml:"ssh,omitempty" json:"ssh,omitempty"` // REQUIRED (FIXME)
-	Firmware           Firmware      `yaml:"firmware,omitempty" json:"firmware,omitempty"`
-	Audio              Audio         `yaml:"audio,omitempty" json:"audio,omitempty"`
-	Video              Video         `yaml:"video,omitempty" json:"video,omitempty"`
-	Provision          []Provision   `yaml:"provision,omitempty" json:"provision,omitempty"`
-	UpgradePackages    *bool         `yaml:"upgradePackages,omitempty" json:"upgradePackages,omitempty"`
-	Containerd         Containerd    `yaml:"containerd,omitempty" json:"containerd,omitempty"`
-	GuestInstallPrefix *string       `yaml:"guestInstallPrefix,omitempty" json:"guestInstallPrefix,omitempty"`
-	Probes             []Probe       `yaml:"probes,omitempty" json:"probes,omitempty"`
-	PortForwards       []PortForward `yaml:"portForwards,omitempty" json:"portForwards,omitempty"`
-	CopyToHost         []CopyToHost  `yaml:"copyToHost,omitempty" json:"copyToHost,omitempty"`
-	Message            string        `yaml:"message,omitempty" json:"message,omitempty"`
-	Networks           []Network     `yaml:"networks,omitempty" json:"networks,omitempty"`
+	MinimumLimaVersion    *string       `yaml:"minimumLimaVersion,omitempty" json:"minimumLimaVersion,omitempty"`
+	VMType                *VMType       `yaml:"vmType,omitempty" json:"vmType,omitempty"`
+	VMOpts                VMOpts        `yaml:"vmOpts,omitempty" json:"vmOpts,omitempty"`
+	OS                    *OS           `yaml:"os,omitempty" json:"os,omitempty"`
+	Arch                  *Arch         `yaml:"arch,omitempty" json:"arch,omitempty"`
+	Images                []Image       `yaml:"images" json:"images"` // REQUIRED
+	CPUType               CPUType       `yaml:"cpuType,omitempty" json:"cpuType,omitempty"`
+	CPUs                  *int          `yaml:"cpus,omitempty" json:"cpus,omitempty"`
+	Memory                *string       `yaml:"memory,omitempty" json:"memory,omitempty"` // go-units.RAMInBytes
+	Disk                  *string       `yaml:"disk,omitempty" json:"disk,omitempty"`     // go-units.RAMInBytes
+	AdditionalDisks       []Disk        `yaml:"additionalDisks,omitempty" json:"additionalDisks,omitempty"`
+	Mounts                []Mount       `yaml:"mounts,omitempty" json:"mounts,omitempty"`
+	MountTypesUnsupported []string      `yaml:"mountTypesUnsupported,omitempty" json:"mountTypesUnsupported,omitempty"`
+	MountType             *MountType    `yaml:"mountType,omitempty" json:"mountType,omitempty"`
+	MountInotify          *bool         `yaml:"mountInotify,omitempty" json:"mountInotify,omitempty"`
+	SSH                   SSH           `yaml:"ssh,omitempty" json:"ssh,omitempty"` // REQUIRED (FIXME)
+	Firmware              Firmware      `yaml:"firmware,omitempty" json:"firmware,omitempty"`
+	Audio                 Audio         `yaml:"audio,omitempty" json:"audio,omitempty"`
+	Video                 Video         `yaml:"video,omitempty" json:"video,omitempty"`
+	Provision             []Provision   `yaml:"provision,omitempty" json:"provision,omitempty"`
+	UpgradePackages       *bool         `yaml:"upgradePackages,omitempty" json:"upgradePackages,omitempty"`
+	Containerd            Containerd    `yaml:"containerd,omitempty" json:"containerd,omitempty"`
+	GuestInstallPrefix    *string       `yaml:"guestInstallPrefix,omitempty" json:"guestInstallPrefix,omitempty"`
+	Probes                []Probe       `yaml:"probes,omitempty" json:"probes,omitempty"`
+	PortForwards          []PortForward `yaml:"portForwards,omitempty" json:"portForwards,omitempty"`
+	CopyToHost            []CopyToHost  `yaml:"copyToHost,omitempty" json:"copyToHost,omitempty"`
+	Message               string        `yaml:"message,omitempty" json:"message,omitempty"`
+	Networks              []Network     `yaml:"networks,omitempty" json:"networks,omitempty"`
 	// `network` was deprecated in Lima v0.7.0, removed in Lima v0.14.0. Use `networks` instead.
 	Env          map[string]string `yaml:"env,omitempty" json:"env,omitempty"`
 	Param        map[string]string `yaml:"param,omitempty" json:"param,omitempty"`

diff --git a/pkg/limayaml/validate.go b/pkg/limayaml/validate.go
@@ -185,6 +185,12 @@ func Validate(y *LimaYAML, warn bool) error {
 		return fmt.Errorf("field `mountType` must be %q or %q or %q, or %q, got %q", REVSSHFS, NINEP, VIRTIOFS, WSLMount, *y.MountType)
 	}
 
+	for _, f := range y.MountTypesUnsupported {
+		if f == *y.MountType {
+			return fmt.Errorf("field `mountType` must not be one of %v (`mountTypesUnsupported`), got %q", y.MountTypesUnsupported, *y.MountType)
+		}
+	}
+
 	if warn && runtime.GOOS != "linux" {
 		for i, mount := range y.Mounts {
 			if mount.Virtiofs.QueueSize != nil {

diff --git a/pkg/version/versionutil/versionutil.go b/pkg/version/versionutil/versionutil.go
@@ -38,3 +38,9 @@ func GreaterThan(limaVersion, oldVersion string) bool {
 	}
 	panic("unreachable")
 }
+
+// GreaterEqual return true if limaVersion >= oldVersion.
+func GreaterEqual(limaVersion, oldVersion string) bool {
+	return strings.TrimPrefix(limaVersion, "v") == strings.TrimPrefix(oldVersion, "v") ||
+		GreaterThan(limaVersion, oldVersion)
+}
diff --git a/website/content/en/docs/faq/_index.md b/website/content/en/docs/faq/_index.md
@@ -32,6 +32,7 @@ weight: 6
 - [Filesystem sharing](#filesystem-sharing)
   - ["Filesystem is slow"](#filesystem-is-slow)
   - ["Filesystem is not writable"](#filesystem-is-not-writable)
+  - ["Filesystem is unmounted after upgrading Lima to v1.0 (pre-release)"](#filesystem-is-unmounted-after-upgrading-lima-to-v10-pre-release)
 - [External projects](#external-projects)
   - ["I am using Rancher Desktop. How to deal with the underlying Lima?"](#i-am-using-rancher-desktop-how-to-deal-with-the-underlying-lima)
 - ["Hints for debugging other problems?"](#hints-for-debugging-other-problems)
@@ -230,6 +231,25 @@ mounts:
 
 Run `limactl edit <INSTANCE>` to open the YAML editor for an existing instance.
 
+#### "Filesystem is unmounted after upgrading Lima to v1.0 (pre-release)"
+
+Lima v1.0 (pre-release) changed the default mount type for QEMU from `reverse-sshfs` to `9p`.
+
+The `9p` mount type is known to be incompatible with the following guest operating systems:
+- AlmaLinux, CentOS Stream, Oracle Linux, and RockyLinux
+- Debian GNU/Linux
+- openSUSE
+
+A new instance of these OS still use `reverse-sshfs` by default.
+However, an existing instance created with a previous version of Lima may potentially need
+running the following command (usually not needed):
+
+```
+limactl edit --mount-type=reverse-sshfs <NAME>
+```
+
+Ubuntu users are not affected by this issue.
+
 ### External projects
 #### "I am using Rancher Desktop. How to deal with the underlying Lima?"