All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Lindig <christian.lindig@citrix.com>
To: Juergen Gross <jgross@suse.com>,
	"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
	"xen-devel@dornerworks.com" <xen-devel@dornerworks.com>
Cc: "Samuel Thibault" <samuel.thibault@ens-lyon.org>,
	"Ian Jackson" <Ian.Jackson@citrix.com>, "Wei Liu" <wl@xen.org>,
	"George Dunlap" <George.Dunlap@citrix.com>,
	"Nick Rosbrook" <rosbrookn@ainfosec.com>,
	"Andrew Cooper" <Andrew.Cooper3@citrix.com>,
	"Jan Beulich" <jbeulich@suse.com>,
	"Julien Grall" <julien@xen.org>,
	"Stefano Stabellini" <sstabellini@kernel.org>,
	"Anthony Perard" <anthony.perard@citrix.com>,
	"Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>,
	"Josh Whitehead" <josh.whitehead@dornerworks.com>,
	"Stewart Hildebrand" <stewart.hildebrand@dornerworks.com>,
	"David Scott" <dave@recoil.org>,
	"Shriram Rajagopalan" <rshriram@cs.ubc.ca>,
	"Yang Hongyang" <imhy.yang@gmail.com>,
	"Edwin Torok" <edvin.torok@citrix.com>
Subject: Re: [PATCH v3 00/38] tools: move most libraries into tools/libs
Date: Tue, 25 Aug 2020 10:54:21 +0000	[thread overview]
Message-ID: <1598352860986.67246@citrix.com> (raw)
In-Reply-To: <20200823093519.18386-1-jgross@suse.com>

[-- Attachment #1: Type: text/plain, Size: 1928 bytes --]


I have no objections. For the OCaml part in particular I previously said that I believe that it would benefit from either be maintained outside the tree (which requires a clean interface) or moving it to use Dune for its build (triggered by the current make build). Any cleanup that is helping here is welcome.

--
Acked-by: Christian Lindig <christian.lindig@citrix.com>??


________________________________
From: Juergen Gross <jgross@suse.com>
Sent: 23 August 2020 10:34
To: xen-devel@lists.xenproject.org; xen-devel@dornerworks.com
Cc: Juergen Gross; Samuel Thibault; Ian Jackson; Wei Liu; George Dunlap; Nick Rosbrook; Andrew Cooper; Jan Beulich; Julien Grall; Stefano Stabellini; Anthony Perard; Marek Marczykowski-Górecki; Josh Whitehead; Stewart Hildebrand; Christian Lindig; David Scott; Shriram Rajagopalan; Yang Hongyang
Subject: [PATCH v3 00/38] tools: move most libraries into tools/libs

Move most remaining libraries under tools/libs, including libxenctrl
and libxl. This is resulting in a lot of cleanup work regarding
building libs and restructuring of the tools directory.

After this huge cleanup all dependencies between libraries are defined
in a single rather small file tools/libs/uselibs.mk, which is used to
create the needed make variables and to control the stubdom build
dependencies as well.

Another bonus of the rework is the automation of setting the versions
of unstable libraries. This removes the need to bump those versions
manually for each Xen release.

libfsimage is not moved by this series, as IMO there are some open
questions:
- should it really be unstable?
- is the support of adding external fs-support used in practice, i.e.
  shouldn't the fs-specific sub-libraries be just included into
  libfsimage instead of being loaded dynamically?

The complete series is available via:

git://xenbits.xen.org/people/jgross/xen.git libbuild-v3





[-- Attachment #2: Type: text/html, Size: 3578 bytes --]

  parent reply	other threads:[~2020-08-25 10:55 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-23  9:34 [PATCH v3 00/38] tools: move most libraries into tools/libs Juergen Gross
2020-08-23  9:34 ` [PATCH v3 01/38] stubdom: add stubdom/mini-os.mk for Xen paths used by Mini-OS Juergen Gross
2020-08-23  9:34 ` [PATCH v3 02/38] tools: switch XEN_LIBXEN* make variables to lower case (XEN_libxen*) Juergen Gross
2020-08-27 10:22   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 03/38] tools: add a copy of library headers in tools/include Juergen Gross
2020-08-27 10:25   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 04/38] tools: don't call make recursively from libs.mk Juergen Gross
2020-08-27 10:28   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 05/38] tools: define ROUNDUP() in tools/include/xen-tools/libs.h Juergen Gross
2020-08-27 10:22   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 06/38] tools: generate most contents of library make variables Juergen Gross
2020-08-27 10:28   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 07/38] stubdom: add correct dependencies for Xen libraries Juergen Gross
2020-08-23  9:47   ` Samuel Thibault
2020-08-23  9:51     ` Jürgen Groß
2020-08-23  9:34 ` [PATCH v3 08/38] stubdom: simplify building xen libraries for stubdoms Juergen Gross
2020-08-23  9:50   ` Samuel Thibault
2020-08-23  9:34 ` [PATCH v3 09/38] tools: drop explicit path specifications for qemu build Juergen Gross
2020-08-27 10:30   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 10/38] tools: tweak tools/libs/libs.mk for being able to support libxenctrl Juergen Gross
2020-08-27 10:52   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 11/38] tools/python: drop libxenguest from setup.py Juergen Gross
2020-08-25 11:07   ` Marek Marczykowski-Górecki
2020-08-23  9:34 ` [PATCH v3 12/38] tools: fix pkg-config file for libxenguest Juergen Gross
2020-08-27 10:34   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 13/38] tools: don't assume libxenguest and libxenctrl to be in same directory Juergen Gross
2020-08-27 10:36   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 14/38] tools/misc: don't use libxenctrl internals from xen-hptool Juergen Gross
2020-08-27 10:37   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 15/38] tools/misc: don't include xg_save_restore.h from xen-mfndump.c Juergen Gross
2020-08-27 10:37   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 16/38] tools/misc: replace PAGE_SIZE with XC_PAGE_SIZE in xen-mfndump.c Juergen Gross
2020-08-27 10:37   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 17/38] tools/misc: drop all libxc internals from xen-mfndump.c Juergen Gross
2020-08-27 10:38   ` Wei Liu
2020-08-23  9:34 ` [PATCH v3 18/38] tools/libxc: remove unused headers xc_efi.h and xc_elf.h Juergen Gross
2020-08-27 10:39   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 19/38] tools/libxc: move xc_[un]map_domain_meminfo() into new source xg_domain.c Juergen Gross
2020-08-27 10:39   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 20/38] tools/libxc: rename all libxenguest sources to xg_* Juergen Gross
2020-08-27 10:56   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 21/38] tools/libxc: rename libxenguest internal headers Juergen Gross
2020-08-27 10:56   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 22/38] tools/misc: rename xc_dom.h do xenctrl_dom.h Juergen Gross
2020-08-25 11:10   ` Marek Marczykowski-Górecki
2020-08-27 10:57   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 23/38] tools/libxc: untangle libxenctrl from libxenguest Juergen Gross
2020-08-27 11:23   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 24/38] tools/xcutils: use official headers in readnotes Juergen Gross
2020-08-27 11:27   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 25/38] tools/xenpaging: remove libxc internals Juergen Gross
2020-08-27 11:27   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 26/38] tools: move libxenctrl below tools/libs Juergen Gross
2020-08-25 11:16   ` Marek Marczykowski-Górecki
2020-08-25 11:26     ` Jürgen Groß
2020-08-27 11:33   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 27/38] tools/libxc: move libxenguest to tools/libs/guest Juergen Gross
2020-08-23  9:53   ` Samuel Thibault
2020-08-27 11:33   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 28/38] tools: split libxenstore into new tools/libs/store directory Juergen Gross
2020-08-27 11:34   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 29/38] tools: split libxenvchan into new tools/libs/vchan directory Juergen Gross
2020-08-27 11:35   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 30/38] tools: split libxenstat into new tools/libs/stat directory Juergen Gross
2020-08-27 11:36   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 31/38] tools/libxl: fix dependencies of libxl tests Juergen Gross
2020-08-27 11:36   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 32/38] tools/libxl: don't include libxl_osdeps.h from libxlutil sources Juergen Gross
2020-08-27 11:47   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 33/38] tools/libxl: add missing items to clean target Juergen Gross
2020-08-27 11:47   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 34/38] tools/libxl: move xl parsing checks to tools/xl Juergen Gross
2020-08-27 11:48   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 35/38] tools/xl: don't assume libxl and libxlutil are in one directory Juergen Gross
2020-08-27 11:49   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 36/38] tools/libxl: move libxenlight to tools/libs/light Juergen Gross
2020-08-27 11:51   ` Wei Liu
2020-08-23  9:35 ` [PATCH v3 37/38] tools: add support for library names other than libxen* Juergen Gross
2020-08-27 11:55   ` Wei Liu
2020-08-27 12:34     ` Jürgen Groß
2020-08-23  9:35 ` [PATCH v3 38/38] tools: move libxlutil to tools/libs/util Juergen Gross
2020-08-25 10:54 ` Christian Lindig [this message]
2020-08-27 11:07 ` [PATCH v3 00/38] tools: move most libraries into tools/libs Wei Liu
2020-08-27 11:24   ` 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=1598352860986.67246@citrix.com \
    --to=christian.lindig@citrix.com \
    --cc=Andrew.Cooper3@citrix.com \
    --cc=George.Dunlap@citrix.com \
    --cc=Ian.Jackson@citrix.com \
    --cc=anthony.perard@citrix.com \
    --cc=dave@recoil.org \
    --cc=edvin.torok@citrix.com \
    --cc=imhy.yang@gmail.com \
    --cc=jbeulich@suse.com \
    --cc=jgross@suse.com \
    --cc=josh.whitehead@dornerworks.com \
    --cc=julien@xen.org \
    --cc=marmarek@invisiblethingslab.com \
    --cc=rosbrookn@ainfosec.com \
    --cc=rshriram@cs.ubc.ca \
    --cc=samuel.thibault@ens-lyon.org \
    --cc=sstabellini@kernel.org \
    --cc=stewart.hildebrand@dornerworks.com \
    --cc=wl@xen.org \
    --cc=xen-devel@dornerworks.com \
    --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.