diff options
| author |   Alex Elder <elder@linaro.org> | 2022-06-13 12:17:58 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2022-06-15 09:07:58 +0100 |
| commit | dbad2fa71914a6a1a41bcba8bdc3e9213a982f82 (patch) | |
| tree | c1899ec82ef6c5c7263b2f4da4ad70386a68f63d /drivers/net/ipa/gsi.h | |
| parent | net: ipa: simplify TX completion statistics (diff) | |
| download | wireguard-linux-dbad2fa71914a6a1a41bcba8bdc3e9213a982f82.tar.xz wireguard-linux-dbad2fa71914a6a1a41bcba8bdc3e9213a982f82.zip | |
net: ipa: stop counting total RX bytes and transactions
In gsi_evt_ring_rx_update(), we update each transaction so its len
field reflects the actual number of bytes received. In the process,
the total number of transactions and bytes processed on the channel
are summed, and added to a running total for the channel.
But we don't actually use those running totals for RX endpoints.
They're maintained for TX channels to support CoDel when they are
associated with a "real" network device.
So stop maintaining these totals for RX endpoints, and update the
comment where the fields are defined to make it clear they're only
valid for TX channels.
Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ipa/gsi.h')
| -rw-r--r-- | drivers/net/ipa/gsi.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ipa/gsi.h b/drivers/net/ipa/gsi.h index 89dac7fc8c4c..bad1a78a96ed 100644 --- a/drivers/net/ipa/gsi.h +++ b/drivers/net/ipa/gsi.h @@ -117,9 +117,9 @@ struct gsi_channel { struct gsi_ring tre_ring; u32 evt_ring_id; + /* The following counts are used only for TX endpoints */ u64 byte_count; /* total # bytes transferred */ u64 trans_count; /* total # transactions */ - /* The following counts are used only for TX endpoints */ u64 queued_byte_count; /* last reported queued byte count */ u64 queued_trans_count; /* ...and queued trans count */ u64 compl_byte_count; /* last reported completed byte count */ |