[PATCH 2/2] Cell OProfile: SPU mutex lock fix, version 4 - Kernel

This is a discussion on [PATCH 2/2] Cell OProfile: SPU mutex lock fix, version 4 - Kernel ; On Monday 25 August 2008, Paul Mackerras wrote: > > > Since rc4 is out now, I understand if you feel more comfortable with > > putting the patch into -next instead of -merge. > > Linus has been getting ...

+ Reply to Thread
Page 2 of 2 FirstFirst 1 2
Results 21 to 23 of 23

Thread: [PATCH 2/2] Cell OProfile: SPU mutex lock fix, version 4

  1. Re: [Cbe-oss-dev] powerpc/cell/oprofile: fix mutex locking for spu-oprofile

    On Monday 25 August 2008, Paul Mackerras wrote:
    >
    > > Since rc4 is out now, I understand if you feel more comfortable with
    > > putting the patch into -next instead of -merge.

    >
    > Linus has been getting stricter about only putting in fixes for
    > regressions and serious bugs (see his recent email to Dave Airlie on
    > LKML for instance). *I assume that the corruption is just in the data
    > that is supplied to userspace and doesn't extend to any kernel data
    > structures.


    That's right, please queue it for -next then.

    > > Note that the second patch is trivial and fixes an oopsable condition
    > > of the kernel, so at least that should still go into 2.6.27.

    >
    > OK, I'll cherry-pick that one for my next batch for Linus.


    Thanks,

    Arnd <><
    --
    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/

  2. Re: [Cbe-oss-dev] powerpc/cell/oprofile: fix mutex locking for spu-oprofile

    On Monday 25 August 2008, Arnd Bergmann wrote:
    > On Monday 25 August 2008, Paul Mackerras wrote:
    > >
    > > > Since rc4 is out now, I understand if you feel more comfortable with
    > > > putting the patch into -next instead of -merge.

    > >
    > > Linus has been getting stricter about only putting in fixes for
    > > regressions and serious bugs (see his recent email to Dave Airlie on
    > > LKML for instance). *I assume that the corruption is just in the data
    > > that is supplied to userspace and doesn't extend to any kernel data
    > > structures.

    >
    > That's right, please queue it for -next then.


    I just realized that this patch never made it into powerpc-next after
    all, neither benh nor paulus version. Whoever is handling it today,
    could you please pull

    master.kernel.org:/pub/scm/linux/kernel/git/arnd/cell-2.6.git merge

    to get this commit below. I have rebased it on top of the current
    benh/powerpc/next branch.

    Thanks,

    Arnd <><

    ---

    commit aa5810fa545515c9f383e3e649bd120bef9c7f29
    Author: Carl Love
    Date: Fri Aug 8 15:38:36 2008 -0700

    powerpc/cell/oprofile: fix mutex locking for spu-oprofile

    The issue is the SPU code is not holding the kernel mutex lock while
    adding samples to the kernel buffer.

    This patch creates per SPU buffers to hold the data. Data
    is added to the buffers from in interrupt context. The data
    is periodically pushed to the kernel buffer via a new Oprofile
    function oprofile_put_buff(). The oprofile_put_buff() function
    is called via a work queue enabling the funtion to acquire the
    mutex lock.

    The existing user controls for adjusting the per CPU buffer
    size is used to control the size of the per SPU buffers.
    Similarly, overflows of the SPU buffers are reported by
    incrementing the per CPU buffer stats. This eliminates the
    need to have architecture specific controls for the per SPU
    buffers which is not acceptable to the OProfile user tool
    maintainer.

    The export of the oprofile add_event_entry() is removed as it
    is no longer needed given this patch.

    Note, this patch has not addressed the issue of indexing arrays
    by the spu number. This still needs to be fixed as the spu
    numbering is not guarenteed to be 0 to max_num_spus-1.

    Signed-off-by: Carl Love
    Signed-off-by: Maynard Johnson
    Signed-off-by: Arnd Bergmann
    Acked-by: Acked-by: Robert Richter

    arch/powerpc/oprofile/cell/pr_util.h | 13 +
    arch/powerpc/oprofile/cell/spu_profiler.c | 4
    arch/powerpc/oprofile/cell/spu_task_sync.c | 236 ++++++++++++++++++++++++---
    drivers/oprofile/buffer_sync.c | 24 ++
    drivers/oprofile/cpu_buffer.c | 15 +
    drivers/oprofile/event_buffer.c | 2
    drivers/oprofile/event_buffer.h | 7
    include/linux/oprofile.h | 16 +
    drivers/oprofile/cpu_buffer.c | 4
    9 files changed, 284 insertions(+), 37 deletions(-)
    --
    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/

  3. Re: [Cbe-oss-dev] powerpc/cell/oprofile: fix mutex locking for spu-oprofile

    On 13.10.08 16:53:28, Arnd Bergmann wrote:
    > On Monday 25 August 2008, Arnd Bergmann wrote:
    > > On Monday 25 August 2008, Paul Mackerras wrote:
    > > >
    > > > > Since rc4 is out now, I understand if you feel more comfortable with
    > > > > putting the patch into -next instead of -merge.
    > > >
    > > > Linus has been getting stricter about only putting in fixes for
    > > > regressions and serious bugs (see his recent email to Dave Airlie on
    > > > LKML for instance). *I assume that the corruption is just in the data
    > > > that is supplied to userspace and doesn't extend to any kernel data
    > > > structures.

    > >
    > > That's right, please queue it for -next then.

    >
    > I just realized that this patch never made it into powerpc-next after
    > all, neither benh nor paulus version. Whoever is handling it today,
    > could you please pull
    >
    > master.kernel.org:/pub/scm/linux/kernel/git/arnd/cell-2.6.git merge
    >
    > to get this commit below. I have rebased it on top of the current
    > benh/powerpc/next branch.


    All powerpc oprofile patches are in this branch:

    git://git.kernel.org/pub/scm/linux/kernel/git/rric/oprofile.git powerpc-for-paul

    Pending patches are:

    Carl Love (1):
    powerpc/cell/oprofile: fix mutex locking for spu-oprofile

    Roel Kluin (1):
    powerpc/cell/oprofile: vma_map: fix test on overlay_tbl_offset

    Please pull from there.

    -Robert

    --
    Advanced Micro Devices, Inc.
    Operating System Research Center
    email: robert.richter@amd.com

    --
    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/

+ Reply to Thread
Page 2 of 2 FirstFirst 1 2