glusterfs.git/xlators/performance/stat-prefetch, branch v3.3.0qa3

Change Copyright current year

2011-08-10T17:57:19+00:00

Change-Id: I2d10f2be44f518f496427f257988f1858e888084
BUG: 3348
Reviewed-on: http://review.gluster.com/200
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

LICENSE: s/GNU Affero General Public/GNU General Public/

2011-08-06T13:33:52+00:00

Change-Id: I3914467611e573cccee0d22df93920cf1b2eb79f
BUG: 3348
Reviewed-on: http://review.gluster.com/182
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

performance/stat-prefetch: dump cache and list of ops waiting for lookup completion.

2011-06-15T05:37:41+00:00

Signed-off-by: Raghavendra G 
Signed-off-by: Anand Avati 

BUG: 1059 (enhancements for getting statistics from performance translators)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1059

performance/stat-prefetch: return ESTALE if inode's gfid is not the same as cached gfid for a path.

2011-05-31T12:11:05+00:00

Signed-off-by: Raghavendra G 
Signed-off-by: Anand Avati 

BUG: 2927 (Mismatched link/target gfid and ESTALE/ENOENT)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2927

stat-prefetch: fix dirname(3) usage

2011-05-30T11:48:48+00:00

glibc dirname() modify the string it is given and returns it.
glusterfs takes this behavior for granted, and assume that if it
gives a malloc'ed string to dirname(), then it can free()) the
return value.

Here is what SUSv2 says:
http://opengroup.org/onlinepubs/007908799/xsh/dirname.html
"The dirname() function may modify the string pointed to by path,
and may return a pointer to static storage"

At least NetBSD returns a static storage. glusterfs will return it to
a calling function that has the responsability to free it, causing
a SIGSEGV.

Thanks to: Emmanuel Dreyfus 

Signed-off-by: Anand Avati 

BUG: 2923 (NetBSD port)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2923

remove excessive logs due to log enhancement

2011-04-13T07:38:47+00:00

Signed-off-by: Amar Tumballi 
Signed-off-by: Anand Avati 

BUG: 2346 (Log message enhancements in GlusterFS - phase 1)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2346

performance/stat-prefetch: logging enhancements

2011-03-17T18:58:35+00:00

Signed-off-by: Amar Tumballi 
Signed-off-by: Vijay Bellur 

BUG: 2346 (Log message enhancements in GlusterFS - phase 1)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2346

performance/stat-prefetch: whitespace related changes to adhere to coding guidelines.

2011-03-17T18:58:31+00:00

Signed-off-by: Raghavendra G 
Signed-off-by: Amar Tumballi 
Signed-off-by: Vijay Bellur 

BUG: 2346 (Log message enhancements in GlusterFS - phase 1)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2346

check the op_ret for less than zero in sp_create_cbk since posix can send negative values other than -1

2011-02-22T17:22:05+00:00

posix can send -ve values other than -1 in create:

op_ret = setgid_override (this, real_path, &gid);

        if (op_ret < 0) {
                goto out;
        }

In stat-prefetch we check op_ret only for -1 which results in function
proceeding and op_ret may become 0 due to other function calls in
sp_create_cbk, hence crashes in fuse.

Signed-off-by: Raghavendra Bhat 
Signed-off-by: Raghavendra G 
Signed-off-by: Anand V. Avati 

BUG: 2409 (crash in stat prefetch)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2409

check the value of op_ret pointer not the address

2011-01-08T01:43:50+00:00

Signed-off-by: Raghavendra Bhat 
Signed-off-by: Anand V. Avati 

BUG: 221 (stat prefetch implementation)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=221