Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Made aie_ctx to ZOCL_CTX_NOOPS to support multi partition #8448

Merged

Conversation

bisingha-xilinx
Copy link
Collaborator

@bisingha-xilinx bisingha-xilinx commented Sep 24, 2024

Problem solved by the commit

To run multiple graphs with multiple hwctx (graph1->hwctx1, graph2->hwctx2, etc.) from a single xclbin

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

How problem was solved, alternative solutions (if any) and why they were rejected

Made aie_ctx to ZOCL_CTX_NOOPS. It enables to open multiple hwctx with a single xclbin with multi partition, multi graph.

Risks (if any) associated the changes in the commit

low

What has been tested and how, request additional testing if necessary

Please find the table below.

Testname legacy hwctx
complex_graph passed passed
simple_gmio passed passed
simple_graph passed passed
simple_rtp passed passed
simple_vadd(opencl) passed testcase is n/a
simple_vadd passed passed
zynqmp hello world(opencl) passed testcase is n/a
multi hwctx single xclbin with multi graph simple gmio testcase is n/a passed

Documentation impact (if any)

Once the destruction of objects from the userspace are in order we need to work on this aie_ctx.

Signed-off-by: bisingha <bikash.singha@amd.com>
@chvamshi-xilinx chvamshi-xilinx merged commit befb177 into Xilinx:master Sep 24, 2024
18 checks passed
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.

2 participants