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

Move hwctx initializtion of module to xrt::run #7647

Merged
merged 1 commit into from
Jul 29, 2023
Merged

Conversation

stsoe
Copy link
Collaborator

@stsoe stsoe commented Jul 28, 2023

Problem solved by the commit

Ensure that creation of instruction buffer object is specific to a xrt::run object. Treat the module associated with xrt::kernel as a template for all run objects created from the kernel.

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

The instruction buffer may be address patching for the kernel arguments applied prior to starting the kernel. Invoking a kernel creates an xrt::run object and kernel arguments are specific to the run. Setting a argument patches the instruction buffer with the address of the kernel argument buffer objects.

Ensure that creation of instruction buffer object is specific to a
xrt::run object.  Treat the module associated with xrt::kernel as a
template for all run objects created from the kernel.

The instruction buffer may be address patching for the kernel
arguments applied prior to starting the kernel.  Invoking a kernel
creates an xrt::run object and kernel arguments are specific to the
run.  Setting a argument patches the instruction buffer with the address
of the kernel argument buffer objects.

Signed-off-by: Soren Soe <2106410+stsoe@users.noreply.github.com>
@gbuildx
Copy link
Collaborator

gbuildx commented Jul 28, 2023

Build failed :(

@dayeh-xilinx
Copy link

retest this please.

@gbuildx
Copy link
Collaborator

gbuildx commented Jul 29, 2023

Build failed :(

@manikandan-xilinx
Copy link
Collaborator

retest this please.

@gbuildx
Copy link
Collaborator

gbuildx commented Jul 29, 2023

Build Passed!

@stsoe stsoe merged commit c57f9d1 into Xilinx:master Jul 29, 2023
2 checks passed
@stsoe stsoe deleted the instr_bo branch July 29, 2023 23:54
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.

4 participants