Kevin,
I have changed the code of this one after you took the snapshot.
The initial if() is gone and I have kept the switch. We need
to reject on UNLOADED or when LOADED and not sampling format nor buffer.
On Wed, Sep 05, 2007 at 09:10:19AM -0500, Kevin Corry wrote:
> This switch statement in perfmon/perfmon.c doesn't seem to have any effect,
> so we should be be able to remove it.
>
> Signed-off-by: Kevin Corry <[EMAIL PROTECTED]>
> ---
>
> perfmon/perfmon.c | 7 -------
> 1 files changed, 0 insertions(+), 7 deletions(-)
>
> diff --git a/perfmon/perfmon.c b/perfmon/perfmon.c
> index ff0765a..6d1b56e 100644
> --- a/perfmon/perfmon.c
> +++ b/perfmon/perfmon.c
> @@ -1145,13 +1145,6 @@ int __pfm_restart(struct pfm_context *ctx, int
> *complete_needed)
> return -EBUSY;
> }
>
> - switch (state) {
> - case PFM_CTX_MASKED:
> - break;
> - case PFM_CTX_LOADED:
> - if (ctx->smpl_addr && ctx->smpl_fmt->fmt_restart)
> - break;
> - }
> __get_cpu_var(pfm_stats).pfm_restart_count++;
> /*
> * at this point, the context is either LOADED or MASKED
> _______________________________________________
> perfmon mailing list
> [email protected]
> http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/
--
-Stephane
_______________________________________________
perfmon mailing list
[email protected]
http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/
