<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/fs/gfs2/acl.c, branch linux-3.17.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.17.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.17.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2014-03-19T15:53:52Z</updated>
<entry>
<title>GFS2: inline function gfs2_set_mode</title>
<updated>2014-03-19T15:53:52Z</updated>
<author>
<name>Bob Peterson</name>
</author>
<published>2014-03-19T15:51:28Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=733dbc1b21dcfac83b292865a707d6a0d0fc16eb'/>
<id>urn:sha1:733dbc1b21dcfac83b292865a707d6a0d0fc16eb</id>
<content type='text'>
Here is a revised patch based on Steve's feedback:

This patch eliminates function gfs2_set_mode which was only called in
one place, and always returned 0.

Signed-off-by: Bob Peterson &lt;rpeterso@redhat.com&gt;
Signed-off-by: Steven Whitehouse &lt;swhiteho@redhat.com&gt;

</content>
</entry>
<entry>
<title>GFS2: Increase the max number of ACLs</title>
<updated>2014-03-19T15:16:24Z</updated>
<author>
<name>Bob Peterson</name>
</author>
<published>2014-03-19T13:10:21Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=b00263d1cafdd667de56cde47f35d5ee8dd37e14'/>
<id>urn:sha1:b00263d1cafdd667de56cde47f35d5ee8dd37e14</id>
<content type='text'>
This patch increases the maximum number of ACLs from 25 to 300 for
a 4K block size. The value is adjusted accordingly if the block size
is smaller. Note that this is an arbitrary limit with a performance
tradeoff, and that the physical limit is slightly over 500.

Signed-off-by: Bob Peterson &lt;rpeterso@redhat.com&gt;
Signed-off-by: Steven Whitehouse &lt;swhiteho@redhat.com&gt;

</content>
</entry>
<entry>
<title>GFS2: return -E2BIG if hit the maximum limits of ACLs</title>
<updated>2014-03-06T10:39:32Z</updated>
<author>
<name>Jie Liu</name>
</author>
<published>2014-03-04T03:28:39Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=f2113eb8a4ede4016199492f3e10f5a165b04fcd'/>
<id>urn:sha1:f2113eb8a4ede4016199492f3e10f5a165b04fcd</id>
<content type='text'>
Return -E2BIG rather than -EINVAL if hit the maximum size limits of
ACLs, as the former errno is consistent with VFS xattr syscalls.

This is pointed out by Dave Chinner in previous discussion thread:
http://www.spinics.net/lists/linux-fsdevel/msg71125.html

Signed-off-by: Jie Liu &lt;jeff.liu@oracle.com&gt;
Signed-off-by: Steven Whitehouse &lt;swhiteho@redhat.com&gt;

</content>
</entry>
<entry>
<title>gfs2: use generic posix ACL infrastructure</title>
<updated>2014-01-26T04:58:22Z</updated>
<author>
<name>Christoph Hellwig</name>
</author>
<published>2013-12-20T13:16:52Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=e01580bf9e4d0e3bbaead44bd46cdbfe61957732'/>
<id>urn:sha1:e01580bf9e4d0e3bbaead44bd46cdbfe61957732</id>
<content type='text'>
This contains some major refactoring for the create path so that
inodes are created with the right mode to start with instead of
fixing it up later.

Signed-off-by: Christoph Hellwig &lt;hch@lst.de&gt;
Signed-off-by: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
</content>
</entry>
<entry>
<title>fs: make posix_acl_create more useful</title>
<updated>2014-01-26T04:58:18Z</updated>
<author>
<name>Christoph Hellwig</name>
</author>
<published>2013-12-20T13:16:42Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=37bc15392a2363ca822b2c2828e0ccafbea32f75'/>
<id>urn:sha1:37bc15392a2363ca822b2c2828e0ccafbea32f75</id>
<content type='text'>
Rename the current posix_acl_created to __posix_acl_create and add
a fully featured helper to set up the ACLs on file creation that
uses get_acl().

Signed-off-by: Christoph Hellwig &lt;hch@lst.de&gt;
Reviewed-by: Jan Kara &lt;jack@suse.cz&gt;
Signed-off-by: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
</content>
</entry>
<entry>
<title>fs: make posix_acl_chmod more useful</title>
<updated>2014-01-26T04:58:18Z</updated>
<author>
<name>Christoph Hellwig</name>
</author>
<published>2013-12-20T13:16:41Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=5bf3258fd2acd8515450ab8efcd97c9d3b69f7f9'/>
<id>urn:sha1:5bf3258fd2acd8515450ab8efcd97c9d3b69f7f9</id>
<content type='text'>
Rename the current posix_acl_chmod to __posix_acl_chmod and add
a fully featured ACL chmod helper that uses the -&gt;set_acl inode
operation.

Signed-off-by: Christoph Hellwig &lt;hch@lst.de&gt;
Reviewed-by: Jan Kara &lt;jack@suse.cz&gt;
Signed-off-by: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
</content>
</entry>
<entry>
<title>gfs2: Use uid_eq and gid_eq where appropriate</title>
<updated>2013-02-13T14:15:10Z</updated>
<author>
<name>Eric W. Biederman</name>
</author>
<published>2013-02-01T05:56:13Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=6b24c0d279eacfb499854d09ea7f2b69d1721a29'/>
<id>urn:sha1:6b24c0d279eacfb499854d09ea7f2b69d1721a29</id>
<content type='text'>
Where kuid_t values are compared use uid_eq and where kgid_t values
are compared use gid_eq.  This is unfortunately necessary because
of the type safety that keeps someone from accidentally mixing
kuids and kgids with other types.

Cc: Steven Whitehouse &lt;swhiteho@redhat.com&gt;
Signed-off-by: "Eric W. Biederman" &lt;ebiederm@xmission.com&gt;
</content>
</entry>
<entry>
<title>userns: Pass a userns parameter into posix_acl_to_xattr and posix_acl_from_xattr</title>
<updated>2012-09-18T08:01:35Z</updated>
<author>
<name>Eric W. Biederman</name>
</author>
<published>2012-09-11T03:17:44Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=5f3a4a28ec140a90e6058d1d09f6b1f235d485e5'/>
<id>urn:sha1:5f3a4a28ec140a90e6058d1d09f6b1f235d485e5</id>
<content type='text'>
 - Pass the user namespace the uid and gid values in the xattr are stored
   in into posix_acl_from_xattr.

 - Pass the user namespace kuid and kgid values should be converted into
   when storing uid and gid values in an xattr in posix_acl_to_xattr.

- Modify all callers of posix_acl_from_xattr and posix_acl_to_xattr to
  pass in &amp;init_user_ns.

In the short term this change is not strictly needed but it makes the
code clearer.  In the longer term this change is necessary to be able to
mount filesystems outside of the initial user namespace that natively
store posix acls in the linux xattr format.

Cc: Theodore Tso &lt;tytso@mit.edu&gt;
Cc: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Cc: Andreas Dilger &lt;adilger.kernel@dilger.ca&gt;
Cc: Jan Kara &lt;jack@suse.cz&gt;
Cc: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
Signed-off-by: "Eric W. Biederman" &lt;ebiederm@xmission.com&gt;
</content>
</entry>
<entry>
<title>GFS2: Fix sgid propagation when using ACLs</title>
<updated>2012-05-04T13:33:06Z</updated>
<author>
<name>Steven Whitehouse</name>
</author>
<published>2012-05-04T13:33:06Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=f9425ad4e5c8f8f9f6297d0358dc44ca058ffe47'/>
<id>urn:sha1:f9425ad4e5c8f8f9f6297d0358dc44ca058ffe47</id>
<content type='text'>
This cleans up the mode setting code when creating inodes. The
SGID bit was being reset by setattr_copy() when the user creating a
subdirectory was not in the owning group. When ACLs are in use this
SGID bit should have been propagated if the ACL allows creation of
a subdirectory. GFS2's behaviour now matches that of the other ACL
supporting filesystems in this regard.

Signed-off-by: Steven Whitehouse &lt;swhiteho@redhat.com&gt;
</content>
</entry>
<entry>
<title>GFS2: We only need one ACL getting function</title>
<updated>2011-11-23T13:31:51Z</updated>
<author>
<name>Steven Whitehouse</name>
</author>
<published>2011-11-23T13:31:51Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=018a01cd27b3448a7c65272ccb1a1cbab6c2667e'/>
<id>urn:sha1:018a01cd27b3448a7c65272ccb1a1cbab6c2667e</id>
<content type='text'>
There is no need to have two versions of this function with
slightly different arguments.

Signed-off-by: Steven Whitehouse &lt;swhiteho@redhat.com&gt;
</content>
</entry>
</feed>
