* [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling
@ 2020-07-30 23:28 Thinh Nguyen
2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen
` (3 more replies)
0 siblings, 4 replies; 19+ messages in thread
From: Thinh Nguyen @ 2020-07-30 23:28 UTC (permalink / raw)
To: Felipe Balbi, Greg Kroah-Hartman, Thinh Nguyen, linux-usb
Cc: John Youn, stable
This series fixes a couple of driver issues handling ClearFeature(halt)
request:
1) A function driver often uses set_halt() to reject a class driver protocol
command. After set_halt(), the endpoint will be stalled. It can queue new
requests while the endpoint is stalled. However, dwc3 currently drops those
requests after CLEAR_STALL. The driver should only drop started requests. Keep
the pending requests in the pending list to resume and process them after the
host issues ClearFeature(Halt) to the endpoint.
2) DWC3 should issue CLEAR_STALL command _after_ END_TRANSFER command completes.
Thinh Nguyen (3):
usb: dwc3: gadget: Resume pending requests after CLEAR_STALL
usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command
usb: dwc3: gadget: Refactor ep command completion
drivers/usb/dwc3/core.h | 1 +
drivers/usb/dwc3/ep0.c | 16 +++++++++
drivers/usb/dwc3/gadget.c | 85 +++++++++++++++++++++++++++++++----------------
drivers/usb/dwc3/gadget.h | 1 +
4 files changed, 75 insertions(+), 28 deletions(-)
base-commit: e3ee0e740c3887d2293e8d54a8707218d70d86ca
--
2.11.0
^ permalink raw reply [flat|nested] 19+ messages in thread* [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Thinh Nguyen @ 2020-07-30 23:28 ` Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin ` (5 more replies) 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen ` (2 subsequent siblings) 3 siblings, 6 replies; 19+ messages in thread From: Thinh Nguyen @ 2020-07-30 23:28 UTC (permalink / raw) To: Felipe Balbi, Greg Kroah-Hartman, Thinh Nguyen, linux-usb Cc: John Youn, stable The function driver may queue new requests right after halting the endpoint (i.e. queue new requests while the endpoint is stalled). There's no restriction preventing it from doing so. However, dwc3 currently drops those requests after CLEAR_STALL. The driver should only drop started requests. Keep the pending requests in the pending list to resume and process them after the host issues ClearFeature(Halt) to the endpoint. Cc: stable@vger.kernel.org Fixes: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)") Signed-off-by: Thinh Nguyen <thinhn@synopsys.com> --- drivers/usb/dwc3/gadget.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index e44bfc3b5096..fcd0df1e7d3f 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -1566,8 +1566,12 @@ static int __dwc3_gadget_ep_queue(struct dwc3_ep *dep, struct dwc3_request *req) if (dep->flags & DWC3_EP_WAIT_TRANSFER_COMPLETE) return 0; - /* Start the transfer only after the END_TRANSFER is completed */ - if (dep->flags & DWC3_EP_END_TRANSFER_PENDING) { + /* + * Start the transfer only after the END_TRANSFER is completed + * and endpoint STALL is cleared. + */ + if ((dep->flags & DWC3_EP_END_TRANSFER_PENDING) || + (dep->flags & DWC3_EP_STALL)) { dep->flags |= DWC3_EP_DELAY_START; return 0; } @@ -1774,9 +1778,6 @@ int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol) list_for_each_entry_safe(req, tmp, &dep->started_list, list) dwc3_gadget_move_cancelled_request(req); - list_for_each_entry_safe(req, tmp, &dep->pending_list, list) - dwc3_gadget_move_cancelled_request(req); - if (!(dep->flags & DWC3_EP_END_TRANSFER_PENDING)) { dep->flags &= ~DWC3_EP_DELAY_START; dwc3_gadget_ep_cleanup_cancelled_requests(dep); -- 2.11.0 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen @ 2020-08-01 23:18 ` Sasha Levin 2020-08-06 1:24 ` Sasha Levin ` (4 subsequent siblings) 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-01 23:18 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.7.11, v5.4.54. v5.7.11: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.54: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin @ 2020-08-06 1:24 ` Sasha Levin 2020-08-09 15:53 ` Sasha Levin ` (3 subsequent siblings) 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-06 1:24 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.7.11, v5.4.54. v5.7.11: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.54: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin 2020-08-06 1:24 ` Sasha Levin @ 2020-08-09 15:53 ` Sasha Levin 2020-08-13 16:25 ` Sasha Levin ` (2 subsequent siblings) 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-09 15:53 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8, v5.7.13, v5.4.56. v5.8: Build OK! v5.7.13: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.56: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen ` (2 preceding siblings ...) 2020-08-09 15:53 ` Sasha Levin @ 2020-08-13 16:25 ` Sasha Levin 2020-08-19 23:56 ` Sasha Levin 2020-08-26 13:53 ` Sasha Levin 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-13 16:25 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8, v5.7.14, v5.4.57. v5.8: Build OK! v5.7.14: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.57: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen ` (3 preceding siblings ...) 2020-08-13 16:25 ` Sasha Levin @ 2020-08-19 23:56 ` Sasha Levin 2020-08-26 13:53 ` Sasha Levin 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-19 23:56 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8.1, v5.7.15, v5.4.58. v5.8.1: Build OK! v5.7.15: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.58: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen ` (4 preceding siblings ...) 2020-08-19 23:56 ` Sasha Levin @ 2020-08-26 13:53 ` Sasha Levin 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-26 13:53 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8.2, v5.7.16, v5.4.59. v5.8.2: Build OK! v5.7.16: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.59: Failed to apply! Possible dependencies: 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Thinh Nguyen 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen @ 2020-07-30 23:28 ` Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin ` (5 more replies) 2020-08-06 1:24 ` [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Sasha Levin 2020-08-17 18:06 ` Thinh Nguyen 3 siblings, 6 replies; 19+ messages in thread From: Thinh Nguyen @ 2020-07-30 23:28 UTC (permalink / raw) To: Felipe Balbi, Greg Kroah-Hartman, Thinh Nguyen, linux-usb Cc: John Youn, stable According the programming guide (for all DWC3 IPs), when the driver handles ClearFeature(halt) request, it should issue CLEAR_STALL command _after_ the END_TRANSFER command completes. The END_TRANSFER command may take some time to complete. So, delay the ClearFeature(halt) request control status stage and wait for END_TRANSFER command completion interrupt. Only after END_TRANSFER command completes that the driver may issue CLEAR_STALL command. Cc: stable@vger.kernel.org Fixes: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)") Signed-off-by: Thinh Nguyen <thinhn@synopsys.com> --- drivers/usb/dwc3/core.h | 1 + drivers/usb/dwc3/ep0.c | 16 ++++++++++++++++ drivers/usb/dwc3/gadget.c | 43 +++++++++++++++++++++++++++++++++---------- drivers/usb/dwc3/gadget.h | 1 + 4 files changed, 51 insertions(+), 10 deletions(-) diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index 2f04b3e42bf1..eb026c9cca28 100644 --- a/drivers/usb/dwc3/core.h +++ b/drivers/usb/dwc3/core.h @@ -706,6 +706,7 @@ struct dwc3_ep { #define DWC3_EP_IGNORE_NEXT_NOSTREAM BIT(8) #define DWC3_EP_FORCE_RESTART_STREAM BIT(9) #define DWC3_EP_FIRST_STREAM_PRIMED BIT(10) +#define DWC3_EP_PENDING_CLEAR_STALL BIT(11) /* This last one is specific to EP0 */ #define DWC3_EP0_DIR_IN BIT(31) diff --git a/drivers/usb/dwc3/ep0.c b/drivers/usb/dwc3/ep0.c index 59f2e8c31bd1..92bc1044e7ab 100644 --- a/drivers/usb/dwc3/ep0.c +++ b/drivers/usb/dwc3/ep0.c @@ -524,6 +524,11 @@ static int dwc3_ep0_handle_endpoint(struct dwc3 *dwc, ret = __dwc3_gadget_ep_set_halt(dep, set, true); if (ret) return -EINVAL; + + /* ClearFeature(Halt) may need delayed status */ + if (!set && (dep->flags & DWC3_EP_END_TRANSFER_PENDING)) + return USB_GADGET_DELAYED_STATUS; + break; default: return -EINVAL; @@ -1042,6 +1047,17 @@ static void dwc3_ep0_do_control_status(struct dwc3 *dwc, __dwc3_ep0_do_control_status(dwc, dep); } +void dwc3_ep0_send_delayed_status(struct dwc3 *dwc) +{ + unsigned int direction = !dwc->ep0_expect_in; + + if (dwc->ep0state != EP0_STATUS_PHASE) + return; + + dwc->delayed_status = false; + __dwc3_ep0_do_control_status(dwc, dwc->eps[direction]); +} + static void dwc3_ep0_end_control_data(struct dwc3 *dwc, struct dwc3_ep *dep) { struct dwc3_gadget_ep_cmd_params params; diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index fcd0df1e7d3f..8c89444efd9f 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -1764,6 +1764,19 @@ int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol) return 0; } + dwc3_stop_active_transfer(dep, true, true); + + list_for_each_entry_safe(req, tmp, &dep->started_list, list) + dwc3_gadget_move_cancelled_request(req); + + if (dep->flags & DWC3_EP_END_TRANSFER_PENDING) { + dep->flags |= DWC3_EP_PENDING_CLEAR_STALL; + return 0; + } + + dep->flags &= ~DWC3_EP_DELAY_START; + dwc3_gadget_ep_cleanup_cancelled_requests(dep); + ret = dwc3_send_clear_stall_ep_cmd(dep); if (ret) { dev_err(dwc->dev, "failed to clear STALL on %s\n", @@ -1772,16 +1785,6 @@ int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol) } dep->flags &= ~(DWC3_EP_STALL | DWC3_EP_WEDGE); - - dwc3_stop_active_transfer(dep, true, true); - - list_for_each_entry_safe(req, tmp, &dep->started_list, list) - dwc3_gadget_move_cancelled_request(req); - - if (!(dep->flags & DWC3_EP_END_TRANSFER_PENDING)) { - dep->flags &= ~DWC3_EP_DELAY_START; - dwc3_gadget_ep_cleanup_cancelled_requests(dep); - } } return ret; @@ -2927,6 +2930,26 @@ static void dwc3_endpoint_interrupt(struct dwc3 *dwc, dep->flags &= ~DWC3_EP_END_TRANSFER_PENDING; dep->flags &= ~DWC3_EP_TRANSFER_STARTED; dwc3_gadget_ep_cleanup_cancelled_requests(dep); + + if (dep->flags & DWC3_EP_PENDING_CLEAR_STALL) { + struct dwc3 *dwc = dep->dwc; + + dep->flags &= ~DWC3_EP_PENDING_CLEAR_STALL; + if (dwc3_send_clear_stall_ep_cmd(dep)) { + struct usb_ep *ep0 = &dwc->eps[0]->endpoint; + + dev_err(dwc->dev, "failed to clear STALL on %s\n", + dep->name); + if (dwc->delayed_status) + __dwc3_gadget_ep0_set_halt(ep0, 1); + return; + } + + dep->flags &= ~(DWC3_EP_STALL | DWC3_EP_WEDGE); + if (dwc->delayed_status) + dwc3_ep0_send_delayed_status(dwc); + } + if ((dep->flags & DWC3_EP_DELAY_START) && !usb_endpoint_xfer_isoc(dep->endpoint.desc)) __dwc3_gadget_kick_transfer(dep); diff --git a/drivers/usb/dwc3/gadget.h b/drivers/usb/dwc3/gadget.h index bd85eb7fa9ef..a7791cb827c4 100644 --- a/drivers/usb/dwc3/gadget.h +++ b/drivers/usb/dwc3/gadget.h @@ -113,6 +113,7 @@ int dwc3_gadget_ep0_set_halt(struct usb_ep *ep, int value); int dwc3_gadget_ep0_queue(struct usb_ep *ep, struct usb_request *request, gfp_t gfp_flags); int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol); +void dwc3_ep0_send_delayed_status(struct dwc3 *dwc); /** * dwc3_gadget_ep_get_transfer_index - Gets transfer index from HW -- 2.11.0 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen @ 2020-08-01 23:18 ` Sasha Levin 2020-08-06 1:24 ` Sasha Levin ` (4 subsequent siblings) 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-01 23:18 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.7.11, v5.4.54. v5.7.11: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.54: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin @ 2020-08-06 1:24 ` Sasha Levin 2020-08-09 15:53 ` Sasha Levin ` (3 subsequent siblings) 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-06 1:24 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.7.11, v5.4.54. v5.7.11: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.54: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin 2020-08-06 1:24 ` Sasha Levin @ 2020-08-09 15:53 ` Sasha Levin 2020-08-13 16:25 ` Sasha Levin ` (2 subsequent siblings) 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-09 15:53 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8, v5.7.13, v5.4.56. v5.8: Failed to apply! Possible dependencies: Unable to calculate v5.7.13: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.56: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen ` (2 preceding siblings ...) 2020-08-09 15:53 ` Sasha Levin @ 2020-08-13 16:25 ` Sasha Levin 2020-08-19 23:56 ` Sasha Levin 2020-08-26 13:53 ` Sasha Levin 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-13 16:25 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8, v5.7.14, v5.4.57. v5.8: Failed to apply! Possible dependencies: Unable to calculate v5.7.14: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.57: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen ` (3 preceding siblings ...) 2020-08-13 16:25 ` Sasha Levin @ 2020-08-19 23:56 ` Sasha Levin 2020-08-26 13:53 ` Sasha Levin 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-19 23:56 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8.1, v5.7.15, v5.4.58. v5.8.1: Failed to apply! Possible dependencies: Unable to calculate v5.7.15: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.58: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen ` (4 preceding siblings ...) 2020-08-19 23:56 ` Sasha Levin @ 2020-08-26 13:53 ` Sasha Levin 5 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-26 13:53 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.8.2, v5.7.16, v5.4.59. v5.8.2: Failed to apply! Possible dependencies: Unable to calculate v5.7.16: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.59: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling 2020-07-30 23:28 [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Thinh Nguyen 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen @ 2020-08-06 1:24 ` Sasha Levin 2020-08-17 18:06 ` Thinh Nguyen 3 siblings, 0 replies; 19+ messages in thread From: Sasha Levin @ 2020-08-06 1:24 UTC (permalink / raw) To: Sasha Levin, Thinh Nguyen, Felipe Balbi; +Cc: John Youn, stable, stable Hi [This is an automated email] This commit has been processed because it contains a "Fixes:" tag fixing commit: cb11ea56f37a ("usb: dwc3: gadget: Properly handle ClearFeature(halt)"). The bot has tested the following trees: v5.7.11, v5.4.54. v5.7.11: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") a42d7cef357a ("usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") v5.4.54: Failed to apply! Possible dependencies: 140ca4cfea8a ("usb: dwc3: gadget: Handle stream transfers") 2e6e9e4b2ed7 ("usb: dwc3: gadget: Refactor TRB completion handler") 3eaecd0c2333 ("usb: dwc3: gadget: Handle XferComplete for streams") a42d7cef357a ("usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command") b6842d4938c3 ("usb: dwc3: gadget: Check for in-progress END_TRANSFER") d9feef974e0d ("usb: dwc3: gadget: Continue to process pending requests") e0d19563eb6c ("usb: dwc3: gadget: Wait for transfer completion") NOTE: The patch will not be queued to stable trees until it is upstream. How should we proceed with this patch? -- Thanks Sasha ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling 2020-07-30 23:28 [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Thinh Nguyen ` (2 preceding siblings ...) 2020-08-06 1:24 ` [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Sasha Levin @ 2020-08-17 18:06 ` Thinh Nguyen 2020-08-18 6:56 ` Felipe Balbi 3 siblings, 1 reply; 19+ messages in thread From: Thinh Nguyen @ 2020-08-17 18:06 UTC (permalink / raw) To: Felipe Balbi, Greg Kroah-Hartman, linux-usb; +Cc: John Youn, stable Hi Felipe, Thinh Nguyen wrote: > This series fixes a couple of driver issues handling ClearFeature(halt) > request: > > 1) A function driver often uses set_halt() to reject a class driver protocol > command. After set_halt(), the endpoint will be stalled. It can queue new > requests while the endpoint is stalled. However, dwc3 currently drops those > requests after CLEAR_STALL. The driver should only drop started requests. Keep > the pending requests in the pending list to resume and process them after the > host issues ClearFeature(Halt) to the endpoint. > > 2) DWC3 should issue CLEAR_STALL command _after_ END_TRANSFER command completes. > > > Thinh Nguyen (3): > usb: dwc3: gadget: Resume pending requests after CLEAR_STALL > usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Since you're picking the fix patches for RC cycle, can you pick up these 2 patches of this series also? We can leave the refactoring patch in this series for v5.10. Thanks, Thinh > usb: dwc3: gadget: Refactor ep command completion > > drivers/usb/dwc3/core.h | 1 + > drivers/usb/dwc3/ep0.c | 16 +++++++++ > drivers/usb/dwc3/gadget.c | 85 +++++++++++++++++++++++++++++++---------------- > drivers/usb/dwc3/gadget.h | 1 + > 4 files changed, 75 insertions(+), 28 deletions(-) > > > base-commit: e3ee0e740c3887d2293e8d54a8707218d70d86ca ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling 2020-08-17 18:06 ` Thinh Nguyen @ 2020-08-18 6:56 ` Felipe Balbi 2020-08-18 21:40 ` Thinh Nguyen 0 siblings, 1 reply; 19+ messages in thread From: Felipe Balbi @ 2020-08-18 6:56 UTC (permalink / raw) To: Thinh Nguyen, Greg Kroah-Hartman, linux-usb; +Cc: John Youn, stable [-- Attachment #1: Type: text/plain, Size: 1204 bytes --] Thinh Nguyen <Thinh.Nguyen@synopsys.com> writes: > Hi Felipe, > > Thinh Nguyen wrote: >> This series fixes a couple of driver issues handling ClearFeature(halt) >> request: >> >> 1) A function driver often uses set_halt() to reject a class driver protocol >> command. After set_halt(), the endpoint will be stalled. It can queue new >> requests while the endpoint is stalled. However, dwc3 currently drops those >> requests after CLEAR_STALL. The driver should only drop started requests. Keep >> the pending requests in the pending list to resume and process them after the >> host issues ClearFeature(Halt) to the endpoint. >> >> 2) DWC3 should issue CLEAR_STALL command _after_ END_TRANSFER command completes. >> >> >> Thinh Nguyen (3): >> usb: dwc3: gadget: Resume pending requests after CLEAR_STALL >> usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command > > Since you're picking the fix patches for RC cycle, can you pick up these > 2 patches of this series also? We can leave the refactoring patch in > this series for v5.10. just to be sure: you did run these through usbcv and usb3 compliance suite, right? Which gadget drivers did you use? -- balbi [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 857 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling 2020-08-18 6:56 ` Felipe Balbi @ 2020-08-18 21:40 ` Thinh Nguyen 0 siblings, 0 replies; 19+ messages in thread From: Thinh Nguyen @ 2020-08-18 21:40 UTC (permalink / raw) To: Felipe Balbi, Thinh Nguyen, Greg Kroah-Hartman, linux-usb Cc: John Youn, stable Felipe Balbi wrote: > Thinh Nguyen <Thinh.Nguyen@synopsys.com> writes: > >> Hi Felipe, >> >> Thinh Nguyen wrote: >>> This series fixes a couple of driver issues handling ClearFeature(halt) >>> request: >>> >>> 1) A function driver often uses set_halt() to reject a class driver protocol >>> command. After set_halt(), the endpoint will be stalled. It can queue new >>> requests while the endpoint is stalled. However, dwc3 currently drops those >>> requests after CLEAR_STALL. The driver should only drop started requests. Keep >>> the pending requests in the pending list to resume and process them after the >>> host issues ClearFeature(Halt) to the endpoint. >>> >>> 2) DWC3 should issue CLEAR_STALL command _after_ END_TRANSFER command completes. >>> >>> >>> Thinh Nguyen (3): >>> usb: dwc3: gadget: Resume pending requests after CLEAR_STALL >>> usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command >> Since you're picking the fix patches for RC cycle, can you pick up these >> 2 patches of this series also? We can leave the refactoring patch in >> this series for v5.10. > just to be sure: you did run these through usbcv and usb3 compliance > suite, right? Which gadget drivers did you use? > Yes I did for chapter 9, MSC, and UASP CV. I missed this issue because "stall" was not enabled by default for mass_storage configfs. Without this fix, mass_storage function won't work with "stall" enabled. (Note: for UASP CV, it requires many fixes/enhancements in f_tcm and the target subsystem to work properly.) Thanks, Thinh ^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2020-08-26 13:55 UTC | newest] Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-07-30 23:28 [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Thinh Nguyen 2020-07-30 23:28 ` [PATCH 1/3] usb: dwc3: gadget: Resume pending requests after CLEAR_STALL Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin 2020-08-06 1:24 ` Sasha Levin 2020-08-09 15:53 ` Sasha Levin 2020-08-13 16:25 ` Sasha Levin 2020-08-19 23:56 ` Sasha Levin 2020-08-26 13:53 ` Sasha Levin 2020-07-30 23:28 ` [PATCH 2/3] usb: dwc3: gadget: END_TRANSFER before CLEAR_STALL command Thinh Nguyen 2020-08-01 23:18 ` Sasha Levin 2020-08-06 1:24 ` Sasha Levin 2020-08-09 15:53 ` Sasha Levin 2020-08-13 16:25 ` Sasha Levin 2020-08-19 23:56 ` Sasha Levin 2020-08-26 13:53 ` Sasha Levin 2020-08-06 1:24 ` [PATCH 0/3] usb: dwc3: gadget: Fix halt/clear_stall handling Sasha Levin 2020-08-17 18:06 ` Thinh Nguyen 2020-08-18 6:56 ` Felipe Balbi 2020-08-18 21:40 ` Thinh Nguyen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).