On Thu, Sep 10, 2026 at 3:10 AM Lucas DRAESCHER <[email protected]> wrote:
> Alternatively, if maintaining a separate backpatch variant is undesirable,
> v1 is still available. It doesn't change the `IoMethodOps` struct at all,
> so the ABI question disappears.

I don't object to the v4 approach for master, but I still prefer v1 for
backpatching because it keeps the changes minimal and avoids changing
the IoMethodOps ABI.

I've attached updated v1-based patches. The changes are:
- Rename the cleanup function.
- Replace the NULL check with an assertion and add Assert(!IsUnderPostmaster).
- Remove the debug log message.

How about committing this v1-based fix and backpatching it to v18, then
considering a common cleanup interface separately for master if needed?

Regards,

-- 
Fujii Masao

Attachment: nocfbot-v1-2-PG18-0001-Fix-io_uring-resource-leaks-across-crash-restar.patch
Description: Binary data

Attachment: v1-2-0001-Fix-io_uring-resource-leaks-across-crash-restar.patch
Description: Binary data

Reply via email to