net: tftp: Fix too small block size 71/228771/1
authorAndre Przywara <andre.przywara@arm.com>
Sat, 23 Nov 2019 17:58:59 +0000 (17:58 +0000)
committerMarek Szyprowski <m.szyprowski@samsung.com>
Wed, 25 Mar 2020 09:29:53 +0000 (10:29 +0100)
commit5d0b2006170f6432e7b94309c1e4d54643d77eb6
tree88692c3dc5b68461f9de7e4d1089fa7db9a8a503
parent7c9da953c3e34499b7f1e77c8141466aa1ba52b1
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 <andre.przywara@arm.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
[backport of mainline commit 16cdc2daa07986243751a9f545aa287c60e04ced]
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: Ib62265400f73d6ecd7780e70a84070158e9d8184
net/Kconfig