lkml.org 
[lkml]   [2011]   [Jan]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH -tip 1/2] [PATCH -tip 1/2] sched: Reinstate group names in /proc/sched_debug.
    On Tue, Jan 11, 2011 at 08:46:58PM +0800, Yong Zhang wrote:
    > On Tue, Jan 11, 2011 at 03:41:54PM +0530, Bharata B Rao wrote:
    > > sched: Reinstate group names in /proc/sched_debug.
    > >
    > > Displaying of group names in /proc/sched_debug was dropped in autogroup
    > > patches. Add group names while displaying cfs_rq and tasks information.
    > >
    > > Signed-off-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
    > > ---
    > > kernel/sched_debug.c | 35 +++++++++++++++++++++++++++++++++++
    > > 1 file changed, 35 insertions(+)
    > >
    > > --- a/kernel/sched_debug.c
    > > +++ b/kernel/sched_debug.c
    > > @@ -16,6 +16,8 @@
    > > #include <linux/kallsyms.h>
    > > #include <linux/utsname.h>
    > >
    > > +static DEFINE_SPINLOCK(sched_debug_lock);
    >
    > I don't get your point on introducing this lock.
    > Just to avoid concurrent access?

    Yes.

    > Or could you explain please?

    The group name is generated in a global buffer which is protected by
    this lock. Earlier small sized local buffer (on stack) was used for
    this purpose which wasn't ideal. Also since we can be here from
    interrupt context, I wanted to avoid allocation too. Hence went for
    a global buffer protected by a lock.

    Regards,
    Bharata.


    \
     
     \ /
      Last update: 2011-01-11 14:45    [W:0.033 / U:91.488 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site