Merge pull request #4041 from ashwat287/UpdateReadonlyOption

jandubois · web-flow · commit e18298ad07a8 · 2025-09-16T14:38:56.000-07:00
fix: update the qemu option 'readonly' to 'readonly=on'
diff --git a/pkg/driver/qemu/qemu.go b/pkg/driver/qemu/qemu.go
@@ -935,7 +935,7 @@ func Cmdline(ctx context.Context, cfg Config) (exe string, args []string, err er
 				options += fmt.Sprintf(",path=%s", f.Location)
 				options += fmt.Sprintf(",security_model=%s", *f.NineP.SecurityModel)
 				if !*f.Writable {
-					options += ",readonly"
+					options += ",readonly=on"
 				}
 				args = append(args, "-virtfs", options)
 			case limatype.VIRTIOFS:

Original file line number	Diff line number	Diff line change
`@@ -935,7 +935,7 @@ func Cmdline(ctx context.Context, cfg Config) (exe string, args []string, err er`
`935`	`935`	`options += fmt.Sprintf(",path=%s", f.Location)`
`936`	`936`	`options += fmt.Sprintf(",security_model=%s", *f.NineP.SecurityModel)`
`937`	`937`	`if !*f.Writable {`
`938`		`- options += ",readonly"`
	`938`	`+ options += ",readonly=on"`
`939`	`939`	`}`
`940`	`940`	`args = append(args, "-virtfs", options)`
`941`	`941`	`case limatype.VIRTIOFS:`