lwIP - A Lightweight TCP/IP stack - Bugs: bug #50694, TX exist more pbufs after enable...
You are not allowed to post comments on this tracker with your current authentication level.
bug #50694: TX exist more pbufs after enable LWIP_NETIF_TX_SINGLE_PBUF
Submitter: | Robin <back1127_mtk> | ||
Submitted: | Sat 01 Apr 2017 08:35:28 AM UTC | ||
Category: | TCP | Severity: | 3 - Normal |
Item Group: | None | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | 2.0.2 |
Jump to the original submission
Tue 25 Apr 2017 09:23:58 AM UTC, comment #8: |
Simon Goldschmidt <goldsimon> |
Tue 25 Apr 2017 08:12:14 AM UTC, comment #7: Ah, well I haven't tested that setting, sorry.
|
Simon Goldschmidt <goldsimon> |
Tue 25 Apr 2017 01:23:19 AM UTC, comment #6: For the patch in comment #4:
|
Axel Lin <axellin> |
Mon 24 Apr 2017 07:07:45 PM UTC, comment #5: To ensure linkoutput gets only packets in one piece, you can always add this to your driver:
This handles future error cases, too. |
Simon Goldschmidt <goldsimon> |
Mon 24 Apr 2017 07:02:02 PM UTC, comment #4: Could you try the attached patch? It removes "seg->oversize_left = 0;" on output and should fix "pcb->unsent_oversize" on rexmit. |
Simon Goldschmidt <goldsimon> |
Mon 24 Apr 2017 12:34:46 PM UTC, comment #3: I can confirm Robin's description on comment #2 because I can also run into this case:
|
Axel Lin <axellin> |
Fri 07 Apr 2017 01:03:54 AM UTC, comment #2: yes, i set TCP_OVERSIZE to TCP_MSS. if you set it as 0, it will check fail when lwip init.
|
Robin <back1127_mtk> |
Wed 05 Apr 2017 07:24:44 PM UTC, comment #1: For me, this works. Have you set TCP_OVERSIZE to TCP_MSS? In that case, a segment should always allow adding a full segment of data. |
Simon Goldschmidt <goldsimon> |
Sat 01 Apr 2017 08:35:28 AM UTC, original submission:
if project enable LWIP_NETIF_TX_SINGLE_PBUF
|
Robin <back1127_mtk> |
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 |
---|---|---|---|---|---|
2017-04-25 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2017-04-25 | goldsimon | Status | Works For Me | None | |
2017-04-24 | goldsimon | Attached File | - | Added bug50694_test.diff, #40487 | |
2017-04-05 | goldsimon | Status | None | Works For Me |
Powered by Savane 3.13-3230.
Corresponding source code
Fixed by not executing phase 2 for LWIP_NETIF_TX_SINGLE_PBUF==1.
http://git.savannah.nongnu.org/cgit/lwip.git/commit/?id=dd811bca0689ed41f1cf6a0725f2b858a4a7a5a5