michaelh/zephyr
1
0
Fork 0
Code Releases Activity

samples: net: echo_client: handle net if mtu

Browse source 
UDP portion of the echo_client sends 1 packet with the sample data.
(TCP can send chunks of it depending on the response of the send()
function.)  Not every network interface can send a UDP packet large
enough to handle the size of the sample data.

Let's make sure to account for the network interface MTU when deciding
the amount of sample data to send.

Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/22447

Signed-off-by: Michael Scott <mike@foundries.io>
This commit is contained in:
Michael Scott 2020-02-03 09:43:54 -08:00 • committed by Jukka Rissanen
commit 2c7e420a8e
3 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
samples/net/sockets/echo_client/src/common.h
View file

@ -20,6 +20,7 @@ struct data {
struct k_delayed_work transmit;
u32_t expecting;
u32_t counter;
u32_t mtu;
} udp;
struct {

2
samples/net/sockets/echo_client/src/echo-client.c
View file

@ -198,6 +198,8 @@ static void event_handler(struct net_mgmt_event_callback *cb,
LOG_INF("Network connected");
connected = true;
conf.ipv4.udp.mtu = net_if_get_mtu(iface);
conf.ipv6.udp.mtu = conf.ipv4.udp.mtu;
k_sem_give(&run_app);
return;

3
samples/net/sockets/echo_client/src/udp.c
View file

@ -32,7 +32,8 @@ static int send_udp_data(struct data *data)
do {
data->udp.expecting = sys_rand32_get() % ipsum_len;
} while (data->udp.expecting == 0U);
} while (data->udp.expecting == 0U ||
data->udp.expecting > data->udp.mtu);
ret = send(data->udp.sock, lorem_ipsum, data->udp.expecting, 0);