On Mon, 26 Mar 2007, Jay Vosburgh wrote:
> Andrew Morton <[EMAIL PROTECTED]> wrote:
>
> >On Mon, 26 Mar 2007 21:50:01 -0800 [EMAIL PROTECTED] wrote:
> >
> >> Replace system timer with work queue in monitor functions. The reason for
> >> this change is that bonding handlers calls various sleeping functions from
> >> the timer handler which is not allowed. Because we cannot share the main
> >> workqueue threads (rtnl_lock is used also in linkwatch_event) - new bond
> >> workqueue thread is created.
> >
> >I have a note here that this patch needs additional work.
> >
> >I forget what it was, there has since been no followup and hence I shall now
> >drop this patch.
>
> Andy Gospodarek <[EMAIL PROTECTED]> and I have been working off
> list to come up with a comprehensive scheme to resolve the various
> locking problems in bonding all at once, rather than piecemeal.
> Currently, we believe we've identified all of the problem areas, have
> worked up a set of locking rules, and are now hashing out an
> implementation that (hopefully) doesn't suck.
>
> The patch referenced above is more or less a functional subset
> of the work we're doing, in that we'll also be moving the timers to
> workqueues.
When do you expect to finish your work? I think that common rule is to
have at least some code to fix the problem rather than none.
Jaroslav
-----
Jaroslav Kysela <[EMAIL PROTECTED]>
Linux Kernel Sound Maintainer
ALSA Project, SUSE Labs
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
