From 0c9f335a223845787c05c5f695a33d813a7acc32 Mon Sep 17 00:00:00 2001 From: Michael Grzeschik Date: Tue, 11 Mar 2025 08:51:19 +0900 Subject: [PATCH] usb: gadget: uvc: wake pump everytime we update the free list BugLink: https://bugs.launchpad.net/bugs/2101915 [ Upstream commit adc292d54de9db2e6b8ecb7f81f278bbbaf713e9 ] Since the req_free list will updated if enqueuing one request was not possible it will be added back to the free list. With every available free request in the queue it is a valid case for the pump worker to use it and continue the pending bufferdata into requests for the req_ready list. Fixes: 6acba0345b68 ("usb:gadget:uvc Do not use worker thread to pump isoc usb requests") Signed-off-by: Michael Grzeschik Link: https://lore.kernel.org/r/20240403-uvc_request_length_by_interval-v7-1-e224bb1035f0@pengutronix.de Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sasha Levin [koichiroden: adjusted context due to missing commit: 58da4e57cb29 ("usb: gadget: uvc: rework complete handler")] Signed-off-by: Koichiro Den Signed-off-by: Stefan Bader --- drivers/usb/gadget/function/uvc_video.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/usb/gadget/function/uvc_video.c b/drivers/usb/gadget/function/uvc_video.c index 278a438dd9a7..89305e6c07e4 100644 --- a/drivers/usb/gadget/function/uvc_video.c +++ b/drivers/usb/gadget/function/uvc_video.c @@ -473,6 +473,10 @@ uvc_video_complete(struct usb_ep *ep, struct usb_request *req) * up later. */ list_add_tail(&to_queue->list, &video->req_free); + /* + * There is a new free request - wake up the pump. + */ + queue_work(video->async_wq, &video->pump); } } else { uvc_video_free_request(ureq, ep);