Re: [d-kernel] CONFIG_PREEMPT=y --> CONFIG_PREEMPT_VOLUNTARY=y

[Vitaly Chikunov <vt@altlinux.org>]

On Fri, May 06, 2022 at 09:49:15PM +0300, Dmitry V. Levin wrote:
> On Fri, May 06, 2022 at 06:31:10PM +0300, Vitaly Chikunov wrote:
> > On Fri, May 06, 2022 at 05:59:29PM +0300, Dmitry V. Levin wrote:
> > > On Fri, May 06, 2022 at 05:54:49PM +0300, Vitaly Chikunov wrote:
> > > > On Fri, May 06, 2022 at 05:53:06PM +0300, Vladimir D. Seleznev wrote:
> > > > > On Fri, May 06, 2022 at 05:44:37PM +0300, Vitaly Chikunov wrote:
> > > > > 
> > > > > > Предлагаю так же отключить нашу традиционную std-def опцию CONFIG_PREEMPT=y
> > > > > 
> > > > > Вроде бы CONFIG_PREEMPT=y включен в un-def, а в std-def традиционно
> > > > > выключен.
> > > > 
> > > > Да это и имелось ввиду. Спасибо за поправку.
> > > 
> > > Главное - не выключать CONFIG_PREEMPT_DYNAMIC.
> > 
> > Это другой вариант о котором я не думал, но может стоит сделать и так.
> > 
> > На сколько я понял, по умолчанию эта опция включает PREEMPT_NONE, а не
> > PREEMPT_VOLUNTARY. То есть умолчание хуже чем CONFIG_PREEMPT_VOLUNTARY=y.
> 
> Давайте почитаем код вслух:

Я это читал - но понял так, что он выбирает из тех что включены, начиная с
NONE. Если не включать NONE, то тогда остается выбор только между VOLUNTARY и
PREEMPT. Хотя наверное это хороший выбор.

> 
> config PREEMPT_NONE
>         bool "No Forced Preemption (Server)"
>         select PREEMPT_NONE_BUILD if !PREEMPT_DYNAMIC
> 
> config PREEMPT_VOLUNTARY
>         bool "Voluntary Kernel Preemption (Desktop)"
>         depends on !ARCH_NO_PREEMPT
>         select PREEMPT_VOLUNTARY_BUILD if !PREEMPT_DYNAMIC
> 
> config PREEMPT
>         bool "Preemptible Kernel (Low-Latency Desktop)"
>         depends on !ARCH_NO_PREEMPT
>         select PREEMPT_BUILD
> 
> config PREEMPT_DYNAMIC
>         bool "Preemption behaviour defined on boot"
>         depends on HAVE_PREEMPT_DYNAMIC && !PREEMPT_RT
>         select JUMP_LABEL if HAVE_PREEMPT_DYNAMIC_KEY
>         select PREEMPT_BUILD
>         default y if HAVE_PREEMPT_DYNAMIC_CALL
> 
> static void __init preempt_dynamic_init(void)
> {
>         if (preempt_dynamic_mode == preempt_dynamic_undefined) {
>                 if (IS_ENABLED(CONFIG_PREEMPT_NONE)) {
>                         sched_dynamic_update(preempt_dynamic_none);
>                 } else if (IS_ENABLED(CONFIG_PREEMPT_VOLUNTARY)) {
>                         sched_dynamic_update(preempt_dynamic_voluntary);
>                 } else {
>                         /* Default static call setting, nothing to do */
>                         WARN_ON_ONCE(!IS_ENABLED(CONFIG_PREEMPT));
>                         preempt_dynamic_mode = preempt_dynamic_full;
>                         pr_info("Dynamic Preempt: full\n");
>                 }
>         }
> }
> 
> Получается, что PREEMPT_DYNAMIC по сути включает CONFIG_PREEMPT,
> что соответствует preempt=full.

Не понимаю откуда такой вывод.

ps. Кстати, с точки зрения авторов PREEMPT_NONE - это "server", а
PREEMPT_VOLUNTARY "desktop":

  #if defined(CONFIG_PREEMPT_NONE)
		     "server",
  #elif defined(CONFIG_PREEMPT_VOLUNTARY)
		     "desktop",
  #elif defined(CONFIG_PREEMPT)
		     "preempt",
  #elif defined(CONFIG_PREEMPT_RT)
		     "preempt_rt",
  #else
		     "unknown",
  #endif

> 
> Похоже, что HAVE_PREEMPT_DYNAMIC_CALL включено по умолчанию только на x86*.
> 
> 
> -- 
> ldv
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel