diff options
author | Amar Tumballi <amar@gluster.com> | 2012-02-14 15:05:19 +0530 |
---|---|---|
committer | Vijay Bellur <vijay@gluster.com> | 2012-02-20 00:46:27 -0800 |
commit | c0b8e886cac4ef0f16d5f93adab02229bb1414cd (patch) | |
tree | 03f5bc6550d5f2501d5b968776f4bb0ee8025310 /xlators/performance | |
parent | 975933a25d14cbac861e809b40c6edd01acaa28d (diff) |
iobuf: use 'iobuf_get2()' to get variable sized buffers
added 'TODO' in places where it is missing.
Change-Id: Ia802c94e3bb76930f7c88c990f078525be5459f5
Signed-off-by: Amar Tumballi <amar@gluster.com>
BUG: 765264
Reviewed-on: http://review.gluster.com/388
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vijay@gluster.com>
Diffstat (limited to 'xlators/performance')
-rw-r--r-- | xlators/performance/quick-read/src/quick-read.c | 3 | ||||
-rw-r--r-- | xlators/performance/write-behind/src/write-behind.c | 1 |
2 files changed, 4 insertions, 0 deletions
diff --git a/xlators/performance/quick-read/src/quick-read.c b/xlators/performance/quick-read/src/quick-read.c index 14ff58b5157..c1460b1debf 100644 --- a/xlators/performance/quick-read/src/quick-read.c +++ b/xlators/performance/quick-read/src/quick-read.c @@ -1180,6 +1180,9 @@ qr_readv (call_frame_t *frame, xlator_t *this, fd_t *fd, size_t size, } for (i = 0; i < count; i++) { + /* TODO: Now that we have support for variable + io-buf-sizes, i guess we need to get rid of + default size here */ iobuf = iobuf_get (iobuf_pool); if (iobuf == NULL) { op_ret = -1; diff --git a/xlators/performance/write-behind/src/write-behind.c b/xlators/performance/write-behind/src/write-behind.c index 7c666b40339..be7da2f44ce 100644 --- a/xlators/performance/write-behind/src/write-behind.c +++ b/xlators/performance/write-behind/src/write-behind.c @@ -1901,6 +1901,7 @@ __wb_copy_into_holder (wb_request_t *holder, wb_request_t *request) int ret = -1; if (holder->flags.write_request.virgin) { + /* TODO: check the required size */ iobuf = iobuf_get (request->file->this->ctx->iobuf_pool); if (iobuf == NULL) { goto out; |