netdev - Re: [PATCH 1/6] Convert bonding timers to workqueues

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, 23 Oct 2007 20:42:27 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	Jay Vosburgh <fubar@...ibm.com>
CC:	netdev@...r.kernel.org, andy@...yhouse.net
Subject: Re: [PATCH 1/6] Convert bonding timers to workqueues

Jay Vosburgh wrote:
> 	Convert bonding timers to workqueues.  This converts the various
> monitor functions to run in periodic work queues instead of timers.  This
> patch introduces the framework and convers the calls, but does not resolve
> various locking issues, and does not stand alone.
> 
> Signed-off-by: Andy Gospodarek <andy@...yhouse.net>
> Signed-off-by: Jay Vosburgh <fubar@...ibm.com>
> ---
>  drivers/net/bonding/bond_3ad.c   |    6 +-
>  drivers/net/bonding/bond_3ad.h   |    2 +-
>  drivers/net/bonding/bond_alb.c   |    6 +-
>  drivers/net/bonding/bond_alb.h   |    2 +-
>  drivers/net/bonding/bond_main.c  |  147 ++++++++++++++++++++-----------------
>  drivers/net/bonding/bond_sysfs.c |   63 +++++-----------
>  drivers/net/bonding/bonding.h    |   13 ++--
>  7 files changed, 117 insertions(+), 122 deletions(-)

applied patches 1-6

However, two issues:

1) Credit.  Did Andy write these, as the Signed-off-by lines indicate? 
If so, you should reflect that with a

	From: Andy Gospodarek <andy@...yhouse.net>

as the first line of your email body.  Linus's git tool will 
automatically notice this special first-line header, and appropriately 
credit the right person.

2) vague subject line.  _Every_ subject line is copied verbatim into the 
kernel changelog, preserved as a permanent one-line summary of the 
changeset.  As such, with considered in the context of the entire kernel,

	Convert more locks to _bh, acquire rtnl, for new locking

tells us nothing at all about the portion of the kernel being modified.

In the future, make sure to include "bonding" somewhere in the subject 
line.  Generally a "bonding: " prefix would probably be best, but it can 
fall elsewhere as your patch #1 subject line indicates.

I hand-edited these subject lines to add "bonding: " prefix before I 
applied the patchset.

-
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