pgsql: Fix macro name for io_uring_queue_init_mem check. - Mailing list pgsql-committers

From Masahiko Sawada
Subject pgsql: Fix macro name for io_uring_queue_init_mem check.
Date
Msg-id E1vb1iq-003hpt-1K@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix macro name for io_uring_queue_init_mem check.

Commit f54af9f2679d added a check for
io_uring_queue_init_mem(). However, it used the macro name
HAVE_LIBURING_QUEUE_INIT_MEM in both meson.build and the C code, while
the Autotools build script defined HAVE_IO_URING_QUEUE_INIT_MEM. As a
result, the optimization was never enabled in builds configured with
Autotools, as the C code checked for the wrong macro name.

This commit changes the macro name to HAVE_IO_URING_QUEUE_INIT_MEM in
meson.build and the C code. This matches the actual function
name (io_uring_queue_init_mem), following the standard HAVE_<FUNCTION>
convention.

Backpatch to 18, where the macro was introduced.

Bug: #19368
Reported-by: Evan Si <evsi@amazon.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/19368-016d79a7f3a1c599@postgresql.org
Backpatch-through: 18

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/85d5bd308bf5dc3068a4c28aa0f12cc22272f789

Modified Files
--------------
meson.build                               | 2 +-
src/backend/storage/aio/method_io_uring.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Doc: remove obsolete, confused about rowtype I/O syntax.
Next
From: Masahiko Sawada
Date:
Subject: pgsql: Fix macro name for io_uring_queue_init_mem check.