Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110603190350.GA514@openwall.com>
Date: Fri, 3 Jun 2011 23:03:50 +0400
From: Solar Designer <solar@...nwall.com>
To: owl-dev@...ts.openwall.com
Cc: Eugene Teo <eugeneteo@...il.com>
Subject: Re: add mount options to sysfs (was: segoon's status report - #1 of 15)

Vasiliy,

On Fri, Jun 03, 2011 at 10:38:41PM +0400, Vasiliy Kulikov wrote:
> I've added mount option printing in /proc/mounts and tested different
> remount situations.

Sounds/looks good.

> Solar, I'm waiting for your response re: mode usefullness.  Then I'll
> try to post the patch to LKML one more time.

I think that having both umask= and mode= is not needed for sysfs and
debugfs.  I think umask= should be left, mode= removed.

If I understood correctly, with your implementation (current patch),
umask applies when entries are created initially, and mode on chmod().
If so, mode should simply be gone.

I'll start a new thread for procfs.  Of these, I mostly care about
procfs, because it is of immediate relevance to Owl.

Thanks,

Alexander

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.