Page MenuHomeFreeBSD

lib/libc/tests/rpc: Correctly set timeout
ClosedPublic

Authored by arichardson on Feb 4 2021, 12:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 27, 4:30 AM
Unknown Object (File)
Wed, Jun 25, 1:43 AM
Unknown Object (File)
Fri, Jun 20, 7:50 PM
Unknown Object (File)
Thu, Jun 19, 10:52 AM
Unknown Object (File)
Fri, Jun 13, 8:43 PM
Unknown Object (File)
Tue, Jun 10, 8:21 AM
Unknown Object (File)
Sun, Jun 8, 7:13 AM
Unknown Object (File)
Sat, Jun 7, 8:57 PM
Subscribers

Details

Summary

The rpc_control() API does not accept the CLCR_SET_RPCB_TIMEOUT command,
it only accepts RPC_SVC_CONNMAXREC_GET/RPC_SVC_CONNMAXREC_SET, so it was
not doing anything.
Instead of incorrectly calling this API, use clnt_create_timed() instead.

I noticed this because the test was timing out after 120s in the CheriBSD CI.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 36706
Build 33595: arc lint + arc unit

Event Timeline

ngie requested changes to this revision.Feb 4 2021, 4:31 PM

Could you please wrap the changes in #ifdef __FreeBSD__ so it'd be easier for me to upstream to NetBSD?

This revision now requires changes to proceed.Feb 4 2021, 4:31 PM

Add #ifdef for easier upstreaming

This revision is now accepted and ready to land.Feb 12 2021, 6:45 PM