workqueue: concurrency managed workqueue, take#5
in [Kernel]
|
Prev: [PATCH 02/30] acpi: use queue_work_on() instead of binding workqueue worker to cpu0
Next: [PATCH 6/6] hvc_console: Fix race between hvc_close and hvc_remove
From: Andrew Morton on 14 Jun 2010 18:10 On Mon, 14 Jun 2010 23:37:17 +0200 Tejun Heo <tj(a)kernel.org> wrote: > This is the fifth take of cmwq (concurrency managed workqueue) > patchset. What is a concurrency managed workqueue and why do we want one? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Tejun Heo on 14 Jun 2010 18:20 On 06/14/2010 11:58 PM, Andrew Morton wrote: > On Mon, 14 Jun 2010 23:37:17 +0200 > Tejun Heo <tj(a)kernel.org> wrote: > >> This is the fifth take of cmwq (concurrency managed workqueue) >> patchset. > > What is a concurrency managed workqueue and why do we want one? That goes back about 9 month. You were cc'd there too. :-) http://thread.gmane.org/gmane.linux.kernel/896268 Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Tejun Heo on 14 Jun 2010 18:40 On 06/15/2010 12:31 AM, Daniel Walker wrote: > Could you explain how you view "concurrency" w.r.t this patchset? Hmm... can you be a bit more specific? I don't understand what the question is. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Daniel Walker on 14 Jun 2010 18:40 On Tue, 2010-06-15 at 00:33 +0200, Tejun Heo wrote: > On 06/15/2010 12:31 AM, Daniel Walker wrote: > > Could you explain how you view "concurrency" w.r.t this patchset? > > Hmm... can you be a bit more specific? I don't understand what the > question is. What is you definition of "concurrency" ? and explain try to explain it in terms of your patchset. Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Andrew Morton on 14 Jun 2010 18:40
On Tue, 15 Jun 2010 00:17:25 +0200 Tejun Heo <tj(a)kernel.org> wrote: > On 06/14/2010 11:58 PM, Andrew Morton wrote: > > On Mon, 14 Jun 2010 23:37:17 +0200 > > Tejun Heo <tj(a)kernel.org> wrote: > > > >> This is the fifth take of cmwq (concurrency managed workqueue) > >> patchset. > > > > What is a concurrency managed workqueue and why do we want one? > > That goes back about 9 month. You were cc'd there too. :-) > > http://thread.gmane.org/gmane.linux.kernel/896268 > Nobody's going to remember all that stuff except yourself, and the info might be out of date. So please update and maintain that information and retain it with the patchset. eg: "<NEED SOME BACKING NUMBERS>". And "Please read the patch description of the last patch for more details" is out of date. Because right now I have a bunch of code in my inbox and little (actually "no") idea why anyone might want to merge it into anything. Trying to review a large patchset when you don't have an overall picture of what it's trying to do and how it's trying to do it is rather painful - you have to work all that stuff out from the implementation. It's also error-prone if the implementation doesn't implement that which the author thinks it implements (ie: if it has design bugs). -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |