pm: fix suspend counter increment only on successful validation

The suspend counter was incremented even when VALIDATE_REQ()
failed, causing the counter to grow indefinitely on duplicate pause
requests. This led to continuous validation failures and required
multiple resume calls to exit the suspended state.

Move the counter increment inside the VALIDATE_REQ() conditional block
to ensure it only increments on successful validation.

Before fix: Multiple resume requests needed to match pause count
After fix: Single resume request properly exits suspended state
Reproduce sequence:
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b true
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b true
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b true
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b true
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b true

busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b false
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b false
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b false
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b false
busctl --user introspect org.clight.clight /org/clight/clight | grep Suspended
.Suspended                          property  b         true           emits-change
busctl --user call org.clight.clight /org/clight/clight org.clight.clight Pause b false
busctl --user introspect org.clight.clight /org/clight/clight | grep Suspended
.Suspended                          property  b         false          emits-change

Signed-off-by: LI Qingwu <[email protected]>

Author: Qingwu-Li

src/modules/pm.c

@@ -262,10 +262,10 @@ static void on_suspend_req(suspend_upd *up) {
                 M_PUB(suspend_msg);
             }
         }
+
+       /* Count currently held suspends */
+        if (up->new) {
+            suspend_ctr++;
+        }
     }
-    /* Count currently held suspends */
-    if (up->new) {
-        suspend_ctr++;
-    }
-    DEBUG("Suspend ctr: %d\n", suspend_ctr);
 }