Hi Reinette, On 12/20/22 11:32, Reinette Chatre wrote: > Hi Babu, > > On 12/19/2022 11:50 AM, Moger, Babu wrote: >> Forgot about this. This snippet is going to change. I have tested and works fine. >> How about this? >> >> /* >> * Update MSR_IA32_EVT_CFG_BASE MSR on one of the CPUs in the >> * domain. The MSRs offset from MSR MSR_IA32_EVT_CFG_BASE >> * are scoped at the domain level. Writing any of these MSRs >> * on one CPU is supposed to be observed by all CPUs in the domain. >> */ >> smp_call_function_any(&d->cpu_mask, mon_event_config_write, &mon_info, 1); >> >> > It looks good but please drop the "supposed to be". If there is any uncertainty > then the data should be written to all CPUs, if not then that uncertain text should > be dropped. There is no uncertainty. I will drop "supposed to be". Thanks Babu
