From: Keir Fraser <keir@xen.org>
To: wei.huang2@amd.com, "Zhang, Xiantao" <xiantao.zhang@intel.com>
Cc: Jan Beulich <JBeulich@suse.com>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: LWP Interrupt Handler
Date: Fri, 30 Mar 2012 09:21:05 +0100 [thread overview]
Message-ID: <CB9B2B81.3D111%keir@xen.org> (raw)
In-Reply-To: <4F74E825.9060909@amd.com>
On 29/03/2012 23:54, "Wei Huang" <wei.huang2@amd.com> wrote:
> How about the approach in attached patches? A interrupt handler is
> registered at vector 0xf6 as a delegate for guest VM. Guest VCPU can
> register interrupt handler to receive notification. I will prepare
> formal ones after receiving your comments.
Ugh, no, these ad-hoc dynamically-registered subhandlers are getting beyond
the pale. Therefore, I've just spent half hour routing all our interrupts,
even the direct APIC ones, through do_IRQ(), and now allow everything to be
dynamically allocated.
Please pull up to xen-unstable tip (>= 25113) and use
alloc_direct_apic_vector(). You can find a few callers of it that I added
myself, to give you an idea how to use it. In essence, you can call it from
your SVM-specific code, to allocate a vector for your irq handler. And you
can do it immediately before you poke the vector into your special MSR. You
can call alloc_direct_apic_vector() unconditionally -- it will deal with
ensuring the allocation happens only once.
-- Keir
> Thanks,
> -Wei
>
>
> On 03/25/2012 07:08 PM, Zhang, Xiantao wrote:
>> Please make sure the per-cpu vector is considered in your case. For CPU's
>> built-in event, it always happens on all cpus, but IRQ-based events may only
>> happen on some special cpus which are determined by apic's mode.
>> Xiantao
>>
>>> -----Original Message-----
>>> From: xen-devel-bounces@lists.xen.org [mailto:xen-devel-
>>> bounces@lists.xen.org] On Behalf Of Keir Fraser
>>> Sent: Saturday, March 24, 2012 6:50 AM
>>> To: wei.huang2@amd.com; xen-devel@lists.xen.org; Jan Beulich
>>> Subject: Re: [Xen-devel] LWP Interrupt Handler
>>>
>>> On 23/03/2012 22:03, "Wei Huang"<wei.huang2@amd.com> wrote:
>>>
>>>> I am adding interrupt support for LWP, whose spec is available at
>>>> http://support.amd.com/us/Processor_TechDocs/43724.pdf. Basically OS
>>>> can specify an interrupt vector in LWP_CFG MSR; the interrupt will be
>>>> triggered when event buffer overflows. For HVM guests, I want to
>>>> re-inject this interrupt back into the guest VM. Here is one idea
>>>> similar to virtualized PMU: It first registers a special interrupt
>>>> handler (say on vector 0xf6) using set_intr_gate(). When triggered,
>>>> this handler injects an IRQ (with vector copied from LWP_CFG) into
>>>> guest VM via virtual local APIC. This worked from my test.
>>>>
>>>> But adding a interrupt handler seems to be an overkill. Is there any
>>>> better way to create a dummy interrupt receiver on be-behalf of guest
>>>> VMs? I also looked into IRQ and MSI solutions inside Xen. But most of
>>>> them assume that interrupts are from physical device (but not in this
>>>> LWP case, where interrupt is initiated from CPU itself); so they don't
>>>> fit very well.
>>> I think just allocating a vector is fine. If we get too many we could move
>>> to
>>> dynamic allocation of them.
>>>
>>> -- Keir
>>>
>>>> Thanks,
>>>> -Wei
>>>>
>>>>
>>>
>>>
>>> _______________________________________________
>>> Xen-devel mailing list
>>> Xen-devel@lists.xen.org
>>> http://lists.xen.org/xen-devel
>
next prev parent reply other threads:[~2012-03-30 8:21 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-23 22:03 LWP Interrupt Handler Wei Huang
2012-03-23 22:49 ` Keir Fraser
2012-03-26 0:08 ` Zhang, Xiantao
2012-03-29 22:54 ` Wei Huang
2012-03-30 8:21 ` Keir Fraser [this message]
2012-03-30 19:44 ` Wei Huang
2012-03-30 20:06 ` Keir Fraser
2012-03-30 20:11 ` Wei Huang
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=CB9B2B81.3D111%keir@xen.org \
--to=keir@xen.org \
--cc=JBeulich@suse.com \
--cc=wei.huang2@amd.com \
--cc=xen-devel@lists.xen.org \
--cc=xiantao.zhang@intel.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).