diff options
| author |   Andy Grover <andy.grover@oracle.com> | 2011-01-13 11:40:31 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-08-15 12:04:24 -0700 |
| commit | 11740ef4482914fcd8c9814ef7ceb7085715e554 (patch) | |
| tree | 0507a9a67059c1d06cb410aaad1201f11b85de8b /net/rds/send.c | |
| parent | net/rds: Add a few missing rds_stat_names entries (diff) | |
| download | linux-dev-11740ef4482914fcd8c9814ef7ceb7085715e554.tar.xz linux-dev-11740ef4482914fcd8c9814ef7ceb7085715e554.zip | |
rds: check for excessive looping in rds_send_xmit
Original commit from 2011 updated to include a change by
Yuval Shaia <yuval.shaia@oracle.com>
that adds a new statistic counter "send_stuck_rm"
to capture the messages looping exessively
in the send path.
Signed-off-by: Gerd Rausch <gerd.rausch@oracle.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rds/send.c')
| -rw-r--r-- | net/rds/send.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/net/rds/send.c b/net/rds/send.c index 031b1e97a466..9ce552abf9e9 100644 --- a/net/rds/send.c +++ b/net/rds/send.c @@ -145,6 +145,7 @@ int rds_send_xmit(struct rds_conn_path *cp) LIST_HEAD(to_be_dropped); int batch_count; unsigned long send_gen = 0; + int same_rm = 0; restart: batch_count = 0; @@ -200,6 +201,17 @@ restart: rm = cp->cp_xmit_rm; + if (!rm) { + same_rm = 0; + } else { + same_rm++; + if (same_rm >= 4096) { + rds_stats_inc(s_send_stuck_rm); + ret = -EAGAIN; + break; + } + } + /* * If between sending messages, we can send a pending congestion * map update. |