On 11/14/18 9:35 AM, Tian, Kevin wrote:
>>>> +#ifdef CONFIG_HVM
>>>> +/* There's already a memory_type_changed() in asm/mtrr.h. */
>>>> +static void _memory_type_changed(struct p2m_domain *p2m)
>>>> +{
>>>> +    if ( p2m->memory_type_changed )
>>>> +        p2m->memory_type_changed(p2m);
>>>>  }
>>>>
>>>>  void p2m_memory_type_changed(struct domain *d)
>>> why making whole p2m_memory_type_changed under CONFIG_HVM?
>> It has been requested by Jan and Wei in V3:
>>
>> https://lists.xenproject.org/archives/html/xen-devel/2018-
>> 10/msg02495.html
>>
> then it's good to mention it in patch description.
> 
> Reviewed-by: Kevin Tian <kevin.t...@intel.com>

With George's comments yesterday there'll be at least one more iteration
of the series (probably minus the first patch, which seems ready to go
in) - so I'll update the patch description in the next version.


Thanks,
Razvan

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to