Re: [d-kernel] [PATCH v2 std-def/sisyphus] config: Update some config options

From: Vitaly Chikunov <vt@altlinux.org>
To: ALT Linux kernel packages development <devel-kernel@lists.altlinux.org>
Subject: Re: [d-kernel] [PATCH v2 std-def/sisyphus] config: Update some config options
Date: Sun, 8 May 2022 15:30:23 +0300
Message-ID: <20220508123023.s2rp2jrmi3d3vqr4@altlinux.org> (raw)
In-Reply-To: <20220508122708.zf7k6xiwwy6cjeos@altlinux.org>

On Sun, May 08, 2022 at 03:27:08PM +0300, Vitaly Chikunov wrote:
> On Sun, May 08, 2022 at 03:13:29PM +0300, Dmitry V. Levin wrote:
> > On Sun, May 08, 2022 at 02:14:03PM +0300, Vitaly Chikunov wrote:
> > > Based on suggestions from Alexey V. Vissarionov <gremlin@altlinux.org>,
> > > but not completely following them. All mistakes are mine.
> > > 
> > > - Mostly - add new hardware support.
> > > - Disable some legacy stuff.
> > > - Turn off SHA1 by default, replace with SHA512.
> > > - Set panic=600 by default.
> > [...]
> > > --- a/config
> > > +++ b/config
> > > @@ -99,7 +99,7 @@ CONFIG_NO_HZ_IDLE=y
> > >  # CONFIG_NO_HZ_FULL is not set
> > >  CONFIG_CONTEXT_TRACKING=y
> > >  CONFIG_CONTEXT_TRACKING_FORCE=y
> > > -CONFIG_NO_HZ=y
> > > +# CONFIG_NO_HZ is not set
> > 
> > Посмотрел Documentation/timers/no_hz.rst и не понял, как обосновано это
> > изменение.
> 
> Следующим образом:
> 
>   config NO_HZ
> 	  bool "Old Idle dynticks config"
> 	  help
> 	    This is the old config entry that enables dynticks idle.
> 	    We keep it around for a little while to enforce backward
> 	    compatibility with older config files.
> 
> Фраза "old". Плюс фраза "old obsolete dynticks idle symbol" в коммите,
> который добавляет этот текст:
> 
>   commit 0644ca5c774f1a7033bfc83c7ed0660f74f28b56
>   AuthorDate: Fri Apr 12 16:40:12 2013 +0200
> 
>       Namely we want:
> 
> 	      config NO_HZ # old obsolete dynticks idle symbol
> 		      bool
> 
> 	      config NO_HZ_IDLE # new dynticks idle symbol
> 		      default NO_HZ
> 
> 
> Я так же не нахожу в коде использования CONFIG_NO_HZ.
> 
> (ps. У меня на defconfig она оказалась =y, но сейчас я его перегенерировал и

То есть была =n первый раз.

> она стала =y. Значит тут я ошибся.)
> 
> 
> > 
> > 
> > -- 
> > ldv
> > _______________________________________________
> > devel-kernel mailing list
> > devel-kernel@lists.altlinux.org
> > https://lists.altlinux.org/mailman/listinfo/devel-kernel
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel