On 30/08/18 13:25, Jan Beulich wrote:
>>>> On 30.08.18 at 11:55, <[email protected]> wrote:
>> --- a/xen/arch/x86/Kconfig
>> +++ b/xen/arch/x86/Kconfig
>> @@ -164,3 +164,26 @@ endmenu
>>  source "common/Kconfig"
>>  
>>  source "drivers/Kconfig"
>> +
>> +menu "Deprecated Functionality"
>> +
>> +config LEGACY_PV_LDT_PAGING
>> +    def_bool n
> Can this please simply be "bool" and depend on PV right away

Sure.  (I believe this patch actually predicates CONFIG_PV entirely).

> (maybe the name would then also better have PV_ first, but
> I'll leave that part up to you)? With that
> Reviewed-by: Jan Beulich <[email protected]>

Perhaps drop it to just CONFIG_PV_LDT_PAGING ?  The LEGACY part is
rather redundant.

~Andrew

_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to