XRT icon indicating copy to clipboard operation
XRT copied to clipboard

Passing as a separate module of mgmtpf & xocl to avoid execution confl…

Open raghkris opened this issue 1 year ago • 5 comments

…ict in a custom kernel

Problem solved by the commit

As the 2 drivers xocl/ & xclmgmt/ are being compiled as part of kernel image with static compilation, there was an issue of conflict with two driver registrations. Resolution is to add the string as new parameter in mgmt driver initialization. As part of custom kernel changes, xocl_debug_init() function is being called by both xclmgmt_init() and xocl_init() fucntions, so we differentiated with by passing parameters as module like "mgmtpf" & "xocl", to avoid execution conflict of xocl_debug_init() fucntions. Used separate module string name for both MGMTPF & XOCL to avoid macro and execution at runtime.

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

Risks (if any) associated the changes in the commit

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

Documentation impact (if any)

raghkris avatar Apr 30 '24 10:04 raghkris

raghkris - is not a collaborator Can XRT admins please validate PR

gbuildx avatar Apr 30 '24 10:04 gbuildx

Can one of the admins verify this patch?

gbuildx avatar Apr 30 '24 10:04 gbuildx

Can one of the admins verify this patch?

gbuildx avatar Apr 30 '24 10:04 gbuildx

raghkris - is not a collaborator Can XRT admins please validate PR

gbuildx avatar Apr 30 '24 12:04 gbuildx

raghkris - is not a collaborator Can XRT admins please validate PR

gbuildx avatar May 29 '24 08:05 gbuildx

Closing as these are available in 2022.2

chvamshi-xilinx avatar Jul 24 '24 10:07 chvamshi-xilinx