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.

Sign up for GitHub

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

Jump to bottom

Add support for perf #6

Open
derickson2402 opened this issue Jan 27, 2022 · 1 comment · Fixed by #15
Open

Add support for perf #6

derickson2402 opened this issue Jan 27, 2022 · 1 comment · Fixed by #15
Assignees
@derickson2402
Labels
enhancement New feature or request

Comments

@derickson2402
Copy link
Owner

This post has some tips for getting perf running in Docker. Perf needs special permissions to work, but this is probably not the safest thing to enable by default.

Maybe have the ./caen wrapper script check if you want to use perf, then enable the permissions flags?
@derickson2402 derickson2402 added the enhancement New feature or request label Jan 27, 2022
@derickson2402 derickson2402 self-assigned this Jan 27, 2022
derickson2402 pushed a commit that referenced this issue Feb 6, 2022
Revamp the wrapper script, add installation instructions
bfedea2 
Added the following to the wrapper script

- Automatically detect host OS
- Automatically detect if Docker is installed and running, provide instructions or turn it on if not
- Automatically detect an internet connection and pull the newest version of the container
- Use environment variables to change config
- Verify the user is fine with using Perf with SYS_ADMIN turned on

Also added command to the docs for automatically installing the script

See #8, #6
@derickson2402 derickson2402 mentioned this issue Feb 16, 2022
Merged
derickson2402 added a commit that referenced this issue Feb 20, 2022
@derickson2402
Merge pull request #15 from derickson2402/dev
0fb3670 
Merge changes from dev for release v0.5

- Add functionality and error checking to wrapper script (closes #8)
- Add support for input redirection (closes #2)
- Add support for running Perf command with elevated Docker permissions (closes #6)
- Add cppchecker to C++ container (closes #3)
@derickson2402 derickson2402 mentioned this issue Mar 11, 2022
Closed
@derickson2402
Copy link
Owner Author

Reopening this issue. I have had some issues recently trying to get Perf to work (due to kernel permissions). Also, Perf may not work inside of devcontainer.

@derickson2402 derickson2402 reopened this Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@derickson2402 derickson2402

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Merge changes from dev derickson2402/Dockerized-CAEN
1 participant
@derickson2402