Re: [PATCH] check kernel version for io_method

On 1/14/26 8:22 AM, Steven Niu wrote:
> I agree that you are correct for now. But in future, the IoMethod may have new member. Then we have to process it.
> 
> How about change to like this:
>   + #else
>   +     if (*newval == IOMETHOD_IO_URING)
>   +     {
>   +             GUC_check_errdetail("io_uring not enabled.");
>   +             return false;
>   +     }

No, that would be incorrect. It would not even compile. 
IOMETHOD_IO_URING is not even defined if we do not compile with liburing 
enabled. You can try it out yourself by compiling PostgreSQL without 
io_uring support and then try to enable it. You will get an error and 
that code will never be reached.

Andreas