From: Ingo Molnar <mingo@kernel.org>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Thomas Gleixner <tglx@linutronix.de>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Andrew Morton <akpm@linux-foundation.org>,
Borislav Petkov <bp@alien8.de>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Jonathan Corbet <corbet@lwn.net>
Subject: [Tree, v2] De-clutter the top level directory, move ipc/ => kernel/ipc/, samples/ => Documentation/samples/ and sound/ => drivers/sound/
Date: Tue, 24 Sep 2019 13:31:35 +0200 [thread overview]
Message-ID: <20190924113135.GA82089@gmail.com> (raw)
In-Reply-To: <20190923200818.GA116090@gmail.com>
* Ingo Molnar <mingo@kernel.org> wrote:
> Oh, that's a pleasant surprise, I didn't expect _100%_ support! :-)
>
> So I started working on this today and whipped up three of these
> movements, in a 100% scripted fashion.
>
> You can have a sneak preview at the result in this tree:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.core/toplevel
>
> ...
>
> 2515 files changed, 42476 insertions(+), 42476 deletions(-)
I have pushed out a -v2 version:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.core/toplevel
2523 files changed, 41304 insertions(+), 41302 deletions(-)
Changes relative to -v1:
- Fixed a bunch of bugs that light testing and light review missed:
missed rename patterns and some build bugs as well. This tree has
passed much wider testing, including cross-platform build testing, a
distro kernel package build and it also got some light boot testing,
just in case.
- Split it into finer grained steps (3 instead of 2 patches per
movement), for easier review and bisection testing:
toplevel: Move ipc/ to kernel/ipc/: move the files
toplevel: Move ipc/ to kernel/ipc/: adjust the build system
toplevel: Move ipc/ to kernel/ipc/: adjust comments and documentation
toplevel: Move sound/ to drivers/sound/: move the files
toplevel: Move sound/ to drivers/sound/: adjust the build system
toplevel: Move sound/ to drivers/sound/: adjust comments and documentation
toplevel: Move samples/ to Documentation/samples/: move the files
toplevel: Move samples/ to Documentation/samples/: adjust the build system
toplevel: Move samples/ to Documentation/samples/: adjust comments and documentation
- The changes are now bisection safe if the #1 ('move') and #2 ('build
system') patches are squashed. The final #3 'adjust comments and
documentation' patch is non-functional in the normal kernel build.
I still kept the three steps separate, for reviewability: for many of
the changes the build system changes are lost in the noise of the
file movement diff itself. (See patch-splitting notes further below.)
- Made some of the build system changes less ad-hoc - but it's still all
100% scripted and automated. Added a SOB to the changelogs, but the
changelogs are still barebones. It's on top of Linus's latestest.
- The longer term plan outlined in my first mail is still in flux - the
'scripts/' movement is probaly a bad idea due to its widespread use.
I'm still torn about whether to do a 3-part or 2-part approach for each
directory movement:
- The problem with the 2-part approach that merges the 'pure file move'
and 'build system' patches so that the latter gets lost in the first
one in an almost unreviewable fashion. Someone would have to re-do the
git mv step and generate a diff by hand to see the build system
changes in isolation...
- The problem with the 3-part approach is that it breaks bisection
between the first two patches, although 'git bisect next' would always
step to a working commit, because the bisection build-breakage window
is only one commit wide.
So I'm slightly leaning toward the 3-part approach for the documentation
and review value - but no strong feelings either way.
Anyway, I know everyone is super busy with the merge window, will keep
posting new versions every now and then until you or Linus tells me not
to bother :-)
Thanks,
Ingo
next prev parent reply other threads:[~2019-09-24 11:31 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-17 19:26 [GIT PULL] Kselftest update for Linux 5.4-rc1 Shuah Khan
2019-09-20 16:17 ` Linus Torvalds
2019-09-20 16:26 ` Randy Dunlap
[not found] ` <CAKRRn-edxk9Du70A27V=d3Na73fh=fVvGEVsQRGROrQm05YRrA@mail.gmail.com>
2019-09-20 16:35 ` Brendan Higgins
2019-09-20 16:51 ` Linus Torvalds
2019-09-20 18:03 ` Brendan Higgins
2019-09-20 18:14 ` Linus Torvalds
2019-09-20 18:16 ` Brendan Higgins
2019-09-20 18:06 ` Shuah Khan
2019-09-22 11:25 ` Ingo Molnar
2019-09-22 11:52 ` Greg KH
2019-09-23 14:44 ` Shuah Khan
2019-09-23 19:43 ` Ingo Molnar
2019-09-23 19:52 ` Randy Dunlap
2019-09-23 20:29 ` Shuah Khan
2019-09-23 20:53 ` Ingo Molnar
2019-09-23 21:11 ` Shuah Khan
2019-09-23 20:08 ` [RFC, Tree] De-clutter the top level directory, move ipc/ => kernel/ipc/, samples/ => Documentation/samples/ and sound/ => drivers/sound/ Ingo Molnar
2019-09-24 11:31 ` Ingo Molnar [this message]
2019-09-24 15:28 ` [Tree, v2] " Eric W. Biederman
2019-09-24 18:41 ` Ingo Molnar
2019-09-25 23:00 ` Eric W. Biederman
2019-09-23 23:54 ` [GIT PULL] Kselftest update for Linux 5.4-rc1 Tim.Bird
2019-09-24 8:07 ` Ingo Molnar
2019-09-26 12:52 ` David Sterba
2019-09-27 13:52 ` Pavel Machek
2019-10-03 9:08 ` Masahiro Yamada
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=20190924113135.GA82089@gmail.com \
--to=mingo@kernel.org \
--cc=a.p.zijlstra@chello.nl \
--cc=akpm@linux-foundation.org \
--cc=bp@alien8.de \
--cc=corbet@lwn.net \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).