Skip to content

Add DMA enablement tests#296

Open
vnarapar wants to merge 3 commits into
qualcomm-linux:mainfrom
vnarapar:dmabuf
Open

Add DMA enablement tests#296
vnarapar wants to merge 3 commits into
qualcomm-linux:mainfrom
vnarapar:dmabuf

Conversation

@vnarapar
Copy link
Copy Markdown
Contributor

Added the tests to check DMA configs, Device Tree Validation and upstream kselftests

@vnarapar vnarapar requested a review from smuppand February 15, 2026 19:17
Comment thread Runner/suites/Kernel/Baseport/dmabuf/dmabuf.yaml Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/README.md Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh
@vnarapar vnarapar force-pushed the dmabuf branch 4 times, most recently from 90efb69 to 6b50d76 Compare March 23, 2026 12:47
@smuppand
Copy link
Copy Markdown
Contributor

smuppand commented Mar 24, 2026

This one should be split into 3 commits, unlike the simple one commit and one-testcase PR. It currently combines:

  1. library/helper addition
  2. presence-style dmabuf testcase
  3. functional dmabuf heap kselftest runner

Comment thread Runner/utils/functestlib.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf/dmabuf.yaml
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/README.md
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/README.md Outdated
@smuppand
Copy link
Copy Markdown
Contributor

This one should be split into 3 commits, unlike the simple one commit and one-testcase PR. It currently combines:

  1. library/helper addition
  2. presence-style dmabuf testcase
  3. functional dmabuf heap kselftest runner

This should still be split into 3 commits. Also address the structural commensts. The PR is 1 commit / 7 files / 3 logical chunks

@vnarapar vnarapar changed the title Added tests for dmabuf validations Add DMA enablement tests Apr 5, 2026
Copy link
Copy Markdown
Contributor

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly close the comments that have been addressed with the updated patches.

Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/README.md Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/README.md
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
@vnarapar vnarapar force-pushed the dmabuf branch 2 times, most recently from 963bc70 to a9ffc22 Compare May 7, 2026 11:45
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/dmabuf_heap_kselftest.yaml Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/dmabuf_heap_kselftest.yaml Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
@vnarapar vnarapar force-pushed the dmabuf branch 2 times, most recently from 9876cee to 9e46c25 Compare May 18, 2026 09:25
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
Comment thread Runner/utils/functestlib.sh
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf_heap_kselftest/run.sh Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/dmabuf.yaml Outdated
Comment thread Runner/suites/Kernel/Baseport/dmabuf/run.sh Outdated
@vnarapar vnarapar force-pushed the dmabuf branch 3 times, most recently from 677cb77 to 3641787 Compare May 22, 2026 10:52
@smuppand
Copy link
Copy Markdown
Contributor

I noticed that previous comments haven't been marked as resolved. Please close the comments that have been addressed and take care of any outstanding ones.

@vnarapar vnarapar force-pushed the dmabuf branch 3 times, most recently from f273c2b to efcf58a Compare June 1, 2026 06:59
@vnarapar
Copy link
Copy Markdown
Contributor Author

vnarapar commented Jun 1, 2026

@smuppand all comments are addressed.
FYI, shellcheck errors are not from this PR.

@vnarapar vnarapar force-pushed the dmabuf branch 2 times, most recently from 6379b14 to 785c402 Compare June 1, 2026 08:57
vnarapar added 3 commits June 1, 2026 15:58
Add the helper function to check kernel optional config

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
This test validates the DMA-BUF subsystem configuration on Qualcomm
platforms, including kernel configuration, device tree setup, and system
interfaces

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Runs the `dmabuf-heap` binary from the Linux kernel selftests
suite to validate DMA-BUF heap functionality

Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
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