There already is an environment variable 'tftptimeout' for setting individuabl block query timeout value, but there is none for setting the maximum TFTP timeout count, which can also help with unreliable network, TFTP server, or client hardware.

Add environment variable 'tftptimeoutcount'. As per the comments about the global variable tftp_timeout_count_max, make sure tftptimeoutcount is nonnegative.

Signed-off-by: Albert ARIBAUD (3ADEV) albert.aribaud@3adev.fr --- README | 8 ++++++++ net/tftp.c | 10 ++++++++++ 2 files changed, 18 insertions(+)

diff --git a/README b/README index b564640..c9dea37 100644 --- a/README +++ b/README @@ -5474,6 +5474,14 @@ List of environment variables (most likely not complete): faster in networks with high packet loss rates or with unreliable TFTP servers.

+ tftptimeoutcountmax - maximum count of TFTP timeouts (no + unit, minimum value = 0). Defines how many timeouts + can happen during a single file transfer before that + transfer is aborted. The default is 10, and 0 means + 'no timeouts allowed'. Increasing this value may help + downloads succeed with high packet loss rates, or with + unreliable TFTP servers or client hardware. + vlan - When set to a value < 4095 the traffic over Ethernet is encapsulated/received over 802.1q VLAN tagged frames. diff --git a/net/tftp.c b/net/tftp.c index 3e99e73..fd44eb2 100644 --- a/net/tftp.c +++ b/net/tftp.c @@ -715,6 +715,16 @@ void tftp_start(enum proto_t protocol) timeout_ms = 1000; }

+ ep = getenv("tftptimeoutcountmax"); + if (ep != NULL) + timeout_count_max = simple_strtol(ep, NULL, 10); + + if (timeout_count_max < 0) { + printf("TFTP timeout count max (%d ms) negative, set to 0\n", + timeout_count_max); + timeout_count_max = 0; + } + debug("TFTP blocksize = %i, timeout = %ld ms\n", tftp_block_size_option, timeout_ms);