Changes in / [968:652713ce3596:967:bbc08d58325a] in freeDiameter

Files:

• include/freeDiameter/libfdcore.h (modified) (1 diff)
• libfdcore/fdcore-internal.h (modified) (1 diff)
• libfdcore/fdd.l (modified) (1 diff)
• libfdcore/p_sr.c (modified) (3 diffs)
• libfdcore/peers.c (modified) (2 diffs)

include/freeDiameter/libfdcore.h

@@ -385 +385 @@
 int fd_peer_get_state(struct peer_hdr *peer);
 
-/* 
- * FUNCTION:    fd_peer_get_load_pending
- *
- * PARAMETERS:
- *  peer        : The peer which load to read
- *
- * DESCRIPTION: 
- *   Returns the current number of requests sent to this peer
- *  that have not been answered yet. This is an empirical indication
- *  of the workload of this peer.
- *
- * RETURN VALUE:
- *  0  : The load parameter has been updated. (it should have a positive value always)
- * !0  : An error occurred
- */
-int fd_peer_get_load_pending(struct peer_hdr *peer, int * load);
-
 /*
  * FUNCTION:    fd_peer_validate_register

libfdcore/fdcore-internal.h

@@ -126 +126 @@
         struct fd_list  srs; /* requests ordered by hop-by-hop id */
         struct fd_list  exp; /* requests that have a timeout set, ordered by timeout */
-        int             cnt; /* number of requests in the srs list */
         pthread_mutex_t mtx; /* mutex to protect these lists */
         pthread_cond_t  cnd; /* cond var used by the thread that handles timeouts */

libfdcore/fdd.l

@@ -52 +52 @@
         yylloc->first_column = yylloc->last_column + 1;                 \
         yylloc->last_column = yylloc->first_column + yyleng - 1;        \
-        fd_log_debug(                                                   \
+        TRACE_DEBUG_ERROR(                                              \
                 "(%d:%d-%d:%d) matched rule %d, length=%d, txt='%s'\n", \
                 yylloc->first_line, yylloc->first_column,               \

libfdcore/p_sr.c

@@ -226 +226 @@
         *req = NULL;
         fd_list_insert_before(next, &sr->chain);
-        srlist->cnt++;
         srl_dump("Saved new request, ", &srlist->srs);
         
@@ -282 +281 @@
                 /* Unlink */
                 fd_list_unlink(&sr->chain);
-                srlist->cnt--;
                 fd_list_unlink(&sr->expire);
                 *req = sr->req;
@@ -302 +300 @@
                 struct sentreq * sr = (struct sentreq *)(srlist->srs.next);
                 fd_list_unlink(&sr->chain);
-                srlist->cnt--;
                 fd_list_unlink(&sr->expire);
                 if (fd_msg_is_routable(sr->req)) {

libfdcore/peers.c

@@ -257 +257 @@
 }
 
-/* Return the value of srlist->cnt */
-int fd_peer_get_load_pending(struct peer_hdr *peer, int * load)
-{
-        struct fd_peer * p = (struct fd_peer *)peer;
-        TRACE_ENTRY("%p %p", peer, load);
-        CHECK_PARAMS(CHECK_PEER(peer) && load);
-        
-        CHECK_POSIX( pthread_mutex_lock(&p->p_sr.mtx) );
-        *load = p->p_sr.cnt;
-        CHECK_POSIX( pthread_mutex_unlock(&p->p_sr.mtx) );
-        
-        return 0;
-}
-
-
 /* Destroy a structure once cleanups have been performed (fd_psm_abord, ...) */
 int fd_peer_free(struct fd_peer ** ptr)
@@ -410 +395 @@
         }
 
-        fd_log_debug(">  %s\t%s\t[%dsr]", STATE_STR(fd_peer_getstate(peer)), peer->p_hdr.info.pi_diamid, peer->p_sr.cnt);
+        fd_log_debug(">  %s\t%s", STATE_STR(fd_peer_getstate(peer)), peer->p_hdr.info.pi_diamid);
         if (details > INFO) {
                 fd_log_debug("\t(rlm:%s)", peer->p_hdr.info.runtime.pir_realm ?: "<unknown>");