Re: [PATCH] io_uring/waitid: remove #ifdef CONFIG_COMPAT

[Pavel Begunkov]

On 2/24/25 17:55, Jens Axboe wrote:
> On 2/24/25 10:53 AM, Caleb Sander Mateos wrote:
> > On Mon, Feb 24, 2025 at 9:44 AM Jens Axboe <axboe@xxxxxxxxx> wrote:
> > > On 2/24/25 10:23 AM, Caleb Sander Mateos wrote:
> > > > io_is_compat() is already defined to return false if CONFIG_COMPAT is
> > > > disabled. So remove the additional #ifdef CONFIG_COMPAT guards. Let the
> > > > compiler optimize out the dead code when CONFIG_COMPAT is disabled.
> > >
> > > Would you mind if I fold this into Pavel's patch? I can keep it
> > > standalone too, just let me know.
> >
> > Fine by me, though I thought Pavel was suggesting keeping it separate:
> > https://lore.kernel.org/io-uring/da109d01-7aab-4205-bbb1-f5f1387f1847@xxxxxxxxx/T/#u
>
> I'm reading it as he has other stuff that will go on top. I don't see
> any reason to double stage this part, might as well remove the
> CONFIG dependency at the same time, if it's doable.
>
> Pavel?

I'm not sure why you'd want that, but I don't mind

--
Pavel Begunkov