All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juergen Gross <jgross@suse.com>
To: xen-devel@lists.xenproject.org
Cc: "Andrew Cooper" <andrew.cooper3@citrix.com>,
	"Anthony PERARD" <anthony.perard@vates.tech>,
	"Michal Orzel" <michal.orzel@amd.com>,
	"Jan Beulich" <jbeulich@suse.com>,
	"Julien Grall" <julien@xen.org>,
	"Roger Pau Monné" <roger.pau@citrix.com>,
	"Stefano Stabellini" <sstabellini@kernel.org>,
	"Daniel P. Smith" <dpsmith@apertussolutions.com>,
	"Samuel Thibault" <samuel.thibault@ens-lyon.org>
Subject: Re: [PATCH v8 0/9] remove libxenctrl usage from xenstored
Date: Tue, 25 Feb 2025 12:10:06 +0100	[thread overview]
Message-ID: <6e63a858-9554-440b-92f1-55dc34256e0b@suse.com> (raw)
In-Reply-To: <20250204113407.16839-1-jgross@suse.com>


[-- Attachment #1.1.1: Type: text/plain, Size: 1845 bytes --]

Ping? Especially ...

On 04.02.25 12:33, Juergen Gross wrote:
> Xenstored is using libxenctrl for only one purpose: to get information
> about state of domains.
> 
> This patch series is removing that dependency by introducing a new
> stable interface which can be used by xenstored instead.
> 
> There was a RFC series sent out 3 years ago, which I have taken as a
> base and by addressing all comments from back then.
> 
> The main differences since that RFC series are:
> 
> - Instead of introducing an new main hypercall for a stable management
>    interface I have just added a new domctl sub-op, as requested in 2021.
> 
> - I have added a new library libxenmanage for easy use of the new
>    stable hypervisor interface. Main motivation for adding the library
>    was the recent attempt to decouple oxenstored from the Xen git tree.
>    By using the new library, oxenstored could benefit in the same way as
>    xenstored from the new interface: it would be possible to rely on
>    stable libraries only.
> 
> - Mini-OS has gained some more config options recently, so it was rather
>    easy to make xenstore[pvh]-stubdom independent from libxenctrl, too.
> 
> Please note that the last 4 patches can be committed only after the
> related Mini-OS patch "config: add support for libxenmanage" has gone in
> AND the Mini-OS commit-id has been updated in Config.mk accordingly! The
> Mini-OS patch has been Acked already, so it can go in as soon as patch
> 5 of this series (the one introducing libxenmanage) has been committed.
> 
> As patches 1 and 2 change current behavior, Jan didn't want to give his
> Ack (he didn't reject the patch either). So I'm asking other "Rest"
> maintainers to look at those patches specifically.

... patches 1 and 2 could need an additional opinion.


Juergen

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3743 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

  parent reply	other threads:[~2025-02-25 11:10 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-04 11:33 [PATCH v8 0/9] remove libxenctrl usage from xenstored Juergen Gross
2025-02-04 11:33 ` [PATCH v8 1/9] xen/events: don't allow binding a global virq from any domain Juergen Gross
2025-03-11  9:35   ` Julien Grall
2025-03-11  9:51     ` Jürgen Groß
2025-03-11 19:04       ` Julien Grall
2025-03-12  7:05         ` Jürgen Groß
2025-02-04 11:34 ` [PATCH v8 2/9] xen/events: allow setting of global virq handler only for unbound virqs Juergen Gross
2025-02-04 11:34 ` [PATCH v8 3/9] xen: add bitmap to indicate per-domain state changes Juergen Gross
2025-03-06 15:41   ` Jan Beulich
2025-03-06 15:52     ` Jürgen Groß
2025-02-04 11:34 ` [PATCH v8 4/9] xen: add new domctl get_changed_domain Juergen Gross
2025-02-04 11:34 ` [PATCH v8 5/9] tools/libs: add a new libxenmanage library Juergen Gross
2025-02-04 11:34 ` [PATCH v8 6/9] tools/xenstored: use new stable interface instead of libxenctrl Juergen Gross
2025-02-04 11:34 ` [PATCH v8 7/9] docs: update xenstore migration stream definition Juergen Gross
2025-03-11  9:43   ` Julien Grall
2025-03-11  9:58     ` Jürgen Groß
2025-03-11 18:52       ` Julien Grall
2025-02-04 11:34 ` [PATCH v8 8/9] tools/xenstored: use unique_id to identify new domain with same domid Juergen Gross
2025-03-12 17:40   ` Jason Andryuk
2025-03-13  9:52     ` Jürgen Groß
2025-02-04 11:34 ` [PATCH v8 9/9] tools/xenstored: use xenmanage_poll_changed_domain() Juergen Gross
2025-03-12 17:45   ` Jason Andryuk
2025-03-13  9:53     ` Jürgen Groß
2025-02-25 11:10 ` Juergen Gross [this message]
2025-03-05  8:23   ` [PATCH v8 0/9] remove libxenctrl usage from xenstored Juergen Gross
2025-03-05 23:32     ` Stefano Stabellini
2025-03-06 13:13       ` Jan Beulich
2025-03-06 13:27         ` Jürgen Groß
2025-03-06 13:53           ` Jan Beulich
2025-03-06 13:56             ` Jürgen Groß

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=6e63a858-9554-440b-92f1-55dc34256e0b@suse.com \
    --to=jgross@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=anthony.perard@vates.tech \
    --cc=dpsmith@apertussolutions.com \
    --cc=jbeulich@suse.com \
    --cc=julien@xen.org \
    --cc=michal.orzel@amd.com \
    --cc=roger.pau@citrix.com \
    --cc=samuel.thibault@ens-lyon.org \
    --cc=sstabellini@kernel.org \
    --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.