public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Jerome Forissier <jerome.forissier@linaro.org>
Cc: u-boot@lists.denx.de,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	Javier Tia <javier.tia@linaro.org>,
	Raymond Mao <raymond.mao@linaro.org>,
	Maxim Uvarov <muvarov@gmail.com>,
	Tim Harvey <tharvey@gateworks.com>,
	Joe Hershberger <joe.hershberger@ni.com>,
	Ramon Fried <rfried.dev@gmail.com>,
	Simon Glass <sjg@chromium.org>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>,
	Mattijs Korpershoek <mkorpershoek@baylibre.com>,
	Eddie James <eajames@linux.ibm.com>,
	Sean Anderson <seanga2@gmail.com>,
	Marek Vasut <marek.vasut+renesas@mailbox.org>,
	Siddharth Vadapalli <s-vadapalli@ti.com>
Subject: Re: [PATCH v7 00/20] Introduce the lwIP network stack
Date: Fri, 2 Aug 2024 12:32:49 -0600	[thread overview]
Message-ID: <20240802183249.GO3794063@bill-the-cat> (raw)
In-Reply-To: <cover.1722615735.git.jerome.forissier@linaro.org>

[-- Attachment #1: Type: text/plain, Size: 2570 bytes --]

On Fri, Aug 02, 2024 at 06:26:27PM +0200, Jerome Forissier wrote:

> This is a rework of a patch series by Maxim Uvarov: "net/lwip: add lwip
> library for the network stack" [1]. The goal is to introduce the lwIP TCP/IP
> stack [2] [3] as an alternative to the current implementation in net/,
> selectable with Kconfig, and ultimately keep only lwIP if possible. Some
> reasons for doing so are:
> - Make the support of HTTPS in the wget command easier. Javier T. and
> Raymond M. (CC'd) have some additional lwIP and Mbed TLS patches to do
> so. With that it becomes possible to fetch and launch a distro installer
> such as Debian etc. using a secure, authenticated connection directly
> from the U-Boot shell. Several use cases:
>   * Authentication: prevent MITM attack (third party replacing the
> binary with a different one)
>   * Confidentiality: prevent third parties from grabbing a copy of the
> image as it is being downloaded
>   * Allow connection to servers that do not support plain HTTP anymore
> (this is becoming more and more common on the Internet these days)
> - Possibly benefit from additional features implemented in lwIP
> - Less code to maintain in U-Boot
> 
> Prior to applying this series, the lwIP stack needs to be added as a
> Git subtree with the following command:
> 
>  $  git subtree add --squash --prefix lib/lwip/lwip https://git.savannah.gnu.org/git/lwip.git STABLE-2_2_0_RELEASE

On Pi 3 I'm again / still seeing:
========================================== FAILURES ===========================================
___________________________________ test_efi_helloworld_net ___________________________________
test/py/tests/test_efi_loader.py:163: in test_efi_helloworld_net
    assert expected_text in output
E   AssertionError: assert 'Hello, world' in 'No UEFI binary known at 200000'
------------------------------------ Captured stdout call -------------------------------------
U-Boot> tftpboot 200000 EFI/arm64/helloworld.efi
Using smsc95xx_eth device
TFTP from server 192.168.1.10; our IP address is 192.168.1.100
Filename 'EFI/arm64/helloworld.efi'.
Load address: 0x200000
Loading: #
         883.8 KiB/s
done
Bytes transferred = 4528 (11b0 hex)
U-Boot> U-Boot> crc32 200000 $filesize
crc32 for 00200000 ... 002011af ==> 2b466005
U-Boot> U-Boot> bootefi 200000
No UEFI binary known at 200000
U-Boot>
=================================== short test summary info ===================================

On a configuration that works fine with the legacy network stack.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  parent reply	other threads:[~2024-08-02 18:33 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-02 16:26 [PATCH v7 00/20] Introduce the lwIP network stack Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 01/20] flash: prefix error codes with FL_ Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 02/20] net: introduce alternative implementation as net-lwip/ Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 03/20] configs: replace '# CONFIG_NET is not set' with CONFIG_NO_NET=y Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 04/20] net: fec_mxc_init(): do not ignore return status of fec_open() Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 05/20] net: split include/net.h into net{, -common, -legacy, -lwip}.h Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 06/20] net: eth-uclass: add function eth_start_udev() Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 07/20] net-lwip: build lwIP Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 08/20] net-lwip: add DHCP support and dhcp commmand Jerome Forissier
2024-08-02 19:33   ` Tom Rini
2024-08-07 12:41     ` Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 09/20] net-lwip: add TFTP support and tftpboot command Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 10/20] net-lwip: add ping command Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 11/20] net-lwip: add dns command Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 12/20] net: split cmd/net.c into cmd/net.c and cmd/net-common.c Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 13/20] net-lwip: add wget command Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 14/20] net-lwip: lwIP wget supports user defined port in the uri, so allow it Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 15/20] cmd: bdinfo: enable -e when CONFIG_CMD_NET_LWIP=y Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 16/20] configs: add qemu_arm64_lwip_defconfig Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 17/20] lwip: tftp: add support of blksize option to client Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 18/20] net-lwip: add TFTP_BLOCKSIZE Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 19/20] CI: add qemu_arm64_lwip to the test matrix Jerome Forissier
2024-08-02 16:26 ` [PATCH v7 20/20] MAINTAINERS: net-lwip: add myself as a maintainer Jerome Forissier
2024-08-02 18:32 ` Tom Rini [this message]
2024-08-05 18:18   ` [PATCH v7 00/20] Introduce the lwIP network stack Jerome Forissier
2024-08-05 18:20     ` Tom Rini
2024-08-06 22:29       ` Jerome Forissier
2024-08-06 21:12   ` Simon Glass
2024-08-06 21:19     ` Tom Rini
2024-08-06 21:51       ` Simon Glass
2024-08-06 22:28         ` Jerome Forissier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240802183249.GO3794063@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=eajames@linux.ibm.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=javier.tia@linaro.org \
    --cc=jerome.forissier@linaro.org \
    --cc=joe.hershberger@ni.com \
    --cc=marek.vasut+renesas@mailbox.org \
    --cc=mkorpershoek@baylibre.com \
    --cc=muvarov@gmail.com \
    --cc=raymond.mao@linaro.org \
    --cc=rfried.dev@gmail.com \
    --cc=s-vadapalli@ti.com \
    --cc=seanga2@gmail.com \
    --cc=sjg@chromium.org \
    --cc=tharvey@gateworks.com \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox