• Suzuki K Poulose's avatar
    perf cs-etm: Fix bitmap for option · 6fc5baf5
    Suzuki K Poulose authored
    When set option with macros ETM_OPT_CTXTID and ETM_OPT_TS, it wrongly
    takes these two values (14 and 28 prespectively) as bit masks, but
    actually both are the offset for bits.  But this doesn't lead to
    further failure due to the AND logic operation will be always true for
    ETM_OPT_CTXTID / ETM_OPT_TS.
    
    This patch defines new independent macros (rather than using the
    "config" bits) for requesting the "contextid" and "timestamp" for
    cs_etm_set_option().
    Signed-off-by: default avatarSuzuki Poulouse <suzuki.poulose@arm.com>
    Reviewed-by: default avatarMike Leach <mike.leach@linaro.org>
    Cc: Al Grant <al.grant@arm.com>
    Cc: Daniel Kiss <daniel.kiss@arm.com>
    Cc: Denis Nikitin <denik@chromium.org>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: John Garry <john.garry@huawei.com>
    Cc: Jonathan Corbet <corbet@lwn.net>
    Cc: Leo Yan <leo.yan@linaro.org>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Suzuki Poulouse <suzuki.poulose@arm.com>
    Cc: Will Deacon <will@kernel.org>
    Cc: coresight@lists.linaro.org
    Cc: linux-arm-kernel@lists.infradead.org
    Cc: linux-doc@vger.kernel.org
    Link: http://lore.kernel.org/lkml/20210206150833.42120-5-leo.yan@linaro.org
    [ Extract the change as a separate patch for easier review ]
    Signed-off-by: default avatarLeo Yan <leo.yan@linaro.org>
    Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    6fc5baf5
cs-etm.c 23.3 KB