Bug 676100

Summary:	Please replace /sbin/mount.tmpfs by C code in /bin/mount itself
Product:	[Fedora] Fedora	Reporter:	Lennart Poettering <lpoetter>
Component:	util-linux	Assignee:	Karel Zak <kzak>
Status:	CLOSED UPSTREAM	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	dwalsh, eparis, jonathan, kzak
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-02-09 13:25:48 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Lennart Poettering 2011-02-08 20:21:31 UTC

The shell script that is /sbin/mount.tmpfs shows up very unfavourably in bootchart:

http://www.harald-hoyer.de/personal/blog/2011-02-03-fedora-rawhide-bootcharts

It would be fantastic if we could get rid of that script (and especially the grep in it), and the selinux label magic could be done in mount instead, in simple and fast C code.

Comment 1 Karel Zak 2011-02-09 09:14:46 UTC

Note that the mount.tmpfs script is originally developed for bug #476964.

Comment 2 Karel Zak 2011-02-09 13:25:48 UTC

Fixed by upstream commit 086f6471089b3487f46309b401867b3dae7d1551. The fedora version will be updated after upgrade to the final v2.19 (next week).

Comment 3 Lennart Poettering 2011-02-18 17:34:08 UTC

Awesome, thank you. You rock!

/me strikes one more item of his list.