>> + >> + /* events are matched against occurrences of e=ev1[,ev2,..] pattern */ >> + ret = regcomp(®, "e=([_[:alnum:]-],?)+", REG_EXTENDED); >> + if (ret) { >> + PMU_LOG(ERR, "Failed to compile event matching regexp"); >> + return -EINVAL; >> + } >> + >> + for (;;) { >> + if (regexec(®, pattern, 1, &rmatch, 0)) >> + break; >> + > >As with log parameters. Regex is harder to work with than a glob based syntax.
That regex is for --trace and --trace parameter is inherited from tracing library and that itself takes a regex. I don't that patch should tinker with existing behavior.