[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Kconfig: defaults for UNSUPPORTED
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
- Date: Thu, 3 Mar 2022 15:29:01 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=UPACNY/YM8vRwcBogPoDlodhILeMLXLUuKMzwdU0nhI=; b=CvO1f/KKwDYapBmoRrbol12qRgtpetSP+JjfU85bI8aZN1T+id26485dzFPzTr/F92UybhtiLwkL5VYS7CQ91jcL9oLmzdA/YIY+cYoD01DsQ5ytl/90Q3/G5U2V85X1NUQiJ5Eq9m6bhhyN4HQ+OH5Mgf0a0J/JTod9NOlALkMcpRj4e28rUgZekh6AifXTIyqAxQqtvxaNeWVw0JZHKkEFkAcU9GTZafojCHUVhpeFUnD0k5PqZGFsZ49n+2SgVjJhwm3Hl3KNkXsyuoP2VY0POi7Vwbn2OYvx/XjoMB8pp/scsU5aetNqG9nozI4e26YdzC0QChCLitBfZmlgUg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oJX0D+d4ugGFFnxLFrweu7GFjiL+ZPEFOT00nwYLBO4L1S0j66x8sWKQ6sE2xqjXkR3Rplv8G6qGIU55bmtRVE96lXxJY8yNSgzpoR0VCcaBZ5+1aDW790iS+73kcA58LaZWnDWtUqLIuA7mJ8l+dnAP9yIL55zfvkhfzF4KiyM6Y/NFsoBC1yzs44cVdj8Q25gmy4FkLYZPxQz4DDL6mMtP6Z2z2tcVgMDi7Fxs2eP/1F5TXEBU8oPe5E4UxF/v0QtyWllCb0oy5BUdukJqSNw+QhCSLIAdmTHYQr060NKaSKu6pNSZU3qFFpMlBNSDbFnQCSb15Q0Mu/72UbHeWA==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Thu, 03 Mar 2022 15:29:23 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Thread-index: AQHYLUXTrzQbHj+avkybd4n9OCZXjaytzEMA
- Thread-topic: Kconfig: defaults for UNSUPPORTED
Hi,
> On 1 Mar 2022, at 08:24, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> Hello,
>
> when commit d96e5e6c1214 added UNSUPPORTED, it left x86'es TBOOT
> default untouched. This means we default-enable an unsupported
> setting, which doesn't look to be what's generally wanted. I can
> see defaulting to DEBUG as reasonable, and SCHED_NULL's defaulting
> to enabled when PV_SHIM can imo also be justified (there it's
> rather that UNSUPPORTED is inapplicable for the shim case, and the
> adjustment was also done subsequent to the named commit).
>
> Shouldn't we therefore have a rule of thumb that UNSUPPORTED
> entries only ever have no "default" (implying "n") or default to
> no more than DEBUG?
In general that would definitely make sense yes even though there might be
exceptions due to for example a dependency to an other unsupported parameter.
I would definitely agree with this.
Cheers
Bertrand
>
> Thanks for opinions,
> Jan
>
>
|
