If concerned about syn flooding, those rules should be on any router with a public IP address. With one huge recommendation, set up a test router with your PPPoE setup and make sure the rules don’t block any PPPoE traffic they’re not supposed to (I’m not familiar with PPPoE, but I have enabled rules that have had major effects on our traffic).
As far as if the rules are set up properly, check the rules to see if the flags are set properly, or to make it easier, you could just export the config for your firewall filter and compare it to the rules you are looking at:
The rule appears to be correct, but Winbox does not show everything on the line display, if you want to post the output of those lines after running the code I posted, I can say that they are correct.
The other thing I’m seeing is since you are using PPPoE behind a router, you might want to consider changing the “limit” parameter to “dst-limit” as described in this post: Does the SYN protect chain really protect anything?
So the code would be: