diff options
| author | Milind Changire <mchangir@redhat.com> | 2017-07-07 10:49:23 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2017-07-21 06:22:21 +0000 |
| commit | 08ee8541cfc9096a7f1cb40db7d7df763256d535 (patch) | |
| tree | a4622446ba7980f1d98dbd3963ed7eb02084801f | |
| parent | e01783d871fbbf5a598d3bbf984ea98bafa5c10f (diff) | |
rpc: include current second in timed out frame cleanup
Problem:
frames which time out at current second are missed out
Solution:
change test to include frames timing out at current second
i.e. timeout <= current.tv_sec
instead of
timeout < current.tv_sec
Change-Id: I459d47856ade2b657a0289e49f7f63da29186d6e
BUG: 1468433
Signed-off-by: Milind Changire <mchangir@redhat.com>
Reviewed-on: https://review.gluster.org/17722
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
| -rw-r--r-- | rpc/rpc-lib/src/rpc-clnt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/rpc/rpc-lib/src/rpc-clnt.c b/rpc/rpc-lib/src/rpc-clnt.c index ef8f764b62f..e34d2ca47ac 100644 --- a/rpc/rpc-lib/src/rpc-clnt.c +++ b/rpc/rpc-lib/src/rpc-clnt.c @@ -32,7 +32,7 @@ __saved_frames_get_timedout (struct saved_frames *frames, uint32_t timeout, if (!list_empty(&frames->sf.list)) { tmp = list_entry (frames->sf.list.next, typeof (*tmp), list); - if ((tmp->saved_at.tv_sec + timeout) < current->tv_sec) { + if ((tmp->saved_at.tv_sec + timeout) <= current->tv_sec) { bailout_frame = tmp; list_del_init (&bailout_frame->list); frames->count--; |