From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dan Carpenter <dan.carpenter@oracle.com>
Subject: Re: vfs: Don't copy mount bind mounts of /proc/<pid>/ns/mnt between
 namespaces
Date: Tue, 3 Sep 2013 02:10:01 +0300
Message-ID: <20130902231001.GQ19256@mwanda>
References: <20130902140531.GA5450@elgon.mountain>
 <874na3nf07.fsf@xmission.com>
 <20130902204038.GO19256@mwanda>
 <8738pmnb6z.fsf@xmission.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: linux-fsdevel@vger.kernel.org
To: "Eric W. Biederman" <ebiederm@xmission.com>
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from userp1040.oracle.com ([156.151.31.81]:25046 "EHLO
	userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1752655Ab3IBXKK (ORCPT
	<rfc822;linux-fsdevel@vger.kernel.org>);
	Mon, 2 Sep 2013 19:10:10 -0400
Content-Disposition: inline
In-Reply-To: <8738pmnb6z.fsf@xmission.com>
Sender: linux-fsdevel-owner@vger.kernel.org
List-ID: <linux-fsdevel.vger.kernel.org>

On Mon, Sep 02, 2013 at 02:52:20PM -0700, Eric W. Biederman wrote:
> So yes it does look like the case that except for the first iteration
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> the test at the top of the while loop is now redundant, because of the
> if (!q) break; that I added.

Ah... I see the confusion now.

The test is redundant on the first iteration as well.  The test is
entirely redundant.  That's the point.

regards,
dan carpenter