Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Mike Looijmans <mike.looijmans@topic.nl>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices
Date: Thu, 18 Dec 2014 15:38:56 +0100	[thread overview]
Message-ID: <5492E700.3050000@topic.nl> (raw)
In-Reply-To: <CAP9ODKrXLd9mQpJco47ddwOhZOARN3PYY9Ci6Se0XBsmTQM7Lw@mail.gmail.com>

On 12/18/2014 01:55 PM, Otavio Salvador wrote:
> Hello Mike,
>
> On Thu, Dec 18, 2014 at 4:31 AM, Mike Looijmans <mike.looijmans@topic.nl> wrote:
>> On 12/17/2014 11:08 PM, Gary Thomas wrote:
>>>
>>> On 2014-12-17 15:03, Burton, Ross wrote:
>>>>
>>>> Hi Mike,
>>>>
>>>> On 5 December 2014 at 09:58, Mike Looijmans <mike.looijmans@topic.nl
>>>> <mailto:mike.looijmans@topic.nl>> wrote:
>>>>
>>>>      Upon inserting a USB stick or similar device, mdev will run
>>>>      an automounter script that mounts valid partitions on
>>>>      /media/<device>. The script first checks /etc/fstab entries
>>>>      so that mounting on UUID or LABEL or using custom mount options
>>>>      is still possible. If /etc/fstab does not contain particular
>>>>      mount options, the script will create (and remove) the mountpoint
>>>>      automatically.
>>>>      The script also supports full disk partitions (devices without
>>>>      partition table).
>>>>
>>>>
>>>> Thanks for this, but can this script be put into a separate recipe to
>>>> make
>>>> it even easier for people who don't want it, or want their own version?
>>>> The
>>>> prior art here is the udev
>>>> automounting script which is in udev-extraconf.
> ...
>
> I didn't try but what happens in mdev if the script does not exists? I
> mean keep mdev.conf as is (ready for use) and move the script to
> another package which can be installed or not.

I just tried, removed the script and inserted a stick. It still created 
/dev/sda and /dev/sda1 so it seems nothing really bad will happen then.

>> On the satellite settop boxes, /media is a 64k tmpfs mount. This is because
>> when a mount fails, the box may end up writing a TV recording to wherever
>> /media happens to be. It that's flash or a big tmpfs, the box will commit
>> suicide in mere seconds by either filling the flash or eating up the
>> precious RAM. And for some things, the box really needs a big tmpfs, so just
>> reducing the volatiles' sizes wasn't an option.
>
> Nice 'solution' to avoid a more serious problem :-D

Well, it got rid of tons of "check if this and that" code in the recorder code 
which occasionally would still be fooled into recording onto flash or RAM.

>> Having said that, if this is blocking, I'll do a s./media./run/media.g on
>> the file, getting this mainlined has higher priority.
>
> I prefer to have it behaving as udev as much as possible, by default.
> However, the script could source a defaults file (if it exists) to
> change the mounting point for example and allow for those special
> systems to be handle without forking.

Yeah, something like "MDEV_AUTOMOUNT_ROOT=/media" in /etc/default/mdev should 
do the trick.



Met vriendelijke groet / kind regards,

Mike Looijmans
System Expert


TOPIC Embedded Systems
Eindhovenseweg 32-C, NL-5683 KH Best
Postbus 440, NL-5680 AK Best
Telefoon: (+31) (0) 499 33 69 79
Telefax:  (+31) (0) 499 33 69 70
E-mail: mike.looijmans@topic.nl
Website: www.topic.nl

Please consider the environment before printing this e-mail

Topic zoekt gedreven (embedded) software specialisten!
http://topic.nl/vacatures/topic-zoekt-software-engineers/



  reply	other threads:[~2014-12-18 14:38 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-04 10:27 [PATCH 0/2] Busybox mdev improvements Mike Looijmans
2014-12-04 10:27 ` [PATCH 1/2] busybox-mdev: Install missing find-touchscreen.sh Mike Looijmans
2014-12-04 10:27 ` [PATCH 2/2] busybox/find-touchscreen.sh: Simplify script and recognize USB devices Mike Looijmans
2014-12-04 20:47 ` [PATCH 0/2] Busybox mdev improvements Otavio Salvador
2014-12-05  6:18   ` Mike Looijmans
2014-12-05 11:27     ` Otavio Salvador
2014-12-05 12:57       ` Mike Looijmans
2014-12-05 14:13         ` Otavio Salvador
2014-12-05  9:58 ` [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices Mike Looijmans
2014-12-05  9:58   ` [PATCH 2/2] busybox-mdev: Add hotplug kernel module support to mdev.conf Mike Looijmans
2014-12-17 22:01   ` [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices Burton, Ross
2014-12-18 13:14     ` Mike Looijmans
2014-12-17 22:03   ` Burton, Ross
2014-12-17 22:05     ` Burton, Ross
2014-12-17 22:08     ` Gary Thomas
2014-12-18  6:31       ` Mike Looijmans
2014-12-18 12:55         ` Otavio Salvador
2014-12-18 14:38           ` Mike Looijmans [this message]
2014-12-18  6:23     ` Mike Looijmans
2014-12-18 13:30       ` Burton, Ross
2014-12-18 13:54         ` Mike Looijmans
2014-12-18 14:17 ` [PATCH v2 0/4] Busybox mdev improvements Mike Looijmans
2014-12-18 14:17   ` [PATCH v2 1/4] busybox-mdev: Install missing find-touchscreen.sh Mike Looijmans
2014-12-18 14:43     ` Otavio Salvador
2014-12-18 14:17   ` [PATCH v2 2/4] busybox/find-touchscreen.sh: Simplify script and recognize USB devices Mike Looijmans
2014-12-18 14:17   ` [PATCH v2 4/4] busybox-mdev: Support automatic mounting of block devices Mike Looijmans
2015-01-01  2:49     ` Khem Raj
2015-01-01  9:07       ` Mike Looijmans
2015-01-01 13:08         ` Peter A. Bigot
2015-01-01 19:21           ` Mike Looijmans
2014-12-18 14:19   ` [PATCH v2 3/4] busybox-mdev: Add hotplug kernel module support to mdev.conf Mike Looijmans
2014-12-18 14:44     ` Otavio Salvador

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5492E700.3050000@topic.nl \
    --to=mike.looijmans@topic.nl \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio@ossystems.com.br \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox