mirror of
https://github.com/Dr-Noob/cpufetch.git
synced 2026-03-24 23:40:39 +01:00
[v1.06][X86] Bugfix: set affinity in --accurate-pp
There are cases where measure_frequency is called after binding the process to a specific core via bind_to_cpu (e.g., when iterating over modules in hybrid architectures). Thus, in measure_frequency we must set the affinity of the newly created threads, ensuring they are binded to the right core.
This commit is contained in:
@@ -116,8 +116,25 @@ int64_t measure_frequency(struct cpuInfo* cpu) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
pthread_t* compute_th = malloc(sizeof(pthread_t) * cpu->topo->total_cores);
|
pthread_t* compute_th = malloc(sizeof(pthread_t) * cpu->topo->total_cores);
|
||||||
|
cpu_set_t cpus;
|
||||||
|
pthread_attr_t attr;
|
||||||
|
if ((ret = pthread_attr_init(&attr)) != 0) {
|
||||||
|
printErr("pthread_attr_init: %s", strerror(ret));
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
|
||||||
for(int i=0; i < cpu->topo->total_cores; i++) {
|
for(int i=0; i < cpu->topo->total_cores; i++) {
|
||||||
ret = pthread_create(&compute_th[i], NULL, compute_function, NULL);
|
// We might have called bind_to_cpu previously, binding the threads
|
||||||
|
// to a specific core, so now we must make sure we run the new thread
|
||||||
|
// on the correct core.
|
||||||
|
CPU_ZERO(&cpus);
|
||||||
|
CPU_SET(i, &cpus);
|
||||||
|
if ((ret = pthread_attr_setaffinity_np(&attr, sizeof(cpu_set_t), &cpus)) != 0) {
|
||||||
|
printErr("pthread_attr_setaffinity_np: %s", strerror(ret));
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
|
||||||
|
ret = pthread_create(&compute_th[i], &attr, compute_function, NULL);
|
||||||
|
|
||||||
if(ret != 0) {
|
if(ret != 0) {
|
||||||
fprintf(stderr, "Error creating thread\n");
|
fprintf(stderr, "Error creating thread\n");
|
||||||
|
|||||||
Reference in New Issue
Block a user