From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf1-f193.google.com ([209.85.210.193]:41835 "EHLO mail-pf1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730583AbfKFE3l (ORCPT ); Tue, 5 Nov 2019 23:29:41 -0500 Date: Wed, 6 Nov 2019 09:59:25 +0530 From: Bhaskar Chowdhury Subject: Re: [PATCH] scripts:prune-kernel:remove old kernels and modules dir from system Message-ID: <20191106042922.GD20442@Gentoo> References: <20191102063036.28601-1-unixbhaskar@gmail.com> <50680c37-9e85-0050-c1e1-700260a0471c@infradead.org> <20191105023243.GA16635@fieldses.org> <20191106031027.GA20442@Gentoo> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="EP0wieDxd4TSJjHq" Content-Disposition: inline In-Reply-To: Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Masahiro Yamada Cc: "J. Bruce Fields" , Randy Dunlap , Michal Marek , Linux Kbuild mailing list , Linux Kernel Mailing List --EP0wieDxd4TSJjHq Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 13:03 Wed 06 Nov 2019, Masahiro Yamada wrote: >On Wed, Nov 6, 2019 at 12:10 PM Bhaskar Chowdhury = wrote: >> >> On 11:53 Wed 06 Nov 2019, Masahiro Yamada wrote: >> >On Tue, Nov 5, 2019 at 11:32 AM J. Bruce Fields = wrote: >> >> >> >> On Mon, Nov 04, 2019 at 06:03:13PM -0800, Randy Dunlap wrote: >> >> > On 11/1/19 11:30 PM, Bhaskar Chowdhury wrote: >> >> > > This patch allow you to remove old kernels and associated modules >> >> > > directory from the system.You can do it at once with the -r flag >> >> > > and interactively with the -i flag. >> >> > > >> >> > > Signed-off-by: Bhaskar Chowdhury >> >> > > --- >> >> > > scripts/prune-kernel | 82 +++++++++++++++++++++++++++++++++++---= ------ >> >> > > 1 file changed, 65 insertions(+), 17 deletions(-) >> >> > >> >> > Hi, >> >> > I believe that this script now does what the patch author intends i= t to do. >> >> > It does have a few whitespace issues, but no big deals. (see below) >> >> >> >> My original comment stands: looks like it prompts for full module path >> >> and kernel versions which means it's no more convenient than just doi= ng >> >> an "ls" and then removing the ones you want to. (In fact, with "rm" >> >> you'd also get the benefit of tab completion....) >> >> >> >> It's quite different from the original script and I don't really see = the >> >> advantage. >> >> >> >> --b. >> > >> >I am with Bruce. >> > >> >This patch is trying to replace everything >> >with worse code. >> Well,Masahiro, >> I won't mind dropping the idea, which you already concluded.But, would >> you care to let me know how worse the code seems to be???? > > >As far as I understood this script, >it is useful to delete stale versions with a single command. > >scripts/prune-kernel 5.2-rc1 5.2-rc2 5.2-rc3 > > >This patch is dropping the 'for f in "$@"' loop, >so you would end up with running this script multiple times. > >scripts/prune-kernel -r 5.2-rc1 5.2-rc1 >scripts/prune-kernel -r 5.2-rc2 5.2-rc2 >scripts/prune-kernel -r 5.2-rc3 5.2-rc3 > Yes, true. You don't want all the stale kernels to be discarded at once, do you??? > >What is funny is, it takes the kernel_version and modules_version >separately. Lack of farsight of mine ...will take your suggestion. Should have realized it before, but was writing for different purpose. >And now it requires -r option for the default behavior. Because other flags are involved and they do different things. >I see nothing cool overall. > Meh... I am interpreting you "cool" literal way...which you certainly didn't mean it...do you?? > > >J. Bruce Fields suggested: >"But if somebody does actually use it as-is, it'd be nicer to keep the >current behavior and add an option ("-i" or something) for the >interactive behavior." > Yes, that's true ...and now I understood. On the contrary ,if I recollect properly, Bruce ,once told that he won't if do something which will be useful to the wider users..not sure that stands anymore. > >I want to see a patch if and only if >you can add -i without intrusively changing the current code. > Hopefully , you wish get fulfilled sooner than later. > >Masahiro > > > Bhaskar > >> > >> >BTW. >> >Bruce, >> >Does the current script expect RHEL or something? >> >I do not see 'new-kernel-pkg' on my Ubuntu machine. >> > >> >It would still work with 'new-kernel-pkg: command not found' >> >warning. >> > >> >We could bypass it if we like. >> > >> >command -v new-kernel-pkg && new-kernel-pkg --remove $f >> > >> > >> > >> >Masahiro Yamada >> > >> > >> > >> >> > >> >> > Tested-by: Randy Dunlap >> >> > >> >> > >> >> > > diff --git a/scripts/prune-kernel b/scripts/prune-kernel >> >> > > index e8aa940bc0a9..01d0778db71f 100755 >> >> > > --- a/scripts/prune-kernel >> >> > > +++ b/scripts/prune-kernel >> >> > > @@ -1,21 +1,69 @@ >> >> > > #!/bin/bash >> >> > > # SPDX-License-Identifier: GPL-2.0 >> >> > > +#This script will remove old kernels and modules directory relat= ed to it. >> >> > > +#"-r" or "--remove" show how to silently remove old kernel and m= odules dir. >> >> > > +# "-h" or "--help" show how to use this script or show without p= arameter. >> >> > > +#"-i" or "--interactive" show how to remove interactively. >> >> > > + >> >> > > +flag=3D$1 >> >> > > +kernel_version=3D$2 >> >> > > +modules_version=3D$3 >> >> > > +boot_dir=3D/boot >> >> > > +modules_dir=3D/lib/modules >> >> > > + >> >> > > +remove_old_kernel() { >> >> > > + cd $boot_dir >> >> > > + rm -If vmlinuz-$kernel_version System.map-$kernel_version con= fig-$kernel_version >> >> > > + return 0 >> >> > > +} >> >> > > + >> >> > > +remove_old_modules_dir() { >> >> > > + cd $modules_dir >> >> > > + rm -rf $modules_version >> >> > > + return 0 >> >> > > +} >> >> > > + >> >> > > +usage() { >> >> > > + printf "Usage: $(basename $0) [-ri]\n" >> >> > > + printf "\n -r or --remove kernel_version modules_version\n" >> >> > > + printf "\n -i or --interactive do as interactive way\n" >> >> > > + return 0 >> >> > > +} >> >> > > + >> >> > > +case "$flag" in >> >> > > + -i | --interactive) >> >> > > + printf "\nEnter kernel version to remove or blank/emp= ty to exit:" >> >> > > + read kernel_version >> >> > > + if [[ $kernel_version !=3D "" ]]; then >> >> > > + remove_old_kernel >> >> > > + printf "\nRemoved kernel version:$kernel_vers= ion from the system.\n\n" >> >> > >> >> > space after ':' >> >> > >> >> > drop one \n above. >> >> > >> >> > > + printf "Please give the full modules director= y name to remove:" >> >> > > + read modules_version >> >> > > + if [[ $modules_version !=3D "" ]]; then >> >> > > + remove_old_modules_dir >> >> > > + printf "\n\nRemoved modules directory= :$modules_version from the system.\n\n" >> >> > >> >> > space after ':' >> >> > >> >> > drop one \n above. >> >> > >> >> > > + else >> >> > > + exit 1 >> >> > > + fi >> >> > > + fi >> >> > > + ;; >> >> > > + -h | --help) >> >> > > + usage >> >> > > + exit 0 >> >> > > + ;; >> >> > > + -r | --remove) >> >> > > + if [[ $# -ne 3 ]]; then >> >> > > + printf "You need to provide kernel version a= nd modules directory name.\n" >> >> > > + exit 1 >> >> > > + else >> >> > > + remove_old_kernel >> >> > > + remove_old_modules_dir >> >> > > + fi >> >> > > + ;; >> >> > > + *) >> >> > > + usage >> >> > > + exit 1 >> >> > > + ;; >> >> > > +esac >> >> > > >> >> > > -# because I use CONFIG_LOCALVERSION_AUTO, not the same version a= gain and >> >> > > -# again, /boot and /lib/modules/ eventually fill up. >> >> > > -# Dumb script to purge that stuff: >> >> > > >> >> > >> >> > OK, the former script's loop is removed.. good. >> >> > But the 2 preceding blank lines are not removed, so the script >> >> > now ends with 2 unnecessary blank lines. >> >> > >> >> > > -for f in "$@" >> >> > > -do >> >> > > - if rpm -qf "/lib/modules/$f" >/dev/null; then >> >> > > - echo "keeping $f (installed from rpm)" >> >> > > - elif [ $(uname -r) =3D "$f" ]; then >> >> > > - echo "keeping $f (running kernel) " >> >> > > - else >> >> > > - echo "removing $f" >> >> > > - rm -f "/boot/initramfs-$f.img" "/boot/System.map= -$f" >> >> > > - rm -f "/boot/vmlinuz-$f" "/boot/config-$f" >> >> > > - rm -rf "/lib/modules/$f" >> >> > > - new-kernel-pkg --remove $f >> >> > > - fi >> >> > > -done >> >> > > -- >> >> > >> >> > >> >> > -- >> >> > ~Randy >> > >> > >> > >> >-- >> >Best Regards >> > >> >Masahiro Yamada > > > >--=20 >Best Regards >Masahiro Yamada --EP0wieDxd4TSJjHq Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCAAdFiEEnwF+nWawchZUPOuwsjqdtxFLKRUFAl3CTB8ACgkQsjqdtxFL KRW5NAgA0uVcwzRFLEvGndfy6LOeW2pEg7NLPhU6RiPRpAdGpvf4U02sO6H7B3DR TC47joFXWmi8cbrQVQYyjlCspa2EenGfaD6qQWLtuYUoEux++H2CWsd1cyw1U6UX LYv5Ibs3htO0LRjt+0r79Rt+v6RMucrs0A1Z3zrYrTJOPAhyQzaPJ6GZv06asx9Z A9dC+An5s2tlirX3oKp/ERRBliR5WLHR+G3Yiae3JtwtjYtVXBgs9RuVjrrwsblJ PcULONU318O8M2WGjNkASmnyN18TqSBtZltive5e7F+CnNNZ58QkwMe+zSJeYbcS 89FEJ3oG87N+PHp5XOXg+PJ9k1x7Cg== =UI0O -----END PGP SIGNATURE----- --EP0wieDxd4TSJjHq--