From mboxrd@z Thu Jan 1 00:00:00 1970 From: sukadev-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org Subject: Re: [PATCH 5/7]: Determine pts_ns from a pty's inode. Date: Wed, 26 Mar 2008 07:55:21 -0700 Message-ID: <20080326145521.GA24292@us.ibm.com> References: <20080325035904.GB27451@us.ibm.com> <20080325042541.GE27864@us.ibm.com> <20080325151705.GE9561@sergelap.austin.ibm.com> <20080325211406.GA5817@sergelap.austin.ibm.com> <20080326020328.GA11747@us.ibm.com> <20080326025038.GA24538@sergelap.austin.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20080326025038.GA24538-6s5zFf/epYLPQpwDFJZrxKsjOiXwFzmk@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: "Serge E. Hallyn" Cc: Containers , Matt Helsley , Pavel Emelianov List-Id: containers.vger.kernel.org Serge E. Hallyn [serue-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org] wrote: | > | I suppose you could just create /dev/pts/ptmx and /dev/pts/tty. | > | Recommend that in containers /dev/ptmx and /dev/tty be symlinks | > | into /dev/pts. Applications don't need to change. If | > | ptmx_open() sees that inode->i_sb is a devptsfs, it gets the | > | namespace from the sb. If not, then it was a device in /dev | > | and it gets the nmespace from current. | > | > But we would still depend on user-space remounting /dev/pts after | > the clone right ? Until they do that we would access the parent | > container's /dev/pts/ptmx ? | | Yes. Which is the right thing to do imo. Hmm, that sounds reasonable, although slightly inconsistent with pid-ns, where pid starts at 1 regardless of whether /proc is remounted. But even so, if user fails to establish the symlink, clones the pts ns and tries to create a pty, we would end up with different pts nses again ? i.e /dev/ptmx is still a char dev in root fs clone(pts_ns) ( In child, (before remount /dev/pts)) open("/dev/ptmx") open("/dev/pts/0") Since ptmx is not in devpts, we use current_pts_ns() or child-pts-ns Since /dev/pts is not remounted in child, we get the parent pts-ns from If we can somehow detect the incorrect configuration and fail either open, we should be ok :-) inode. Suka