On 25.01.2021 18:17, Andrew Cooper wrote:
> On 25/01/2021 15:08, Jan Beulich wrote:
>> On 21.01.2021 22:27, Andrew Cooper wrote:
>>> --- a/xen/include/xen/sched.h
>>> +++ b/xen/include/xen/sched.h
>>> @@ -257,6 +257,10 @@ struct vcpu
>>> /* vPCI per-vCPU area, used to store data for long running operations.
>>> */
>>> struct vpci_vcpu vpci;
>>>
>>> + struct {
>>> + struct page_info *buf;
>>> + } vmtrace;
>> While perhaps minor, I'm unconvinced "buf" is a good name
>> for a field of this type.
>
> Please suggest a better one then. This one is properly namespaced as
> v->vmtrace.buf which is the least bad option I could come up with.
"pg", "page", or "pages" are all names we use for variables and
fields of this type. I don't see at all how the namespacing
helps it - to me "v->vmtrace.buf" is still a pointer to the
actual buffer, not the struct page_info describing the 1st page
of it.
Jan
