ITTI: remove useless log_scheduler() function
The primary motivation here is to remove a VLA, as in the parent commits. Closer inspection revealed that (1) it is only used by the LTE UE, and (2) it only logs some data which we likely don't need. So I simply removed this function.
This commit is contained in:
@@ -475,41 +475,5 @@ typedef struct timer_elm_s {
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
void log_scheduler(const char* label)
|
|
||||||
{
|
|
||||||
int policy = sched_getscheduler(0);
|
|
||||||
struct sched_param param;
|
|
||||||
if (sched_getparam(0, ¶m) == -1)
|
|
||||||
{
|
|
||||||
LOG_E(HW, "sched_getparam: %s\n", strerror(errno));
|
|
||||||
abort();
|
|
||||||
}
|
|
||||||
|
|
||||||
cpu_set_t cpu_set;
|
|
||||||
if (sched_getaffinity(0, sizeof(cpu_set), &cpu_set) == -1)
|
|
||||||
{
|
|
||||||
LOG_E(HW, "sched_getaffinity: %s\n", strerror(errno));
|
|
||||||
abort();
|
|
||||||
}
|
|
||||||
int num_cpus = sysconf(_SC_NPROCESSORS_ONLN);
|
|
||||||
if (num_cpus < 1)
|
|
||||||
{
|
|
||||||
LOG_E(HW, "sysconf(_SC_NPROCESSORS_ONLN): %s\n", strerror(errno));
|
|
||||||
abort();
|
|
||||||
}
|
|
||||||
char buffer[num_cpus];
|
|
||||||
for (int i = 0; i < num_cpus; i++)
|
|
||||||
{
|
|
||||||
buffer[i] = CPU_ISSET(i, &cpu_set) ? 'Y' : '-';
|
|
||||||
}
|
|
||||||
|
|
||||||
LOG_A(HW, "Scheduler policy=%d priority=%d affinity=[%d]%.*s label=%s\n",
|
|
||||||
policy,
|
|
||||||
param.sched_priority,
|
|
||||||
num_cpus,
|
|
||||||
num_cpus,
|
|
||||||
buffer,
|
|
||||||
label);
|
|
||||||
}
|
|
||||||
} // extern "C"
|
} // extern "C"
|
||||||
|
|
||||||
|
|||||||
@@ -566,7 +566,6 @@ int timer_remove(long timer_id);
|
|||||||
int signal_handle(int *end);
|
int signal_handle(int *end);
|
||||||
int signal_mask(void);
|
int signal_mask(void);
|
||||||
|
|
||||||
void log_scheduler(const char *label);
|
|
||||||
#ifdef __cplusplus
|
#ifdef __cplusplus
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|||||||
@@ -1740,8 +1740,6 @@ void *UE_thread(void *arg) {
|
|||||||
oai_exit=1;
|
oai_exit=1;
|
||||||
}
|
}
|
||||||
|
|
||||||
log_scheduler(__func__);
|
|
||||||
|
|
||||||
while (!oai_exit) {
|
while (!oai_exit) {
|
||||||
AssertFatal ( 0== pthread_mutex_lock(&UE->proc.mutex_synch), "");
|
AssertFatal ( 0== pthread_mutex_lock(&UE->proc.mutex_synch), "");
|
||||||
int instance_cnt_synch = UE->proc.instance_cnt_synch;
|
int instance_cnt_synch = UE->proc.instance_cnt_synch;
|
||||||
|
|||||||
Reference in New Issue
Block a user