lkml.org 
[lkml]   [2012]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCHSET] blkcg: unify blkgs for different policies
    On Thu, Feb 02, 2012 at 12:36:09PM -0800, Tejun Heo wrote:
    > Hello,
    >
    > On Thu, Feb 02, 2012 at 02:29:58PM -0500, Vivek Goyal wrote:
    > > On Wed, Feb 01, 2012 at 01:19:05PM -0800, Tejun Heo wrote:
    > >
    > > [..]
    > > >
    > > > * use unified stats updated under queue lock and drop percpu stats
    > > > which should fix locking / context bug across percpu allocation.
    > >
    > > Does that mean that stat updation will happen under queue lock even if
    > > there are no throttling rules? That will introduce extra queue lock on
    > > fast path those who have throttling compiled in but are not using (common
    > > case for distributions).
    >
    > No, I don't think extra locking would be necessary at all. We end up
    > grabbing queue_lock anyway. It's just the matter of where to update
    > the stats.

    What about bio based drivers? They might have their own internal locking
    and not relying on queue lock. And conceptually, we first throttle the
    bio, update the stats and then call into driver. So are you planning
    to move blk_throtl_bio() call into request function of individual driver
    to avoid taking queue lock twice.

    In fact even for request based drivers, if bio is being merged onto
    existing request on plug, we will not take any queue lock. So I am not
    sure how would you avoid extra queue lock for this case.

    Thanks
    Vivek


    \
     
     \ /
      Last update: 2012-02-02 21:45    [W:8.499 / U:0.124 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site