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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux