diff options
| author | Krutika Dhananjay <kdhananj@redhat.com> | 2016-05-09 16:53:19 +0530 |
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2016-05-31 02:13:34 -0700 |
| commit | b5590f1db641a22ea515c9c4760a593d3342ffa0 (patch) | |
| tree | edd698ef366306d9f37a37c8a0f69bc6103016b9 /xlators/storage | |
| parent | c8da5669a15ed6944cceb9d003789ff333754bff (diff) | |
storage/posix: Print offset,size and gfid too when readv fails
... for better debuggability.
Here's a sample log:
[2016-05-31 05:18:34.893951] E [MSGID: 113040] [posix.c:3089:posix_readv]
0-dis-posix: read failed on gfid=3744d34b-f798-4ce2-b421-cc632a9903bd,
fd=0x7f8b3c015abc, offset=131072 size=131072 [Invalid argument]
Change-Id: I069c17fecbb01a4a2b5dea1eb482c7f56a191fd3
BUG: 1341009
Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com>
Reviewed-on: http://review.gluster.org/14271
Smoke: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Ashish Pandey <aspandey@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Diffstat (limited to 'xlators/storage')
| -rw-r--r-- | xlators/storage/posix/src/posix.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c index 8e1c12e1515..9ee2b2ed274 100644 --- a/xlators/storage/posix/src/posix.c +++ b/xlators/storage/posix/src/posix.c @@ -3084,7 +3084,9 @@ posix_readv (call_frame_t *frame, xlator_t *this, if (op_ret == -1) { op_errno = errno; gf_msg (this->name, GF_LOG_ERROR, errno, P_MSG_READ_FAILED, - "read failed on fd=%p", fd); + "read failed on gfid=%s, fd=%p, offset=%"PRIu64" " + "size=%"GF_PRI_SIZET"", uuid_utoa (fd->inode->gfid), fd, + offset, size); goto out; } |