Re: [PATCH 2/2] test: dm: nfs: add a regression test for an oversized NFS read length

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
>