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

Hi Shah, Quentin,

On 13/08/2026 18:41, shj wrote:
> 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

My bad. I recently changed my workflow from git pw to b4 and I obviously
applied only one patch when my intent was to apply the series :-/

I will apply both patches to for-main for the next -rc. Sorry for the
inconvenience.

-- 
Jerome

> 
> Thanks,
> Shah
>