diff options
| author | Raghavendra G <raghavendra@gluster.com> | 2009-09-25 02:55:12 +0000 |
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2009-09-28 18:57:13 -0700 |
| commit | 03b5a87b2f4d641c798440923b255ca30a13bf47 (patch) | |
| tree | cc0dc4becbfd7267e6fe988ca440e7818a953f20 | |
| parent | 009fc3e3e921517a7b986f7b5a73676f43edb358 (diff) | |
| download | glusterfs-03b5a87b2f4d641c798440923b255ca30a13bf47.tar.gz glusterfs-03b5a87b2f4d641c798440923b255ca30a13bf47.tar.xz glusterfs-03b5a87b2f4d641c798440923b255ca30a13bf47.zip | |
libglusterfsclient: don't destroy the iobuf pool during fini.
- some of the iobufs in pool will be cached by io-cache and hence refcount
will not be zero. Hence the assertion of ref being zero in
__iobuf_arena_destroy fails. Commenting out iobuf_pool_destroy during fini, till
we introduce proper cleanup in all translators, thereby allowing io-cache
to release the buffers it has held.
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 283 (booster aborts complaining the refcount of iobuf is not zero during glusterfs_umount_all.)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=283
| -rwxr-xr-x | libglusterfsclient/src/libglusterfsclient.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libglusterfsclient/src/libglusterfsclient.c b/libglusterfsclient/src/libglusterfsclient.c index 419944af6b..8649b743b4 100755 --- a/libglusterfsclient/src/libglusterfsclient.c +++ b/libglusterfsclient/src/libglusterfsclient.c @@ -1866,7 +1866,7 @@ glusterfs_fini (glusterfs_handle_t handle) FREE (ctx->gf_ctx.cmd_args.volume_name); FREE (ctx->gf_ctx.pool); FREE (ctx->gf_ctx.event_pool); - iobuf_pool_destroy (ctx->gf_ctx.iobuf_pool); + /* iobuf_pool_destroy (ctx->gf_ctx.iobuf_pool); */ ((gf_timer_registry_t *)ctx->gf_ctx.timer)->fin = 1; xlator_graph_fini (ctx->gf_ctx.graph); |