glusterfs.git/xlators/storage, branch v4.0dev1

posix: Ignore disk space reserve check for internal FOPS

2017-10-30T15:53:46+00:00

Problem: Currently disk space reserve check is applicable for internal FOP
         also it needs to be ignore for internal FOP.

Solution: Update the DISK_SPACE_CHECK_AND_GOTO macro at posix component.
          Macro will call only while key "GLUSTERFS_INTERNAL_FOP_KEY"
          exists in xdata.

BUG: 1506083
Change-Id: I2b0840bbf4fa14bc247855b024ca136773d68d16
Signed-off-by: Mohit Agrawal

geo-rep: Filter out volume-mark xattr

2017-10-13T16:26:05+00:00

The volume-mark xattr, maintained at brick root
of slave volume is specific to geo-replication
and should be filtered out for all other clients.
It should also be filtered out from list getxattr
from all mounts including geo-rep mount as it
might cause rsync to read and set.

Change-Id: If9eb5a3af18051083c853e70d93b2819e8eea222
BUG: 1500433
Signed-off-by: Kotresh HR

storage/posix: Adding implementation for posix_do_futimes

2017-10-05T12:57:35+00:00

Adding the implementation for the posix_do_futimes function which is
not complete in the current implementation and giving the ENOSYS error.

Change-Id: I9cfc95a7ea293b0a2df8efd4ac80d0120b3120e4
BUG: 1350406
Signed-off-by: karthik-us

Posix: fix additional inode ref

2017-09-29T06:36:49+00:00

Problem: In the iteration, the inode is being ref-ed twice and
unref-ed once. this leads to ref leak.

Fix: assign the parent to the inode instead of referencing it.

Change-Id: Ib154b12d38ad68220f8f5288bbc50081beccc2b9
BUG: 1496379
Signed-off-by: hari gowtham

Posix: Unused value

2017-09-26T13:07:18+00:00

In line number 5179 we are storing -1 into op_ret variable. Before
we use the value we are overwriting the same variable in line
number 5339. So we are removing the value assigning statement.

Change-Id: I8c6dae9f6b0f9f1e3c09d0744d451b9296d12db8
BUG: 789278
Signed-off-by: Sanju Rakonde

Fix the type of len, a warning is triggered

2017-09-08T16:16:54+00:00

clang show this warning on FreeBSD:

  posix.c:133:17: warning: comparison of unsigned expression < 0
  is always false [-Wtautological-compare]

A closer look on the code show that sys_readlink is returning a ssize_t,
not a size_t, who is unsigned.

Change-Id: Idb6a440ea44cc2168b0dd85f791c7955caa67c8c
BUG: 1488909
Signed-off-by: Michael Scherer 
Reviewed-on: https://review.gluster.org/18220
Tested-by: Michael Scherer 
CentOS-regression: Gluster Build System 
Smoke: Gluster Build System 
Reviewed-by: Jeff Darcy

feature/posix: Enabled gfid2path by default

2017-08-29T12:52:46+00:00

Enable gfid2path feature by default. The basic
performance tests are carried out and it doesn't
show significant depreciation. The results are
updated in issue.

Updates: #139
Change-Id: I5f1949a608d0827018ef9d548d5d69f3bb7744fd
Signed-off-by: Kotresh HR 
Reviewed-on: https://review.gluster.org/17950
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Aravinda VK 
Reviewed-by: Amar Tumballi

posix: add null gfid checks

2017-08-08T12:54:25+00:00

...in file/dir creation and lookup codepaths. The check is relaxed for
fops coming from trash xlator at the moment until trash has client side
logic to send the create fops with gfid-req.

Also fixed the missing trash pid assignment in creates sent by trash
xlator. Without this, truncated files won't be moved to .trashcan.

Change-Id: Ieddd7f0634850e7c7010e4fbb4ad1eead35888c8
BUG: 1478297
Signed-off-by: Ravishankar N 
Reviewed-on: https://review.gluster.org/17975
Smoke: Gluster Build System 
Reviewed-by: jiffin tony Thottan 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra Bhat

cluster/dht: Check for open fd only on EBADF

2017-08-08T10:21:18+00:00

DHT fd based fops used to check if the fd was open
on the cached subvol before winding the call. However,
this introduced a performance regression of about
30% for reads.

This check was introduced to handle cases where files
were migrated while IOs were happening. As this is not
the common case, dht will now check if the fd is
open on the cached subvol only if the call fails
with EBADF.

This will prevent a performance hit where a rebalance
is not running.

Change-Id: I2035a858d63c3fcd22bb634055bbb0ad01686808
BUG: 1476665
Signed-off-by: N Balachandran 
Reviewed-on: https://review.gluster.org/17976
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Susant Palai 
Reviewed-by: Raghavendra G

posix: add sanity checks for removing the gfid symlink for directories

2017-08-04T15:54:01+00:00

...during mkdir and rmdir. Otherwise, during entry self-heal, the
directory could be left out without a .glusterfs symlink causing fops like
opendir, readdir to fail.  The only chance the missing symlink will be
created is when a fresh lookup comes on it.

Change-Id: I2e1cf1bce8962ea80187edd8f6d73e0a09cf9f8e
BUG: 1477169
Signed-off-by: Ravishankar N 
Reviewed-on: https://review.gluster.org/17945
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra Bhat