Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754156AbYKSSPj (ORCPT ); Wed, 19 Nov 2008 13:15:39 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752751AbYKSSPa (ORCPT ); Wed, 19 Nov 2008 13:15:30 -0500 Received: from nf-out-0910.google.com ([64.233.182.186]:24766 "EHLO nf-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752543AbYKSSP3 (ORCPT ); Wed, 19 Nov 2008 13:15:29 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=LdJTWIJ+4HM5oX9fvTQCk+VyhCdAzN8o0tgQdUEvKR1+32h62f+HcdFLCY2satSUTa GFHXgm7r7SljswK0zUDze3UP3vDqDleM9zFp8ol4RA+YvdoQysSKpq2WaeSJwUnY6WLT xNyqkxgSvu/C13zJkuq7H8bNoklVPYd/gEYpU= Date: Wed, 19 Nov 2008 19:14:57 +0100 From: Marcin Slusarz To: Jan Kara Cc: Andrew Morton , linux-kernel@vger.kernel.org Subject: Re: [PATCH] udf: implement mode and dmode mounting options Message-ID: <20081119181427.GA29507@joi> References: <20081116195129.GA5920@joi> <20081118162205.bc61e2b4.akpm@linux-foundation.org> <20081119151605.GA29820@duck.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20081119151605.GA29820@duck.suse.cz> User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2041 Lines: 48 On Wed, Nov 19, 2008 at 04:16:06PM +0100, Jan Kara wrote: > On Tue 18-11-08 16:22:05, Andrew Morton wrote: > > On Sun, 16 Nov 2008 20:52:19 +0100 > > Marcin Slusarz wrote: > > > > > "dmode" allows overriding permissions of directories and > > > "mode" allows overriding permissions of files. > > > > > > Signed-off-by: Marcin Slusarz > > > Cc: Jan Kara > > > --- > > > Documentation/filesystems/udf.txt | 2 ++ > > > fs/udf/inode.c | 11 +++++++++-- > > > fs/udf/super.c | 31 ++++++++++++++++++++++++++++++- > > > fs/udf/udf_sb.h | 2 ++ > > > 4 files changed, 43 insertions(+), 3 deletions(-) > > > > > > diff --git a/Documentation/filesystems/udf.txt b/Documentation/filesystems/udf.txt > > > index fde829a..902b95d 100644 > > > --- a/Documentation/filesystems/udf.txt > > > +++ b/Documentation/filesystems/udf.txt > > > @@ -24,6 +24,8 @@ The following mount options are supported: > > > > > > gid= Set the default group. > > > umask= Set the default umask. > > > + mode= Set the default file permissions. > > > + dmode= Set the default directory permissions. > > > > Can we give these the same names and usage as their fatfs equivalents? > Yes, making these options dmask and fmask would be more consistent I > think. Martin? You can't add permissions by umask/fmask/dmask. I have one DVD with top level dir permissions set to 0000 and there's no way to read it now (as an user) and adding mask options wouldn't change it. I should mention in a changelog that the names and semantics of these options are copied from isofs. I can create a second patch which adds mask options but I think modes are needed too. What do you think? Marcin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/