search

baidu / dperf

dperf is a 100Gbps network load tester.
https://dperf.org
Apache License 2.0
4.93k stars 509 forks source link

add vhost-net support #459

Closed fkcrazy001 closed 1 month ago

fkcrazy001 commented 3 months ago
  1. add vhost-net as kni support
  2. bugfix of kni queue racing
  3. del KNI_ENABLE macro, because we can support kni in all cases
pengjianzhang commented 3 months ago

  1. include 会不会在高版本DPDK编译不过？

  2. 注释保持C风格，多行注释格式如下： 
    /*
 * aaa
 * bbb
 */
  3. 局部变量保持在函数前面定义
  4. 一个变量一行，赋初始值，变量按照size排序（不考虑指针）

逻辑还没细看，待补充

fkcrazy001 commented 3 months ago

  1. include 会不会在高版本DPDK编译不过？

  2. 注释保持C风格
  3. 局部变量保持在函数前面定义

逻辑还没细看，待补充

  1. include 已修改为只在 kni.c 中包含， vhost-net 模块不需要，之前是从kni.c中拷贝过来忘记删除了

    2-3 已修改

pengjianzhang commented 2 months ago

The spaces at the end of the line need to be removed code style reference https://dperf.org/doc/html/code-style

fkcrazy001 commented 2 months ago

才注意到评论。今天验证下这些问题

fkcrazy001 commented 2 months ago

在我的vscode中，似乎没有对齐问题，我再研究下

pengjianzhang commented 2 months ago

行尾还是有空格

pengjianzhang commented 1 month ago

一些格式问题，请查邮箱

pengjianzhang commented 1 month ago

还是有些行尾空格

fkcrazy001 commented 1 month ago

已修改，将新的commit合并并重新提了一个pr。

https://github.com/baidu/dperf/pull/481