On 9/13/22 17:25, Athira Rajeev wrote:
> For PERF_SAMPLE_BRANCH_STACK sample type, different branch_sample_type,
> ie branch filters are supported. The testcase "bhrb_filter_map_test"
> tests the valid and invalid filter maps in different powerpc platforms.
> Update this testcase to include scenario to cover multiple branch
> filters at sametime. Since powerpc doesn't support multiple filters at
> sametime, expect failure during perf_event_open.
> 
> Reported-by: Disha Goel <disg...@linux.vnet.ibm.com>
> Signed-off-by: Athira Rajeev <atraj...@linux.vnet.ibm.com>
> ---
>  .../powerpc/pmu/sampling_tests/bhrb_filter_map_test.c    | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git 
> a/tools/testing/selftests/powerpc/pmu/sampling_tests/bhrb_filter_map_test.c 
> b/tools/testing/selftests/powerpc/pmu/sampling_tests/bhrb_filter_map_test.c
> index 8182647c63c8..605669d4e4cb 100644
> --- 
> a/tools/testing/selftests/powerpc/pmu/sampling_tests/bhrb_filter_map_test.c
> +++ 
> b/tools/testing/selftests/powerpc/pmu/sampling_tests/bhrb_filter_map_test.c
> @@ -96,6 +96,15 @@ static int bhrb_filter_map_test(void)
>               }
>       }
>  

Patch looks good to me.

Reviewed-By: Kajol Jain <kj...@linux.ibm.com>

Thanks,
Kajol Jain


> +     /*
> +      * Combine filter maps which includes a valid branch filter and an 
> invalid branch
> +      * filter. Example: any ( PERF_SAMPLE_BRANCH_ANY) and save_type
> +      * (PERF_SAMPLE_BRANCH_TYPE_SAVE).
> +      * The perf_event_open should fail in this case.
> +      */
> +     event.attr.branch_sample_type = PERF_SAMPLE_BRANCH_ANY | 
> PERF_SAMPLE_BRANCH_TYPE_SAVE;
> +     FAIL_IF(!event_open(&event));
> +
>       return 0;
>  }
>  

Reply via email to