Commit 4b5bdf5c authored by Roberto Nibali's avatar Roberto Nibali Committed by David S. Miller

[IPVS]: Cleanup IP_VS_DBG statements.

From: Roberto Nibali <ratz@drugphish.ch>

The attached patch (against current -GIT) is a cleanup patch which does
following:

o lookup debug messages shifted back to 9
o added more informational value to flags and refcnt since those
entries can be in multiple referenced structures
o cleanup 80 char violation

It's the prepatch to the session pool implementation and helps very much
to debug and monitor important variables and structures regarding the
threshold limitation and persistency without the thousands of lookup
messages which noone is interested in.
Signed-off-by: default avatarHorms <horms@verge.net.au>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent fd30333d
...@@ -221,7 +221,7 @@ struct ip_vs_conn *ip_vs_conn_in_get ...@@ -221,7 +221,7 @@ struct ip_vs_conn *ip_vs_conn_in_get
if (!cp && atomic_read(&ip_vs_conn_no_cport_cnt)) if (!cp && atomic_read(&ip_vs_conn_no_cport_cnt))
cp = __ip_vs_conn_in_get(protocol, s_addr, 0, d_addr, d_port); cp = __ip_vs_conn_in_get(protocol, s_addr, 0, d_addr, d_port);
IP_VS_DBG(7, "lookup/in %s %u.%u.%u.%u:%d->%u.%u.%u.%u:%d %s\n", IP_VS_DBG(9, "lookup/in %s %u.%u.%u.%u:%d->%u.%u.%u.%u:%d %s\n",
ip_vs_proto_name(protocol), ip_vs_proto_name(protocol),
NIPQUAD(s_addr), ntohs(s_port), NIPQUAD(s_addr), ntohs(s_port),
NIPQUAD(d_addr), ntohs(d_port), NIPQUAD(d_addr), ntohs(d_port),
...@@ -256,7 +256,7 @@ struct ip_vs_conn *ip_vs_ct_in_get ...@@ -256,7 +256,7 @@ struct ip_vs_conn *ip_vs_ct_in_get
out: out:
ct_read_unlock(hash); ct_read_unlock(hash);
IP_VS_DBG(7, "template lookup/in %s %u.%u.%u.%u:%d->%u.%u.%u.%u:%d %s\n", IP_VS_DBG(9, "template lookup/in %s %u.%u.%u.%u:%d->%u.%u.%u.%u:%d %s\n",
ip_vs_proto_name(protocol), ip_vs_proto_name(protocol),
NIPQUAD(s_addr), ntohs(s_port), NIPQUAD(s_addr), ntohs(s_port),
NIPQUAD(d_addr), ntohs(d_port), NIPQUAD(d_addr), ntohs(d_port),
...@@ -297,7 +297,7 @@ struct ip_vs_conn *ip_vs_conn_out_get ...@@ -297,7 +297,7 @@ struct ip_vs_conn *ip_vs_conn_out_get
ct_read_unlock(hash); ct_read_unlock(hash);
IP_VS_DBG(7, "lookup/out %s %u.%u.%u.%u:%d->%u.%u.%u.%u:%d %s\n", IP_VS_DBG(9, "lookup/out %s %u.%u.%u.%u:%d->%u.%u.%u.%u:%d %s\n",
ip_vs_proto_name(protocol), ip_vs_proto_name(protocol),
NIPQUAD(s_addr), ntohs(s_port), NIPQUAD(s_addr), ntohs(s_port),
NIPQUAD(d_addr), ntohs(d_port), NIPQUAD(d_addr), ntohs(d_port),
...@@ -393,8 +393,9 @@ ip_vs_bind_dest(struct ip_vs_conn *cp, struct ip_vs_dest *dest) ...@@ -393,8 +393,9 @@ ip_vs_bind_dest(struct ip_vs_conn *cp, struct ip_vs_dest *dest)
cp->flags |= atomic_read(&dest->conn_flags); cp->flags |= atomic_read(&dest->conn_flags);
cp->dest = dest; cp->dest = dest;
IP_VS_DBG(9, "Bind-dest %s c:%u.%u.%u.%u:%d v:%u.%u.%u.%u:%d " IP_VS_DBG(7, "Bind-dest %s c:%u.%u.%u.%u:%d v:%u.%u.%u.%u:%d "
"d:%u.%u.%u.%u:%d fwd:%c s:%u flg:%X cnt:%d destcnt:%d\n", "d:%u.%u.%u.%u:%d fwd:%c s:%u conn->flags:%X conn->refcnt:%d "
"dest->refcnt:%d\n",
ip_vs_proto_name(cp->protocol), ip_vs_proto_name(cp->protocol),
NIPQUAD(cp->caddr), ntohs(cp->cport), NIPQUAD(cp->caddr), ntohs(cp->cport),
NIPQUAD(cp->vaddr), ntohs(cp->vport), NIPQUAD(cp->vaddr), ntohs(cp->vport),
...@@ -432,8 +433,9 @@ static inline void ip_vs_unbind_dest(struct ip_vs_conn *cp) ...@@ -432,8 +433,9 @@ static inline void ip_vs_unbind_dest(struct ip_vs_conn *cp)
if (!dest) if (!dest)
return; return;
IP_VS_DBG(9, "Unbind-dest %s c:%u.%u.%u.%u:%d v:%u.%u.%u.%u:%d " IP_VS_DBG(7, "Unbind-dest %s c:%u.%u.%u.%u:%d v:%u.%u.%u.%u:%d "
"d:%u.%u.%u.%u:%d fwd:%c s:%u flg:%X cnt:%d destcnt:%d\n", "d:%u.%u.%u.%u:%d fwd:%c s:%u conn->flags:%X conn->refcnt:%d "
"dest->refcnt:%d\n",
ip_vs_proto_name(cp->protocol), ip_vs_proto_name(cp->protocol),
NIPQUAD(cp->caddr), ntohs(cp->cport), NIPQUAD(cp->caddr), ntohs(cp->cport),
NIPQUAD(cp->vaddr), ntohs(cp->vport), NIPQUAD(cp->vaddr), ntohs(cp->vport),
...@@ -573,7 +575,7 @@ static void ip_vs_conn_expire(unsigned long data) ...@@ -573,7 +575,7 @@ static void ip_vs_conn_expire(unsigned long data)
ip_vs_conn_hash(cp); ip_vs_conn_hash(cp);
expire_later: expire_later:
IP_VS_DBG(7, "delayed: refcnt-1=%d conn.n_control=%d\n", IP_VS_DBG(7, "delayed: conn->refcnt-1=%d conn->n_control=%d\n",
atomic_read(&cp->refcnt)-1, atomic_read(&cp->refcnt)-1,
atomic_read(&cp->n_control)); atomic_read(&cp->n_control));
......
...@@ -426,7 +426,7 @@ ip_vs_schedule(struct ip_vs_service *svc, const struct sk_buff *skb) ...@@ -426,7 +426,7 @@ ip_vs_schedule(struct ip_vs_service *svc, const struct sk_buff *skb)
return NULL; return NULL;
IP_VS_DBG(6, "Schedule fwd:%c c:%u.%u.%u.%u:%u v:%u.%u.%u.%u:%u " IP_VS_DBG(6, "Schedule fwd:%c c:%u.%u.%u.%u:%u v:%u.%u.%u.%u:%u "
"d:%u.%u.%u.%u:%u flg:%X cnt:%d\n", "d:%u.%u.%u.%u:%u conn->flags:%X conn->refcnt:%d\n",
ip_vs_fwd_tag(cp), ip_vs_fwd_tag(cp),
NIPQUAD(cp->caddr), ntohs(cp->cport), NIPQUAD(cp->caddr), ntohs(cp->cport),
NIPQUAD(cp->vaddr), ntohs(cp->vport), NIPQUAD(cp->vaddr), ntohs(cp->vport),
......
...@@ -448,7 +448,7 @@ ip_vs_service_get(__u32 fwmark, __u16 protocol, __u32 vaddr, __u16 vport) ...@@ -448,7 +448,7 @@ ip_vs_service_get(__u32 fwmark, __u16 protocol, __u32 vaddr, __u16 vport)
out: out:
read_unlock(&__ip_vs_svc_lock); read_unlock(&__ip_vs_svc_lock);
IP_VS_DBG(6, "lookup service: fwm %u %s %u.%u.%u.%u:%u %s\n", IP_VS_DBG(9, "lookup service: fwm %u %s %u.%u.%u.%u:%u %s\n",
fwmark, ip_vs_proto_name(protocol), fwmark, ip_vs_proto_name(protocol),
NIPQUAD(vaddr), ntohs(vport), NIPQUAD(vaddr), ntohs(vport),
svc?"hit":"not hit"); svc?"hit":"not hit");
...@@ -598,7 +598,7 @@ ip_vs_trash_get_dest(struct ip_vs_service *svc, __u32 daddr, __u16 dport) ...@@ -598,7 +598,7 @@ ip_vs_trash_get_dest(struct ip_vs_service *svc, __u32 daddr, __u16 dport)
*/ */
list_for_each_entry_safe(dest, nxt, &ip_vs_dest_trash, n_list) { list_for_each_entry_safe(dest, nxt, &ip_vs_dest_trash, n_list) {
IP_VS_DBG(3, "Destination %u/%u.%u.%u.%u:%u still in trash, " IP_VS_DBG(3, "Destination %u/%u.%u.%u.%u:%u still in trash, "
"refcnt=%d\n", "dest->refcnt=%d\n",
dest->vfwmark, dest->vfwmark,
NIPQUAD(dest->addr), ntohs(dest->port), NIPQUAD(dest->addr), ntohs(dest->port),
atomic_read(&dest->refcnt)); atomic_read(&dest->refcnt));
...@@ -805,7 +805,7 @@ ip_vs_add_dest(struct ip_vs_service *svc, struct ip_vs_dest_user *udest) ...@@ -805,7 +805,7 @@ ip_vs_add_dest(struct ip_vs_service *svc, struct ip_vs_dest_user *udest)
dest = ip_vs_trash_get_dest(svc, daddr, dport); dest = ip_vs_trash_get_dest(svc, daddr, dport);
if (dest != NULL) { if (dest != NULL) {
IP_VS_DBG(3, "Get destination %u.%u.%u.%u:%u from trash, " IP_VS_DBG(3, "Get destination %u.%u.%u.%u:%u from trash, "
"refcnt=%d, service %u/%u.%u.%u.%u:%u\n", "dest->refcnt=%d, service %u/%u.%u.%u.%u:%u\n",
NIPQUAD(daddr), ntohs(dport), NIPQUAD(daddr), ntohs(dport),
atomic_read(&dest->refcnt), atomic_read(&dest->refcnt),
dest->vfwmark, dest->vfwmark,
...@@ -950,7 +950,8 @@ static void __ip_vs_del_dest(struct ip_vs_dest *dest) ...@@ -950,7 +950,8 @@ static void __ip_vs_del_dest(struct ip_vs_dest *dest)
atomic_dec(&dest->svc->refcnt); atomic_dec(&dest->svc->refcnt);
kfree(dest); kfree(dest);
} else { } else {
IP_VS_DBG(3, "Moving dest %u.%u.%u.%u:%u into trash, refcnt=%d\n", IP_VS_DBG(3, "Moving dest %u.%u.%u.%u:%u into trash, "
"dest->refcnt=%d\n",
NIPQUAD(dest->addr), ntohs(dest->port), NIPQUAD(dest->addr), ntohs(dest->port),
atomic_read(&dest->refcnt)); atomic_read(&dest->refcnt));
list_add(&dest->n_list, &ip_vs_dest_trash); list_add(&dest->n_list, &ip_vs_dest_trash);
......
...@@ -426,7 +426,7 @@ set_tcp_state(struct ip_vs_protocol *pp, struct ip_vs_conn *cp, ...@@ -426,7 +426,7 @@ set_tcp_state(struct ip_vs_protocol *pp, struct ip_vs_conn *cp,
struct ip_vs_dest *dest = cp->dest; struct ip_vs_dest *dest = cp->dest;
IP_VS_DBG(8, "%s %s [%c%c%c%c] %u.%u.%u.%u:%d->" IP_VS_DBG(8, "%s %s [%c%c%c%c] %u.%u.%u.%u:%d->"
"%u.%u.%u.%u:%d state: %s->%s cnt:%d\n", "%u.%u.%u.%u:%d state: %s->%s conn->refcnt:%d\n",
pp->name, pp->name,
(state_off==TCP_DIR_OUTPUT)?"output ":"input ", (state_off==TCP_DIR_OUTPUT)?"output ":"input ",
th->syn? 'S' : '.', th->syn? 'S' : '.',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment