netdev - Re: [PATCH V2 net-next 14/15] drivers/net/typhoon.c: Use (pr|netdev)

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:	Mon, 22 Feb 2010 19:22:29 -0500
From:	David Dillow <dave@...dillows.org>
To:	David Miller <davem@...emloft.net>
Cc:	joe@...ches.com, netdev@...r.kernel.org
Subject: Re: [PATCH V2 net-next 14/15] drivers/net/typhoon.c: Use
 (pr|netdev)_<level> macro helpers

On Mon, 2010-02-22 at 15:43 -0800, David Miller wrote:
> Man you are picky :-/  Are you a control freak?

Sometimes, yes. Aren't you? ;)

> You can't even let Joe write his commit comments consistently with the
> ones he is using for all the other similar network driver transmations?

That's fine, I was offering a suggestion on improving the comments that
I felt were in line with messages in the past from you, Ingo, Linus, and
others regarding the quality of commit messages.

> What makes doing the same exact transformations to your favorite
> driver so special that it deserves a specially constructed commit
> message?

I expect Joe to continue to submit high quality patches, and to grow
into larger roles than the already important one he is doing now. He's
doing good work, and he's been easy to work with. I'd just like to see
the commit messages improve beyond giving a bullet list of changes one
could get from a quick scan of the patch -- adding the context and why
of the changes rather than a mechanical listing.

It doesn't matter for this driver, with its minimal user base. And it
may not be worth the effort for the transformation series. But I think
he's going to move into much larger projects and I believe it would be a
good habit to develop.

--
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