Skip to content

Add "xl info" command

Andre Przywara xen-devel 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest4 patches 7 repliesIn review
  2. v14 patches 14 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4libxl: extend physinfo structuretools/libxl/libxl.c tools/libxl/libxl.h+8 -04 replies
2/4libxl: add sched_get_id functiontools/libxl/libxl.c tools/libxl/libxl.h+15 -00 replies
3/4libxl: add version_info functiontools/libxl/libxl.c tools/libxl/libxl.h+64 -02 replies
3/4libxl: add version_info functiontools/libxl/libxl.c tools/libxl/libxl.h+64 -00 replies
4/4xl: add "xl info" commandtools/libxl/xl.c+114 -00 replies

Reviews trailers per patch

None of the 5 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchVincent Hanquez
SeriesA-b
1/4
2/4
3/4
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCHv2 0/4] Add "xl info" command

The "info" subcommand is missing from the xl tool.
The attached patchset adds support for this, extending libxl on the way to provide the necessary info only by using own functions. On my system the output of xm info and xl info was identical, I omitted the recent NUMA additions from xl info for now and will provide the necessary patches later.

Reworked version due to comments from Ian and Vincent (thanks for the review!) Changes to version 1:
- use Xen public headers for constants instead of (re-)defining them
- comment the meaning of the return value of get_sched_id()
- remove the masking feature for get_version_info
- make the version_info structure a member of libxl_ctx and fill it on demand, omitting the need to free it manually

Please apply!

Regards,
Andre.

Signature · 3 lines
Andre Przywara
AMD-Operating System Research Center (OSRC), Dresden, Germany
Tel: +49 351 488-3567-12