[PATCH] initramfs-framework/udev: remove mount.sh to avoid /run/media/XXX

[PATCH] initramfs-framework/udev: remove mount.sh to avoid /run/media/XXX

[hongxu]

The udev invokes script hook mount.sh to mount /run/media/XXX. In
initramfs-framework, if udev was not killed in time, the partition
will be mounted to /run/media/XXX before switch_root.

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/recipes-core/initrdscripts/initramfs-framework/udev | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-core/initrdscripts/initramfs-framework/udev b/meta/recipes-core/initrdscripts/initramfs-framework/udev
index 8b62080c68..3c4dd0ec9b 100644
--- a/meta/recipes-core/initrdscripts/initramfs-framework/udev
+++ b/meta/recipes-core/initrdscripts/initramfs-framework/udev
@@ -49,6 +49,9 @@ udev_run() {
 	# Workaround if console=null, systemd-udevd needs valid stdin, stdout and stderr to work
 	sh -c "exec 4< /dev/console" || { exec 0> /dev/null; exec 1> /dev/null; exec 2> /dev/null; }
 
+	# Avoid /run/media/XXX was mounted by udev
+	rm -f /etc/udev/scripts/mount.sh
+
 	$_UDEV_DAEMON --daemon
 	udevadm trigger --action=add
 	udevadm settle
-- 
2.18.2

Re: [PATCH] initramfs-framework/udev: remove mount.sh to avoid /run/media/XXX

[Richard Purdie]

On Wed, 2020-05-27 at 21:43 +0800, Hongxu Jia wrote:
> The udev invokes script hook mount.sh to mount /run/media/XXX. In
> initramfs-framework, if udev was not killed in time, the partition
> will be mounted to /run/media/XXX before switch_root.
> 
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>  meta/recipes-core/initrdscripts/initramfs-framework/udev | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/meta/recipes-core/initrdscripts/initramfs-framework/udev 
> b/meta/recipes-core/initrdscripts/initramfs-framework/udev
> index 8b62080c68..3c4dd0ec9b 100644
> --- a/meta/recipes-core/initrdscripts/initramfs-framework/udev
> +++ b/meta/recipes-core/initrdscripts/initramfs-framework/udev
> @@ -49,6 +49,9 @@ udev_run() {
>  	# Workaround if console=null, systemd-udevd needs valid stdin,
> stdout and stderr to work
>  	sh -c "exec 4< /dev/console" || { exec 0> /dev/null; exec 1>
> /dev/null; exec 2> /dev/null; }
>  
> +	# Avoid /run/media/XXX was mounted by udev
> +	rm -f /etc/udev/scripts/mount.sh
> +
>  	$_UDEV_DAEMON --daemon
>  	udevadm trigger --action=add
>  	udevadm settle

Somehow I think this patch is causing:

https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/976

(the runqemu.RunqemuTests.test_boot_machine_iso and
runqemu.RunqemuTests.test_boot_deploy_hddimg failures)

Cheers,

Richard

Re: [PATCH] initramfs-framework/udev: remove mount.sh to avoid /run/media/XXX

[hongxu]

On 5/29/20 6:06 AM, Richard Purdie wrote:
> On Wed, 2020-05-27 at 21:43 +0800, Hongxu Jia wrote:
>> The udev invokes script hook mount.sh to mount /run/media/XXX. In
>> initramfs-framework, if udev was not killed in time, the partition
>> will be mounted to /run/media/XXX before switch_root.
>>
>> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>> ---
>>   meta/recipes-core/initrdscripts/initramfs-framework/udev | 3 +++
>>   1 file changed, 3 insertions(+)
>>
>> diff --git a/meta/recipes-core/initrdscripts/initramfs-framework/udev
>> b/meta/recipes-core/initrdscripts/initramfs-framework/udev
>> index 8b62080c68..3c4dd0ec9b 100644
>> --- a/meta/recipes-core/initrdscripts/initramfs-framework/udev
>> +++ b/meta/recipes-core/initrdscripts/initramfs-framework/udev
>> @@ -49,6 +49,9 @@ udev_run() {
>>   	# Workaround if console=null, systemd-udevd needs valid stdin,
>> stdout and stderr to work
>>   	sh -c "exec 4< /dev/console" || { exec 0> /dev/null; exec 1>
>> /dev/null; exec 2> /dev/null; }
>>   
>> +	# Avoid /run/media/XXX was mounted by udev
>> +	rm -f /etc/udev/scripts/mount.sh
>> +
>>   	$_UDEV_DAEMON --daemon
>>   	udevadm trigger --action=add
>>   	udevadm settle
> Somehow I think this patch is causing:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/976
>
> (the runqemu.RunqemuTests.test_boot_machine_iso and
> runqemu.RunqemuTests.test_boot_deploy_hddimg failures)

Sorry for the inconvenient

//Hongxu

> Cheers,
>
> Richard
>