Hi Quentin, Jerome,
On 13.08.26 18:11, Quentin Schulz wrote:
On 8/13/26 5:42 PM, shj wrote:
Hi Jerome,
On 12.08.26 17:06, Jerome Forissier wrote:
With the series applied to for-main I got a CI error, see:
https://eur02.safelinks.protection.outlook.com/?
url=https%3A%2F%2Fgit.u-boot-project.org%2Fu-boot%2Fcustodians%2Fu-
boot-net%2F-
%2Fjobs%2F68484&data=05%7C02%7Cquentin.schulz%40cherry.de%7Cfcf4355cfe314a023a7e08def9538abe%7C5e0e1b5221b54e7b83bb514ec460677e%7C0%7C0%7C639222334948504305%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C80000%7C%7C%7C&sdata=nNfbYn4Y0KW9fswuFVBAVh5ceKMSMFfACtCcOHpIdgw%3D&reserved=0
I applied both NFS patches on top of current for-main and ran the
sandbox tests (I forgot to enable CONFIG_CMD_NFS in the sandbox
config in the previous nfs patch, and the nfs test is skipped,
exactly as in your job, sending v2). But overall I can't reproduce a
failure, the failing test in that job, dm_test_dhcp6_serverid_reject,
isn't on for- main. I applied the serverid series (fix + test)
verbatim on current for-main (b752e441790) and
t_dm_dm_test_dhcp6_serverid_reject passed.
Could you tell me which tree/commit you saw this on (the job checked
out for-main at 37f4c339, which I can't find on for-main now), or
point me at your merged test/dm/eth.c? I'll reproduce against exactly
that.
https://git.u-boot-project.org/u-boot/custodians/u-boot-net/-/commits/37f4c339a4b20e8b60370d6b0f73560c04feef7c
Thanks,
Shah
Thanks for the commit. I reproduced test failing on 37f4c339. On that
tree the test "test: dm: eth: add DHCPv6 oversized option regression
tests" is applied, but "net: dhcp6: bound received DUID option lengths"
is not, so the assert trips. Cherry-picking that fix onto 37f4c339
should makes the test pass
Thanks,
Shah
