summaryrefslogtreecommitdiffstats
path: root/meta/recipes-extended/watchdog/files/fix-ping-failure.patch
diff options
context:
space:
mode:
Diffstat (limited to 'meta/recipes-extended/watchdog/files/fix-ping-failure.patch')
-rw-r--r--meta/recipes-extended/watchdog/files/fix-ping-failure.patch65
1 files changed, 65 insertions, 0 deletions
diff --git a/meta/recipes-extended/watchdog/files/fix-ping-failure.patch b/meta/recipes-extended/watchdog/files/fix-ping-failure.patch
new file mode 100644
index 0000000000..f5976eb5cf
--- /dev/null
+++ b/meta/recipes-extended/watchdog/files/fix-ping-failure.patch
@@ -0,0 +1,65 @@
1Fix ping mode failure
2
3Upstream-Status: Pending
4
5When watchdog works on ping mode, the system will be rebooted since
6watchdog can not receive the expected ECOREPLY on a setting interval.
7
8Ping mode uses a raw socket to send a ECO packet, then uses select()
9to wait and recvfrom() to receive the ECOREPLY packet, if select()
10shows the data is ready, and the data is not the expected ECOREPLY,
11and waiting time is not overdue, it will continue use select() and
12recvfrom().
13
14Problem is that the raw socket can receive any icmp packets, if we do
15not set filters, and there are many icmp packets on socket, this
16program will not find its interested ECOREPLY packet in a special
17interval, which makes the ping mode fail.
18
19
20Other program is that watchdog sometime can not reach the call of
21recvfrom to try to receive packets since tv_sec of struct timeval
22of select parameter is 0.
23
24The timeout of select() is the result of ping interval minusing the
25time of calling gettimeofday spending, when ping interval is 1 second,
26and the call of gettimeofday() spends several useconds, the tv_sec of
27struct timeval of select parameter must be 0, at that condition, we
28should it is valid of tv_sec of struct timeval of select parameter be 0
29
30Signed-off-by: Roy.Li <rongqing.li@windriver.com>
31---
32 src/net.c | 2 +-
33 src/watchdog.c | 5 ++++-
34 2 files changed, 5 insertions(+), 2 deletions(-)
35
36Index: watchdog-5.13/src/watchdog.c
37===================================================================
38--- watchdog-5.13.orig/src/watchdog.c 2013-02-01 03:15:44.000000000 -0800
39+++ watchdog-5.13/src/watchdog.c 2013-03-11 22:27:48.741657881 -0700
40@@ -28,6 +28,7 @@
41 #include <sys/types.h>
42 #include <sys/ioctl.h>
43 #include <linux/oom.h>
44+#include <linux/icmp.h>
45 #include <linux/watchdog.h>
46 #include <string.h>
47
48@@ -567,6 +568,8 @@
49 pid_t child_pid;
50 int oom_adjusted = 0;
51 struct stat s;
52+ struct icmp_filter filt;
53+ filt.data = ~(1<<ICMP_ECHOREPLY);
54
55 #if USE_SYSLOG
56 char *opts = "d:i:n:Ffsvbql:p:t:c:r:m:a:";
57@@ -703,7 +706,7 @@
58 perror(progname);
59 exit(1);
60 }
61-
62+ setsockopt(net->sock_fp, SOL_RAW, ICMP_FILTER, (char*)&filt, sizeof(filt));
63 /* this is necessary for broadcast pings to work */
64 (void) setsockopt(net->sock_fp, SOL_SOCKET, SO_BROADCAST, (char *)&hold, sizeof(hold));
65