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:	Mon, 15 Oct 2007 12:08:59 -0700
From:	Jay Vosburgh <fubar@...ibm.com>
To:	Jeff Garzik <jgarzik@...ox.com>
cc:	netdev@...r.kernel.org, andy@...yhouse.net
Subject: Re: [PATCH 1/6] Convert bonding timers to workqueues 

Jeff Garzik <jgarzik@...ox.com> wrote:

>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>
>
>"does not stand alone" == it is not bisectable?  That's a problem.

	The patch will compile fine (this is true for any point in the
series), but it's possible for bonding to deadlock or misbehave when
doing certain operations.  Those problems are addressed in the later
patches.

	I didn't want to just vomit out one big mega-patch that has the
whole set of changes, because that's harder to understand, and the later
patches generally address discrete issues.

	Does that make you more or less nervous about its bisectability?
I can repost the whole thing as a big blob if that's what you'd prefer.

>Anyway, we have more fun fish to fry:  after applying the IPoIB bonding
>patchset, this no longer applies...

	Yah, I'll rebase it (once I know your blob vs. not-blob
preference).  I wasn't sure how long the IPoIB mystery bug hunt was
going to take.

	-J

---
	-Jay Vosburgh, IBM Linux Technology Center, fubar@...ibm.com
-
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