Skip to content

CR-1273889: disable UBSAN for the out-of-tree zocl module - #10044

Merged
chvamshi-xilinx merged 1 commit into
Xilinx:masterfrom
ManojTakasi:cr-1273889-repro-master
Sep 15, 2026
Merged

chvamshi-xilinx merged 1 commit into
Xilinx:masterfrom
ManojTakasi:cr-1273889-repro-master

Conversation

@ManojTakasi

Copy link
Copy Markdown
Collaborator

Problem solved by the commit

https://jira.xilinx.com/browse/CR-1273889 ENABLE_KERNEL_DEBUG turns on kernel UBSAN, which embeds absolute kernel-source paths in zocl.ko and fails Yocto do_package_qa [buildpaths].

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

Disabling UBSAN only for zocl keeps the kernel sanitizers and avoids hiding the QA check.

Risks (if any) associated the changes in the commit

low

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

Documentation impact (if any)

@ManojTakasi
ManojTakasi force-pushed the cr-1273889-repro-master branch 2 times, most recently from 6e3e096 to 5cbbc5f Compare September 7, 2026 08:47
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

2 similar comments
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

ENABLE_KERNEL_DEBUG turns on kernel UBSAN, which embeds absolute
kernel-source paths in zocl.ko and fails Yocto do_package_qa [buildpaths].

Signed-off-by: Takasi, Manoj <Manoj.Takasi@amd.com>
@ManojTakasi
ManojTakasi force-pushed the cr-1273889-repro-master branch from 5cbbc5f to e59ed80 Compare September 7, 2026 08:58
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@chvamshi-xilinx
chvamshi-xilinx merged commit 35de88c into Xilinx:master Sep 15, 2026
22 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