From b4f6c7606e6c99a2b5fa6808d97bdf9f3c76586b Mon Sep 17 00:00:00 2001
From: wuying39 <wuying53@huawei.com>
Date: Tue, 2 Dec 2025 18:46:04 +0800
Subject: [PATCH] modify the verification of SPE event filter

---
 pmu/pmu.cpp | 33 +++++++++++++++++----------------
 1 file changed, 17 insertions(+), 16 deletions(-)

diff --git a/pmu/pmu.cpp b/pmu/pmu.cpp
index 6fd27d4..3825a6c 100644
--- a/pmu/pmu.cpp
+++ b/pmu/pmu.cpp
@@ -204,7 +204,8 @@ static int CheckCgroupNameList(unsigned numCgroup, char** cgroupName)
     return SUCCESS;
 }
 
-static bool CheckValidSpeEvtFilter(SpeEventFilter val) {
+static bool CheckValidSpeEvtFilter(SpeEventFilter val)
+{
     switch (val) {
         case SPE_EVENT_NONE:
         case SPE_EVENT_RETIRED:
@@ -217,21 +218,21 @@ static bool CheckValidSpeEvtFilter(SpeEventFilter val) {
     }
 }
 
-static bool CheckValidSpeFilter(SpeFilter val) {
-    switch (val) {
-        case SPE_FILTER_NONE:
-        case TS_ENABLE:
-        case PA_ENABLE:
-        case PCT_ENABLE:
-        case JITTER:
-        case BRANCH_FILTER:
-        case LOAD_FILTER:
-        case STORE_FILTER:
-        case SPE_DATA_ALL:
-            return true;
-        default:
-            return false;
-    }
+static bool CheckValidSpeFilter(SpeFilter val)
+{
+    const unsigned long validMask =
+        SPE_FILTER_NONE |
+        TS_ENABLE |
+        PA_ENABLE |
+        PCT_ENABLE |
+        JITTER |
+        BRANCH_FILTER |
+        LOAD_FILTER |
+        STORE_FILTER;
+    if ((val & ~validMask) == 0) {
+        return true;
+    }
+    return false;
 }
 
 static int CheckCollectTypeConfig(enum PmuTaskType collectType, struct PmuAttr *attr)
-- 
Gitee