[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] x86/vm_event: Added support for VM_EVENT_REASON_INTERRUPT
On 11/09/2016 01:17 PM, Jan Beulich wrote: >>>> On 09.11.16 at 10:42, <rcojocaru@xxxxxxxxxxxxxxx> wrote: >> +static bool svm_get_pending_event(struct vcpu *v, struct hvm_trap *info) >> +{ >> + struct vmcb_struct *vmcb = v->arch.hvm_svm.vmcb; >> + >> + if ( vmcb->eventinj.fields.v ) >> + return false; >> + >> + info->vector = vmcb->eventinj.fields.vector; >> + info->type = vmcb->eventinj.fields.type; >> + info->error_code = vmcb->eventinj.fields.errorcode; >> + info->cr2 = v->arch.hvm_vcpu.guest_cr[2]; > > I'd prefer for this last part to be put into generic code (i.e. the > wrapper). Actually, doing this: static inline bool hvm_get_pending_event(struct vcpu *v, struct hvm_trap *info) { info->cr2 = v->arch.hvm_vcpu.guest_cr[2]; return hvm_funcs.get_pending_event(v, info); } leads to "error: dereferencing pointer to incomplete type" about v->, so to do this an additional #include will be necessary. Is that acceptable? Thanks, Razvan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |