Re: [PATCH] base-files: remove symbolic link from /etc/mtab to /proc/mounts

[Mark Hatle <mark.hatle@windriver.com>]

On 7/31/13 4:16 AM, Guo Chunrong-B40290 wrote:
> Hello,
>
> I add libhugetlbfs package into my rootfs and run the below command in my board .
> root@p1021rdb:~# hugeadm --create-mounts
>      hugeadm:WARNING: Unable to add entry /var/lib/hugetlbfs/pagesize-4MB to /etc/mtab, error: Invalid argument
>      hugeadm:WARNING: Unable to add entry /var/lib/hugetlbfs/pagesize-16MB to /etc/mtab, error: Invalid argument

This looks like the hugeadm may be broken in some way.  The writing to the 
mounts and behavior should be based on the way the standard mount does it.

If the hugeadm uses standard mount resources, then the entries it adds should 
have showed up into the /proc/mounts automatically.

--Mark

> root@p1022ds:~# ls /etc/mtab -l
> lrwxrwxrwx 1 root root 12 Jul 31 07:28 /etc/mtab -> /proc/mounts
> root@p1022ds:~# ls /proc/mounts
> /proc/mounts
> root@p1022ds:~# ls /proc/mounts -l
> lrwxrwxrwx 1 root root 11 Jul 31 07:58 /proc/mounts -> self/mounts
>
>
>
> thanks,
> chunrong
>
>
>
> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org [mailto:openembedded-core-bounces@lists.openembedded.org] On Behalf Of ChenQi
> Sent: Wednesday, July 31, 2013 4:47 PM
> To: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] base-files: remove symbolic link from /etc/mtab to /proc/mounts
>
> On 07/31/2013 04:19 PM, Chunrong Guo wrote:
>>       *If /etc/mtab is a symlink to /proc/self/mounts,
>>        mounting fails because addmntent returns an error.
>>        As a consequence, the Private directory is unusable
>>        because /proc/self/mounts is read-only feature.
>
> Could you please detail your environment a little bit?
> Where does this error come from?
>
> Besides, if you're going to make this change, please take a read-only rootfs into consideration. There should be more to fix.
>
> Best Regards,
> Chen Qi
>
>> Signed-off-by: Chunrong Guo <B40290@freescale.com>
>> ---
>>    meta/recipes-core/base-files/base-files_3.0.14.bb |    1 -
>>    1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb
>> b/meta/recipes-core/base-files/base-files_3.0.14.bb
>> index 054fefa..1843bdd 100644
>> --- a/meta/recipes-core/base-files/base-files_3.0.14.bb
>> +++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
>> @@ -100,7 +100,6 @@ do_install () {
>>    	install -m 0644 ${WORKDIR}/host.conf ${D}${sysconfdir}/host.conf
>>    	install -m 0644 ${WORKDIR}/motd ${D}${sysconfdir}/motd
>>
>> -	ln -sf /proc/mounts ${D}${sysconfdir}/mtab
>>    }
>>
>>    do_install_basefilesissue () {
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>