From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752889AbeCMQvv (ORCPT ); Tue, 13 Mar 2018 12:51:51 -0400 Received: from mx2.mailbox.org ([80.241.60.215]:49118 "EHLO mx2.mailbox.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752384AbeCMQvs (ORCPT ); Tue, 13 Mar 2018 12:51:48 -0400 Date: Tue, 13 Mar 2018 17:51:33 +0100 From: Christian Brauner To: "Eric W. Biederman" Cc: Christian Brauner , containers@lists.linux-foundation.org, torvalds@linux-foundation.org, viro@zeniv.linux.org.uk, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/4 v5] devpts: handle bind-mounts correctly Message-ID: <20180313165133.GA17319@mailbox.org> References: <20180313121713.32551-1-christian.brauner@ubuntu.com> <87y3iw0vz7.fsf@xmission.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <87y3iw0vz7.fsf@xmission.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 13, 2018 at 11:41:00AM -0500, Eric W. Biederman wrote: > Christian Brauner writes: > > > Hey everyone, > > > > This is the fith iteration of this patch. Per-patch changes are > > summarized in the individual patches: > > Acked-by: "Eric W. Biederman" > > I guess we need to answer the question do I pick up this series or does > gregkh. This series would seem to fit in his tty tree. > > You haven't copied gregkh so you may need to resend and do that. I'll resend and CC grekh now. Christian > > Eric > > > > ChangeLog v4->v5: > > * added non-functional patch to document devpts_mntget(). > > Reason for putting this in a separate patch is that it allows you, > > Linus and Eric, to simply drop it if judged useless. > > * reverse error handling logic to further simplify > > * dput() dentry in the non-function patch. This was not really a problem > > since the following patch included a fix for it. But better to get it > > right in all individual patches. > > * I did another rewrite of the problem analysis for > > posterity in the patch "Subject: [PATCH 2/3 v3] devpts: resolve devpts > > bind-mounts" and in this cover letter. > > > > ChangeLog v3->v4: > > * small logical simplifications > > * add test that bind-mounts of /dev/pts/ptmx to locations that do not > > resolve to a valid slave pty path under the originating devpts mount > > fail > > > > ChangeLog v2->v3: > > * rewritten commit message to thoroughly analyse the problem for > > posterity in the patch "Subject: [PATCH 2/3 v3] devpts: resolve devpts > > bind-mounts" and in this cover letter. > > * extended selftests to test for correct handling of /dev/pts/ptmx > > bind-mounts to /dev/ptmx and non-standard devpts mounts such as > > mount -t devpts devpts /mnt > > > > ChangeLog v1->v2: > > * see individual patches > > ChangeLog v0->v1: > > * see individual patches > > > > Christian Brauner (4): > > devpts: hoist out check for DEVPTS_SUPER_MAGIC > > devpts: resolve devpts bind-mounts > > devpts: comment devpts_mntget() > > selftests: add devpts selftests > > > > fs/devpts/inode.c | 66 +++-- > > tools/testing/selftests/Makefile | 1 + > > tools/testing/selftests/filesystems/.gitignore | 1 + > > tools/testing/selftests/filesystems/Makefile | 2 +- > > tools/testing/selftests/filesystems/devpts_pts.c | 313 +++++++++++++++++++++++ > > 5 files changed, 363 insertions(+), 20 deletions(-) > > create mode 100644 tools/testing/selftests/filesystems/devpts_pts.c > _______________________________________________ > Containers mailing list > Containers@lists.linux-foundation.org > https://lists.linuxfoundation.org/mailman/listinfo/containers