buglwIP - A Lightweight TCP/IP stack - Bugs: bug #64638, "iperf -r" does not work...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #64638: "iperf -r" does not work for all "-M"

Submitter:  Hardy Griech <rgrr>
Submitted:  Fri 08 Sep 2023 06:25:38 PM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Planned Release:  None
lwIP version:  git head

Wed 27 Sep 2023 07:04:53 PM UTC, comment #1: 

Could you provide a littel more input? I don't know if "txlen != 0" is correct. This should probably be somehow connected to the "-M" size or we risk having low throughput when we're low on memory. The current "txlen" check is there for a reason, after all!

Simon Goldschmidt <goldsimon>
Group administrator
Fri 08 Sep 2023 06:25:38 PM UTC, original submission:  

This is true at least for the combination lwIP/TinyUSB.

On my Linux host, "iperf -r" did not work for small values of "-M <mss>".  Reason was that the loop in lwiperf_tcp_client_send_more() is to restrictive.  It aborts when "txlen >= (TCP_MSS / 2)".

For my setup it is needed, that this has to be changed to "txlen != 0", because otherwise there is under described circumstances no transmission and thus no lwiperf_tcp_client_sent() which retriggers send_more.

Should I provide the patch?

Hardy Griech <rgrr>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by rgrr (Submitted the item)
  •  

    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.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code