From: Andre Przywara Date: Sat, 23 Nov 2019 17:58:59 +0000 (+0000) Subject: net: tftp: Fix too small block size X-Git-Tag: v2020.10~454^2~15 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=16cdc2daa07986243751a9f545aa287c60e04ced;p=platform%2Fkernel%2Fu-boot.git net: tftp: Fix too small block size Commit b618b3707633 ("net: Convert CONFIG_TFTP_BLOCKSIZE to Kconfig") accidentally set the default *option* TFTP block size to 512 bytes, even though the comment in the code says that this is a terrible choice. Most boards didn't define the symbol before, so they got the default block size of 1468 bytes before, but now use 512 bytes, which is also the fallback. This leads to both abysmal performance and a lot of hashes printed on the screen (one character for every 5K), which is both annoying and slow over serial links. Set the default block size in Kconfig back to the value it had before. This improves TFTP performance from 2.8 MB/s to 6.9 MB/s on a Pine64. Fixes: b618b3707633 ("net: Convert CONFIG_TFTP_BLOCKSIZE to Kconfig") Signed-off-by: Andre Przywara Acked-by: Joe Hershberger --- diff --git a/net/Kconfig b/net/Kconfig index 68cecf7..a07f674 100644 --- a/net/Kconfig +++ b/net/Kconfig @@ -31,7 +31,7 @@ config IP_DEFRAG config TFTP_BLOCKSIZE int "TFTP block size" - default 512 + default 1468 help Default TFTP block size.