All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keir Fraser <keir.xen@gmail.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	David Vrabel <david.vrabel@citrix.com>,
	Jan Beulich <JBeulich@suse.com>,
	Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [RFC PATCH V3] Implement 3-level event channel in Xen
Date: Thu, 28 Feb 2013 05:58:39 +0000	[thread overview]
Message-ID: <CD54A28F.4CD9C%keir.xen@gmail.com> (raw)
In-Reply-To: <20130227231927.GA13549@zion.uk.xensource.com>

On 27/02/2013 23:19, "Wei Liu" <wei.liu2@citrix.com> wrote:

> On Wed, Feb 27, 2013 at 07:49:34PM +0000, Keir Fraser wrote:
>> On 27/02/2013 17:01, "Jan Beulich" <JBeulich@suse.com> wrote:
>> 
>>>>>> On 27.02.13 at 15:33, Wei Liu <wei.liu2@citrix.com> wrote:
>>>> Keir, Jan, Ian and David, are you happy with this design in general? I
>>>> would
>>>> like to have explicit ACK / NACK on this if possible, as feature freeze for
>>>> 4.3 is quite close now.
>>> 
>>> The patches look reasonable (apart from the comments I just gave
>>> on some of them), but other than Keir I'm not that eager to see this
>>> one go in for 4.3 in order to then likely be replaced by an
>>> implementation of David's design in the 4.4 cycle.
>> 
>> If this went in for 4.3, a re-design would really have to be implemented and
>> measurably prove its worth to make it in as another replacement.
> 
> Even this design does not go in for 4.3, any new design will still have
> to be implemented and measurable prove its worth, as with this design.

I believe there's a requirement to solve the dom0 event-channel limit for
4.3. Either of the proposed designs obviously solves that, without need for
implement and test. Of course, we need to do that to test for correctness
and performance regressions anyway. But I mean with that primary requirement
satisfied, yet another evtchn ABI really then has to have some compelling
advantages to get committed.

 -- Keir

> The value of this design lies in that a) it's straight forward, easy to
> prove its correctness (or wrongness), b) it meets our need for now, c)
> a new interface will always be necessary, be it this design or any other
> design, d) it buys time for any better designs to become mature.
> 
> I understand your (and Jan's) concern for burdens of maintaining
> different ABIs, that's why the interface has been made to allow user to
> query enabled / supported ABIs. If we need to add / remove ABIs, it's
> just a matter of setting some feature bits.
> 
> I don't mean to push this design to get merged, it's up to maintainers
> to decide. Actually I'm fine with any decision. Explicit Ack / Nack will
> help me arrange my future work better.
> 
> 
> Wei.
> 
>>  -- Keir
>> 
>>> Jan
>>> 
>> 
>> 

  reply	other threads:[~2013-02-28  5:58 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-27 14:33 [RFC PATCH V3] Implement 3-level event channel in Xen Wei Liu
2013-02-27 14:33 ` [RFC PATCH V3 01/22] Clean up trailing whitespaces Wei Liu
2013-02-28 11:30   ` David Vrabel
2013-02-28 11:36     ` Wei Liu
2013-02-28 11:38     ` Jan Beulich
2013-02-28 12:14       ` Ian Campbell
2013-02-28 12:29         ` Wei Liu
2013-02-28 13:08         ` Jan Beulich
2013-02-27 14:33 ` [RFC PATCH V3 02/22] Dynamically allocate d->evtchn Wei Liu
2013-02-27 16:35   ` Jan Beulich
2013-02-27 14:33 ` [RFC PATCH V3 03/22] Move event channel macros / struct definition to proper place Wei Liu
2013-02-27 14:33 ` [RFC PATCH V3 04/22] flask: include xen/event.h Wei Liu
2013-02-28 11:20   ` David Vrabel
2013-02-28 11:22     ` Wei Liu
2013-02-27 14:33 ` [RFC PATCH V3 05/22] Change MAX_EVTCHNS macro to max_evtchns inline function Wei Liu
2013-02-28 11:58   ` David Vrabel
2013-02-28 13:59     ` Wei Liu
2013-02-27 14:33 ` [RFC PATCH V3 06/22] Define extended event channel registration interface Wei Liu
2013-02-27 16:42   ` Jan Beulich
2013-02-28 11:25     ` Wei Liu
2013-02-28 11:32       ` Jan Beulich
2013-02-28 11:41         ` Wei Liu
2013-02-28 12:32   ` David Vrabel
2013-02-28 15:04     ` Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 07/22] Add evtchn_extended in struct domain Wei Liu
2013-02-28 11:55   ` David Vrabel
2013-02-27 14:34 ` [RFC PATCH V3 08/22] Calculate max event channels for EVTCHN_EXTENDED_L3 Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 09/22] Bump EVTCHNS_PER_BUCKET to 512 Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 10/22] Add evtchn_is_{pending, masked} and evtchn_clear_pending Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 11/22] Update Xen public header Wei Liu
2013-02-28 12:00   ` David Vrabel
2013-02-27 14:34 ` [RFC PATCH V3 12/22] Add supported extended event channel ABI bitmap Wei Liu
2013-02-27 16:47   ` Jan Beulich
2013-02-28 11:21     ` Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 13/22] Add evtchn_abi_str Wei Liu
2013-02-27 16:51   ` Jan Beulich
2013-02-28 11:28     ` Wei Liu
2013-02-28 11:33       ` Jan Beulich
2013-02-27 14:34 ` [RFC PATCH V3 14/22] Add control structures for 3-level event channel Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 15/22] Genneralized event channel operations Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 16/22] Introduce some macros for event channels Wei Liu
2013-02-27 16:53   ` Jan Beulich
2013-02-27 17:04     ` Ian Campbell
2013-02-28  7:54       ` Jan Beulich
2013-02-28  8:35         ` Ian Campbell
2013-02-28 11:17         ` Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 17/22] Infrastructure to manipulate 3-level event channel pages Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 18/22] Implement EVTCHNOP_register_extended Wei Liu
2013-02-28 12:33   ` David Vrabel
2013-02-27 14:34 ` [RFC PATCH V3 19/22] Enable exteneded event channel ABI query Wei Liu
2013-02-28 12:36   ` David Vrabel
2013-02-27 14:34 ` [RFC PATCH V3 20/22] Implement 3-level event channel routines Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 21/22] Only allow extended event channel on Dom0 and driver domains Wei Liu
2013-02-27 16:58   ` Jan Beulich
2013-02-28 11:19     ` Wei Liu
2013-02-28 12:43   ` David Vrabel
2013-02-28 19:29     ` Wei Liu
2013-02-27 14:34 ` [RFC PATCH V3 22/22] libxl: add evtchn_extended flag Wei Liu
2013-02-28 12:48   ` David Vrabel
2013-03-01 11:55   ` Ian Jackson
2013-02-27 16:28 ` [RFC PATCH V3] Implement 3-level event channel in Xen Keir Fraser
2013-02-27 17:01 ` Jan Beulich
2013-02-27 19:49   ` Keir Fraser
2013-02-27 23:19     ` Wei Liu
2013-02-28  5:58       ` Keir Fraser [this message]
2013-02-28  7:23     ` Jan Beulich
2013-03-01 12:00       ` Ian Jackson

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=CD54A28F.4CD9C%keir.xen@gmail.com \
    --to=keir.xen@gmail.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=david.vrabel@citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.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.