From: Max Krasnyansky <maxk@qualcomm.com>
To: Peter Zijlstra <a.p.zijlstra@chello.nl>, oleg@tv-sign.ru, jarkao2@o2.pl
Cc: linux-kernel@vger.kernel.org
Subject: workqueue flush_work() patches
Date: Fri, 27 Jun 2008 10:40:14 -0700 [thread overview]
Message-ID: <486525FE.1060509@qualcomm.com> (raw)
Peter,
Oleg,
I'm not sure if you guys saw my last email on this. So I'll restart the
thread.
If you guys are ok with the summary I provided below I can put all
Oleg's patches into some git tree, test them on my boxes and resend to
Andrew. I was also going to go over the users of flush_queued_work() and
convert them to cancel_work_sync() and/or flush_work(). So I need to
know if we want to go ahead with the flush_work() patches.
Please see summary below and let me know what you guys think.
Thanx
Max
----
Peter Zijlstra wrote:
> Anyway, I think before we go further down this road, we'd better see
> if anybody actually needs this. Not that theorizing about this problem
> isn't fun,... but... :-)
Let me see if I can sum up current state of affairs. Looks like people
are in general ok with Oleg's patches. Fancier stuff is much more
complex and may not be needed.
Combining Oleg's patches with auditing current flush_scheduled_work()
users and fixing them to use cancel_work_sync() (and in some cases
flush_work()) gives us desired behaviour. Which is:
1. minimizing flush overhead
2. handling (actually avoiding) work queue thread starvation
Does that sound right ? Or did I miss something in the discussion ?
If that sounds right we should resend the patches to Andrew with formal
ACKs because I do not seem them in mainline, linux-next or -mm.
Thanks
Max
next reply other threads:[~2008-06-27 17:40 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-27 17:40 Max Krasnyansky [this message]
2008-06-27 17:56 ` workqueue flush_work() patches Oleg Nesterov
2008-06-27 18:07 ` Max Krasnyansky
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=486525FE.1060509@qualcomm.com \
--to=maxk@qualcomm.com \
--cc=a.p.zijlstra@chello.nl \
--cc=jarkao2@o2.pl \
--cc=linux-kernel@vger.kernel.org \
--cc=oleg@tv-sign.ru \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.