[Bug 221067] vfs.root.mountfrom.options="ro" did not take effect for a zfs root filesystem

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

Mark Linimon <***@FreeBSD.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Assignee|freebsd-***@FreeBSD.org |freebsd-***@FreeBSD.org
Keywords| |patch
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-***@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-fs
To unsubscribe, send any mail to "freebsd-fs-***@freebsd.org"

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

--- Comment #2 from Andriy Gapon <***@FreeBSD.org> ---
If I am reading comment #0 correctly, the reporter wants
vfs.root.mountfrom.options="ro" to imply the read-only import of the pool.
I think that that's not what vfs.root.mountfrom.options is for. We could add a
different knob for that as it might be useful in certain situations.
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-***@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-fs
To unsubscribe, send any mail to "freebsd-fs-***@freebsd.org"

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

--- Comment #3 from Allan Jude <***@FreeBSD.org> ---
(In reply to Andriy Gapon from comment #2)
I would tend to agree, having the entire pool be readonly would be unexpected
when the sysctl suggests only the root filesystem will be readonly.
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-***@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-fs
To unsubscribe, send any mail to "freebsd-fs-***@freebsd.org"

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

--- Comment #4 from David NewHamlet <***@gmail.com> ---
(In reply to Allan Jude from comment #1)

I hit this issue when trying to use a zfs as rootfs which use a compressed
md_image as underlayer device. A compressed md_image can not be opened in write
mode(see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221066).

So in this case I want to open the whole zfs pool in readonly.

A new kenv entry to control this behavior also make sense.
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-***@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-fs
To unsubscribe, send any mail to "freebsd-fs-***@freebsd.org"

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

Alexander Vereeken <***@protonmail.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Keywords|patch |

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

Mark Linimon <***@FreeBSD.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #184793|0 |1
is obsolete| |

--- Comment #5 from Mark Linimon <***@FreeBSD.org> ---
Created attachment 257791
--> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=257791&action=edit
prototype fix for reference

^Triage: attempt to rebase patch.

[b***@freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221067

Mark Linimon <***@FreeBSD.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@FreeBSD.org

--- Comment #6 from Mark Linimon <***@FreeBSD.org> ---
(In reply to Mark Linimon from comment #5)
Note that this was insufficient: in the new ZFS codebase, this file is at:

./sys/contrib/openzfs/module/zfs/spa.c

To submitter: given that we are on the new codebase, is this aging PR still
relevant?