bartoszek/zephyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net/arp: There is no need to fill in the header in all frags

Browse Source 
Only the first one requires it. Same issue as already fixed in ethernet
side.

Signed-off-by: Tomasz Bursztyka <tomasz.bursztyka@linux.intel.com>
This commit is contained in:
Tomasz Bursztyka 2018-06-08 14:32:23 +02:00 committed by Anas Nashif
parent c0109fd690
commit 9e0cfaf0a7
1 changed files with 3 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
subsys/net/ip/l2/ethernet/arp.c
View File

@ -198,7 +198,6 @@ struct net_pkt *net_arp_prepare(struct net_pkt *pkt)
{
struct arp_entry *entry, *free_entry = NULL, *non_pending = NULL;
struct ethernet_context *ctx;
struct net_buf *frag;
struct net_linkaddr *ll;
struct net_eth_hdr *hdr;
struct in_addr *addr;
@ -290,22 +289,9 @@ struct net_pkt *net_arp_prepare(struct net_pkt *pkt)
net_sprint_ll_addr(ll->addr, sizeof(struct net_eth_addr)),
net_sprint_ipv4_addr(&NET_IPV4_HDR(pkt)->src));
frag = pkt->frags;
while (frag) {
/* If there is no room for link layer header, then
* just send the packet as is.
*/
if (!net_buf_headroom(frag)) {
frag = frag->frags;
continue;
}
hdr = net_eth_fill_header(ctx, pkt, frag,
htons(NET_ETH_PTYPE_IP),
ll->addr, entry->eth.addr);
frag = frag->frags;
}
net_eth_fill_header(ctx, pkt, pkt->frags,
htons(NET_ETH_PTYPE_IP),
ll->addr, entry->eth.addr);
return pkt;
}