]> git.baikalelectronics.ru Git - kernel.git/commit
ceph: Fix up after semantic merge conflict
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 29 Jan 2014 02:06:18 +0000 (18:06 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 29 Jan 2014 02:06:18 +0000 (18:06 -0800)
commit85a4b28d28263501872ec711b434bece8ab79e0e
tree25154cd4c6883b15ff269389e60a392b01c72318
parentabd8565564e633ef1fe5824bef87d6fcb51c9cea
ceph: Fix up after semantic merge conflict

The previous ceph-client merge resulted in ceph not even building,
because there was a merge conflict that wasn't visible as an actual data
conflict: commit cf70bd68f9bf ("ceph: add acl for cephfs") added support
for POSIX ACL's into Ceph, but unluckily we also had the VFS tree change
a lot of the POSIX ACL helper functions to be much more helpful to
filesystems (see for example commits e9e1ceb1586f "fs: add generic
xattr_acl handlers", 061c70001c63 "fs: make posix_acl_chmod more useful"
and aa0844b9e8c7 "fs: make posix_acl_create more useful")

The reason this conflict wasn't obvious was many-fold: because it was a
semantic conflict rather than a data conflict, it wasn't visible in the
git merge as a conflict.  And because the VFS tree hadn't been in
linux-next, people hadn't become aware of it that way.  And because I
was at jury duty this morning, I was using my laptop and as a result not
doing constant "allmodconfig" builds.

Anyway, this fixes the build and generally removes a fair chunk of the
Ceph POSIX ACL support code, since the improved helpers seem to match
really well for Ceph too.  But I don't actually have any way to *test*
the end result, and I was really hoping for some ACK's for this.  Oh,
well.

Not compiling certainly doesn't make things easier to test, so I'm
committing this without the acks after having waited for four hours...
Plus it's what I would have done for the merge had I noticed the
semantic conflict..

Reported-by: Dave Jones <davej@redhat.com>
Cc: Sage Weil <sage@inktank.com>
Cc: Guangliang Zhao <lucienchao@gmail.com>
Cc: Li Wang <li.wang@ubuntykylin.com>
Cc: Christoph Hellwig <hch@infradead.org>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ceph/acl.c
fs/ceph/inode.c
fs/ceph/super.h
fs/ceph/xattr.c