glusterfs.git/xlators/storage/posix/src/posix.c, branch v3.4.0qa8

storage/posix: skip path construction when dentry list is empty

2013-01-26T19:37:04+00:00

This is a minor latency optimization to the readdirp path in
storage/posix. During a recursive list, we hit this codepath with
an empty list once per high-level directory to read when end of
directory is reached. Skip constructing hpath, since we don't do
anything with it in this case.

BUG: 903175
Change-Id: I98d7c65505205d55575f064b1e982700f1320cc0
Signed-off-by: Brian Foster 
Reviewed-on: http://review.gluster.org/4432
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cli: fix descriptions of owner-uid and owner-gid

2013-01-21T18:06:34+00:00

Change-Id: I04c0dd23bc5bc34fd9d7bddb11beeecb8e7e2a49
BUG: 853842
Signed-off-by: Shireesh Anjal 
Reviewed-on: http://review.gluster.org/4398
Reviewed-by: Anand Avati 
Tested-by: Anand Avati

core: remove all the 'inner' functions in codebase

2012-12-20T07:35:29+00:00

* move 'dict_keys_join()' from api/glfs_fops.c to libglusterfs/dict.c
  - also added an argument which is treated as a filter function if
    required, currently useful for fuse.

* now 'make CFLAGS="-std=gnu99 -pedantic" 2>&1 | grep nested' gives
  no output.

Change-Id: I4e18496fbd93ae1d3942026ef4931889cba015e8
Signed-off-by: Amar Tumballi 
BUG: 875913
Reviewed-on: http://review.gluster.org/4187
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

storage/posix: if create returns EXIST, donot set gfid/xattrs

2012-12-04T22:43:34+00:00

Change-Id: I9f2b75b10bde428d36d6516aa09c18e590d17ed9
BUG: 864801
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/4265
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

posix: Fix volume will not start if brick has no volume-id attribute

2012-11-20T08:45:59+00:00

Problem:
If the extended attribute (trusted.glusterfs.volume-id) of a brick  is
absent and  command is executed then
curretly volume-id from the volume file will be set as an extended attribute of
the brick and volume will get started.
But if setup is such that brick is used as a mount point and before
executing the  command, nothing is mounted on
the brick then all the file operations will take place at the brick but actual
intention of the brick is to be used as mount point only.

FIX:
Do not start the volume if extended attribute (trusted.glusterfs.volume-id)
is set absent.

Change-Id: Id2462d87d6087e97e0b8831512fdbc3595f7078b
BUG: 860297
Signed-off-by: Venkatesh Somyajulu 
Reviewed-on: http://review.gluster.org/4202
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

storage/posix: Make rchecksum O_DIRECT friendly

2012-11-20T08:30:25+00:00

Problem:
When posix-aio is enabled to perform aio fd is set with O_DIRECT
whenever possible in read, writev fops. Rchecksum does not take
this into account. If either offset/size/memory-buf passed to
pread in rchecksum fop is not aligned, pread fails with EINVAL.

Fix:
Before doing pread necessary O_DIRECT manipulation is done when
aio is enabled. Memory buffer passed to pread is now page-aligned.

Test:
1) Create replica volume with aio enabled.
2) dd if=/dev/urandom of=a bs=1M count=1
3) kill one of the bricks in the replica pair
4) dd if=/dev/urandom of=a bs=1M count=1
5) bring back the brick. Self-heal succeeds after the change.

The test above checks both rchecksum, writev fops that were
changed in this patch.

Change-Id: I186099a2854d4864c5b48086ab7bc5f1a7b27313
BUG: 866459
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/4134
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

storage/posix: remove dependency on loc->path in posix_lookup()

2012-10-12T01:10:06+00:00

Change-Id: I0a3bc8650d9ff83977be696aa5caf9c7570197fd
Signed-off-by: Amar Tumballi 
BUG: 781318
Reviewed-on: http://review.gluster.org/3997
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

logging: log ENOENT errors in DEBUG mode instead of ERROR or INFO

2012-09-18T05:46:17+00:00

Change-Id: I0a43769223991e4ad5206b4382d737a0c3557bf3
BUG: 851953
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/3934
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

storage/posix: Option to set brick(of a volume)'s root dir's uid/gid

2012-09-14T08:28:45+00:00

CLI
---
gluster volume set VOLNAME owner-uid uid
gluster volume set VOLNAME owner-gid gid
where uid,gid are the owner's user id and group id respectively that
would be set on the root of all brick (backend) fs.

TODO: uid/gid should not be -1. Today we don't validate that in CLI.

Change-Id: Ib6a2fb5e404691c5fe105a89faaeff3e1ab72e91
BUG: 853842
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/3891
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

All: License message change

2012-09-13T20:19:37+00:00

License message changed for server-side, dual license GPLV2 and LGPLv3+.

Change-Id: Ia9e53061b9d2df3b3ef3bc9778dceff77db46a09
BUG: 852318
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/3940
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Anand Avati