075eb6a6d2
When using IOB queue to store readahead data, we use one IOB for each UDP packet. Then if the packets are very small, like 10Bytes per packet, we'll use ~1600 IOBs just for 16KB recv buffer size, which is wasteful and dangerous. So change conn->readahead to a single IOB chain like TCP. Benefits: - Using memory and IOBs more efficiently (small packets are common in UDP) Side effects: - UDP recv buffer size may count the overhead - A little bit drop in performance (<1%, more seek & copy) Signed-off-by: Zhe Weng <wengzhe@xiaomi.com>
101 lines
2.9 KiB
C
101 lines
2.9 KiB
C
/****************************************************************************
|
|
* net/udp/udp_ioctl.c
|
|
*
|
|
* Licensed to the Apache Software Foundation (ASF) under one or more
|
|
* contributor license agreements. See the NOTICE file distributed with
|
|
* this work for additional information regarding copyright ownership. The
|
|
* ASF licenses this file to you under the Apache License, Version 2.0 (the
|
|
* "License"); you may not use this file except in compliance with the
|
|
* License. You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
* License for the specific language governing permissions and limitations
|
|
* under the License.
|
|
*
|
|
****************************************************************************/
|
|
|
|
/****************************************************************************
|
|
* Included Files
|
|
****************************************************************************/
|
|
|
|
#include <nuttx/config.h>
|
|
|
|
#include <stdint.h>
|
|
#include <stdbool.h>
|
|
#include <debug.h>
|
|
#include <errno.h>
|
|
|
|
#include <net/if.h>
|
|
|
|
#include <nuttx/fs/ioctl.h>
|
|
#include <nuttx/mm/iob.h>
|
|
#include <nuttx/net/net.h>
|
|
|
|
#include "udp/udp.h"
|
|
|
|
/****************************************************************************
|
|
* Public Functions
|
|
****************************************************************************/
|
|
|
|
/****************************************************************************
|
|
* Name: udp_ioctl
|
|
*
|
|
* Description:
|
|
* This function performs udp specific ioctl() operations.
|
|
*
|
|
* Parameters:
|
|
* conn The TCP connection of interest
|
|
* cmd The ioctl command
|
|
* arg The argument of the ioctl cmd
|
|
*
|
|
****************************************************************************/
|
|
|
|
int udp_ioctl(FAR struct udp_conn_s *conn, int cmd, unsigned long arg)
|
|
{
|
|
FAR struct iob_s *iob;
|
|
int ret = OK;
|
|
|
|
net_lock();
|
|
|
|
switch (cmd)
|
|
{
|
|
case FIONREAD:
|
|
iob = conn->readahead;
|
|
if (iob)
|
|
{
|
|
uint16_t datalen;
|
|
iob_copyout((FAR uint8_t *)&datalen, iob, sizeof(datalen), 0);
|
|
*(FAR int *)((uintptr_t)arg) = datalen;
|
|
}
|
|
else
|
|
{
|
|
*(FAR int *)((uintptr_t)arg) = 0;
|
|
}
|
|
break;
|
|
case FIONSPACE:
|
|
#ifdef CONFIG_NET_UDP_WRITE_BUFFERS
|
|
# if CONFIG_NET_SEND_BUFSIZE == 0
|
|
*(FAR int *)((uintptr_t)arg) =
|
|
iob_navail(true) * CONFIG_IOB_BUFSIZE;
|
|
# else
|
|
*(FAR int *)((uintptr_t)arg) =
|
|
conn->sndbufs - udp_wrbuffer_inqueue_size(conn);
|
|
# endif
|
|
#else
|
|
*(FAR int *)((uintptr_t)arg) = MIN_UDP_MSS;
|
|
#endif
|
|
break;
|
|
default:
|
|
ret = -ENOTTY;
|
|
break;
|
|
}
|
|
|
|
net_unlock();
|
|
|
|
return ret;
|
|
}
|