[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH v2 0/4] tools/xenstore: add some new features to the documentation


  • To: Juergen Gross <jgross@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Henry Wang <Henry.Wang@xxxxxxx>
  • Date: Fri, 17 Jun 2022 03:24:38 +0000
  • Accept-language: zh-CN, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=8VREWxiJvUhFIjwf3qdTHKNTciRIEdi1fyoHyo3QY8o=; b=RSP9ysFr/VMS7hnK4HDKUwQUhhXBB/D8YawDlHh/9JOEPj20mMr9KWG+PgxCC3BoZURGV9XX+iEAfMTqGL5rz2aSY5vnjpVwxwrpgIEMTgdpuGR/WbE4tge+sSqXwrF0m2hFUjvsDlELG9swk9LvghDK0wgYG2Wx9JoIljDPhM68T/+1d+igIfbEoDTUJxnnZR1QH22PN/FwSK3jaDevRRjP0wMA2alRKE0bBTTS3Zw19ZpTxFtQEFDbIEm04Nb4Wte6dF0Gk/RdqlsKPhMKnKSCxDae88HoJfYSuHyK3lh7kq4lpHS7mEIaUHQuPHAEvywdz+sutTQoHoV0qH8OYw==
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=8VREWxiJvUhFIjwf3qdTHKNTciRIEdi1fyoHyo3QY8o=; b=ncR0StoTqXPR6ZPjVojCwnDCpaGpJ04fL+XDNuBz8DgC5Dn5mOPgZqG0yz5dUvMzfx83DQG3MXtH1zEfw8Id3tbkpIgnZiE2rOJtYMOdJ0qEd3mNhlCXtkFEKPEWRTkk7YJjkDMvkgSa8lwVsB+etpcSat0q3LtHytlNSDWKChmalAEFb2/ql+QTmyh3IKMEEtcOoUxitaniWV4u9KPq3jWjlI/T+HGxYGVRIiiJsheNKx1esv4MmTEhtTOqijvDq8Fjhdr507yrNDCJ0vRHLKQm4AaoTtnCjCgCBBmym90XHpKbrDqAaf+tlvxrUdqt/wsoQgfwY+l4P7QvbOcCBg==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=nHKJV0E5fUB0OlVlv9AHd5ppMBau1uBX/UquojaSpaNIloTcNF6FlmSbPG+SrtZiCnxbExInwPnn2Paw/hyiDLzMn6AhiDD4dEMhQgzmm6X8mKfBxRoSHbR5ZBq9MQSzOkotIQsZEZ7V2qj90R9OZvP+rtNOhLdTza30tw1ZXOXvKuqSdHZ8CtEMdFl3tqkntnIYj/DCGXsXCar2v67G1lBCukfVbW8Q5xWA6RT+zsioE/u1lyPKXZXjN5hQd5VXAf0onXq/i5PhUcCj8phi6QaKgrXnPIcDHKAgCTVQaWEBiVZMgwUxdBFsSP4paqBggmkuT/o2Q9vE3BphDIjA8w==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Wdg9Qnc6hC//Bpj+o7MngaTbsR0Z/Hh7fws5VW6QNk1dQd1uxgkOmAV5XNvoRy3Hg32f5KnvIqYBWZHmSn3iuxHt+XnhVcKZ7+ZwuvEutQWYGLDy1ag6Y0m5bV+/as4kZajnCZV/ktdUP7XYRReuAEo3rbDdtxGO+McIcPYr9JEHSh20ClPgmgbMryp9BKCZu+KY7McybGTukzmxN7Jbjehnijk+I23zJ+gVQOpRlRZWrQeLWPlh55VDC1m+ZKmQevRVa/v4uOdrL1kxwoCywHZ/fMiW2/Xg5PyABnyJpIxiDl6a/pKny7WLlEMrlr/6gZzyHEBr3H6OqYj7z+5hsg==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 17 Jun 2022 03:25:21 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHYcZr+xugQmCaBT0SxSahY0q90Lq1TDssw
  • Thread-topic: [PATCH v2 0/4] tools/xenstore: add some new features to the documentation

Hi,

It seems that this series [1] has been stale for a while with actions needed 
from
the maintainers (review needed). So sending this email as a gentle reminder.
Thanks!

[1] https://patchwork.kernel.org/project/xen-devel/list/?series=645480

Kind regards,
Henry

> -----Original Message-----
> From: Xen-devel <xen-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On Behalf Of
> Juergen Gross
> Subject: [PATCH v2 0/4] tools/xenstore: add some new features to the
> documentation
> 
> In the past there have been spotted some shortcomings in the Xenstore
> interface, which should be repaired. Those are in detail:
> 
> - Using driver domains for large number of domains needs per domain
>   Xenstore quota [1]. The feedback sent was rather slim (one reply),
>   but it was preferring a new set of wire commands.
> 
> - XSA-349 [2] has shown that the current definition of watches is not
>   optimal, as it will trigger lots of events when a single one would
>   suffice: for detecting new backend devices the backends in the Linux
>   kernel are registering a watch for e.g. "/local/domain/0/backend"
>   which will fire for ANY sub-node written below this node (on a test
>   machine this added up to 91 watch events for only 3 devices).
>   This can be limited dramatically by extending the XS_WATCH command
>   to take another optional parameter specifying the depth of
>   subdirectories to be considered for sending watch events ("0" would
>   trigger a watch event only if the watched node itself being written).
> 
> - New features like above being added might make migration of guests
>   between hosts with different Xenstore variants harder, so it should
>   be possible to set the available feature set per domain. For socket
>   connections it should be possible to read the available features.
> 
> - The special watches @introduceDomain and @releaseDomain are rather
>   cumbersome to use, as they only tell you that SOME domain has been
>   introduced/released. Any consumer of those watches needs to scan
>   all domains on the host in order to find out the domid, causing
>   significant pressure on the dominfo hypercall (imagine a system
>   with 1000 domains running and one domain dying - there will be more
>   than 1000 watch events triggered and 1000 xl daemons will try to
>   find out whether "their" domain has died). Those watches should be
>   enhanced to optionally be specific to a single domain and to let the
>   event carry the related domid.
> 
> As some of those extensions will need to be considered in the Xenstore
> migration stream, they should be defined in one go (in fact the 4th one
> wouldn't need that, but it can easily be connected to the 2nd one).
> As such extensions need to be flagged in the "features" in the ring
> page anyway, it is fine to implement them independently.
> 
> Add the documentation of the new commands/features.
> 
> [1]: https://lists.xen.org/archives/html/xen-devel/2020-06/msg00291.html
> [2]: http://xenbits.xen.org/xsa/advisory-349.html
> 
> Changes in V2:
> - added new patch 1
> - remove feature bits for dom0-only features
> - get-features without domid returns Xenstore supported features
> - get/set-quota without domid for global quota access
> 
> Juergen Gross (4):
>   tools/xenstore: modify feature bit specification in xenstore-ring.txt
>   tools/xenstore: add documentation for new set/get-feature commands
>   tools/xenstore: add documentation for new set/get-quota commands
>   tools/xenstore: add documentation for extended watch command
> 
>  docs/misc/xenstore-ring.txt | 10 ++++----
>  docs/misc/xenstore.txt      | 47 ++++++++++++++++++++++++++++++++++---
>  2 files changed, 50 insertions(+), 7 deletions(-)
> 
> --
> 2.35.3
> 




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.