From: Doug Goldstein <cardoe@cardoe.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>, xen-devel@lists.xen.org
Subject: Re: [PATCHv5] 00/28] Kconfig conversion
Date: Tue, 24 Nov 2015 11:03:15 -0600 [thread overview]
Message-ID: <56549853.3020907@cardoe.com> (raw)
In-Reply-To: <56548942.3050206@citrix.com>
[-- Attachment #1.1: Type: text/plain, Size: 2024 bytes --]
On 11/24/15 9:58 AM, Andrew Cooper wrote:
> On 20/11/15 19:09, Doug Goldstein wrote:
>> The following series is a follow on to the Kconfig conversion patch series.
>> There are still more components to convert however this is the bare minimal
>> to get everything working and get the options out of the existing makefiles.
>>
>> The CONFIG_HAS_ variables are there to match the behavior of the Linux
>> CONFIG_HAVE_ variables. The purpose is to say that this hardware/profile/env
>> supports this option while the CONFIG_ variable states that this option was
>> requested on/off by user intervention.
>>
>> The UARTs are now uniformly prefixed as CONFIG_UART_ and dropping most of the
>> CONFIG_HAS_ labeling for them. This means they are now user selectable as
>> requested by Julien Grall in the prior review. The question I've got is
>> the old config was just for selecting defaults. Users could enable the OMAP
>> UART for arm64 for example but I'm not sure if that's valid. Currently this
>> patchset makes those UARTs not user selectable if they were not previously
>> defaulted on. But I would like some feedback on this if possible.
>>
>> Ultimately my goal is to allow for more parts of the hypervisor to be turned
>> off at compile time and potentially make it easier to include more
>> experimental features by others which can be turned off by default. Also to
>> provide the one true location for all possible knobs in the source code.
>>
>> The patch series can be grabbed at:
>> https://github.com/cardoe/xen/tree/kconfig_v5
>
> Having had a play with this, everything appears to be in order.
>
> Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
> Tested-by: Andrew Cooper <andrew.cooper3@citrix.com>
>
> Thankyou very much for your work here.
>
There's now a conflict in staging due to the NUMA changes. Its a simple
conflict and does not affect the resultant build. I will post a v6 to
resolve the conflict to make it easier to merge.
--
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
prev parent reply other threads:[~2015-11-24 17:03 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-20 19:09 [PATCHv5] 00/28] Kconfig conversion Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 01/28] build: import Kbuild/Kconfig from Linux 4.2 Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 02/28] build: build Kconfig and config rules Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 03/28] build: use generated Kconfig options for Xen Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 04/28] build: convert HAS_PASSTHROUGH use to Kconfig Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 05/28] build: convert HAS_DEVICE_TREE " Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 06/28] build: convert HAS_PCI " Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 07/28] build: convert HAS_NS16550 " Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 08/28] build: convert HAS_IOPORTS " Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 09/28] build: convert HAS_ACPI " Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 10/28] build: convert HAS_VIDEO " Doug Goldstein
2015-11-20 19:09 ` [PATCHv5] 11/28] build: convert HAS_VGA " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 12/28] build: convert HAS_CPUFREQ " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 13/28] build: convert HAS_GDBSX " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 14/28] build: convert HAS_PDX " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 15/28] build: convert HAS_KEXEC " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 16/28] build: convert HAS_ARM_HDLCD " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 17/28] build: convert HAS_CADENCE_UART " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 18/28] build: convert HAS_PL011 " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 19/28] build: convert HAS_EXYNOS4210 " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 20/28] build: convert HAS_OMAP " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 21/28] build: convert HAS_SCIF " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 22/28] build: convert HAS_EHCI " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 23/28] build: convert HAS_MEM_ACCESS " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 24/28] build: convert HAS_MEM_PAGING " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 25/28] build: convert HAS_MEM_SHARING " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 26/28] build: convert HAS_GICV3 " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 27/28] build: convert CONFIG_COMPAT " Doug Goldstein
2015-11-20 19:10 ` [PATCHv5] 28/28] build: convert kexec options to CONFIG_KEXEC Doug Goldstein
2015-11-21 15:51 ` [PATCHv5] 00/28] Kconfig conversion Doug Goldstein
2015-11-24 15:58 ` Andrew Cooper
2015-11-24 17:03 ` Doug Goldstein [this message]
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=56549853.3020907@cardoe.com \
--to=cardoe@cardoe.com \
--cc=andrew.cooper3@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.