All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Vrabel <david.vrabel@citrix.com>
To: Paul Durrant <paul.durrant@citrix.com>, xen-devel@lists.xenproject.org
Cc: Keir Fraser <keir@xen.org>, Jan Beulich <jbeulich@suse.com>
Subject: Re: [PATCH v3] x86/hvm: Add per-vcpu evtchn upcalls
Date: Mon, 10 Nov 2014 14:55:06 +0000	[thread overview]
Message-ID: <5460D1CA.3090905@citrix.com> (raw)
In-Reply-To: <1415617996-18632-1-git-send-email-paul.durrant@citrix.com>

On 10/11/14 11:13, Paul Durrant wrote:
> HVM guests have always been confined to using the domain callback
> via (see HVM_PARAM_CALLBACK_IRQ) to receive event notifications.
> This is usually an IOAPIC vector and is only used if the event
> channel is bound to vcpu 0.
> 
> PV-on-HVM Linux uses a local APIC vector for event channel upcall,
> set using HVM_PARAM_CALLBACK_IRQ by ORing in a special bit (bit 57)
> into the value (see params.h), but this mechanism is not suitable
> in the general case since the vector must be the same on all vcpus
> which cannot be guaranteed on some OS (e.g. Windows).

"local APIC vector" doesn't make much sense to me here since PVHVM Linux
bypasses the APIC entirely for event channels.

Reword to:

"PVHVM Linux uses a pre-defined interrupt vector for the event channel
upcall, set using HVM_PARAM_CALLBACK_IRQ by ORing in a special bit
(bit 57) into the value (see params.h), This mechanism does not assert
the interrupt via the local APIC.

This mechanism is not suitable in the general case since Windows (and
potentially other OSes):

- cannot guarantee the same vector for all VCPUs
- require the upcall to be asserted via the local APIC."

or similar.

David

      reply	other threads:[~2014-11-10 14:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-10 11:13 [PATCH v3] x86/hvm: Add per-vcpu evtchn upcalls Paul Durrant
2014-11-10 14:55 ` David Vrabel [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5460D1CA.3090905@citrix.com \
    --to=david.vrabel@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=keir@xen.org \
    --cc=paul.durrant@citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.