[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lwip-devel] Re: [patch #6253] Added csum to struct pbuf.

From: Andrey Volkov
Subject: [lwip-devel] Re: [patch #6253] Added csum to struct pbuf.
Date: Thu, 01 Nov 2007 15:48:29 +0300
User-agent: Thunderbird (Windows/20070728)

Frédéric Bernon wrote:
> Follow-up Comment #2, patch #6253 (project lwip):
>> Is HWD_SUPPORT_CSUM will be appropriate?
>> Andrey
> CHECKSUM_HARDWARE_SUPPORT (to be in the same spirit than other
> CHECKSUM_xxx) ?
Ok, accepted.

> Do you have read "task #6849 : Test how checksum on copy could be
> integrated into the stack" (https://savannah.nongnu.org/task/?6849) ?
Yes, but I introduce this field mainly for next purposes:
 1) speed up reassembly of incoming ip packets. In reassembly case,
    ip payload csum of each fragment should be stored somewhere.
    (IMHO, pbuf is very appropriate for it).
 2) Hardware usually could calculate ip payload/headers checksum,
    but not tcp/udp pseudo checksum, hence ip csum should be stored.
    Also, I dislike to mix different levels of stack in one piece
    of code. I.e. if I've ip checksum in eth driver, I could undef
    since I don't know it in driver level.
 3) In my hwd. driver, I don't 'memcpy' incoming packets, DMA do it
    directly to pbuf.

Andrey Volkov

reply via email to

[Prev in Thread] Current Thread [Next in Thread]