netdev - Re: [GIT] Networking

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] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 25 Oct 2011 13:46:11 +0200
From:	Linus Torvalds <torvalds@...ux-foundation.org>
To:	David Miller <davem@...emloft.net>, Greg KH <gregkh@...e.de>,
	"Eric W. Biederman" <ebiederm@...ssion.com>,
	Mikulas Patocka <mpatocka@...hat.com>
Cc:	akpm@...ux-foundation.org, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [GIT] Networking

On Tue, Oct 25, 2011 at 12:32 PM, David Miller <davem@...emloft.net> wrote:
>
> I have to say I was really impressed with GIT's rename detection.  For
> example, when I'd pull bug fixes from the 'net' tree into this
> 'net-next' tree, GIT would figure it out and cleanly merge in the
> fixes even though the file names were completely different.

Sadly, it doesn't work with files that aren't moved, but are split
into lots of files. So I got a conflict on the drivers/net/Kconfig
file due to the file being moved to tons of different locations. But
the change was pretty trivial, so I think I got that trivially right
too.

> I'll also note that I had to bump GIT's diff.renamelimit to about 1000
> to generate this pull request.

Yeah. You can probably just set it to zero to disable the limiting
entirely. The reason for the rename limit is for crazy use cases on
smaller machines than you'll probably ever use. So the git default
values tend to try to be safe for people who have smaller machines.

> I think that, without the driver renames, this is actually a smaller
> networking pull request than usual for the merge window.

This had an annoying conflict with some of the driver base changes. It
was made doubly annoying by the fact that Greg seems to use some
seriously bad drugs, and creates totally crap commit messages that are
just annoying when you have to look at them because there's some
conflict.

Greg - please fix your crazy tools. Look at this:

   git log fs/sysfs/dir.c

and tell me why the f*&% you have commit messages like this:

  commit 4f72c0cab40536a0be501d85ea4918467ab82ad5
  Author: Mikulas Patocka <mpatocka@...hat.com>
  Date:   Mon Jul 25 17:55:57 2011 -0400

      sysfs: use rb-tree for name lookups

      sysfs: use rb-tree for name lookups

      Use red-black tree for name lookups.

      Signed-off-by: Mikulas Patocka <mpatocka@...hat.com>
      Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>

and there's a *lot* of them. That duplication of the header line is
useless. It is *doubly* useless to then have a "body" of the commit
that just says the same thing *again*. Christ!

EVERY SINGLE COMMIT from Mikulas has that problem. WHY?

So stop duplicating the header line! And make the body of the commit
actually tell us something *useful*.

Anyway, after that rant about really bad practices, let me say that I
did fix up the conflict and I think it's right. But I won't guarantee
it, so please check the changes to fs/sysfs/dir.c.

I added Mikulas, Greg and Eric Biederman to the email, because those
three people seem to be the ones that caused the conflicts in that
file. Please do check out my conflict resolution.

                   Linus
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html