[OE-core] [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
Saul Wold
sgw at linux.intel.com
Mon Apr 7 15:49:44 UTC 2014
On 04/06/2014 03:46 PM, Denys Dmytriyenko wrote:
> ping
>
This change is a little late for 1.6 as we are prepping for the final RC
this week. It's kind of an API change since it moves the mount location
/media, should we be linking /media to /run/media?
There is also an issue with the Freescale PPC BSP, which has a bbappend
for 1.0, so some coordination with them will be needed.
I can revisit this for 1.7 in a week or so.
Sau!
> On Mon, Mar 31, 2014 at 11:54:11PM -0400, Denys Dmytriyenko wrote:
>> From: Denys Dmytriyenko <denys at ti.com>
>>
>> This is done to work around the issue of auto-mounting block devices
>> (i.e. SD cards) when root filesystem is still in read-only mode and
>> creating /media/<device> mount-points by udev is not possible. That
>> is due to udev (/etc/rcS.d/S03udev) getting started earlier than
>> checkroot (/etc/rcS.d/S10checkroot.sh) gets a chance to re-mount the
>> rootfs as read-write.
>>
>> Although, canonical FHS specifies /media/<device> as a mount point
>> for removable media devices, the latest 2.3 version was released in
>> 2004 and since then FreeDesktop/udisks and other tools adopted the
>> new /run/media/<user>/<device> location. That was done to overcome
>> read-only rootfs limitation, since /run is usually a tmpfs mounted
>> partition, plus avoid name-clash between users.
>>
>> For our embedded systems environment we assume single-user operation
>> and hence simplify mount point to just /run/media/<device>. But for
>> proper per-user mounting to /run/media/<user>/<device>, some sort of
>> session management is required along with the tool like udisks, that
>> is out of scope of this simple udev-based auto-mounting.
>>
>> Signed-off-by: Denys Dmytriyenko <denys at ti.com>
>> ---
>> v2 - drop PR, bump PV, elaborate on session/user mounting option
>>
>> meta/recipes-core/udev/udev-extraconf/mount.sh | 14 +++++++-------
>> .../udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} | 2 --
>> 2 files changed, 7 insertions(+), 9 deletions(-)
>> rename meta/recipes-core/udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} (99%)
>>
>> diff --git a/meta/recipes-core/udev/udev-extraconf/mount.sh b/meta/recipes-core/udev/udev-extraconf/mount.sh
>> index cb57e47..3e4f21f 100644
>> --- a/meta/recipes-core/udev/udev-extraconf/mount.sh
>> +++ b/meta/recipes-core/udev/udev-extraconf/mount.sh
>> @@ -20,7 +20,7 @@ done
>> automount() {
>> name="`basename "$DEVNAME"`"
>>
>> - ! test -d "/media/$name" && mkdir -p "/media/$name"
>> + ! test -d "/run/media/$name" && mkdir -p "/run/media/$name"
>> # Silent util-linux's version of mounting auto
>> if [ "x`readlink $MOUNT`" = "x/bin/mount.util-linux" ] ;
>> then
>> @@ -38,12 +38,12 @@ automount() {
>> ;;
>> esac
>>
>> - if ! $MOUNT -t auto $DEVNAME "/media/$name"
>> + if ! $MOUNT -t auto $DEVNAME "/run/media/$name"
>> then
>> - #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/media/$name\" failed!"
>> - rm_dir "/media/$name"
>> + #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/run/media/$name\" failed!"
>> + rm_dir "/run/media/$name"
>> else
>> - logger "mount.sh/automount" "Auto-mount of [/media/$name] successful"
>> + logger "mount.sh/automount" "Auto-mount of [/run/media/$name] successful"
>> touch "/tmp/.automount-$name"
>> fi
>> }
>> @@ -60,7 +60,7 @@ rm_dir() {
>>
>> # No ID_FS_TYPE for cdrom device, yet it should be mounted
>> name="`basename "$DEVNAME"`"
>> -[ -e /sys/block/$name/device/media ] && media_type=`cat /sys/block/$name/device/media`
>> +[ -e /sys/block/$name/device/run/media ] && media_type=`cat /sys/block/$name/device/run/media`
>>
>> if [ "$ACTION" = "add" ] && [ -n "$DEVNAME" ] && [ -n "$ID_FS_TYPE" -o "$media_type" = "cdrom" ]; then
>> if [ -x "$PMOUNT" ]; then
>> @@ -87,5 +87,5 @@ if [ "$ACTION" = "remove" ] && [ -x "$UMOUNT" ] && [ -n "$DEVNAME" ]; then
>>
>> # Remove empty directories from auto-mounter
>> name="`basename "$DEVNAME"`"
>> - test -e "/tmp/.automount-$name" && rm_dir "/media/$name"
>> + test -e "/tmp/.automount-$name" && rm_dir "/run/media/$name"
>> fi
>> diff --git a/meta/recipes-core/udev/udev-extraconf_1.0.bb b/meta/recipes-core/udev/udev-extraconf_1.1.bb
>> similarity index 99%
>> rename from meta/recipes-core/udev/udev-extraconf_1.0.bb
>> rename to meta/recipes-core/udev/udev-extraconf_1.1.bb
>> index 3810b28..d69056d 100644
>> --- a/meta/recipes-core/udev/udev-extraconf_1.0.bb
>> +++ b/meta/recipes-core/udev/udev-extraconf_1.1.bb
>> @@ -4,8 +4,6 @@ LICENSE = "MIT"
>> LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=4d92cd373abda3937c2bc47fbc49d690 \
>> file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
>>
>> -PR = "r16"
>> -
>> SRC_URI = " \
>> file://automount.rules \
>> file://mount.sh \
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core at lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
More information about the Openembedded-core
mailing list