All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jan Beulich" <JBeulich@suse.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Paul Durrant <Paul.Durrant@citrix.com>,
	Anthony PERARD <anthony.perard@citrix.com>,
	Xen-devel <xen-devel@lists.xenproject.org>
Subject: Re: Stabilising some tools only HVMOPs?
Date: Thu, 18 Feb 2016 03:56:20 -0700	[thread overview]
Message-ID: <56C5B16402000078000D3A0D@prv-mh.provo.novell.com> (raw)
In-Reply-To: <20160218104526.GD3723@citrix.com>

>>> On 18.02.16 at 11:45, <wei.liu2@citrix.com> wrote:
> On Thu, Feb 18, 2016 at 03:37:06AM -0700, Jan Beulich wrote:
>> >>> On 18.02.16 at 11:24, <ian.campbell@citrix.com> wrote:
>> > On Wed, 2016-02-17 at 17:28 +0000, Wei Liu wrote:
>> >> The list of tools only HVMOPs used by QEMU are:
>> >> 
>> >>   #define HVMOP_track_dirty_vram    6
>> >>   #define HVMOP_modified_memory    7
>> >>   #define HVMOP_set_mem_type    8
>> >>   #define HVMOP_inject_msi         16
>> >>   #define HVMOP_create_ioreq_server 17
>> >>   #define HVMOP_get_ioreq_server_info 18
>> >>   #define HVMOP_map_io_range_to_ioreq_server 19
>> >>   #define HVMOP_unmap_io_range_from_ioreq_server 20
>> >>   #define HVMOP_destroy_ioreq_server 21
>> >>   #define HVMOP_set_ioreq_server_state 22
>> >> 
>> >> I'm curious about the rationale for making them tools only in the
>> >> first place and what needs to be done to make them stable.
>> > 
>> > FWIW (IMHO, YMMV etc) it is becoming increasing incorrect to consider the
>> > device model as "tools" in the face of disaggregation and support for
>> > (nearly) arbitrary upstream QEMU versions etc.
>> 
>> As just written in the other reply, it depends on what exactly
>> qemu uses: libxc interfaces are fine, since the "tools only"
>> aspect in the public headers is mainly to allow us to alter
>> structure layouts and alike. The "tools only" aspect there in
>> particular is not to preclude entities like qemu (indirectly)
>> invoking such operations - that's instead being dealt with by
>> permission checks.
>> 
>> I.e. as long a qemu doesn't define __XEN_TOOLS__ for its
>> building, I think we're fine.
>> 
> 
> OK, so you're suggesting building stable APIs on top of unstable ones.

No, sorry, I'm not, after having read your and Ian's earlier replies.

Jan

  parent reply	other threads:[~2016-02-18 10:56 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-17 17:28 Stabilising some tools only HVMOPs? Wei Liu
2016-02-18 10:24 ` Ian Campbell
2016-02-18 10:37   ` Jan Beulich
2016-02-18 10:45     ` Wei Liu
2016-02-18 10:53       ` Ian Campbell
2016-02-18 10:55         ` Wei Liu
2016-02-18 10:56       ` Jan Beulich [this message]
2016-02-18 10:31 ` Jan Beulich
2016-02-18 10:36   ` Wei Liu
2016-02-18 10:44   ` Ian Campbell
2016-02-18 10:55     ` Jan Beulich
2016-02-18 10:59       ` Wei Liu
2016-02-18 11:04         ` Jan Beulich
2016-02-18 12:51 ` Wei Liu
2016-02-18 16:28   ` Ian Jackson
2016-02-18 16:29     ` Wei Liu
2016-02-18 16:41     ` Jan Beulich
2016-02-18 16:45       ` Ian Jackson
2016-02-18 16:49       ` Wei Liu
2016-02-18 16:37   ` Ian Campbell
2016-02-19 16:05 ` Domctl and physdevop for passthrough (Was: Re: Stabilising some tools only HVMOPs?) Wei Liu
2016-02-22 11:28   ` Jan Beulich
2016-02-22 11:56     ` Wei Liu
2016-02-23 14:31     ` Wei Liu
2016-02-23 15:46       ` Jan Beulich
2016-02-23 17:09         ` Wei Liu
2016-02-23 17:24           ` Jan Beulich
2016-02-23 17:28             ` Jan Beulich
2016-02-23 17:55             ` Wei Liu
2016-02-29 12:23       ` Wei Liu
2016-02-29 12:29         ` Jan Beulich
2016-02-29 18:12           ` Wei Liu
2016-03-01  7:54             ` Jan Beulich
2016-03-01 10:52               ` Wei Liu
2016-03-01 11:10                 ` Jan Beulich

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=56C5B16402000078000D3A0D@prv-mh.provo.novell.com \
    --to=jbeulich@suse.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=Paul.Durrant@citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=anthony.perard@citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=wei.liu2@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.