lkml.org 
[lkml]   [2012]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH UPDATED 11/11] blkcg: unify blkg's for blkcg policies
On Fri, Feb 03, 2012 at 04:53:49PM -0500, Vivek Goyal wrote:
> Hmm... then blkiocg_reset_stats() will run into lock ordering issue. Can't
> hold queue lock inside blkcg lock. I guess you will do some kind of
> locking trick again as you did for io context logic.

Ummm... I don't know the details yet but decisions like making
policies per-queue are much higher level than locking details. There
are cases where implementation details become problematic enough and
high level decisions are affected but I don't really think that would
be the case here. It's not like reset stats is a hot path. In fact,
I don't even understand why the hell we have it. I guess it's another
blkcg braindamage.

If userland is interested in delta between certain periods, it's
supposed to read values at the start and end of the period and
subtract. Allowing reset might make sense if there's single bearded
admin looking at the raw numbers, but it breaks as soon as tools and
automation are involved. How would you decide who owns those
counters?

And we sadly can't even remove that at this point. Ugh................

--
tejun


\
 
 \ /
  Last update: 2012-02-03 23:17    [W:1.186 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site