Skip to content

Clean up and bug fix for 127 worker limit

Tanner Perkins requested to merge 3689-hdl-worker-127-limit into develop

What this MR does / why we need it:

Bug fix for m_usedImpls vector in runtime code. Fix build engine makefile to where a larger number of of workers can be built into an assembly (for future expansion of worker limit). Change worker index from uchar to ushort (for future expansion of worker limit). Remove regions and uuid from occp admin registers and tools that pull these registers (clean up). Remove configRam from occp memory space, this was legacy and unused, required proper padding to get memory address to align (clean up).

Steps to complete before submitting MR:

  • I have read Contribution guidelines

  • I have ensured I have a changelog written in the imperative form that is meaningful and accurately summarizes the work done

  • I have added the release notes label if applicable (more information below)

  • I have ensured the workflow labels are up to date and will continue to do so up until this work is merged into develop

  • I have added a category, type, and target release label along with all other labels needed

  • My branch is up to date with develop. If it is not, then I have ensured my work does not conflict with other work

  • I agree my bugfix MR does not include new features/enhancements

  • I represent that bugfixes have been locally tested against the most recent major.minor release in which the bug exists

  • (REVIEWER ONLY) I have thoroughly gone through the above steps to ensure that they have been followed

Changelog

  • bug(application, runtime): fix broken usedImpls vector
  • feat(runtime): clean up occp admin registers and memory space

Which issue(s) this MR closes

Closes #3689

Edited by Tanner Perkins

Merge request reports