Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-24973

Performance schema duplicates rarely executed code for mutex operations

Details

    Description

      The PERFORMANCE_SCHEMA wrapper for mutex and rw-lock operations is causing a lot of unlikely code to be inlined in each invocation. The impact of this may have been emphasized in MariaDB 10.6, because InnoDB now uses the common implementation of mutexes and condition variables (MDEV-21452).

      By default, we build with cmake -DPLUGIN_PERFSCHEMA enabled, but at runtime no instrumentation will be enabled. Similar to MDEV-22721, we’d better avoid inlining the rarely executed code in order to reduce the code size and to improve the efficiency of the instruction cache.

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. MDEV-21452 Use condition variables and normal mutexes instead of InnoDB os_event and mutex

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. MDEV-22721 Remove bloat caused by InnoDB logger class

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. MDEV-33181 Performance schema introduces many conditional branches even when disabled at runtime

          • Major - Major loss of function.
          • Open

          Activity

            Ascending order - Click to sort in descending order
            marko Marko Mäkelä added a comment -

            axel, your test with performance schema disabled at runtime did not show any regression. Can you please repeat the test with performance schema enabled at runtime?

            marko Marko Mäkelä added a comment - axel , your test with performance schema disabled at runtime did not show any regression. Can you please repeat the test with performance schema enabled at runtime?
            axel Axel Schwenke added a comment -

            commit 522fd37c3e0 vs. da8bc5ee9b0

            axel Axel Schwenke added a comment - commit 522fd37c3e0 vs. da8bc5ee9b0

            People

              marko Marko Mäkelä
              marko Marko Mäkelä
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.