* [PATCH 0/2] make mrproper cleanup more file @ 2017-04-14 21:54 Stephen Hemminger 2017-04-14 21:54 ` [PATCH 1/2] kbuild: cleanup signing keys with mrproper Stephen Hemminger 2017-04-14 21:54 ` [PATCH 2/2] kbuild: remove initramfs cpio " Stephen Hemminger 0 siblings, 2 replies; 7+ messages in thread From: Stephen Hemminger @ 2017-04-14 21:54 UTC (permalink / raw) To: yamada.masahiro, mmarek; +Cc: linux-kbuild, linux-kernel, Stephen Hemminger The point of mrproper is to cleanup all the machine specific files from the build. Stephen Hemminger (2): kbuild: cleanup signing keys with mrproper kbuild: remove initramfs cpio with mrproper Makefile | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) -- 2.11.0 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] kbuild: cleanup signing keys with mrproper 2017-04-14 21:54 [PATCH 0/2] make mrproper cleanup more file Stephen Hemminger @ 2017-04-14 21:54 ` Stephen Hemminger 2017-05-03 4:37 ` Masahiro Yamada 2017-04-14 21:54 ` [PATCH 2/2] kbuild: remove initramfs cpio " Stephen Hemminger 1 sibling, 1 reply; 7+ messages in thread From: Stephen Hemminger @ 2017-04-14 21:54 UTC (permalink / raw) To: yamada.masahiro, mmarek; +Cc: linux-kbuild, linux-kernel, Stephen Hemminger When 'make mrproper' is run it was supposed to remove the signing keys in the certs directory, but only the filename is given rather than the pathanme which is necessary to cause cleanup. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index efa267a92ba6..04ca211552f7 100644 --- a/Makefile +++ b/Makefile @@ -1274,9 +1274,9 @@ MRPROPER_DIRS += include/config usr/include include/generated \ arch/*/include/generated .tmp_objdiff MRPROPER_FILES += .config .config.old .version .old_version \ Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ - signing_key.pem signing_key.priv signing_key.x509 \ - x509.genkey extra_certificates signing_key.x509.keyid \ - signing_key.x509.signer vmlinux-gdb.py + certs/signing_key.pem certs/signing_key.priv certs/signing_key.x509 \ + certs/x509.genkey certs/extra_certificates certs/signing_key.x509.keyid \ + certs/signing_key.x509.signer vmlinux-gdb.py # clean - Delete most, but leave enough to build external modules # -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] kbuild: cleanup signing keys with mrproper 2017-04-14 21:54 ` [PATCH 1/2] kbuild: cleanup signing keys with mrproper Stephen Hemminger @ 2017-05-03 4:37 ` Masahiro Yamada 2017-05-11 17:25 ` Stephen Hemminger 0 siblings, 1 reply; 7+ messages in thread From: Masahiro Yamada @ 2017-05-03 4:37 UTC (permalink / raw) To: Stephen Hemminger Cc: Michal Marek, Linux Kbuild mailing list, Linux Kernel Mailing List, Stephen Hemminger, David Woodhouse, David Howells +CC David Woodhouse +CC David Howells 2017-04-15 6:54 GMT+09:00 Stephen Hemminger <stephen@networkplumber.org>: > When 'make mrproper' is run it was supposed to remove the signing > keys in the certs directory, but only the filename is given > rather than the pathanme which is necessary to cause cleanup. > > Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> > --- > Makefile | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Makefile b/Makefile > index efa267a92ba6..04ca211552f7 100644 > --- a/Makefile > +++ b/Makefile > @@ -1274,9 +1274,9 @@ MRPROPER_DIRS += include/config usr/include include/generated \ > arch/*/include/generated .tmp_objdiff > MRPROPER_FILES += .config .config.old .version .old_version \ > Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ > - signing_key.pem signing_key.priv signing_key.x509 \ > - x509.genkey extra_certificates signing_key.x509.keyid \ > - signing_key.x509.signer vmlinux-gdb.py > + certs/signing_key.pem certs/signing_key.priv certs/signing_key.x509 \ > + certs/x509.genkey certs/extra_certificates certs/signing_key.x509.keyid \ > + certs/signing_key.x509.signer vmlinux-gdb.py > The logic seems quite simple, but I am not quite sure which file is still valid? [1] signing_key.pem - OK, this should be certs/signing_key.pem and removed by 'make mrproper' [2] signing_key.priv - deprecated by commit fb1179499134 ? [3] signing_key.x509 - OK, this should be certs/signing_key.x509 and removed by 'make mrproper' [4] x509.genkey - this is an intermediate file for generating signing_key.pem, but unneeded for installing external modules. Does it make more sense to delete this by 'make clean'? [5] extra_certificates - I am not sure where this is generated, and used [6] siging_key.x509.keyid - same as [5] [7] signing_key.x509.signer - same as [5] -- Best Regards Masahiro Yamada ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] kbuild: cleanup signing keys with mrproper 2017-05-03 4:37 ` Masahiro Yamada @ 2017-05-11 17:25 ` Stephen Hemminger 2017-06-05 9:12 ` Masahiro Yamada 0 siblings, 1 reply; 7+ messages in thread From: Stephen Hemminger @ 2017-05-11 17:25 UTC (permalink / raw) To: Masahiro Yamada Cc: Michal Marek, Linux Kbuild mailing list, Linux Kernel Mailing List, Stephen Hemminger, David Woodhouse, David Howells On Wed, 3 May 2017 13:37:08 +0900 Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > +CC David Woodhouse > +CC David Howells > > > 2017-04-15 6:54 GMT+09:00 Stephen Hemminger <stephen@networkplumber.org>: > > When 'make mrproper' is run it was supposed to remove the signing > > keys in the certs directory, but only the filename is given > > rather than the pathanme which is necessary to cause cleanup. > > > > Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> > > --- > > Makefile | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/Makefile b/Makefile > > index efa267a92ba6..04ca211552f7 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -1274,9 +1274,9 @@ MRPROPER_DIRS += include/config usr/include include/generated \ > > arch/*/include/generated .tmp_objdiff > > MRPROPER_FILES += .config .config.old .version .old_version \ > > Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ > > - signing_key.pem signing_key.priv signing_key.x509 \ > > - x509.genkey extra_certificates signing_key.x509.keyid \ > > - signing_key.x509.signer vmlinux-gdb.py > > + certs/signing_key.pem certs/signing_key.priv certs/signing_key.x509 \ > > + certs/x509.genkey certs/extra_certificates certs/signing_key.x509.keyid \ > > + certs/signing_key.x509.signer vmlinux-gdb.py > > > > The logic seems quite simple, > but I am not quite sure which file is still valid? > > > [1] signing_key.pem - OK, this should be certs/signing_key.pem > and removed by 'make mrproper' > > [2] signing_key.priv - deprecated by commit fb1179499134 ? > > [3] signing_key.x509 - OK, this should be certs/signing_key.x509 > and removed by 'make mrproper' > > [4] x509.genkey - this is an intermediate file for generating signing_key.pem, > but unneeded for installing external modules. > Does it make more sense to delete this by 'make clean'? > > [5] extra_certificates - I am not sure where this is generated, and used > > [6] siging_key.x509.keyid - same as [5] > > [7] signing_key.x509.signer - same as [5] Retested with current Linus tree: After build, these files are present: signing_key.x509 This file is never cleaned up by clean or proper. The changes that I think are necessary: certs/extra_certificates is no longer used, remove it from Makefile certs/signing_key.x509 should be removed by mrproper ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] kbuild: cleanup signing keys with mrproper 2017-05-11 17:25 ` Stephen Hemminger @ 2017-06-05 9:12 ` Masahiro Yamada 0 siblings, 0 replies; 7+ messages in thread From: Masahiro Yamada @ 2017-06-05 9:12 UTC (permalink / raw) To: Stephen Hemminger Cc: Michal Marek, Linux Kbuild mailing list, Linux Kernel Mailing List, Stephen Hemminger, David Woodhouse, David Howells Hi Stephen, Sorry for my late reply. 2017-05-12 2:25 GMT+09:00 Stephen Hemminger <stephen@networkplumber.org>: > On Wed, 3 May 2017 13:37:08 +0900 > Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > >> +CC David Woodhouse >> +CC David Howells >> >> >> 2017-04-15 6:54 GMT+09:00 Stephen Hemminger <stephen@networkplumber.org>: >> > When 'make mrproper' is run it was supposed to remove the signing >> > keys in the certs directory, but only the filename is given >> > rather than the pathanme which is necessary to cause cleanup. >> > >> > Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> >> > --- >> > Makefile | 6 +++--- >> > 1 file changed, 3 insertions(+), 3 deletions(-) >> > >> > diff --git a/Makefile b/Makefile >> > index efa267a92ba6..04ca211552f7 100644 >> > --- a/Makefile >> > +++ b/Makefile >> > @@ -1274,9 +1274,9 @@ MRPROPER_DIRS += include/config usr/include include/generated \ >> > arch/*/include/generated .tmp_objdiff >> > MRPROPER_FILES += .config .config.old .version .old_version \ >> > Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ >> > - signing_key.pem signing_key.priv signing_key.x509 \ >> > - x509.genkey extra_certificates signing_key.x509.keyid \ >> > - signing_key.x509.signer vmlinux-gdb.py >> > + certs/signing_key.pem certs/signing_key.priv certs/signing_key.x509 \ >> > + certs/x509.genkey certs/extra_certificates certs/signing_key.x509.keyid \ >> > + certs/signing_key.x509.signer vmlinux-gdb.py >> > >> >> The logic seems quite simple, >> but I am not quite sure which file is still valid? >> >> >> [1] signing_key.pem - OK, this should be certs/signing_key.pem >> and removed by 'make mrproper' >> >> [2] signing_key.priv - deprecated by commit fb1179499134 ? >> >> [3] signing_key.x509 - OK, this should be certs/signing_key.x509 >> and removed by 'make mrproper' >> >> [4] x509.genkey - this is an intermediate file for generating signing_key.pem, >> but unneeded for installing external modules. >> Does it make more sense to delete this by 'make clean'? >> >> [5] extra_certificates - I am not sure where this is generated, and used >> >> [6] siging_key.x509.keyid - same as [5] >> >> [7] signing_key.x509.signer - same as [5] > > Retested with current Linus tree: > > After build, these files are present: > signing_key.x509 > > This file is never cleaned up by clean or proper. > > The changes that I think are necessary: > certs/extra_certificates is no longer used, remove it from Makefile > > certs/signing_key.x509 should be removed by mrproper I agree. I'd like obsolete files dropped from the list before fixing the file paths. -- Best Regards Masahiro Yamada ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] kbuild: remove initramfs cpio with mrproper 2017-04-14 21:54 [PATCH 0/2] make mrproper cleanup more file Stephen Hemminger 2017-04-14 21:54 ` [PATCH 1/2] kbuild: cleanup signing keys with mrproper Stephen Hemminger @ 2017-04-14 21:54 ` Stephen Hemminger 2017-05-03 4:10 ` Masahiro Yamada 1 sibling, 1 reply; 7+ messages in thread From: Stephen Hemminger @ 2017-04-14 21:54 UTC (permalink / raw) To: yamada.masahiro, mmarek; +Cc: linux-kbuild, linux-kernel, Stephen Hemminger When 'make mrproper' is done, it should also remove the initramfs cpio file. I ran into this while doing test build on one machine, followed by make mrproper and rsync to a target machine. The build on the target machine would succeed but be unbootable because of the bad initramfs. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> --- Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 04ca211552f7..954292695bf6 100644 --- a/Makefile +++ b/Makefile @@ -1276,7 +1276,8 @@ MRPROPER_FILES += .config .config.old .version .old_version \ Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ certs/signing_key.pem certs/signing_key.priv certs/signing_key.x509 \ certs/x509.genkey certs/extra_certificates certs/signing_key.x509.keyid \ - certs/signing_key.x509.signer vmlinux-gdb.py + certs/signing_key.x509.signer vmlinux-gdb.py \ + usr/initramfs_data.cpio.gz # clean - Delete most, but leave enough to build external modules # -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] kbuild: remove initramfs cpio with mrproper 2017-04-14 21:54 ` [PATCH 2/2] kbuild: remove initramfs cpio " Stephen Hemminger @ 2017-05-03 4:10 ` Masahiro Yamada 0 siblings, 0 replies; 7+ messages in thread From: Masahiro Yamada @ 2017-05-03 4:10 UTC (permalink / raw) To: Stephen Hemminger Cc: Michal Marek, Linux Kbuild mailing list, Linux Kernel Mailing List, Stephen Hemminger Hi Stephen, 2017-04-15 6:54 GMT+09:00 Stephen Hemminger <stephen@networkplumber.org>: > When 'make mrproper' is done, it should also remove the initramfs cpio > file. I ran into this while doing test build on one machine, followed > by make mrproper and rsync to a target machine. The build on the target > machine would succeed but be unbootable because of the bad initramfs. I think initramfs_data.cpio.* is unneeded for external modules. So, shouldn't it removed by 'make clean', instead of 'make mrproper'? > Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> > --- > Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 04ca211552f7..954292695bf6 100644 > --- a/Makefile > +++ b/Makefile > @@ -1276,7 +1276,8 @@ MRPROPER_FILES += .config .config.old .version .old_version \ > Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ > certs/signing_key.pem certs/signing_key.priv certs/signing_key.x509 \ > certs/x509.genkey certs/extra_certificates certs/signing_key.x509.keyid \ > - certs/signing_key.x509.signer vmlinux-gdb.py > + certs/signing_key.x509.signer vmlinux-gdb.py \ > + usr/initramfs_data.cpio.gz > As you see usr/Makefile, datafile_y = initramfs_data.cpio$(suffix_y) The suffix could be .gz, .bz2, .xz, etc. Why only initramfs_data.cpio.gz? -- Best Regards Masahiro Yamada ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-06-05 9:12 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-04-14 21:54 [PATCH 0/2] make mrproper cleanup more file Stephen Hemminger 2017-04-14 21:54 ` [PATCH 1/2] kbuild: cleanup signing keys with mrproper Stephen Hemminger 2017-05-03 4:37 ` Masahiro Yamada 2017-05-11 17:25 ` Stephen Hemminger 2017-06-05 9:12 ` Masahiro Yamada 2017-04-14 21:54 ` [PATCH 2/2] kbuild: remove initramfs cpio " Stephen Hemminger 2017-05-03 4:10 ` Masahiro Yamada
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).