ALT Linux kernel packages development
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: devel-kernel@lists.altlinux.org
Subject: Re: [d-kernel] CONFIG_PREEMPT=y --> CONFIG_PREEMPT_VOLUNTARY=y
Date: Sat, 7 May 2022 10:25:49 +0300
Message-ID: <20220507072549.GB3440@altlinux.org> (raw)
In-Reply-To: <20220507004904.aqpof32nijy3pieg@altlinux.org>

On Sat, May 07, 2022 at 03:49:04AM +0300, Vitaly Chikunov wrote:
> On Sat, May 07, 2022 at 12:16:19AM +0300, Vitaly Chikunov wrote:
> > 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
> 
> Эти пункты внутри
> 
>   choice
> 	  prompt "Preemption Model"
> 	  default PREEMPT_NONE
> 
> То есть из них можно выбрать только 1. Судя по коду ниже это и будет
> preempt mode по умолчанию. Но,
> 
> > > 
> > > 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
> 
> Это было в 5.17, а в 5.15:
> 
>   Symbol: PREEMPT_DYNAMIC [=n]                                                      
>   Type  : bool                                                                      
>   Defined at kernel/Kconfig.preempt:85                                              
>   Selected by [n]:                                                                  
>     - PREEMPT [=n] && <choice> && !ARCH_NO_PREEMPT [=n] && HAVE_PREEMPT_DYNAMIC [=y]
> 
> То есть для него нужен PREEMPT=y. А значит, для старых ядер, PREEMPT_DYNAMIC
> можно включить только при выбранном PREEMPT и умолчанием всегда будет
> PREEMPT.

Да, поведение поменялось в результате коммитов v5.16-rc1~197^2~48 и
v5.16-rc1~10^2, и в новых ядрах можно выбрать и PREEMPT_DYNAMIC, и
умолчание, отличное от PREEMPT.

Возможно, эти коммиты можно сбэкпортить.


-- 
ldv


  reply	other threads:[~2022-05-07  7:25 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-06 14:44 Vitaly Chikunov
2022-05-06 14:53 ` Vladimir D. Seleznev
2022-05-06 14:54   ` Vitaly Chikunov
2022-05-06 14:59     ` Dmitry V. Levin
2022-05-06 15:31       ` Vitaly Chikunov
2022-05-06 18:49         ` Dmitry V. Levin
2022-05-06 21:16           ` Vitaly Chikunov
2022-05-07  0:49             ` Vitaly Chikunov
2022-05-07  7:25               ` Dmitry V. Levin [this message]
2022-05-06 15:39     ` Andrey Savchenko

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=20220507072549.GB3440@altlinux.org \
    --to=ldv@altlinux.org \
    --cc=devel-kernel@lists.altlinux.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

ALT Linux kernel packages development

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-kernel/0 devel-kernel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel-kernel devel-kernel/ http://lore.altlinux.org/devel-kernel \
		devel-kernel@altlinux.org devel-kernel@altlinux.ru devel-kernel@altlinux.com
	public-inbox-index devel-kernel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-kernel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git