diff options
| author | Alex Elder <elder@linaro.org> | 2022-02-03 18:09:26 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2022-02-04 11:16:08 +0100 |
| commit | 5d6ac24fb10f04681ff584cd1bb5c2ac22ebe417 (patch) | |
| tree | 715273fa3351412e63819a548c99bf204e6753be /drivers/net/ipa | |
| parent | net: ipa: kill replenish_backlog (diff) | |
| download | linux-5d6ac24fb10f04681ff584cd1bb5c2ac22ebe417.tar.xz linux-5d6ac24fb10f04681ff584cd1bb5c2ac22ebe417.zip | |
net: ipa: replenish after delivering payload
Replenishing is now solely driven by whether transactions are
available for a channel, and it doesn't really matter whether
we replenish before or after we deliver received packets to the
network stack.
Replenishing before delivering the payload adds a little latency.
Eliminate that by requesting a replenish after the payload is
delivered.
Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ipa')
| -rw-r--r-- | drivers/net/ipa/ipa_endpoint.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/ipa/ipa_endpoint.c b/drivers/net/ipa/ipa_endpoint.c index 9d875126a360..a236edf5bf06 100644 --- a/drivers/net/ipa/ipa_endpoint.c +++ b/drivers/net/ipa/ipa_endpoint.c @@ -1341,10 +1341,8 @@ static void ipa_endpoint_rx_complete(struct ipa_endpoint *endpoint, { struct page *page; - ipa_endpoint_replenish(endpoint); - if (trans->cancelled) - return; + goto done; /* Parse or build a socket buffer using the actual received length */ page = trans->data; @@ -1352,6 +1350,8 @@ static void ipa_endpoint_rx_complete(struct ipa_endpoint *endpoint, ipa_endpoint_status_parse(endpoint, page, trans->len); else if (ipa_endpoint_skb_build(endpoint, page, trans->len)) trans->data = NULL; /* Pages have been consumed */ +done: + ipa_endpoint_replenish(endpoint); } void ipa_endpoint_trans_complete(struct ipa_endpoint *endpoint, |