Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New package: LACT-0.5.4 #50027

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Calandracas606
Copy link

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, x86_64-glibc
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-musl
    • aarch64 (cross)
    • aarch64-musl (cross)

@@ -0,0 +1,3 @@
#!/bin/sh
exec 2>&1
exec chpst -n -10 lact daemon
Copy link
Author

Choose a reason for hiding this comment

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

use a nice value of -10 because that is what's used in the sample systemd service

Comment on lines +9 to +14
- // Eagerly release memory
- // `load_controllers` allocates and deallocates the entire PCI ID database,
- // this tells the os to release it right away, lowering measured memory usage (the actual usage is low regardless as it was already deallocated)
- unsafe {
- libc::malloc_trim(0);
- }
Copy link
Author

Choose a reason for hiding this comment

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

musl does not have malloc_trim

Copy link
Contributor

Choose a reason for hiding this comment

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

It might be worth considering applying this patch only on musl and leaving the code intact on glibc.

pkgname=LACT
version=0.5.4
revision=1
archs="x86_64* aarch64*"
Copy link
Author

Choose a reason for hiding this comment

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

Uses an AMDGPU cargo crate which only compiles on 64bit

Don't think this matters, since using a modern amd gpu on 32bit arm seems extemely unlikely, but not working on i686 is unfortuneate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants