lwIP PPP netif и Ethernet netif HW/SW checksum, проблемы с аппаратным расчётом контрольной суммы Опции

[eisufu]

Создаю два интерфейса - один Ethernet (STM32F4x7 Ethernet controller настроен на hardware checksum), другой - PPP over serial. lwIP в lwipopts.h имеет настройки того, как будет считаться контрольная сумма: аппаратно или программно. Конечно, для PPP железо не считает контрольную сумму, интерфейс-то — "виртуальный". Попробовал следующее: везде в коде lwIP, где встречаются CHECKSUM_GEN_* (IP, TCP, UDP, ICMP) добавить код

Код

#if CHECKSUM_GEN_*
  <code>
#else
if ((netif->flags | NETIF_FLAG_POINTTOPOINT) != 0) {
  тот же самый <code>
}
#endif

но сгодилось это лишь для ICMP, TCP уже такое не позволит, так как в функции, содержащие код:

Код

#if CHECKSUM_GEN_*
  <code>
#endif

не передаётся netif *, для которого формируется пакет TCP. Да и вообще пришёл к выводу, что контрольную сумму надо считать централизованно для PPP (на камне есть периферия, которая быстро считает CRC), а для остальных netif - в зависимости от дефайнов CHECKSUM_GEN_*, зависящих от CHECKSUM_BY_HARDWARE.
Масса исходников — сходу не могу понять, где это "узкое место" где проходят все пакеты для каждого из IP, TCP, UDP, ICMP. Каково красивое решение?

[eisufu]

Утверждение не выполнялось в pbuf_free — нельзя передавать NULL аргументом, а вызывалась pbuf_free из ethernetif.c:ethernetif_input, которая изменена в соответствии с :

CODE

void ethernetif_input( void * pvParameters )
{
struct pbuf *p;

for( ;; )
{

if (xSemaphoreTake( s_xSemaphore, emacBLOCK_TIME_WAITING_FOR_INPUT)==pdTRUE)
{
GET_NEXT_FRAGMENT:
p = low_level_input( s_pxNetIf );
if (ERR_OK != s_pxNetIf->input( p, s_pxNetIf))
{
if (p != NULL) {
pbuf_free(p);
p = NULL;
}
}
else
{
xSemaphoreTake(s_xSemaphore, 0);
goto GET_NEXT_FRAGMENT;
}
}
}
}

от себя добавил только строку "if (p != NULL) {" стабильно держит напор скрипта (Lua):

CODE

local socket = require('socket')

local client
client, err = socket.udp()
if err then
error(err)
end
local rport = 22222
local raddress = "172.16.0.100"
_, err = client:setpeername(raddress, rport)
if err then
error(err)
end
local MAX_PAYLOAD_SIZE = 1472
local payload = string.rep('@', MAX_PAYLOAD_SIZE)
local n = 0
while true do
_, err = client:send(payload) -- never blocks
if err then
print(err)
else
n = n + 1
print(string.format('%d', n))
end
end

кстати

Код

cat /dev/random | nc -u 172.16.0.100 22222

не работает как надо, так как UDP датаграммы с payload больше 1472 - фрагментируются (и не успевают собраться). Нагрузка на процессор была ~54%. Теперь выдаёт >>90% c CHECKSUM_BY_HARDWARE:

Код

udpTestTask             2530110         7%
counterTask             26552           <1%
IDLE            1871340         5%
blinkTask               319             <1%
tcpip_thread            13769936                38%
Eth_if          17729671                49%
Tmr Svc         24              <1%

[scifi]

Утверждение не выполнялось в pbuf_free — нельзя передавать NULL аргументом, а вызывалась pbuf_free из ethernetif.c:ethernetif_input, которая изменена в соответствии с ...

Как и следовало ожидать: накосячил пейсатель драйвера. Кто автор? Пошлите ему лучики благодарности.
Кстати, весьма вероятно, что этот драйвер - не более чем демонстрация, и серьёзно не тестировался. Легко могут быть и другие баги.

Update:
Кстати, я недавно написал свой драйвер для STM32 Ethernet MAC и uIP. Получилось всего 200 строк. Для lwip должно быть аналогично. Возможно, проще отказаться от чужого драйвера и сделать свой.