Skip to content

support cpu count > 64 issue6142 #6197

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

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

DaChengTechnology
Copy link
Contributor

add MutiThread Module

@tencent-adm
Copy link
Member

tencent-adm commented Jul 21, 2025

CLA assistant check
All committers have signed the CLA.

@DaChengTechnology
Copy link
Contributor Author

DaChengTechnology commented Jul 21, 2025

some layer not support.It is necessary for @nihui to determine whether he can accept this style of code.

@nihui nihui closed this Jul 22, 2025
@nihui nihui reopened this Jul 22, 2025
@nihui nihui closed this Jul 22, 2025
@nihui nihui reopened this Jul 22, 2025
Copy link
Member

@nihui nihui left a comment

Choose a reason for hiding this comment

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

The goal of issue 6142 is to add CPU topology information retrieval and thread affinity handling.
You should not modify the way OpenMP is used in the layer.

@github-actions github-actions bot added the test label Jul 29, 2025
@DaChengTechnology
Copy link
Contributor Author

The goal of issue 6142 is to add CPU topology information retrieval and thread affinity handling. You should not modify the way OpenMP is used in the layer.

Windows thread affinity handling is contian two pamaes, the one indicate that cpu groups,other one indicate that cpu core mark.
how do you use that on OpenMP`s program?

@DaChengTechnology
Copy link
Contributor Author

now, ncnn lib use openMP maybe not in thread affinity.

@DaChengTechnology DaChengTechnology requested a review from nihui July 29, 2025 14:14
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