我正在使用Xilinx Ethernetlite(LWIP)设计。仅当buf = 32时,我才能通过以太网将数据从KC板传输到PC(Hercules)。但是我的实际缓冲区大小是1024。如何将缓冲区大小从32增加到1024
我无法确定错误是在代码中还是在大力士中。要读取大力神中的值(整数),我正在执行此功能。
最初,我将从Hercules将Hello命令发送到Board,然后Board接受该请求。此后,开发板将数据(整数值)输出到Hercules。
itua的C代码
char* itoa(int val, int base)
{
static char buf[32] = {0}; //buf size
int i = 30;
for(; val && i ; --i, val /= base)
buf[i] = "0123456789abcdef"[val % base];
return &buf[i+1];
}
修改后的代码
#define DAQ_FIFO_DEPTH 128
int transfer_data()
{
return 0;
}
err_t tcp_write_u32_string(struct tcp_pcb *pcb, unsigned char prefix, u32_t value)
{
unsigned char buf[11]; /* enough room for prefix and value. */
err_t result;
u16_t len;
unsigned char *p = buf + sizeof buf;
do {
/* ASCII encoding: '0' = 48, '1' = 49, ..., '9' = 57. */
*(--p) = 48 + (value % 10u);
value /= 10;
} while (value);
if (prefix)
*(--p) = prefix;
len = buf + sizeof buf - p;
if (tcp_sndbuf(pcb) < len)
{
result = tcp_output(pcb);
if (result != ERR_OK)
return result;
}
return tcp_write(pcb, p, len, TCP_WRITE_FLAG_COPY | TCP_WRITE_FLAG_MORE);
}
err_t send_list(struct tcp_pcb *pcb, const u32_t data[], u16_t len)
{
static const char newline[2] = { 13, 10 }; /* ASCII \r\n */
err_t result;
if (len > 0) {
u16_t i;
result = tcp_write_u32_string(pcb, 0, data[0]);
if (result != ERR_OK)
return result;
for (i = 1; i < len; i++)
{
/* ASCII comma is code 44. (Use 32 for space, or 9 for tab.) */
result = tcp_write_u32_string(pcb, 44, data[i]);
if (result != ERR_OK)
return result;
}
}
result = tcp_write(pcb, newline, 2, 0);
if (result)
return result;
return tcp_output(pcb);
}
int application_connection(void *arg, struct tcp_pcb *conn, err_t err)
{
struct netif *netif = arg; /* Because of tcp_arg(, netif). */
u32_t data[DAQ_FIFO_DEPTH];
u32_t i, n;
if (err != ERR_OK) {
tcp_abort(conn);
return ERR_ABRT;
}
err = daq_setup();
if (err != ERR_OK)
{
tcp_abort(conn);
return ERR_ABRT;
}
while (1)
{
xemacif_input(netif);
tcp_tmr();
tcp_output(conn);
n = daq_acquire(data, DAQ_FIFO_DEPTH);
if (n > DAQ_FIFO_DEPTH)
break;
if (tcp_write(conn, data, n * sizeof data[0], TCP_WRITE_FLAG_COPY) != ERR_OK)
break;
}
// daq_close();
/* Close the TCP connection. */
if (tcp_close(conn) == ERR_OK)
return ERR_OK;
/* Close failed. Abort it, then. */
tcp_abort(conn);
return ERR_ABRT;
}
int application_main(struct netif *netif, unsigned int port)
{
struct tcp_pcb *pcb;
err_t err;
pcb = tcp_new();
if (!pcb) {
/* Out of memory error */
return -1;
}
err = tcp_bind(pcb, IP_ADDR_ANY, port);
if (err != ERR_OK) {
/* TCP error */
return -1;
}
pcb = tcp_listen_with_backlog(pcb, 1);
if (!pcb) {
/* Out of memory. */
return -1;
}
tcp_arg(pcb, netif);
tcp_accept(pcb, application_connection);
while (1)
xemacif_input(netif);
}
答案 0 :(得分:0)
因此,这是Xilinx forums上的讨论的延续?
itoa()
函数将无符号整数(存储在int
中)转换为缓冲区buf
中的前30个左右字符。
recv_callback()
函数几乎没有意义。
对aurora_rx_main()
的调用记录为“功能调用”,这没有什么用(因为我们不知道它的作用),甚至它的返回值也被完全忽略了。
第一个for
循环出于调试目的在u32
中转储了前100个DestinationBuffer[]
的内容,因此代码与手头的任务无关。但是,我们不知道谁填充了DestinationBuffer
。 aurora_rx_main()
调用可能已填充也可能未填充;我们都没有被告知。
(tcp_*()
函数似乎遵循Wikia中lwIP Wiki中描述的API。)
如果p
参数为NULL,则调用tcp_close(tcpb)
,然后调用tcp_recv(tcpb, NULL)
。这使所有事情变得毫无意义:为什么在关闭后尝试接收任何内容(以及为什么使用NULL参数)?
下一部分同样令人困惑。看起来if
测试检查TCP发送缓冲区的大小是否超过1024字节。如果不是,则释放p
缓冲区。否则,for
循环会尝试将u32
中的每个DestinationBuffer
转换为字符串,然后将该字符串写入TCP缓冲区;但是,它使用常数1
而不是适当的api标志,甚至不检查附加到TCP发送缓冲区是否有效。
总而言之,这看起来像一堆复制粘贴的代码,无济于事。不仅没有必要在itoa函数中增加缓冲区大小,{u32
即使转换为int
,也始终可以容纳12个字符(不包括减号或末尾的nul字节,因此总共要包含13个字符),但与应该解决的问题完全无关。
根本问题是代码太可怕了。修改它就像将车身填充物放在一块旧的口香糖上,以试图“固定”它。正确的解决方法是完全删除那些垃圾代码,并改用更好的代码。
编辑:OP声明他们是新程序员,因此,以上注释应被视为对所显示代码的直接,诚实的看法,而与OP本身无关。让我们看看我们是否可以帮助OP生成更好的代码。
首先,显示的itoa()
功能很愚蠢。假设实际上是将u32_t
中的DestinationBuffer
作为十进制字符串发回,那么最好实现一个辅助函数来进行转换。由于该值前面带有逗号(或其他分隔符),因此我们也可以对其进行琐碎的添加。由于它将使用tcp_write()
发送,因此我们将结合以下功能:
err_t tcp_write_u32_string(struct tcp_pcb *pcb,
unsigned char prefix, /* 0 for none */
u32_t value)
{
/* Because 0 <= u32_t <= 4294967295, the value itself is at most 10 digits long. */
unsigned char buf[11]; /* enough room for prefix and value. */
err_t result;
u16_t len;
unsigned char *p = buf + sizeof buf;
/* Construct the value first, from right to left. */
do {
/* ASCII encoding: '0' = 48, '1' = 49, ..., '9' = 57. */
*(--p) = 48 + (value % 10u);
value /= 10;
} while (value);
/* Prepend the prefix, if any. */
if (prefix)
*(--p) = prefix;
/* Calculate the length of this part. */
len = buf + sizeof buf - p;
/* If the TCP buffer does not have enough free space, flush it. */
if (tcp_sendbuf(pcb) < len) {
result = tcp_output(pcb);
if (result != ERR_OK)
return result;
}
/* Append the buffer to the TCP send buffer.
We also assume the packet is not done yet. */
return tcp_write(pcb, p, len, TCP_WRITE_FLAG_COPY | TCP_WRITE_FLAG_MORE);
}
要从指定数组发送num
u32_t
作为十进制字符串,并以换行符结尾,您可以使用
err_t send_list(struct tcp_pcb *pcb,
const u32_t data[],
u16_t len)
{
static const char newline[2] = { 13, 10 }; /* ASCII \r\n */
err_t result;
if (len > 0) {
u16_t i;
/* The first number has no prefix. */
result = tcp_write_u32_string(pcb, 0, data[0]);
if (result != ERR_OK)
return result;
/* The following numbers have a comma prefix. */
for (i = 1; i < len; i++) {
/* ASCII comma is code 44. (Use 32 for space, or 9 for tab.) */
result = tcp_write_u32_string(pcb, 44, data[i]);
if (result != ERR_OK)
return result;
}
}
/* We add a final newline.
Note that this one can be referenced,
and it does complete what we wanted to send thus far. */
result = tcp_write(pcb, newline, 2, 0);
if (result)
return result;
/* and flush the buffer, so the packet gets sent right now. */
return tcp_output(pcb);
}
现在,我还没有为Xilinx编写C或根本没有使用lwIP堆栈,因此上述代码是盲目的编写。但是,我非常有信心它会起作用(除非有任何错别字或想法,否则,请在评论中进行举报,然后我将进行验证和修复)。
两个缓冲区(buf
和newline
)被声明为static
,因此尽管它们仅在各自的函数中可见,但它们的值在全局范围内有效。 / p>
由于TCP是流协议,因此不必将每个响应都适合单个数据包。除了11个字符(每个数字及其前缀字符)和2个字符(换行符)缓冲区外,您唯一需要的大缓冲区是TCP发送缓冲区(maximum transmission unit或maximum segment size我不确定lwIP如何在内部使用缓冲区),通常在536和1518字节之间。
以上两个函数试图在数字之间分割数据包,但这仅仅是因为比准确地填充每个数据包更容易。如果下一个(逗号和)值适合缓冲区,则将其添加到缓冲区。否则,首先刷新缓冲区,然后将下一个(逗号和)值添加到缓冲区。
在接收方,您应该使用例如netcat。 (我不知道Hercules是应用程序还是本地计算机的名称。)由于TCP是流协议,因此接收方无法(可靠地)检测到数据包边界在哪里(不同于UDP数据报)。实际上,TCP连接只是两个数据流,每个数据流都是一种方式,而拆分为数据包只是应用程序程序员无需担心的协议详细信息。对于lwIP,由于它是一个低级别的库,因此需要多加注意,但是从上面的代码可以看出,实际上并没有那么多。
在评论中,OP解释说它们不是非常有经验,并且总体目的是使设备接受TCP连接,并通过以下方式将数据流(由单独的采集板采集的样本)作为无符号的32位整数传输连接。
因为我很想拥有其中的一块FPGA板(我有几个任务可以看一下是否可以卸载到FPGA),但是没有资源可以得到,所以我将在这里概述整个应用。请注意,我唯一需要继续的信息是Xilinx OS和库文档集合(UG643)(PDF)的2018年版本。看来OP希望使用原始API来获得高性能。
将样本转换为文本是愚蠢的,尤其是在需要高性能的情况下。我们应该只使用原始二进制文件,以及KC705使用的任何字节序。 (我没有从文档中快速看到它,但是我怀疑它是低端的)。
根据文档,原始API main()
与以下内容类似:
int main(void)
{
/* MAC address. Use an unique one. */
unsigned char mac[6] = { 0x00, 0x0A, 0x35, 0x00, 0x01, 0x02 };
struct netif *netif = NULL;
ip_addr_t ipaddr, netmask, gateway;
/* Define IP address, netmask, and gateway. */
IP4_ADDR(&ipaddr, 192, 168, 1, 1);
IP4_ADDR(&netmask, 255, 255, 255, 0);
IP4_ADDR(&gateway, 0, 0, 0, 0);
/* Initialize lwIP networking stack. */
lwip_init();
/* Add this networking interface, and make it the default one */
if (!xemac_add(netif, &ipaddr, &netmask, &gateway, mac, EMAC_BASEADDR)) {
printf("Error adding network interface\n\r");
return -1;
}
netif_set_default(netif);
platform_enable_interrupts();
/* Bring the network interface up (activate it) */
netif_set_up(netif);
/* Our application listens on port 7. */
return application_main(netif, 7);
}
在文档示例中,您会看到对return application_main(netif);
的调用,而不是start_application()
,然后是一个定期调用xemacif_input(netif)
的无限循环。这只是意味着application_main()
必须定期调用xemacif_input(netif)
才能接收数据。 (lwIP文档指出,我们还应定期调用sys_check_timeouts()
或tcp_tmr()
。)
请注意,我已经省略了错误报告printfs,并且它不会从错误中恢复,而是会返回(从main()
开始);我不确定这是否会导致KC705重新启动还是什么。
int application_main(struct netif *netif, unsigned int port)
{
struct tcp_pcb *pcb;
err_t err;
pcb = tcp_new();
if (!pcb) {
/* Out of memory error */
return -1;
}
/* Listen for incoming connections on the specified port. */
err = tcp_bind(pcb, IP_ADDR_ANY, port);
if (err != ERR_OK) {
/* TCP error */
return -1;
}
pcb = tcp_listen_with_backlog(pcb, 1);
if (!pcb) {
/* Out of memory. */
return -1;
}
/* The accept callback function gets the network interface
structure as the extra parameter. */
tcp_arg(pcb, netif);
/* For each incoming connection, call application_connection(). */
tcp_accept(pcb, application_connection);
/* In the mean time, process incoming data. */
while (1)
xemacif_input(netif);
}
对于端口的每个TCP连接,我们都会调用application_connection()
。此功能可设置数据采集板,并在接收者需要的时间内传输数据。
/* How many DAQ samples to process in each batch.
* Should be around the DAQ FIFO depth or so, I think. */
#define DAQ_FIFO_DEPTH 128
err_t application_connection(void *arg, struct tcp_pcb *conn, err_t err)
{
struct netif *netif = arg; /* Because of tcp_arg(, netif). */
u32_t data[DAQ_FIFO_DEPTH];
u32_t i, n;
/* Drop the connection if there was an error. */
if (err != ERR_OK) {
tcp_abort(conn);
return ERR_ABRT;
}
/* Setup the data aquisition. */
err = daq_setup();
if (err != ERR_OK) {
tcp_abort(conn);
return ERR_ABRT;
}
/* Data acquisition to TCP loop. */
while (1) {
/* Keep the networking stack running. */
xemacif_input(netif);
tcp_tmr();
/* Tell the networking stack to output what it can. */
tcp_output(conn);
/* Acquire up to DAQ_FIFO_DEPTH samples. */
n = daq_acquire(data, DAQ_FIFO_DEPTH);
if (n > DAQ_FIFO_DEPTH)
break;
/* Write data as-is to the tcp buffer. */
if (tcp_write(conn, data, n * sizeof data[0], TCP_WRITE_FLAG_COPY) != ERR_OK)
break;
}
/* Stop data acquisition. */
daq_close();
/* Close the TCP connection. */
if (tcp_close(conn) == ERR_OK)
return ERR_OK;
/* Close failed. Abort it, then. */
tcp_abort(conn);
return ERR_ABRT;
}
还有三个要实现的功能:daq_setup()
,应设置数据采集和FIFO; daq_acquire(u32_t *data, u32_t count)
最多将count
个样本存储到data[]
,并返回实际存储的样本数-最好是只耗尽FIFO,而不是等待新样本到达-最后到达daq_close()
,这将停止数据获取。
我相信它们应该是这样的:
XLlFifo daq_fifo;
err_t daq_setup(void)
{
XLlFifo_Config *config = NULL;
config = XLlFifo_LookupConfig(DAQ_FIFO_ID);
if (!config)
return ERR_RTE;
if (XLlFifo_CfgInitialize(&daq_fifo, config, config->BaseAddress) != XST_SUCCESS)
return ERR_RTE;
}
u32_t daq_acquire(u32_t *data, u32_t max)
{
u32_t len, have;
have = XLlFifo_iRxGetLen(&daq_fifo);
if (have < 1)
return 0;
else
if (have < max)
max = have;
for (len = 0; len < max; len++)
data[len] = XLlFifo_RxGetWork(&daq_fifo);
return len;
}
err_t daq_close(void)
{
/* How to stop the FIFO? Do we need to? */
}
就是这样。