This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH 7/7] xen: Enable event channel of PV extension of

To: Sheng Yang <sheng@xxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 7/7] xen: Enable event channel of PV extension of HVM
From: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Date: Mon, 08 Mar 2010 23:00:52 -0800
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, Ian Pratt <Ian.Pratt@xxxxxxxxxxxxx>, Jeremy Fitzhardinge <Jeremy.Fitzhardinge@xxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxx>, Konrad Rzeszutek Wilk <kliw@xxxxxxxxxx>
Delivery-date: Mon, 08 Mar 2010 23:01:36 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <201003090953.16985.sheng@xxxxxxxxxxxxxxx>
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>
References: <1268032732-8025-1-git-send-email-sheng@xxxxxxxxxxxxxxx> <1268032732-8025-8-git-send-email-sheng@xxxxxxxxxxxxxxx> <alpine.DEB.2.00.1003081701160.16237@kaball-desktop> <201003090953.16985.sheng@xxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20100301 Fedora/3.0.3-1.fc12 Lightning/1.0b2pre Thunderbird/3.0.3
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.

I don't think its a sound idea to rush this stuff upstream, for several reasons:

  1. It is currently based on 2.6.33, which means it will not get into
     any distros in the near future.  If it were based on 2.6.32 then
it has a much greater chance of being accepted into distros. (They may decide to backport the work, but it would be helpful if
     that were done in advance.)
  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.
  3. The code has had very little exposure within the Xen community,
     especially since the Xen-side patches have not been accepted into
     xen-unstable yet, let alone a released hypervisor.  On the kernel
     side, it would help if the patches were based on 2.6.32 (or even
     .31) so they can be merged into the xen.git kernels people are
     actually using.
  4. The kernel Xen code is already complicated with a number of
     different code paths according to architecture, options, features
     etc.  I would like to see some strong concrete evidence that these
     changes make a worthwhile improvement on a real workload before
     adding yet another thing which needs to be tested and maintained


Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>