lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20091120175142.GD13136@tux>
Date:	Fri, 20 Nov 2009 09:51:42 -0800
From:	"Luis R. Rodriguez" <lrodriguez@...eros.com>
To:	Alan Jenkins <alan-jenkins@...fmail.co.uk>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Luis Rodriguez <Luis.Rodriguez@...eros.com>,
	"sam@...nborg.org" <sam@...nborg.org>,
	"greg@...ah.com" <greg@...ah.com>,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"mcgrof@...il.com" <mcgrof@...il.com>
Subject: Re: [PATCH 4/4] kconfig: streamline_config.pl: let users specify
	the list of desired modules

On Fri, Nov 20, 2009 at 07:50:54AM -0800, Alan Jenkins wrote:
> This allows "make localmodconfig MODULES_LIST=lsmod.out",
> where "lsmod.out" may have been created on a different system.
> 
> You can also use this with a manually editted file which
> simply lists the desired modules.

The current localmodconfig is designed to be run on your own system,
not an external system. The changes outlined here give no guidelines
to users what they should do if they want to use this on another system
and a novice user may just assume an old distribution config file
can be used here along with an lsmod output. If this is the case
it should be documented as such but I don't think this is the case.
Consider the case of someone trying to build a kernel for a single
processor ARM or x86 netbook/nettop on a beefy quad core x86_64.

Think this is definitely better than my approach though, that's for sure :)

Thanks for working on it :)

> Signed-off-by: Alan Jenkins <alan-jenkins@...fmail.co.uk>
> ---
>  README                               |    8 ++++++++
>  scripts/kconfig/Makefile             |    3 ++-
>  scripts/kconfig/streamline_config.pl |   17 ++++++++++++++---
>  3 files changed, 24 insertions(+), 4 deletions(-)
> 
> diff --git a/README b/README
> index 737838f..9481dd5 100644
> --- a/README
> +++ b/README
> @@ -175,6 +175,14 @@ CONFIGURING the kernel:
>                            Like above, but avoids cluttering the screen
>                            with questions already answered.
>                            Additionally updates the dependencies.
> +
> +       "make localmodconfig"
> +                          Minimize your existing ./.config file, based
> +                          on the kernel modules currently loaded on your
> +                          system.  You can also specify the list of
> +                          modules manually by appending an option
> +                          "MODULES_LIST=lsmod.out".

Best to add a plain entry for localmodconfig separately and then just
add the new stuff for MODULES_LIST on another patch.

> +
>         "make defconfig"   Create a ./.config file by using the default
>                            symbol values from either arch/$ARCH/defconfig
>                            or arch/$ARCH/configs/${PLATFORM}_defconfig,
> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
> index 6d69c7c..6a1e41d 100644
> --- a/scripts/kconfig/Makefile
> +++ b/scripts/kconfig/Makefile
> @@ -30,7 +30,7 @@ silentoldconfig: $(obj)/conf
>         $< -s $(Kconfig)
> 
>  localmodconfig: $(obj)/streamline_config.pl $(obj)/conf
> -       $(Q)perl $< $(Kconfig) > .tmp.config
> +       $(Q)perl $< $(Kconfig) $(MODULES_LIST) > .tmp.config
>         $(Q)if [ -f .config ]; then                             \
>                         cmp -s .tmp.config .config ||           \
>                         (mv -f .config .config.old.1;           \
> @@ -114,6 +114,7 @@ help:
>         @echo  '  gconfig         - Update current config utilising a GTK based front-end'
>         @echo  '  oldconfig       - Update current config utilising a provided .config as base'
>         @echo  '  localmodconfig  - Update current config disabling modules not loaded'
> +       @echo  '                    (or modules not listed in the file MODULES_LIST)'
>         @echo  '  localyesconfig  - Update current config converting local mods to core'
>         @echo  '  silentoldconfig - Same as oldconfig, but quietly, additionally update deps'
>         @echo  '  randconfig      - New config with random answer to all options'
> diff --git a/scripts/kconfig/streamline_config.pl b/scripts/kconfig/streamline_config.pl
> index f5b44c1..9fde7fd 100644
> --- a/scripts/kconfig/streamline_config.pl
> +++ b/scripts/kconfig/streamline_config.pl
> @@ -40,7 +40,7 @@ use warnings;
>  #
>  #    cd /usr/src/linux-2.6.10
>  #    cp /boot/config-2.6.10-1-686-smp .config
> -#    ~/bin/streamline_config > config_strip
> +#    ~/bin/streamline_config arch/x86/Kconfig > config_strip

Why was x86 added here?

>  #    mv .config config_sav
>  #    mv config_strip .config
>  #    make oldconfig
> @@ -122,9 +122,15 @@ my %objects;
>  my $var;
>  my $cont = 0;
> 
> +if ($#ARGV < 0 || $#ARGV > 1) {
> +    die "usage: streamline_config.pl <Kconfig> [<modules.list>]"
> +}
> +
>  # Get the top level Kconfig file (passed in)
>  my $kconfig = $ARGV[0];
> 
> +my $modules_list = $ARGV[1];
> +
>  # prevent recursion
>  my %read_kconfigs;
> 
> @@ -251,8 +257,13 @@ foreach my $makefile (@makefiles) {
> 
>  my %modules;
> 
> -# see what modules are loaded on this system
> -open(LIN,"/sbin/lsmod|") || die "Cant lsmod";
> +# see what modules are loaded
> +if ($modules_list) {
> +    open(LIN,$modules_list) || die "Can't open $modules_list";
> +} else {
> +    open(LIN,"/sbin/lsmod|") || die "Can't lsmod";
> +}
> +
>  while (<LIN>) {
>         next if (/^Module/);  # Skip the first line.
>         if (/^(\S+)/) {
> --
> 1.6.3.3
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ