From: Doug Goldstein <cardoe@cardoe.com>
To: Ian Campbell <ian.campbell@citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>,
xen-devel@lists.xen.org
Subject: Re: Kbuild and Kconfig
Date: Thu, 3 Sep 2015 08:58:05 -0500 [thread overview]
Message-ID: <55E851ED.5010305@cardoe.com> (raw)
In-Reply-To: <1441274169.26292.323.camel@citrix.com>
[-- Attachment #1.1: Type: text/plain, Size: 2951 bytes --]
On 9/3/15 4:56 AM, Ian Campbell wrote:
> On Wed, 2015-09-02 at 19:29 +0100, Andrew Cooper wrote:
>> On 02/09/15 18:50, Doug Goldstein wrote:
>>> I just wanted to bring this to a top level post since Jonathan
>>> Creekmore
>>> and myself have talked with a few maintainers in different threads and
>>> on IRC about potentially using Kconfig and/or Kbuild for Xen. Basically
>>> I would like to get a rough idea on what the Xen community wants the
>>> system to look like before starting work on it to both save myself time
>>> and save maintainers review cycles. So that being said rough proposal
>>> as
>>> follows:
>>>
>>> * target only the xen/ directory tree (i.e. not the toolstack, stubdoms
>>> or docs)
>>> * split top level config bits to not affect xen/ tree (currently only
>>> XSM_ENABLE / FLASK_ENABLE do)
>>> * convert xen/ to Kbuild first and merge this in (since Kconfig relies
>>> on Kbuild-y bits which can be undone but if we're going to go to Kbuild
>>> in the end why undo it and then redo it)
>>> * convert existing xen/ config bits into Kconfig and merge that in
>>>
>>> Jonathan and I, in a former life, converted a project to Kbuild and
>>> Kconfig successfully. I have looked at starting with
>>> https://github.com/masahir0y/kbuild_skeleton while the tree is fairly
>>> old it does separate out the build bits from the Linux specific bits
>>> pretty nicely while removing module support which arguably is the most
>>> complicated part. Alternatively we could start with Linux 4.2 if that's
>>> more desirable.
>>
>> Thinking longterm, it would be nice to have xen, tools and stubdoms
>> covered by a system like this
>
> Is the proposal here then to abandon autoconf for the tools subtree in
> favour of Kconfig? Or maybe to somehow hybridize autoconf (for e.g. library
> and feature detection) with Kconfig (for user selection of options)? I'm
> not sure how I feel about either of those approaches, they certainly both
> need careful consideration, and the second in particular regarding the
> interactions...
>
> FWIW it seems to me that the link between things which are optional in Xen
> and which are optional in the tools is (or should be) pretty loose. i.e.
> the tools today _always_ support XSM and correctly handle the errors from
> Xen if it is not enabled there. Personally I think this is the right way to
> do things. Likewise Xen doesn't care if the tools have particular opinions
> on the qemu to use or whatever.
>
> IOW I'm not sure have xen and tools use a common .config would make sense.
>
> Ian.
>
>
So with my initial approach of targeting the xen/ directory how you
described it is how it would work. The optional items would be separate
and I really think in a lot of cases they are separate like you
describe. (e.g. turn off XSM in the hypervisor but not in the tools and
the tools gracefully handle that).
--
Doug Goldstein
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2015-09-03 13:58 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-02 17:50 Kbuild and Kconfig Doug Goldstein
2015-09-02 18:29 ` Andrew Cooper
2015-09-02 20:28 ` Doug Goldstein
2015-09-02 20:50 ` Andrew Cooper
2015-09-04 11:54 ` Stefano Stabellini
2015-09-03 9:56 ` Ian Campbell
2015-09-03 10:09 ` Tim Deegan
2015-09-03 10:26 ` Jan Beulich
2015-09-03 13:58 ` Doug Goldstein [this message]
2015-09-04 10:59 ` Andrew Cooper
2015-09-04 11:37 ` Ian Campbell
2015-09-03 9:56 ` Tim Deegan
2015-09-03 14:13 ` Doug Goldstein
2015-09-03 10:31 ` Jan Beulich
2015-09-03 14:04 ` Doug Goldstein
2015-09-03 15:00 ` Jan Beulich
2015-09-18 19:31 ` Doug Goldstein
2015-09-21 8:01 ` Jan Beulich
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=55E851ED.5010305@cardoe.com \
--to=cardoe@cardoe.com \
--cc=andrew.cooper3@citrix.com \
--cc=ian.campbell@citrix.com \
--cc=xen-devel@lists.xen.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.