MDEV-35785: Use same innodb_log_file_mmap description on all systems
authorOtto Kekäläinen <otto@debian.org>
Sun, 12 Jan 2025 21:31:16 +0000 (13:31 -0800)
committerOtto Kekäläinen <otto@debian.org>
Sun, 12 Jan 2025 22:10:40 +0000 (14:10 -0800)
On most 64-bit architectures `mariadbd --help --verbose` said:

    Whether ib_logfile0 resides in persistent memory or
    should initially be memory-mapped

On 32-bit architectures and some 64-bit (e.g. s390x) it said:

    Whether ib_logfile0 should initially be memory-mapped

Having the help test vary with the same source code build with the same
build flags and dependencies seems confusing. Unify the text to always
be the same for clarity.

Ideally the embedded help text would be the exact same first sentence
as the wiki docs[1] but use this custom text now for better clarity.

[1] https://mariadb.com/kb/en/innodb-system-variables/#innodb_log_file_mmap

Forwarded: no

Gbp-Pq: Name MDEV-35785-Use-same-innodb_log_file_mmap-description.patch

storage/innobase/handler/ha_innodb.cc

index 1b2df276818e05c649b3db917c75ac8261714473..87c8bc331d6df167c2c9c862bcd48494615f0c35 100644 (file)
@@ -19424,12 +19424,9 @@ static MYSQL_SYSVAR_UINT(log_buffer_size, log_sys.buf_size,
   "Redo log buffer size in bytes.",
   NULL, NULL, 16U << 20, 2U << 20, log_sys.buf_size_max, 4096);
 
-  static constexpr const char *innodb_log_file_mmap_description=
-    "Whether ib_logfile0"
-# ifdef HAVE_PMEM
-    " resides in persistent memory or"
-# endif
-    " should initially be memory-mapped";
+static constexpr const char *innodb_log_file_mmap_description=
+  "Whether ib_logfile0 on InnoDB startup and recovery is memory-mapped, or in persistent memory (on supported platforms).";
+
 static MYSQL_SYSVAR_BOOL(log_file_mmap, log_sys.log_mmap,
   PLUGIN_VAR_OPCMDARG | PLUGIN_VAR_READONLY,
   innodb_log_file_mmap_description,