diff options
| author | stephane.eranian@hp.com <stephane.eranian@hp.com> | 2005-07-28 05:18:00 -0700 |
|---|---|---|
| committer | Tony Luck <tony.luck@intel.com> | 2005-08-10 16:21:58 -0700 |
| commit | 6bf11e8c708f0e512ed733fc65a50770c5bc7b54 (patch) | |
| tree | efa54f2f19580f330ae71c3a9476d2261b172142 /arch/ia64 | |
| parent | bc68552faad0e134eb22281343d5ae5a4873fa80 (diff) | |
| download | op-kernel-dev-6bf11e8c708f0e512ed733fc65a50770c5bc7b54.zip op-kernel-dev-6bf11e8c708f0e512ed733fc65a50770c5bc7b54.tar.gz | |
[IA64] fix perfmon context load
The PFM_LOAD_CONTEXT may fail silently and cause a session
to remain reserved even though it should not. This can happen
when the commands succeeds in reserving the session but fails
when it actually tries to attach to the load_pid. In that case,
the command has failed but will return 0. More importantly,
the session will remain reserved. This patch fixes the problem.
Signed-off-by: <stephane.eranian@hp.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64')
| -rw-r--r-- | arch/ia64/kernel/perfmon.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/arch/ia64/kernel/perfmon.c b/arch/ia64/kernel/perfmon.c index b8ebb8e..f1201ac 100644 --- a/arch/ia64/kernel/perfmon.c +++ b/arch/ia64/kernel/perfmon.c @@ -4312,6 +4312,7 @@ pfm_context_load(pfm_context_t *ctx, void *arg, int count, struct pt_regs *regs) DPRINT(("before cmpxchg() old_ctx=%p new_ctx=%p\n", thread->pfm_context, ctx)); + ret = -EBUSY; old = ia64_cmpxchg(acq, &thread->pfm_context, NULL, ctx, sizeof(pfm_context_t *)); if (old != NULL) { DPRINT(("load_pid [%d] already has a context\n", req->load_pid)); |
