[lkml]   [2010]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 5/7] xen: Make event channel work with PV featured HVM
    On Mon, 2010-02-08 at 08:05 +0000, Sheng Yang wrote:
    > + if (xen_hvm_pv_evtchn_enabled()) {
    > + if (enable_hvm_pv(HVM_PV_EVTCHN))
    > + return -EINVAL;
    > +[...]
    > + set_callback_via(callback_via);
    > +
    > + x86_platform_ipi_callback = do_hvm_pv_evtchn_intr;

    Why this indirection via X86_PLATFORM_IPI_VECTOR?

    Apart from that why not use CALLBACKOP_register subop CALLBACKTYPE_event
    pointing to xen_hypervisor_callback the same as a full PV guest?

    This would remove all the evtchn related code from HVMOP_enable_pv which
    I think should be eventually unnecessary as an independent hypercall
    since all HVM guests should simply be PV capable by default -- the
    hypervisor only needs to track if the guest has made use of specific PV
    functionality, not the umbrella "is PV" state.


     \ /
      Last update: 2010-02-09 12:55    [W:0.018 / U:30.340 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site