On 8/29/24 16:05, Simon Glass wrote: > Hi Jerome, > > On Fri, 23 Aug 2024 at 07:50, Jerome Forissier > <jerome.foriss...@linaro.org> wrote: >> >> Enable NET_LWIP by default for testing purposes. SANDBOX doesn't support >> NET_LWIP so default to NET in this case. > > Sandbox needs to support NET_LWIP. I agree in principle, but AFAICT it is not a trivial task. Does it have to be done in this series or can it be dealt with later? I would need to look into this more closely to give details, but if I remember correctly the sandbox tests expect a precise sequence of network packets. For example when doing a TFTP tests, it expects an ARP packet to go out first, but when lwIP is used the ARP might not go out because the IP may already be in the cache. So the sandbox is definitely not a black box in that respect which makes things a bit more difficult. Thanks, -- Jerome
- [PATCH v9 29/37] lwip: tftp: add support of blksize optio... Jerome Forissier
- [PATCH v9 30/37] net-lwip: add TFTP_BLOCKSIZE Jerome Forissier
- [PATCH v9 31/37] CI: add qemu_arm64_lwip to the test matr... Jerome Forissier
- [PATCH v9 32/37] MAINTAINERS: net-lwip: add myself as a m... Jerome Forissier
- [PATCH v9 33/37] Kconfig: fix undefined symbols (g_dnl*) ... Jerome Forissier
- [PATCH v9 34/37] configs: use syntax CONFIG_FOO=n in tool... Jerome Forissier
- Re: [PATCH v9 34/37] configs: use syntax CONFIG_FOO=... Tom Rini
- Re: [PATCH v9 34/37] configs: use syntax CONFIG_... Jerome Forissier
- [PATCH v9 35/37] [TESTING] Kconfig: enable NET_LWIP by de... Jerome Forissier
- Re: [PATCH v9 35/37] [TESTING] Kconfig: enable NET_L... Simon Glass
- Re: [PATCH v9 35/37] [TESTING] Kconfig: enable N... Jerome Forissier
- Re: [PATCH v9 35/37] [TESTING] Kconfig: enab... Simon Glass
- Re: [PATCH v9 35/37] [TESTING] Kconfig: ... Ilias Apalodimas
- Re: [PATCH v9 35/37] [TESTING] Kcon... Simon Glass
- Re: [PATCH v9 35/37] [TESTING] ... Tom Rini
- Re: [PATCH v9 35/37] [TESTING] ... Simon Glass
- Re: [PATCH v9 35/37] [TESTING] ... Jerome Forissier
- [PATCH v9 36/37] [TESTING] configs: set CONFIG_NET=y for ... Jerome Forissier
- [PATCH v9 20/37] net-lwip: add DHCP support and dhcp comm... Jerome Forissier