Skip to content

Conversation

@rajendra-desai-ni
Copy link
Contributor

@rajendra-desai-ni rajendra-desai-ni commented Dec 22, 2025

Summary of Changes

Adding cgroups v2 support to LabVIEW RT 2026 caused some test failures in LabVIEW RT. Hence, restrict it to use cgroups v1 instead of v2 for now.

Justification

AB#3515413
This commit also needs to be cherry-picked into nilrt/26.0/scarthgap branch.

Testing

  • I have built the core package feed with this PR in place. (bitbake packagefeed-ni-core)
  • bitbake nilrt-base-system-image
  • Built the BSI and installed it on the PXI-8881 target along with LVRT 2026. Ran all the tests under "C:\sync\penguin\2023\test\trunk\tests_auto\functionality\Utility\CPU and Memory" and all of them passed.

Procedure

Adding cgroups v2 support to LabVIEW RT 2026 caused
some API breakages in LabVIEW. Hence, restrict it to
use cgroups v1 instead of v2.

Signed-off-by: Rajendra Desai <rajendra.desai@emerson.com>
@rajendra-desai-ni rajendra-desai-ni requested a review from a team December 22, 2025 14:00
# LabVIEW RT 2025 and prior versions only support cgroups v1
[ -n "$major_version" ] && [ "$major_version" -le 25 ] && echo 1 || echo 2
# LabVIEW RT 2026 and prior versions only support cgroups v1
[ -n "$major_version" ] && [ "$major_version" -le 26 ] && echo 1 || echo 2
Copy link
Contributor

Choose a reason for hiding this comment

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

Adding a comment to remind ourselves not to merge this accidentally. The v2 code is functional and this change is likely not needed.

@gratian
Copy link
Contributor

gratian commented Jan 20, 2026

The associated bug has been resolved and this PR is no longer needed. Closing.

@gratian gratian closed this Jan 20, 2026
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.

3 participants