lwIP - A Lightweight TCP/IP stack - Patches: patch #6253, Added csum to struct pbuf.
You are not allowed to post comments on this tracker with your current authentication level.
patch #6253: Added csum to struct pbuf.
Submitter: | Andrey Volkov <avolkov> | ||
Submitted: | Wed 31 Oct 2007 05:41:09 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Wont Do | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | None |
Jump to the original submission
Fri 26 Mar 2010 04:20:05 PM UTC, comment #17: |
Simon Goldschmidt <goldsimon> |
Thu 29 Oct 2009 09:48:23 PM UTC, comment #16: I know this is still on 'In Progress', but I haven't had time for this since nearly two years now. However, I'm still planning for this to get in to 1.4.0!
|
Simon Goldschmidt <goldsimon> |
Thu 29 Oct 2009 08:37:30 PM UTC, comment #15: Do you want to expand this and include it in 1.4.0? I've implemented hardware checksum offloading for both UDP and TCP. The changes were minimal for my method of having the checksum at the time I wanted to send the data (tcp_enqueue and udp_sendto).
|
Bill Auerbach <billauerbach> |
Thu 28 Aug 2008 10:52:38 AM UTC, comment #14:
|
Simon Goldschmidt <goldsimon> |
Wed 27 Aug 2008 12:26:19 PM UTC, comment #13: Hi Simon
|
Andrey Volkov <avolkov> |
Tue 26 Aug 2008 08:29:41 PM UTC, comment #12: Andrey, I know this comment is a little late, I'm only starting to work on this again now:
|
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 03:28:53 PM UTC, comment #11: Simon Goldschmidt wrote:
Yes, it is misunderstandings from your side ;).
|
Andrey Volkov <avolkov> |
Thu 01 Nov 2007 02:38:54 PM UTC, comment #10: Oh, I hate it when it turns out we've been fervently agreeing all along :-).
|
Jonathan Larmour <jifl> |
Thu 01 Nov 2007 02:27:31 PM UTC, comment #9:
|
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 02:07:44 PM UTC, comment #8:
|
Jonathan Larmour <jifl> |
Thu 01 Nov 2007 01:54:53 PM UTC, comment #7: [ Moving mail from lwip-devel to the patch, to keep discussion in one place ]
|
Jonathan Larmour <jifl> |
Thu 01 Nov 2007 01:11:44 PM UTC, comment #6: Andrey Volkov wrote:
|
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 01:01:02 PM UTC, comment #5:
|
Kieran Mansley <kieranm> |
Thu 01 Nov 2007 12:47:48 PM UTC, comment #4:
|
Simon Goldschmidt <goldsimon> |
Wed 31 Oct 2007 07:56:20 PM UTC, comment #3: struct pbuf is very good right now for being exactly 32 bytes, which is very amenable to keeping pbuf buffer pools aligned on hardware that supports DMA - usually there are alignment constraints, particularly if caching is also involved, in which case you have to keep packet data and other data (including struct pbuf itself) in different cache lines. Now this isn't something which is very easy in lwIP at the moment, but only because it's been deliberately postponed until after 1.3. I think there's a task for it somewhere. But I have made my own modifications in my source base, and when the time comes after 1.3, I was going to raise the issue.
|
Jonathan Larmour <jifl> |
Wed 31 Oct 2007 07:14:39 PM UTC, comment #2:
|
Frédéric Bernon <fbernon> |
Wed 31 Oct 2007 05:52:16 PM UTC, comment #1: I'm pretty sure that some developers will ask you to enable that with an option in opt.h/lwipopts.h (if they hardware doesn't support it, it's not necessary to add this code and to increase pbuf struct).
|
Frédéric Bernon <fbernon> |
Wed 31 Oct 2007 05:41:09 PM UTC, original submission:
Hello!
|
Andrey Volkov <avolkov> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 5 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-03-26 | goldsimon | Open/Closed | Open | Closed | |
2010-03-26 | goldsimon | Status | In Progress | Wont Do | |
2008-08-28 | goldsimon | Status | None | In Progress | |
Assigned to | None | goldsimon | |||
2007-10-31 | avolkov | Attached File | - | Added pbuf_csum.diff, #14264 |
Powered by Savane 3.13-d3ae.
Corresponding source code
Won't do (see task #6849): it doesn't work with pbuf_header and we have another way to do it for TX.