xen-devel
Re: [Xen-devel] [PATCH 7/7] xen: Enable event channel of PV extension of
To: |
Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, Jeremy Fitzhardinge <Jeremy.Fitzhardinge@xxxxxxxxxx> |
Subject: |
Re: [Xen-devel] [PATCH 7/7] xen: Enable event channel of PV extension of HVM |
From: |
Sheng Yang <sheng@xxxxxxxxxxxxxxx> |
Date: |
Wed, 10 Mar 2010 10:56:42 +0800 |
Cc: |
Jeremy Fitzhardinge <jeremy@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Ian Pratt <Ian.Pratt@xxxxxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxx>, Konrad Rzeszutek Wilk <kliw@xxxxxxxxxx> |
Delivery-date: |
Tue, 09 Mar 2010 18:56:05 -0800 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<alpine.DEB.2.00.1003090954010.4869@kaball-desktop> |
List-help: |
<mailto:xen-devel-request@lists.xensource.com?subject=help> |
List-id: |
Xen developer discussion <xen-devel.lists.xensource.com> |
List-post: |
<mailto:xen-devel@lists.xensource.com> |
List-subscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
List-unsubscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
Organization: |
Intel Opensource Technology Center |
References: |
<1268032732-8025-1-git-send-email-sheng@xxxxxxxxxxxxxxx> <1268128085.11737.59761.camel@xxxxxxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1003090954010.4869@kaball-desktop> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
User-agent: |
KMail/1.12.2 (Linux/2.6.31-19-generic; KDE/4.3.2; x86_64; ; ) |
On Tuesday 09 March 2010 18:22:31 Stefano Stabellini wrote:
> On Tue, 9 Mar 2010, Ian Campbell wrote:
> > On Tue, 2010-03-09 at 07:00 +0000, Jeremy Fitzhardinge wrote:
> > > On 03/08/2010 05:53 PM, Sheng Yang wrote:
> > > > On Tuesday 09 March 2010 01:10:56 Stefano Stabellini wrote:
> > > >> I think that mapping interrupts into VIRQs is a bad idea: you should
> > > >> map interrupts into pirqs instead, the code exists already on the
> > > >> kernel side so we don't need to do any (ugly) change ther.
> > > >
> > > > The code existed in the pv_ops dom0 side, but not in the upstream
> > > > Linux. The latter is our target. We want this work to be accepted by
> > > > upstream Linux soon.
> > >
> > > 2. It has significant overlaps with the current xen.git development
> > > which is also targeted for upstream. There's no point in
> > > creating an unnecessary duplicate mechanism when the infrastructure
> > > will be in place anyway.
> >
> > There's also nothing stopping us from upstreaming portions of the "dom0"
> > patchset ahead of full dom0 support if it is useful for some domU
> > feature.
>
> Indeed.
> You just need the pirq mappings and few other things, it shouldn't be
> too hard.
> At this point I am going to do that myself after this patch series is
> completed.
I think we can leave the controversial thing later. At least, we want a
framework for PV extension of HVM. We can work together to determine what is
the better way for evtchn, as well as porting pirqs. (And the later MSI work
may also depends on it)
But I think the former 6 patches can be taken as the base. It won't overlapped
with others. And PV clocksource is definitely important for HVM guest. We
would like to get the first 6 ones checked in Linux upstream first, then we
can work on others. And it would the work easier for us later.
So Jeremy, how about a even more simplified version only contained framework
and pv clocksource? I think it's pretty elegant and solid first step into
Linux upstream.
If you agree, I would send the updated patchset soon, only framework and pv
clocksource.
--
regards
Yang, Sheng
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|