> > +static int initialize_mem_bw_imc(void)
> > +{
> > +   int imc, j;
> > +
> > +   imcs = num_of_imcs();
> > +   if (imcs < 0)
> 
> I think this condition should be "<=". Pls check.

Thank you for pointing it out. It makes sense to change it to "<=". Will fix it 
in next version.

> > +           return imcs;
> > +
> > +   /* Initialize perf_event_attr structures for all iMC's */
> > +   for (imc = 0; imc < imcs; imc++) {
> > +           for (j = 0; j < 2; j++)
> > +                   membw_initialize_perf_event_attr(imc, j);
> > +   }
> > +
> > +   return 0;
> > +}

Thanks and Regards,
Arshiya

Reply via email to