glusterfs.git/xlators/performance/md-cache, branch v3.3.0qa39

md-cache: fix dict memory leak in mdc_inode_xatt_update

2012-04-12T21:21:39+00:00

We currently either dict_ref() or dict_copy() into mdc->xattr.
The double dict_ref() call path is obvious. Alternatively,
dict_copy() does not ref the dict in the situation where it
creates a new dict, but we only call dict_copy() in this
context when we already have a dict, which should mean the
extra ref is unnecessary.

BUG: 801582
Change-Id: I72e3f1a16f942f04752c033812d13d50ee7c2c2c
Signed-off-by: Brian Foster 
Reviewed-on: http://review.gluster.com/3138
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

md-cache: fix time comparison to support 0 timeout

2012-04-02T18:25:12+00:00

The time comparison currently causes caching when the current
time matches the timeout. This causes caching to occur even
when the timeout parameter is set to 0. Fix the condition to
support the no caching scenario. A longer term approach is to
use gettimeofday() for more accurate granularity, but this
apparently fixes a known bug.

Change-Id: I6552c5961743edb22a1aec2e6a3a6e041d0647b2
BUG: 809193
Signed-off-by: Brian Foster 
Reviewed-on: http://review.gluster.com/3066
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

core: adding extra data for fops

2012-03-22T23:40:27+00:00

with this change, the xlator APIs will have a dictionary as extra
argument, which is passed between all the layers. This can be
utilized for overloading in some of the operations.

Change-Id: I58a8186b3ef647650280e63f3e5e9b9de7827b40
Signed-off-by: Amar Tumballi 
BUG: 782265
Reviewed-on: http://review.gluster.com/2960
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Fix memory leaks found in readdir

2012-03-01T19:42:56+00:00

Change-Id: I0e221e4de9ee12586b09cd8bf7f394e9d4b88a11
BUG: 765785
Reviewed-on: http://review.gluster.com/2853
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-by: Pranith Kumar Karampuri 
Reviewed-by: Vijay Bellur

perf/md-cache: hold lock on modification of md_cache structure

2012-03-01T03:45:45+00:00

as it is possible that multiple fops can alter the content of
md-cache structure from inode-ctx, we need to safe-guard the
issue of corruption due to race conditions.

Change-Id: Iea051f8f6adff7690d6d60f3cf82eda75150b449
Signed-off-by: Amar Tumballi 
BUG: 798179
Reviewed-on: http://review.gluster.com/2834
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/md-cache: Change strdup to gf_strdup

2012-02-21T10:28:54+00:00

Change-Id: Ic96e7d38b988efab49f723c9d1d46f65037a84ed
BUG: 795682
Reviewed-on: http://review.gluster.com/2783
Reviewed-by: Anand Avati 
Tested-by: Gluster Build System

performance/md-cache: Add mem_acct_init

2012-02-21T07:22:12+00:00

Change-Id: I1b91ee0bde160762548c39646ceb96c3c778e06c
BUG: 795651
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2778
Reviewed-by: Vijay Bellur 
Tested-by: Gluster Build System

md-cache: replace stat-prefetch with md-cache in a backward compatible way

2012-02-21T05:14:49+00:00

- preserve CLI set option key as "performance.stat-prefetch"
- create a symlink stat-prefetch.so to point to md-cache.so

Change-Id: Ib95e7c30073f13ae04c39e9466967ba1db5a0614
BUG: 765785
Reviewed-on: http://review.gluster.com/2714
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

md-cache: meta-data caching translator

2012-02-21T05:14:28+00:00

This is a metadata caching translator which is well integrated with
glusterfs core framework and leverages some of the recent protocol
changes to do a better job at caching.

- It uses the attributes returned along callbacks of all calls to
  update its attribute cache as frequently as possible without
  issuing calls on its own (i.e, very low overhead)

- It caches attributes returned via readdirp into the inode contexts
  corresponding to those entries (i.e, well integrated)

- It caches and updates xattrs and not just inode attributes (i.e,
  eliminates the need for a separate xattr-prefetch translator)

In its current form it has a timeout based consistency model

Change-Id: I891f6225c1a4c08bb111e287571d5f6d326dbe97
BUG: 765785
Reviewed-on: http://review.gluster.com/2713
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati