Skip to content

Enable CONFIG_SCHEDSTATS for systemd-bootchart#24

Closed
pavel-shirshov wants to merge 3 commits intosonic-net:masterfrom
pavel-shirshov:pavelsh/CONFIG_SCHEDSTATS
Closed

Enable CONFIG_SCHEDSTATS for systemd-bootchart#24
pavel-shirshov wants to merge 3 commits intosonic-net:masterfrom
pavel-shirshov:pavelsh/CONFIG_SCHEDSTATS

Conversation

@pavel-shirshov
Copy link
Copy Markdown

@pavel-shirshov pavel-shirshov commented Jul 3, 2017

we can use systemd-bootchart to understand cpu/memory/io usage. But currently it doesn't work because CONFIG_SCHEDSTATS is disabled in kernel.

@msftclas
Copy link
Copy Markdown

msftclas commented Jul 3, 2017

@pavel-shirshov,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Jul 3, 2017

I am not sure the kernel build uses this config file.

@pavel-shirshov
Copy link
Copy Markdown
Author

Make the change as a patch

@pavel-shirshov
Copy link
Copy Markdown
Author

This change breaks ABI.

dgsudharsan pushed a commit to dgsudharsan/sonic-linux-kernel that referenced this pull request May 12, 2025
…2412 (sonic-net#24)

```<br>* f9ee9d3 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-linux-kernel into 202412 (2025-03-01) [Sonic Automation]
* 8bd11c2 - (origin/202411) Revert "[optoe] Reset page select byte to 0 before upper memory access on page 0h" (sonic-net#468) (2025-02-26) [mssonicbld]
* a5e9588 - [optoe] Reset page select byte to 0 before upper memory access on page 0h (sonic-net#464) (2025-02-15) [mssonicbld]<br>```
dal00 pushed a commit to kamelnetworks/sonic-linux-kernel that referenced this pull request Jul 20, 2025
Signed-off-by: maipbui maibui@microsoft.com
Why I did it
subprocess is used with shell=True, which is very dangerous for shell injection.
How I did it
remove shell=True, use shell=False
How to verify it
Pass UT
Manual test
dal00 pushed a commit to kamelnetworks/sonic-linux-kernel that referenced this pull request Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants