* refs/remotes/upstream/pull/17175/head:
client: add blank space between function name and inode number for log
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Currently, it doesn't take the client_lock at all, which is problematic
as make_request may very well end up unlocking it. Rename the current
function to _getcwd, and add a new getcwd wrapper that takes the mutex
before calling _getcwd.
This fixes: http://tracker.ceph.com/issues/21082
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Valgrind is unhappy about our turning on supplimentary group handling
with fuse by default. The problem is that we end up calling delete to
free the supplimentary gids list, but fuse uses malloc to allocate it.
Note that I was initially concerned that I needed to use malloc and
free there to handle the case of userland calling ceph_userperm_new,
but we leave freeing the pointer up to the caller in that case.
Convert fuse to use new/delete to allocate and free the group lists
instead.
Tracker: http://tracker.ceph.com/issues/21065
Signed-off-by: Jeff Layton <jlayton@redhat.com>
* refs/remotes/upstream/pull/16959/head:
client: skip lookupname if writing to unlinked file
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Zheng Yan <zyan@redhat.com>
* refs/remotes/upstream/pull/17053/head:
qa: add chown test clearing S_ISGID and S_ISUID
ceph-fuse: load supplementary groups by default
client: clear suid/sgid bits on non-zero write
client: add missing space in debug msg
cephfs: clear suid/sgid if regular file is exe
client: refactor clear set uid/gid if -1
Reviewed-by: Zheng Yan <zyan@redhat.com>
* refs/remotes/upstream/pull/17054/head:
mds: Initializing uninitiazed members SnapServer
Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
the APIs in librbd receive string type as parameter, so there is no need to convert string to char* in image-meta function
Signed-off-by: PCzhangPC <pengcheng.zhang@easystack.cn>
if any of the objects fails to be deleted due to pg reset after latest
osdmap, the pg recovery delete reply won't be sent to the primary OSD.
in that case, we should delete the reply.
Fixes: http://tracker.ceph.com/issues/20913
Signed-off-by: Kefu Chai <kchai@redhat.com>
CrushTester::test() will reset testing range to [0, 1023] whenever
min_x or max_x is negative and the constructor of CrushTester will
always default min_x and max_x to -1.
Thus to set the test range correctly, you have to specify both min_x and max_x.
Local test shows this patch shall decrease the time consumed by the crush
smoke testing to approximate 1/20 of those without this.
For exmaple:
crush somke test duration: 0.668354 seconds ->
crush somke test duration: 0.012592 seconds
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
rgw: exit early if rgw_bucket_set_attrs() fails
Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Abhishek Lekshmanan <abhishek.lekshmanan@gmail.com>
