Skip to content

Fix for CR-1258762#33

Merged
jvillarre merged 2 commits intoXilinx:masterfrom
snigdha-gupta:xdp_CR-1258762
Mar 18, 2026
Merged

Fix for CR-1258762#33
jvillarre merged 2 commits intoXilinx:masterfrom
snigdha-gupta:xdp_CR-1258762

Conversation

@snigdha-gupta
Copy link
Copy Markdown
Collaborator

Summary

Align the AIE Status plugin with the AIE Profile and AIE Trace plugins so it correctly supports the hw context (register-xclbin) flow in addition to the legacy load-xclbin flow

Changes

  • Call continueXDPConfig(hw_context_flow) at the start of updateAIEDevice and return early when config should be skipped (same as profile/trace).
  • When hw_context_flow is true, detect Full ELF flow and return with a warning; AIE Status is not supported for that flow.
  • Add the same xdp_mode / device_id checks (ZOCL vs XDNA vs EDGE) as in profile/trace for non-x86, non-client builds.
  • Use getDeviceContextUniqueId(handle) in getDeviceIDFromHandle for a stable device ID in both flows instead of addDevice("ve2_device") / getDebugIpLayoutPath(handle).
  • For REGISTER_XCLBIN_STYLE: call updateDeviceFromCoreDeviceHwCtxFlow(..., HalDevice(...)); for load-xclbin: call updateDeviceFromHandle(..., HalDevice(handle), handle) so the device wrapper is set in both paths.

Testing

  • Verified behavior on VCK190 hw_context flow

Signed-off-by: snigupta <snigupta@amd.com>
Signed-off-by: snigupta <snigupta@amd.com>
@snigdha-gupta snigdha-gupta requested a review from jvillarre March 18, 2026 17:01
@jvillarre jvillarre merged commit 8bae84e into Xilinx:master Mar 18, 2026
4 checks passed
@snigdha-gupta snigdha-gupta deleted the xdp_CR-1258762 branch March 18, 2026 19:39
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