luchianmihai/walnux
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

rpmsg_ping.c: change check data to fluctuation value

Browse source 
We can found the error happend packet more accurately based
on the data value.

Signed-off-by: Yongrong Wang <wangyongrong@xiaomi.com>
This commit is contained in:
Yongrong Wang 2024-07-17 13:58:57 +08:00 • committed by GUIDINGLI
parent 5e96e72cb6
commit b96ad59d69
1 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
drivers/rpmsg/rpmsg_ping.c
View file

@ -44,7 +44,6 @@
#define RPMSG_PING_SEND_NOACK 3 #define RPMSG_PING_SEND_NOACK 3
#define RPMSG_PING_SEND_ACK 4 #define RPMSG_PING_SEND_ACK 4
#define RPMSG_PING_ACK 5 #define RPMSG_PING_ACK 5
#define RPMSG_PING_CHECK_DATA 0xee
/**************************************************************************** /****************************************************************************
* Private Types * Private Types
@ -82,7 +81,7 @@ static int rpmsg_ping_ept_cb(FAR struct rpmsg_endpoint *ept,
data_len = msg->len - sizeof(struct rpmsg_ping_msg_s) + 1; data_len = msg->len - sizeof(struct rpmsg_ping_msg_s) + 1;
for (i = 0; i < data_len; i++) for (i = 0; i < data_len; i++)
{ {
if (msg->data[i] != RPMSG_PING_CHECK_DATA) if (msg->data[i] != msg->data[0])
{ {
syslog(LOG_ERR, "receive data error at %zu of %zu\n", syslog(LOG_ERR, "receive data error at %zu of %zu\n",
i, data_len); i, data_len);
@ -108,8 +107,8 @@ static int rpmsg_ping_ept_cb(FAR struct rpmsg_endpoint *ept,
return 0; return 0;
} }
static int rpmsg_ping_once(FAR struct rpmsg_endpoint *ept, static int rpmsg_ping_once(FAR struct rpmsg_endpoint *ept, int len,
int len, int ack, uint32_t *buf_len) int ack, uint32_t *buf_len, char i)
{ {
FAR struct rpmsg_ping_msg_s *msg; FAR struct rpmsg_ping_msg_s *msg;
int ret; int ret;
@ -147,8 +146,7 @@ static int rpmsg_ping_once(FAR struct rpmsg_endpoint *ept,
if (msg->cmd == RPMSG_PING_SEND_CHECK) if (msg->cmd == RPMSG_PING_SEND_CHECK)
{ {
memset(msg->data, RPMSG_PING_CHECK_DATA, memset(msg->data, i, len - sizeof(struct rpmsg_ping_msg_s) + 1);
len - sizeof(struct rpmsg_ping_msg_s) + 1);
} }
nxsem_init(&sem, 0, 0); nxsem_init(&sem, 0, 0);
@ -228,7 +226,7 @@ int rpmsg_ping(FAR struct rpmsg_endpoint *ept,
{ {
clock_t tm = perf_gettime(); clock_t tm = perf_gettime();
send_len = rpmsg_ping_once(ept, ping->len, ping->ack, &buf_len); send_len = rpmsg_ping_once(ept, ping->len, ping->ack, &buf_len, i);
if (send_len < 0) if (send_len < 0)
{ {
return send_len; return send_len;