From: "Jürgen Groß" <jgross@suse.com>
To: Jason Andryuk <jason.andryuk@amd.com>, xen-devel@lists.xenproject.org
Cc: Julien Grall <julien@xen.org>,
Anthony PERARD <anthony.perard@vates.tech>
Subject: Re: [PATCH v8 9/9] tools/xenstored: use xenmanage_poll_changed_domain()
Date: Thu, 13 Mar 2025 10:53:01 +0100 [thread overview]
Message-ID: <68390678-c278-492c-9a2b-7248a91deddc@suse.com> (raw)
In-Reply-To: <11885cfa-42be-42d2-821c-ebb8db304340@amd.com>
[-- Attachment #1.1.1: Type: text/plain, Size: 686 bytes --]
On 12.03.25 18:45, Jason Andryuk wrote:
> On 2025-02-04 06:34, Juergen Gross wrote:
>> Instead of checking each known domain after having received a
>> VIRQ_DOM_EXC event, use the new xenmanage_poll_changed_domain()
>> function for directly getting the domid of a domain having changed
>> its state.
>>
>> A test doing "xl shutdown" of 1000 guests has shown to reduce the
>> consumed cpu time of xenstored by 6% with this change applied.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>
> Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>
>
> I see at least some dom_invalid uses goes away, so maybe don't bother changing it.
I'll do the change.
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 --]
next prev parent reply other threads:[~2025-03-13 9:53 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ß [this message]
2025-02-25 11:10 ` [PATCH v8 0/9] remove libxenctrl usage from xenstored Juergen Gross
2025-03-05 8:23 ` 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=68390678-c278-492c-9a2b-7248a91deddc@suse.com \
--to=jgross@suse.com \
--cc=anthony.perard@vates.tech \
--cc=jason.andryuk@amd.com \
--cc=julien@xen.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.