lkml.org 
[lkml]   [2010]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [Xen-devel] [PATCH 03/12] evtchn delivery on HVM
    > +static int __cpuinit xen_hvm_cpu_notify(struct notifier_block *self,
    > + unsigned long action, void *hcpu)
    > +{
    > + int cpu = (long)hcpu;
    > + switch (action) {
    > + case CPU_UP_PREPARE:
    > + per_cpu(xen_vcpu, cpu) = &HYPERVISOR_shared_info->vcpu_info[cpu];
    > + break;

    Is there no need to do anything if the CPUs are brought down?

    .. snip..

    > +/* the callback vector mechanism is a newer alternative way of receiving
    > + * event channel notifications from Xen: we can receive vector callbacks
    > + * on any vcpus and we don't need any PCI or IO APIC support */

    I am not an expert in English, but I think you need to revise this
    comment.
    > +void xen_callback_vector(void)
    > +{
    > + int rc;
    > + uint64_t callback_via;
    > + if (xen_have_vector_callback) {
    > + callback_via = HVM_CALLBACK_VECTOR(XEN_HVM_EVTCHN_CALLBACK);
    > + rc = xen_set_callback_via(callback_via);
    > + if (rc) {
    > + printk(KERN_ERR "request for callback vector failed\n");

    Perhaps mention which type? Say 'Request for Xen HVM callback vector
    failed.\n' ?


    \
     
     \ /
      Last update: 2010-06-15 15:47    [W:0.024 / U:1.076 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site