lkml.org 
[lkml]   [2012]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 06/11] blkcg: move refcnt to blkcg core
    Hello,

    On Thu, Feb 02, 2012 at 05:07:14PM -0500, Vivek Goyal wrote:
    > On Wed, Feb 01, 2012 at 01:19:11PM -0800, Tejun Heo wrote:
    > > Currently, blkcg policy implementations manage blkg refcnt duplicating
    > > mostly identical code in both policies. This patch moves refcnt to
    > > blkg and let blkcg core handle refcnt and freeing of blkgs.
    > >
    > > * cfq blkgs now also get freed via RCU.
    >
    > This can lead to situation where cfq root group (policy data) is still
    > around (yet to be freed after rcu perioed) but cfq has gone away
    > (cfq_exit_queue() followed by cfq_exit()). Does it matter? If some future
    > code is accessing cfqg under rcu, it can become a problem.

    If RCU is being used, the scope of access allowed under it should be
    defined strictly. It's basically adding another layer of locking and
    like any other locking, if the boundary isn't clearly defined, it's
    gonna cause a problem. For blkcg, from the current usage, it's enough
    to guarantee access to blkg proper including policy specific data and
    under that scope, it doesn't matter whether request or cgroup goes
    away.

    Thanks.

    --
    tejun


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