Skip to content

Conversation

@lukamac
Copy link
Contributor

@lukamac lukamac commented Sep 22, 2025

My take on the doublebuffering performance issue.
Passes the testDmas.py test for Mchan and L3 DMAs.

Related PR #114

Added

Changed

Fixed

PR Merge Checklist

  1. The PR is rebased on the latest devel commit and pointing to devel.
  2. Your PR reviewed and approved.
  3. All checks are passing.
  4. The CHANGELOG.md file has been updated.
  5. If the docker was modified, change back its link after review.

@Xeratec
Copy link
Member

Xeratec commented Sep 23, 2025

Nice, I think we have a very similar approach. However, there seem to be some issues with the DMA allocation and freeing part. I will take some of your code and move it to my PR ;) I suggest using my code as a base version and converge on a version we both like.

Siracusa_testEmptyKernel_doublebuffer_Luka

@Xeratec
Copy link
Member

Xeratec commented Sep 27, 2025

Closed in favor of #114

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