[PATCH 0/6] remove libxenctrl usage from xenstored

Juergen Gross posted 6 patches 1 month, 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20241023131005.32144-1-jgross@suse.com
stubdom/Makefile                    |   8 +-
stubdom/mini-os.mk                  |   1 +
tools/9pfsd/io.c                    |   5 +-
tools/flask/policy/modules/dom0.te  |   2 +-
tools/include/xen-barrier.h         |  51 +++++++++
tools/include/xenctrl.h             |  28 +----
tools/include/xenmanage.h           |  98 ++++++++++++++++
tools/libs/Makefile                 |   1 +
tools/libs/ctrl/Makefile            |   2 +-
tools/libs/manage/Makefile          |  10 ++
tools/libs/manage/Makefile.common   |   1 +
tools/libs/manage/core.c            | 170 ++++++++++++++++++++++++++++
tools/libs/manage/libxenmanage.map  |   8 ++
tools/libs/uselibs.mk               |   2 +
tools/xenstored/Makefile            |   2 +-
tools/xenstored/Makefile.common     |   2 +-
tools/xenstored/core.h              |   1 -
tools/xenstored/domain.c            |  52 ++++-----
tools/xenstored/lu.c                |   1 +
tools/xenstored/lu_daemon.c         |   1 +
xen/common/domain.c                 |  92 +++++++++++++++
xen/common/domctl.c                 |  19 +++-
xen/common/event_channel.c          |  11 +-
xen/include/public/domctl.h         |  33 ++++++
xen/include/xen/event.h             |   6 +
xen/include/xen/sched.h             |   7 ++
xen/include/xsm/dummy.h             |   8 ++
xen/include/xsm/xsm.h               |   6 +
xen/xsm/dummy.c                     |   1 +
xen/xsm/flask/hooks.c               |   7 ++
xen/xsm/flask/policy/access_vectors |   2 +
31 files changed, 566 insertions(+), 72 deletions(-)
create mode 100644 tools/include/xen-barrier.h
create mode 100644 tools/include/xenmanage.h
create mode 100644 tools/libs/manage/Makefile
create mode 100644 tools/libs/manage/Makefile.common
create mode 100644 tools/libs/manage/core.c
create mode 100644 tools/libs/manage/libxenmanage.map
[PATCH 0/6] remove libxenctrl usage from xenstored
Posted by Juergen Gross 1 month, 1 week ago
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.

- By moving the CPU barrier definitions out of xenctrl.h into a new
  dedicated header xenstored code no longer needs to #include xenctrl.h,
  thus removing any xenctrl reference from xenstored code.

Please note that the last patch 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!

Juergen Gross (6):
  xen: add a domain unique id to each domain
  xen: add bitmap to indicate per-domain state changes
  xen: add new domctl get_changed_domain
  tools/libs: add a new libxenmanage library
  tools: add a dedicated header file for barrier definitions
  tools/xenstored: use new stable interface instead of libxenctrl

 stubdom/Makefile                    |   8 +-
 stubdom/mini-os.mk                  |   1 +
 tools/9pfsd/io.c                    |   5 +-
 tools/flask/policy/modules/dom0.te  |   2 +-
 tools/include/xen-barrier.h         |  51 +++++++++
 tools/include/xenctrl.h             |  28 +----
 tools/include/xenmanage.h           |  98 ++++++++++++++++
 tools/libs/Makefile                 |   1 +
 tools/libs/ctrl/Makefile            |   2 +-
 tools/libs/manage/Makefile          |  10 ++
 tools/libs/manage/Makefile.common   |   1 +
 tools/libs/manage/core.c            | 170 ++++++++++++++++++++++++++++
 tools/libs/manage/libxenmanage.map  |   8 ++
 tools/libs/uselibs.mk               |   2 +
 tools/xenstored/Makefile            |   2 +-
 tools/xenstored/Makefile.common     |   2 +-
 tools/xenstored/core.h              |   1 -
 tools/xenstored/domain.c            |  52 ++++-----
 tools/xenstored/lu.c                |   1 +
 tools/xenstored/lu_daemon.c         |   1 +
 xen/common/domain.c                 |  92 +++++++++++++++
 xen/common/domctl.c                 |  19 +++-
 xen/common/event_channel.c          |  11 +-
 xen/include/public/domctl.h         |  33 ++++++
 xen/include/xen/event.h             |   6 +
 xen/include/xen/sched.h             |   7 ++
 xen/include/xsm/dummy.h             |   8 ++
 xen/include/xsm/xsm.h               |   6 +
 xen/xsm/dummy.c                     |   1 +
 xen/xsm/flask/hooks.c               |   7 ++
 xen/xsm/flask/policy/access_vectors |   2 +
 31 files changed, 566 insertions(+), 72 deletions(-)
 create mode 100644 tools/include/xen-barrier.h
 create mode 100644 tools/include/xenmanage.h
 create mode 100644 tools/libs/manage/Makefile
 create mode 100644 tools/libs/manage/Makefile.common
 create mode 100644 tools/libs/manage/core.c
 create mode 100644 tools/libs/manage/libxenmanage.map

-- 
2.43.0