diff options
author | Patrick Schaaf <netdev@bof.de> | 2011-02-11 14:01:12 +0100 |
---|---|---|
committer | Simon Horman <horms@verge.net.au> | 2011-02-16 14:53:33 +0900 |
commit | 41ac51eeda58a85b8a06d748cce7035cc77deebd (patch) | |
tree | d647f55796e3b7cb0d5a9f86429ce89877afacec /net/netfilter/ipvs/ip_vs_sed.c | |
parent | 6cb90db502c5f276c8d6256762cc3acde4d3bd9d (diff) | |
download | op-kernel-dev-41ac51eeda58a85b8a06d748cce7035cc77deebd.zip op-kernel-dev-41ac51eeda58a85b8a06d748cce7035cc77deebd.tar.gz |
ipvs: make "no destination available" message more informative
When IP_VS schedulers do not find a destination, they output a terse
"WLC: no destination available" message through kernel syslog, which I
can not only make sense of because syslog puts them in a logfile
together with keepalived checker results.
This patch makes the output a bit more informative, by telling you which
virtual service failed to find a destination.
Example output:
kernel: [1539214.552233] IPVS: wlc: TCP 192.168.8.30:22 - no destination available
kernel: [1539299.674418] IPVS: wlc: FWM 22 0x00000016 - no destination available
I have tested the code for IPv4 and FWM services, as you can see from
the example; I do not have an IPv6 setup to test the third code path
with.
To avoid code duplication, I put a new function ip_vs_scheduler_err()
into ip_vs_sched.c, and use that from the schedulers instead of calling
IP_VS_ERR_RL directly.
Signed-off-by: Patrick Schaaf <netdev@bof.de>
Signed-off-by: Simon Horman <horms@verge.net.au>
Diffstat (limited to 'net/netfilter/ipvs/ip_vs_sed.c')
-rw-r--r-- | net/netfilter/ipvs/ip_vs_sed.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/ipvs/ip_vs_sed.c b/net/netfilter/ipvs/ip_vs_sed.c index 1ab75a9..89ead246 100644 --- a/net/netfilter/ipvs/ip_vs_sed.c +++ b/net/netfilter/ipvs/ip_vs_sed.c @@ -87,7 +87,7 @@ ip_vs_sed_schedule(struct ip_vs_service *svc, const struct sk_buff *skb) goto nextstage; } } - IP_VS_ERR_RL("SED: no destination available\n"); + ip_vs_scheduler_err(svc, "no destination available"); return NULL; /* |