> -----Original Message----- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Wednesday, October 12, 2016 9:45 PM > To: Wu, Feng <feng...@intel.com> > Cc: andrew.coop...@citrix.com; dario.faggi...@citrix.com; > george.dun...@eu.citrix.com; Tian, Kevin <kevin.t...@intel.com>; xen- > de...@lists.xen.org > Subject: Re: [PATCH v5 4/7] VMX: Make sure PI is in proper state before > install > the hooks > > >>> 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? Thanks, Feng _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
