Re: [PATCH v5 2/6] x86: allow reading MSR_IA32_TSC with XENPF_resource_op

["Jan Beulich" <JBeulich@suse.com>]

>>> On 23.01.15 at 09:42, <chao.p.peng@linux.intel.com> wrote:
> Except that MSR_IA32_TSC will looks heterogeneous. But since we already
> treat it as a special case, I have no problem here. And if we do so,
> even the first patch to add irq_disable ability is not needed. Sounds to
> me that the MSR_IA32_TSC read should always imply irq disabled. Codes
> would be like this:

Looks reasonable, minus some formatting issues and ...

>         case XEN_RESOURCE_OP_MSR_READ:
>             if ( unlikely(entry->idx == MSR_IA32_TSC) )
>             { 
>                 entry->val = get_s_time_fixed(tsc);
>                 ret = 0;
>             }   
>             else
>             {
>                 bool_t  read_tsc = 0;
>                 if ( i < ra->nr_done - 1 ) 
>                 {   
>                     xenpf_resource_entry_t *next = ra->entries + i + 1;
>                     if ( unlikely(next->idx == MSR_IA32_TSC) )

... this simply being entry[1].idx.

Jan

>                         read_tsc = 1;
>                 }
> 
>                 if ( unlikely(read_tsc) )
>                     local_irq_save(irqflags);
> 
>                 ret = rdmsr_safe(entry->idx, entry->val);
> 
>                 if ( unlikely(read_tsc) )
>                 {
>                     rdtscll(tsc);
>                     local_irq_restore(irqflags);
>                 }
>             }
>             break;
>         case XEN_RESOURCE_OP_MSR_WRITE:
> 
> Chao