Re: [PATCH][jethro] systemd: re-enable mount propagation for udevd

[Alejandro Hernandez <alejandro.hernandez@linux.intel.com>]

Sorry Joshua I screwed up sending the email, this is meant to be for 
Robert Yang since its for jethro

On 05/26/2016 03:04 PM, Alejandro Hernandez wrote:
> From: Roy Li <rongqing.li@windriver.com>
>
> With MountFlags=slave, those mounts then become private to the systemd-udevd
> namespace and are no longer accessible from outside the namespace, which is
> not expected
>
> [YOCTO #8613]
>
> (From OE-Core rev: 73f43d857fe0102033f25491007b6dbe3d5fa8ee)
>
> Signed-off-by: Roy Li <rongqing.li@windriver.com>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> (cherry picked from commit f2092e67ea880301058396b831a9a18905317d0d)
> Signed-off-by: Alejandro Hernandez <alejandro.hernandez@linux.intel.com>
> ---
>   ...dev-re-enable-mount-propagation-for-udevd.patch | 31 ++++++++++++++++++++++
>   meta/recipes-core/systemd/systemd_225.bb           |  1 +
>   2 files changed, 32 insertions(+)
>   create mode 100644 meta/recipes-core/systemd/systemd/udev-re-enable-mount-propagation-for-udevd.patch
>
> diff --git a/meta/recipes-core/systemd/systemd/udev-re-enable-mount-propagation-for-udevd.patch b/meta/recipes-core/systemd/systemd/udev-re-enable-mount-propagation-for-udevd.patch
> new file mode 100644
> index 0000000..23e22d4
> --- /dev/null
> +++ b/meta/recipes-core/systemd/systemd/udev-re-enable-mount-propagation-for-udevd.patch
> @@ -0,0 +1,31 @@
> +From: Michael Biebl <biebl@debian.org>
> +Date: Sat, 27 Sep 2014 04:19:24 +0200
> +Subject: udev: re-enable mount propagation for udevd
> +
> +Upstream-Status: Backport [http://http.debian.net/debian/pool/main/s/systemd/systemd_215-17+deb8u4.debian.tar.xz]
> +
> +laptop-mode-tools remounts file systems from within a udev rule to apply
> +certain mount options. With MountFlags=slave, those mounts then become private
> +to the systemd-udevd namespace and are no longer accessible from outside the
> +namespace.
> +While the root cause is the broken behaviour of laptop-mode-tools, with mount
> +propagation turned off, this can result in a read-only root file system.
> +Therefore revert the relevant parts from commit
> +c2c13f2df42e0691aecabe3979ea81cd7faa35c7 to re-enable mount propagation for
> +udevd.
> +
> +Once affected packages have been fixed, this patch should be dropped
> +again.
> +
> +Closes: #762018
> +diff --git a/units/systemd-udevd.service.in b/units/systemd-udevd.service.in
> +index e7216d6..1e9a600 100644
> +--- a/units/systemd-udevd.service.in
> ++++ b/units/systemd-udevd.service.in
> +@@ -21,6 +21,5 @@ Sockets=systemd-udevd-control.socket systemd-udevd-kernel.socket
> + Restart=always
> + RestartSec=0
> + ExecStart=@rootlibexecdir@/systemd-udevd
> +-MountFlags=slave
> + KillMode=mixed
> + WatchdogSec=1min
> diff --git a/meta/recipes-core/systemd/systemd_225.bb b/meta/recipes-core/systemd/systemd_225.bb
> index 18c2448..a2122e0 100644
> --- a/meta/recipes-core/systemd/systemd_225.bb
> +++ b/meta/recipes-core/systemd/systemd_225.bb
> @@ -46,6 +46,7 @@ SRC_URI = "git://github.com/systemd/systemd.git;protocol=git \
>              file://init \
>              file://run-ptest \
>              file://rules-whitelist-hd-devices.patch \
> +           file://udev-re-enable-mount-propagation-for-udevd.patch \
>             "
>   SRC_URI_append_qemuall = " file://qemuall_io_latency-core-device.c-Change-the-default-device-timeout-to-2.patch"
>