1. 10 Feb, 2021 1 commit
  2. 07 Feb, 2021 5 commits
  3. 05 Feb, 2021 7 commits
    • Marko Mäkelä's avatar
      MDEV-21452 fixup: Introduce trx_t::mutex_is_owner() · 487fbc2e
      Marko Mäkelä authored
      When we replaced trx_t::mutex with srw_mutex
      in commit 38fd7b7d
      we lost the SAFE_MUTEX instrumentation.
      Let us introduce a replacement and restore the assertions.
      487fbc2e
    • Marko Mäkelä's avatar
      455514c8
    • Marko Mäkelä's avatar
      MDEV-24789: Reduce sizeof(trx_lock_t) · 3e45f8e3
      Marko Mäkelä authored
      trx_lock_t::cond: Use pthread_cond_t directly, because no instrumentation
      will ever be used. This saves sizeof(void*) and removes some duplicated
      inline code.
      
      trx_lock_t::was_chosen_as_wsrep_victim: Fold into
      trx_lock_t::was_chosen_as_deadlock_victim.
      
      trx_lock_t::cancel, trx_lock_t::rec_cached, trx_lock_t::table_cached:
      Use only one byte of storage, reducing memory alignment waste.
      
      On AMD64 GNU/Linux, MDEV-24671 caused a sizeof(trx_lock_t) increase
      of 48 bytes (plus the PLUGIN_PERFSCHEMA overhead of trx_lock_t::cond).
      These changes should save 32 bytes.
      3e45f8e3
    • Marko Mäkelä's avatar
      Cleanup: Reduce some lock_sys.mutex contention · 465bdabb
      Marko Mäkelä authored
      lock_table(): Remove the constant parameter flags=0.
      
      lock_table_resurrect(): Merge lock_table_ix_resurrect() and
      lock_table_x_resurrect().
      
      lock_rec_lock(): Only acquire LockMutexGuard if lock_table_has()
      does not hold.
      465bdabb
    • Marko Mäkelä's avatar
      MDEV-24731 fixup: bogus assertion · de407e7c
      Marko Mäkelä authored
      DeadlockChecker::search(): Move a bogus assertion into a condition.
      If the current transaction is waiting for a table lock (on something
      else than an auto-increment lock), it is well possible that other
      transactions are holding not only a conflicting lock, but also an
      auto-increment lock.
      
      This mistake was noticed during the testing of MDEV-24731, but it was
      accidentally introduced in commit 5f463857.
      
      lock_wait_end(): Remove an unused variable, and add an assertion.
      de407e7c
    • Marko Mäkelä's avatar
      MDEV-24781 fixup: Adjust innodb.innodb-index-debug · c42ee8a7
      Marko Mäkelä authored
      Now that an INSERT into an empty table is replicated more efficiently
      during online ALTER, an old test case started to fail. Let us disable
      the MDEV-515 logic for the critical INSERT statement.
      c42ee8a7
    • Thirunarayanan Balathandayuthapani's avatar
      MDEV-24781 Assertion `mode == 16 || mode == 12 || fix_block->page.status !=... · 597510ad
      Thirunarayanan Balathandayuthapani authored
      MDEV-24781 Assertion `mode == 16 || mode == 12 || fix_block->page.status != buf_page_t::FREED' failed in buf_page_get_low
      
      This is caused by commit 3cef4f8f
      (MDEV-515). dict_table_t::clear() frees all the blob during
      rollback of bulk insert.But online log tries to read the
      freed blob while applying the log. It can be fixed if we
      truncate the online log during rollback of bulk insert operation.
      597510ad
  4. 04 Feb, 2021 1 commit
    • Marko Mäkelä's avatar
      MDEV-24731 Excessive mutex contention in DeadlockChecker::check_and_resolve() · 5f463857
      Marko Mäkelä authored
      The DeadlockChecker expects to be able to freeze the waits-for graph.
      Hence, it is best executed somewhere where we are not holding any
      additional mutexes.
      
      lock_wait(): Defer the deadlock check to this function, instead
      of executing it in lock_rec_enqueue_waiting(), lock_table_enqueue_waiting().
      
      DeadlockChecker::trx_rollback(): Merge with the only caller,
      check_and_resolve().
      
      LockMutexGuard: RAII accessor for lock_sys.mutex.
      
      lock_sys.deadlocks: Replaces lock_deadlock_found.
      
      trx_t: Clean up some comments.
      5f463857
  5. 03 Feb, 2021 1 commit
    • Monty's avatar
      MDEV-24750 Various corruptions caused by Aria subsystem... · eacefbca
      Monty authored
      The test case was setting aria_sort_buffer_size to MAX_ULONGLONG-1
      which was not handled gracefully by my_malloc() or safemalloc().
      Fixed by ensuring that the malloc functions returns 0 if the size
      is too big.
      I also added some protection to Aria repair:
      - Limit sort_buffer_size to 16G (after that a bigger sort buffer will
        not help that much anyway)
      - Limit sort_buffer_size also according to sort file size. This will
        help by not allocating less memory if someone sets the buffer size too
        high.
      eacefbca
  6. 02 Feb, 2021 3 commits
  7. 01 Feb, 2021 3 commits
  8. 30 Jan, 2021 3 commits
  9. 29 Jan, 2021 5 commits
    • Sergei Petrunia's avatar
      MDEV-24739: Assertion `root->weight >= ...' failed in SEL_ARG::tree_delete · 73c43ee9
      Sergei Petrunia authored
      Also update the SEL_ARG graph weight in:
      - sel_add()
      - SEL_ARG::clone()
      
      Make key_{and,or}_with_limit() to also verify weight for the arguments
      (There is no single point to verify SEL_ARG graphs constructed from
      conditions that are not AND-OR formulas, so we hope that those are
      connected with AND/OR and do it here).
      73c43ee9
    • Marko Mäkelä's avatar
      MDEV-24661: Remove the test innodb.innodb_wl6326_big · a70a47f2
      Marko Mäkelä authored
      The purpose of the test was to ensure that the SX (update) mode of
      index tree and buffer page latches are being used.
      
      The test has become unstable, possibly due to changes related to
      buf_pool.mutex and buf_pool.page_hash, or to the use of MDL in the
      purge of transaction history.
      
      In 10.6, the test depends on instrumentation that was refactored
      or removed in MDEV-24142.
      
      The use of different latching modes can better be indirectly observed
      through high-concurrency benchmarks. For MDEV-14637, a performance test
      was conducted where the finer-grained latching and
      BTR_CUR_FINE_HISTORY_LENGTH were removed. It caused a 20% performance
      regression for UPDATE and somewhat smaller for INSERT.
      
      Any new problem with latching granularity should be easily caught by
      performance testing, or by stress tests with Random Query Generator.
      a70a47f2
    • Vladislav Vaintroub's avatar
      MDEV-24685 - remove IO thread states output from SHOW ENGINE INNODB STATUS · d8373fea
      Vladislav Vaintroub authored
      There are no IO threads anymore.
      d8373fea
    • Sergei Petrunia's avatar
      MDEV-9750: Quick memory exhaustion with 'extended_keys=on' ... · c3672038
      Sergei Petrunia authored
      (Variant #5, full patch, for 10.5)
      
      Do not produce SEL_ARG graphs that would yield huge numbers of ranges.
      Introduce a concept of SEL_ARG graph's "weight". If we are about to
      produce a graph whose "weight" exceeds the limit, remove the parts
      of SEL_ARG graph that represent the biggest key parts. Do so until
      the graph's is within the limit.
      
      Includes
      - debug code to verify SEL_ARG graph weight
      - A user-visible @@optimizer_max_sel_arg_weight to control the optimization
      - Logging the optimization into the optimizer trace.
      c3672038
    • sjaakola's avatar
      MDEV-24721 galera.mysql-wsrep-bugs-607 test failure · a2eb974b
      sjaakola authored
      The implementation for MDEV-17048 apperas to be direct copy from mysql version.
      The group commit works differently in mariadb and the assert in wsrep_unregister_from_group_commit() is too strict.
      
      The reason is that in: Wsrep_high_priority_service::log_dummy_write_set(), the transaction will undergo full rollback:
          {
            cs.before_rollback();
            cs.after_rollback();
          }
      
      After that, the client's transaction state is set to be:  wsrep::transaction::s_aborted.
      The execution then continues execution by:
      
      ...
       wsrep_register_for_group_commit(m_thd);
      ...
       wsrep_unregister_from_group_commit(m_thd);
      
      The bogus assert in wsrep_unregister_from_group_commit() allows only transactions states of :s_ordered_commit or s_aborting.
      
      As the fix, I brought back the same assert as is present in MariaDB 10.4 version.
      a2eb974b
  10. 28 Jan, 2021 6 commits
    • Anel Husakovic's avatar
      MDEV-24093: Detect during mysql_upgrade if type_mysql_json.so is needed and load it · 85130c5a
      Anel Husakovic authored
      a. The change makes `mariadb-upgrade` detect if `MYSQL_JSON` data type is needed.
      b. Install the data type if it's not installed.
      c. Uninstalls the data type once finished.
      d. Create `.opt` and `.inc` files `have_type_mysql_json` and adapt the
      tests
      
      Reviewed by: vicentiu@mariadb.org
      85130c5a
    • Marko Mäkelä's avatar
      MDEV-24715 Assertion !node->table->skip_alter_undo in CREATE...REPLACE SELECT · c411393a
      Marko Mäkelä authored
      In commit 3cef4f8f (MDEV-515)
      we inadvertently broke CREATE TABLE...REPLACE SELECT statements
      by wrongly disabling row-level undo logging.
      
      select_create::prepare(): Only invoke extra(HA_EXTRA_BEGIN_ALTER_COPY)
      if no special treatment of duplicates is needed.
      c411393a
    • Marko Mäkelä's avatar
      MDEV-24564 Statistics are lost after ALTER TABLE · 6d1f1b61
      Marko Mäkelä authored
      Ever since commit 007f68c3,
      ALTER TABLE no longer invokes handler::open() after
      handler::commit_inplace_alter_table().
      
      ha_innobase::reload_statistics(): Reload or recompute statistics
      after ALTER TABLE.
      
      innodb_notify_tabledef_changed(): A new function to invoke
      ha_innobase::reload_statistics().
      
      handlerton::notify_tabledef_changed(): Add the parameter handler*
      so that ha_innobase::reload_statistics() can be invoked.
      
      ha_partition::notify_tabledef_changed(),
      partition_notify_tabledef_changed(): Pass through the call
      to any partitions or subpartitions.
      
      This is based on code that was supplied by Monty.
      6d1f1b61
    • Vlad Lesin's avatar
      744e9752
    • Thirunarayanan Balathandayuthapani's avatar
      MDEV-24695 Encryption modifies a freed page · 6e80a34d
      Thirunarayanan Balathandayuthapani authored
      During recovery, InnoDB fails if it tries to apply a FREE_PAGE
      and WRITE record to the page. InnoDB encryption thread accesses
      the freed page and writes redo log for it.
      
      This is similar to commit deadec4e (MDEV-24569)
      InnoDB is missing buf_page_free() while freeing the segment.
      To avoid accessing of freed page in buffer pool, InnoDB should
      mark the pages as FREED while freeing the segment. Also to
      avoid reading of freed page, InnoDB should check the
      allocation bitmap page.
      
      fseg_free_step(): Mark the page in buffer pool as FREED
      
      fseg_free_step_not_header(): Mark the page in buffer pool as FREED
      
      buf_dump(): Ignore the freed pages while dumping the buffer pool content
      
      fil_crypt_get_page_throttle_func(): Skip the rotation for FREED page
      to avoid the assert failure during recovery
      
      fil_crypt_rotate_page(): Skip the rotation for the freed page
      
      Reviewed-by: Marko Mäkelä
      6e80a34d
    • Marko Mäkelä's avatar
      c6308355
  11. 27 Jan, 2021 5 commits
    • Marko Mäkelä's avatar
      Cleanup: Remove many C-style lock_get_ accessors · 68b28193
      Marko Mäkelä authored
      Let us prefer member functions to the old C-style accessor functions.
      Also, prefer bitwise AND operations for checking multiple flags.
      68b28193
    • Marko Mäkelä's avatar
      Cleanup: Remove lock_get_size() · cbb0a60c
      Marko Mäkelä authored
      cbb0a60c
    • Thirunarayanan Balathandayuthapani's avatar
      MDEV-24693 LeakSanitizer: detected memory leaks in mem_heap_create_block_func... · 700ae20d
      Thirunarayanan Balathandayuthapani authored
      MDEV-24693 LeakSanitizer: detected memory leaks in mem_heap_create_block_func / fts_optimize_create_msg
      
      - This issue is caused by the commit bf1f9b59
      (MDEV-24638). Delay the creation of SYNC message in
      fts_optimize_request_sync_table. So that InnoDB can avoid creating
      the message if the table already has SYNC message in fts_optimize_wq queue
      700ae20d
    • Marko Mäkelä's avatar
      MDEV-24700 Assertion "lock not found"==0 in lock_table_x_unlock() · 5dd028f8
      Marko Mäkelä authored
      After an ignored INSERT IGNORE statement into an empty table, we would
      wrongly use the MDEV-515 table-level undo logging for a subsequent
      REPLACE statement.
      
      ha_innobase::reset_template(): Clear m_prebuilt->ins_node->bulk_insert
      on every statement boundary.
      
      ha_innobase::start_stmt(): Invoke end_bulk_insert().
      
      ha_innobase::extra(): Avoid accessing m_prebuilt->trx. Do not call
      thd_to_trx(). Invoke end_bulk_insert() and try to reset bulk_insert
      when changing the REPLACE or IGNORE settings.
      
      trx_mod_table_time_t::WAS_BULK: Use a distinct value from BULK.
      
      trx_undo_report_row_operation(): Add debug assertions.
      
      Note: Some calls to end_bulk_insert() may be redundant, but statement
      boundaries are not always clear in the API (especially in the
      presence of LOCK TABLES or stored procedures).
      5dd028f8
    • Marko Mäkelä's avatar
      MDEV-20612: Speed up lock_table_other_has_incompatible() · 121d0f7f
      Marko Mäkelä authored
      dict_table_t::n_lock_x_or_s: Keep track of LOCK_S or LOCK_X on the table.
      
      lock_table_other_has_incompatible(): In the likely case that no
      transaction is waiting for or holding LOCK_S or LOCK_X on the table,
      return early: conflicts cannot exist.
      
      This is based on the idea of Zhai Weixiang, who reported MySQL Bug #72948.
      
      lock_table_has_to_wait_in_queue(), lock_table_dequeue():
      Extend the optimization, inspired by
      mysql/mysql-server@bb7191d6cbe47e15923143e194c03406cff9024b
      by Jakub Łopuszański.
      121d0f7f