commit:     7cb8d943236fe651ac54c64f8167f7c4369f649c
Author:     William Hubbs <w.d.hubbs <AT> gmail <DOT> com>
AuthorDate: Tue Aug 14 15:21:27 2018 +0000
Commit:     William Hubbs <williamh <AT> gentoo <DOT> org>
CommitDate: Tue Aug 14 15:21:27 2018 +0000
URL:        https://gitweb.gentoo.org/proj/openrc.git/commit/?id=7cb8d943

Stop mounting efivarfs read-only

We do not need to do this any longer since all supported linux kernels
make efivarfs immutable and the tools that manipulate it are aware of
this feature.

This fixes https://github.com/openrc/openrc/issues/238.

 NEWS.md         | 7 +++++++
 init.d/sysfs.in | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/NEWS.md b/NEWS.md
index e9605031..3767c18b 100644
--- a/NEWS.md
+++ b/NEWS.md
@@ -11,6 +11,13 @@ The only place I know that this was used was Gentoo 
Baselayout 1.x, so
 it shouldn't affect anyone since baselayout-1 has been dead for a few
 years.
 
+Since all supported Linux kernel versions now make efivarfs immutable
+and all of the tools that access efivarfs are aware of this, we no
+longer mount efivarfs read-only. See the following github issue for more
+information:
+
+https://github.com/openrc/openrc/issues/238
+
 ## OpenRC 0.37
 
 start-stop-daemon now supports logging stdout and stderr of daemons to

diff --git a/init.d/sysfs.in b/init.d/sysfs.in
index f0d02e5c..89707120 100644
--- a/init.d/sysfs.in
+++ b/init.d/sysfs.in
@@ -101,7 +101,7 @@ mount_misc()
        if [ -d /sys/firmware/efi/efivars ] &&
                ! mountinfo -q /sys/firmware/efi/efivars; then
                ebegin "Mounting efivarfs filesystem"
-               mount -n -t efivarfs -o ro \
+               mount -n -t efivarfs -o ${sysfs_opts} \
                        efivarfs /sys/firmware/efi/efivars 2> /dev/null
                eend 0
        fi

Reply via email to