[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC V4 4/5] xen, libxc: Request page fault injection via libxc

>>> On 04.08.14 at 17:00, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
> On 08/04/2014 05:26 PM, Jan Beulich wrote:
>>>>> On 04.08.14 at 13:30, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
>>> +    __vmread(VM_ENTRY_INTR_INFO, &ev);
>>> +
>>> +    if ( (ev & INTR_INFO_VALID_MASK) &&
>>> +         hvm_event_needs_reinjection((ev >> 8) & 7, ev & 0xff) )
>> Are there no manifest constants for all these plain numbers?
> If there are, vmx_vmcs_save() in vmx.c (line 416) doesn't use them. I've
> copied that part verbatim.

And that's precisely the problem: As long as there's exactly one use
site, the need for manifest constants is questionable (i.e. largely
cosmetic). As soon as there are multiple places, connecting them
together is largely impossible without naming these numbers - only
that way you have a reasonable chance to find the clone of the
original should the original be found to need tweaking.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.