On 4/7/24 7:03 PM, Ming Lei wrote: > sqe->flags is u8, and now we have used 7 bits, so take the last one for > extending purpose. > > If bit7(IOSQE_HAS_EXT_FLAGS_BIT) is 1, it means this sqe carries ext flags > from the last byte(.ext_flags), or bit23~bit16 of sqe->uring_cmd_flags for > IORING_OP_URING_CMD. > > io_slot_flags() return value is converted to `ULL` because the affected bits > are beyond 32bit now. If we're extending flags, which is something we arguably need to do at some point, I think we should have them be generic and not spread out. If uring_cmd needs specific flags and don't have them, then we should add it just for that. -- Jens Axboe