lwIP - A Lightweight TCP/IP stack - Patches: patch #6460, inet_chksum minor optimizations
You are not allowed to post comments on this tracker with your current authentication level.
patch #6460: inet_chksum minor optimizations
Submitter: | Bill Auerbach <billauerbach> | ||
Submitted: | Tue 11 Mar 2008 09:38:59 PM UTC | ||
Category: | None | Priority: | 3 - Low |
Status: | Done | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | None |
Mon 23 Jun 2008 08:09:14 PM UTC, comment #5: |
Simon Goldschmidt <goldsimon> |
Tue 17 Jun 2008 08:07:45 PM UTC, comment #4: I have just checked in my interpretation of this patch. I intentionally didn't change the default checksum implementation: it isn't fast anyway and can stay as a reference implementation. |
Simon Goldschmidt <goldsimon> |
Thu 27 Mar 2008 06:15:35 PM UTC, comment #3: As long as we don't seriously mess with the checksum functions, I think it's good to speed them up:
|
Simon Goldschmidt <goldsimon> |
Tue 25 Mar 2008 01:28:27 PM UTC, comment #2: Do you have a handle on the performance boost this might give? I'm a little reluctant to fiddle with the checksum algorithm as it's currently known to work, and I'm guessing that most ports will write their own (for higher native performance) anyway. |
Kieran Mansley <kieranm> |
Tue 11 Mar 2008 11:06:27 PM UTC, comment #1: I'm not in a position to test it, but it looks OK to me. |
David Empson <dempson> |
Tue 11 Mar 2008 09:38:59 PM UTC, original submission:
inet_chksum.c - Remove unnecessary while loops folding upper and lower checksum words. Unroll 4 other while loops since the test to run the loop is about as expensive as just doing the fold.
|
Bill Auerbach <billauerbach> |
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 6 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2008-06-23 | goldsimon | Status | Ready For Test | Done | |
Open/Closed | Open | Closed | |||
2008-06-17 | goldsimon | Status | None | Ready For Test | |
Assigned to | None | goldsimon | |||
2008-03-11 | billauerbach | Attached File | - | Added inet_chksum_c.diff, #15239 | |
Carbon-Copy | - | Added -email is unavailable- |
Powered by Savane 3.13-3230.
Corresponding source code
no comments / objections -> closed