[arch-projects] [RFC][devtools][PATCH] mkarchroot: use bind mount instead of symlink for /dev/ptmx
Gerardo Exequiel Pozzi
vmlinuz386 at yahoo.com.ar
Thu Mar 8 21:57:13 EST 2012
On 03/08/2012 11:52 PM, Allan McRae wrote:
> On 09/03/12 12:43, Gerardo Exequiel Pozzi wrote:
>> We need /dev/ptmx -> /dev/pts/ptmx (for devpts -o newinstance)
>> Other way to do this thing is via bind mount (as said kernel doc[devpts.txt]).
>>
>> This should be done in this way at least for /dev as devtmpfs in the chroot.
>> Since we can not touch /dev (devtmpfs), because devtmpfs is "singleton",
>> just use bind method and avoid interference.
>>
>> Do it the same for both modes of /dev (tmpfs) and (devtmpfs) to keep it simple.
>>
>> Currently devpts in chroot is not working without this when using /dev as devtmpfs,
>> this fixes this issue (opening /dev/ptmx, creates devices nodes on outside /dev/pts)
>>
>> Signed-off-by: Gerardo Exequiel Pozzi<vmlinuz386 at yahoo.com.ar>
>> ---
>> mkarchroot.in | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/mkarchroot.in b/mkarchroot.in
>> index d45cfba..efe0398 100644
>> --- a/mkarchroot.in
>> +++ b/mkarchroot.in
>> @@ -117,7 +117,7 @@ chroot_mount() {
>>
>> [[ -e "${working_dir}/dev/pts" ]] || mkdir "${working_dir}/dev/pts"
>> mount -t devpts devpts "${working_dir}/dev/pts" -o newinstance,ptmxmode=666
>> - ln -s pts/ptmx "${working_dir}/dev/ptmx"
>> + mount -o bind ${working_dir}/dev/pts/ptmx" "${working_dir}/dev/ptmx"
> Quote is missing...
oopss, bad patch, thanks.
>> [[ -e $cache_dir ]] || mkdir -p "${cache_dir}"
>> [[ -e "${working_dir}/${cache_dir}" ]] || mkdir -p "${working_dir}/${cache_dir}"
>> @@ -148,6 +148,7 @@ copy_hostconf () {
>> chroot_umount () {
>> umount "${working_dir}/proc"
>> umount "${working_dir}/sys"
>> + umount "${working_dir}/dev/ptmx"
>> umount "${working_dir}/dev/pts"
>> umount "${working_dir}/dev/shm"
>> umount "${working_dir}/dev"
>
--
Gerardo Exequiel Pozzi
\cos^2\alpha + \sin^2\alpha = 1
More information about the arch-projects
mailing list