Messages in this thread | | | Date | Wed, 28 Sep 2005 15:35:01 +0900 | From | KUROSAWA Takahiro <> | Subject | Re: [PATCH 1/3] CPUMETER: add cpumeter framework to the CPUSETS |
| |
On Tue, 27 Sep 2005 19:22:17 +1000 Nick Piggin <nickpiggin@yahoo.com.au> wrote:
> It could have come a long way since then, but this code looks > much neater than the code I reviewed.
I'm glad to hear that! But the cpu resource controller has some problems, for example:
- The controller only controls the time_slice value and doesn't care the balance of cpus (the controller leaves balancing to the existing balancing code in the scheduler). So far I don't have any good idea to solve this.
- The controller holds a spinlock once per 1 second. I don't know this is permissive or not, but the current scheduler doesn't hold any spinlocks normally, so...
-- KUROSAWA, Takahiro - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |