Skip to content

Comments

Add RDMA bridge optional parameters in SDK JSON config#397

Merged
ko80 merged 1 commit intoOpenVisualCloud:mainfrom
ko80:add-rdma-options-sdk
Jun 4, 2025
Merged

Add RDMA bridge optional parameters in SDK JSON config#397
ko80 merged 1 commit intoOpenVisualCloud:mainfrom
ko80:add-rdma-options-sdk

Conversation

@ko80
Copy link
Collaborator

@ko80 ko80 commented Jun 4, 2025

  • Add the provider string parameter, options are tcp or verbs, default tcp.
  • Add the numEndpoints integer parameter, range 1..8, default 1.
  • If one or both parameters are not equal, fail compatibility check.
  • Implement JSON parameter parsing.
  • Propagate parameters all the way down to Media Proxy and Agent.
  • Make both parameters appear in create_bridge() to allow passing them to configuration of RDMA brodge classes.
  • Update SDK documentation.
  • Add corresponding parameters rdma_provider and rdma_num_endpoints in FFmpeg Plugin.
  • Update FFmpeg Plugin documentation.

* Add the `provider` string parameter, options are `tcp` or `verbs`,
default `tcp`.
* Add the `numEndpoints` integer parameter, range 1..8, default 1.
* If one or both parameters are not equal, fail compatibility check.
* Implement JSON parameter parsing.
* Propagate parameters all the way down to Media Proxy and Agent.
* Make both parameters appear in create_bridge() to allow passing
them to configuration of RDMA brodge classes.
* Update SDK documentation.
* Add corresponding parameters `rdma_provider` and
`rdma_num_endpoints` in FFmpeg Plugin.
* Update FFmpeg Plugin documentation.

Signed-off-by: Konstantin Ilichev <konstantin.ilichev@intel.com>
@ko80 ko80 marked this pull request as ready for review June 4, 2025 19:06
@ko80 ko80 requested review from moleksy and soopel as code owners June 4, 2025 19:06
@ko80 ko80 removed the request for review from soopel June 4, 2025 19:06
@ko80 ko80 merged commit 64ef465 into OpenVisualCloud:main Jun 4, 2025
9 checks passed
@ko80 ko80 removed the request for review from moleksy June 4, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant