>>> On 17.10.16 at 08:26, <feng...@intel.com> wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Wednesday, October 12, 2016 9:45 PM >> >>> On 11.10.16 at 02:57, <feng...@intel.com> wrote: >> > static void pi_desc_init(struct vcpu *v) >> > { >> > - uint32_t dest; >> > - >> > v->arch.hvm_vmx.pi_desc.nv = posted_intr_vector; >> > >> > - dest = cpu_physical_id(v->processor); >> > - >> > - if ( x2apic_enabled ) >> > - v->arch.hvm_vmx.pi_desc.ndst = dest; >> > - else >> > - v->arch.hvm_vmx.pi_desc.ndst = MASK_INSR(dest, >> PI_xAPIC_NDST_MASK); >> > + /* >> > + * Mark NDST as invalid, then we can use this invalid value as a >> > + * marker to whether update NDST or not in vmx_pi_hooks_assign(). >> > + */ >> > + v->arch.hvm_vmx.pi_desc.ndst = 0xffffffff; >> >> I think I had at the same time asked to make this a #define, so the >> two (currently) instance can be connected together. > > Sorry, Maybe I didn't get that. Do you mean I need to define a Macro > for 0xffffffff, so we can use it here and in vmx.c?
Yes. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel