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

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