It seems crazy to me that group IDs (being integers) can be anything
other than [0 .. num_groups - 1], but the specification is written
with full generality here.
The code was already querying the group ID values originally, and
assigning those to each group->id slot in the metric_group_info_t
structure. But after that, the code had been assuming it could just
use values from 0 .. num_groups-1.
Fix this by carefully using group_index values ([0..num_groups-1])
when indexing into the various arrays and group->id values when
passing ID values to the various performance-monitor API functions.
GLuint num_counters;
GLuint max_active_counters;
GLuint num_counters;
GLuint max_active_counters;
char **counter_names;
GLuint *counter_types;
char **counter_names;
GLuint *counter_types;
0, NULL);
/* Get counter numbers */
0, NULL);
/* Get counter numbers */
- group->counters = xmalloc (group->num_counters * sizeof (GLuint));
+ group->counter_ids = xmalloc (group->num_counters * sizeof (GLuint));
glGetPerfMonitorCountersAMD (group->id, NULL, NULL,
group->num_counters,
glGetPerfMonitorCountersAMD (group->id, NULL, NULL,
group->num_counters,
/* Get counter names */
group->counter_names = xmalloc (group->num_counters * sizeof (char *));
/* Get counter names */
group->counter_names = xmalloc (group->num_counters * sizeof (char *));
for (i = 0; i < group->num_counters; i++) {
glGetPerfMonitorCounterInfoAMD (group->id,
for (i = 0; i < group->num_counters; i++) {
glGetPerfMonitorCounterInfoAMD (group->id,
GL_COUNTER_TYPE_AMD,
&group->counter_types[i]);
GL_COUNTER_TYPE_AMD,
&group->counter_types[i]);
}
glGetPerfMonitorCounterStringAMD (group->id,
}
glGetPerfMonitorCounterStringAMD (group->id,
0, &length, NULL);
group->counter_names[i] = xmalloc (length + 1);
glGetPerfMonitorCounterStringAMD (group->id,
0, &length, NULL);
group->counter_names[i] = xmalloc (length + 1);
glGetPerfMonitorCounterStringAMD (group->id,
length + 1, NULL,
group->counter_names[i]);
}
length + 1, NULL,
group->counter_names[i]);
}
free (group->counter_types);
free (group->counter_names);
free (group->counter_types);
free (group->counter_names);
- free (group->counters);
+ free (group->counter_ids);
metrics_info->groups = xmalloc (metrics_info->num_groups * sizeof (metrics_group_info_t));
for (i = 0; i < metrics_info->num_groups; i++)
metrics_info->groups = xmalloc (metrics_info->num_groups * sizeof (metrics_group_info_t));
for (i = 0; i < metrics_info->num_groups; i++)
- metrics_group_info_init (&metrics_info->groups[i], i);
+ metrics_group_info_init (&metrics_info->groups[i], group_ids[i]);
glSelectPerfMonitorCountersAMD(monitor->id,
GL_TRUE, group->id,
num_counters,
glSelectPerfMonitorCountersAMD(monitor->id,
GL_TRUE, group->id,
num_counters,
}
/* Start the queries */
}
/* Start the queries */
p += sizeof(var);
context_t *ctx = ¤t_context;
p += sizeof(var);
context_t *ctx = ¤t_context;
+ metrics_info_t *info = &ctx->metrics_info;
unsigned char *p = (unsigned char *) result;
while (p < ((unsigned char *) result) + size)
{
unsigned char *p = (unsigned char *) result;
while (p < ((unsigned char *) result) + size)
{
- GLuint group_id, counter_id, counter_index;
+ GLuint group_id, group_index;
+ GLuint counter_id, counter_index;
metrics_group_info_t *group;
uint32_t value;
unsigned i;
metrics_group_info_t *group;
uint32_t value;
unsigned i;
CONSUME (counter_id);
CONSUME (value);
CONSUME (counter_id);
CONSUME (value);
- assert (group_id < ctx->metrics_info.num_groups);
- group = &ctx->metrics_info.groups[group_id];
+ for (i = 0; i < info->num_groups; i++) {
+ if (info->groups[i].id == i)
+ break;
+ }
+ group_index = i;
+ assert (group_index < info->num_groups);
+ group = &info->groups[group_index];
for (i = 0; i < group->num_counters; i++) {
for (i = 0; i < group->num_counters; i++) {
- if (group->counters[i] == counter_id)
+ if (group->counter_ids[i] == counter_id)
break;
}
counter_index = i;
assert (counter_index < group->num_counters);
break;
}
counter_index = i;
assert (counter_index < group->num_counters);
- ctx->op_metrics[op].counters[group_id][counter_index] += value;
+ ctx->op_metrics[op].counters[group_index][counter_index] += value;
metrics_info_t *info = &ctx->metrics_info;
metrics_group_info_t *group;
const char *op_string;
metrics_info_t *info = &ctx->metrics_info;
metrics_group_info_t *group;
const char *op_string;
- unsigned i, group_id, counter;
+ unsigned i, group_index, counter;
double value;
/* Since we sparsely fill the array based on program
double value;
/* Since we sparsely fill the array based on program
metric->time_ns / total * 100);
printf ("[");
metric->time_ns / total * 100);
printf ("[");
- for (group_id = 0; group_id < info->num_groups; group_id++) {
- group = &info->groups[group_id];
+ for (group_index = 0; group_index < info->num_groups; group_index++) {
+ group = &info->groups[group_index];
for (counter = 0; counter < group->num_counters; counter++) {
for (counter = 0; counter < group->num_counters; counter++) {
- value = metric->counters[group_id][counter];
+ value = metric->counters[group_index][counter];
if (value == 0.0)
continue;
printf ("%s: %.2f ", group->counter_names[counter],
if (value == 0.0)
continue;
printf ("%s: %.2f ", group->counter_names[counter],