On 11/12/24 3:44 AM, Anuj Gupta wrote: >> +utilization than polling. Similarly, this feature also requires the devices >> +to support polling configuration. > This feature would work if a device doesn't have polled queues,right? > The performance might be suboptimal in that case, but the userspace won't > get any errors. The usersapce will get error in this case. Hybrid must be dependent on iopoll, if hybrid polling is set separately,it will not pass detection and an error will be returned. Therefore, device end cnnfigured are required for polling. ...... > This patch mostly looks fine. But the code here seems to be largely > duplicated from "test/io_uring_passthrough.c" and "test/iopoll.c". > Can we consider adding the hybrid poll test as a part of the existing > tests as it seems that it would only require passing a extra flag > during ring setup. Make sense, I will try to do this in exsiting test file. Thank you very much. -- hexue
