From d373d7084e4d132fa596fd5def1b9a9843e1410f Mon Sep 17 00:00:00 2001
From: Jaegeuk Kim jaegeuk@samsung.com
Date: Fri, 6 Dec 2013 15:00:58 +0900
Subject: [PATCH] f2fs: add unlikely() macro for compiler more
aggressively
This patch adds unlikely() macro into the most of codes.
The basic rule
-Original Message-
From: Chao Yu [mailto:chao2...@samsung.com]
Sent: Tuesday, December 10, 2013 9:52 AM
To: jaegeuk@samsung.com
Cc: linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org;
linux-f2fs-devel@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 3/3 V3] f2fs:
Christoph,
nice work, and a pretty diffstat.
I see that get_acl and set_acl are being defined in some but not all symlink
inode operations (for example, btrfs them while ext4 does not), and that
posix_acl_xattr_set() doesn't check if set_acl is defined. Symlinks cannot
have ACLs, so set_acl
On Tue, Dec 03, 2013 at 12:00:07AM +0100, Jan Kara wrote:
Hum, this changes the cluster locking. Previously ocfs2_acl_get() used
from ocfs2_acl_chmod() grabbed cluster wide inode lock. Now getting of ACL
isn't protected by the inode lock. That being said the cluster locking
around setattr
2013-12-08 (일), 01:14 -0800, Christoph Hellwig:
On Fri, Dec 06, 2013 at 10:37:34AM +0900, Jaegeuk Kim wrote:
f2fs caches a new mode bit for a while to make the consistency between
xattr's acl mode and the inode mode.
Can you explain what exactly you're trying to do there? I've been
On Wed, Dec 04, 2013 at 12:12:37PM +, Steven Whitehouse wrote:
error = posix_acl_equiv_mode(acl, mode);
+ if (error 0)
Andy Price has pointed out a missing return error; here
- if (error = 0) {
- posix_acl_release(acl);
+
On Fri, Dec 06, 2013 at 10:37:34AM +0900, Jaegeuk Kim wrote:
f2fs caches a new mode bit for a while to make the consistency between
xattr's acl mode and the inode mode.
Can you explain what exactly you're trying to do there? I've been
trying to unwrap what's going on and can't really see the
Hi,
On Sun, 2013-12-01 at 03:59 -0800, Christoph Hellwig wrote:
plain text document attachment
(0016-gfs2-use-generic-posix-ACL-infrastructure.patch)
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