gve: Handle alternate miss completions
authorJeroen de Borst <jeroendb@google.com>
Thu, 17 Nov 2022 16:27:01 +0000 (08:27 -0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 21 Nov 2022 10:52:14 +0000 (10:52 +0000)
The virtual NIC has 2 ways of indicating a miss-path
completion. This handles the alternate.

Signed-off-by: Jeroen de Borst <jeroendb@google.com>
Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/google/gve/gve_adminq.h
drivers/net/ethernet/google/gve/gve_desc_dqo.h
drivers/net/ethernet/google/gve/gve_tx_dqo.c

index b9ee8be73f96b87dce49a1b5f54933ebec5712c7..cf29662e6ad11a5cd56f17d9735ae6098d6817f5 100644 (file)
@@ -154,6 +154,7 @@ enum gve_driver_capbility {
        gve_driver_capability_gqi_rda = 1,
        gve_driver_capability_dqo_qpl = 2, /* reserved for future use */
        gve_driver_capability_dqo_rda = 3,
+       gve_driver_capability_alt_miss_compl = 4,
 };
 
 #define GVE_CAP1(a) BIT((int)a)
@@ -164,7 +165,8 @@ enum gve_driver_capbility {
 #define GVE_DRIVER_CAPABILITY_FLAGS1 \
        (GVE_CAP1(gve_driver_capability_gqi_qpl) | \
         GVE_CAP1(gve_driver_capability_gqi_rda) | \
-        GVE_CAP1(gve_driver_capability_dqo_rda))
+        GVE_CAP1(gve_driver_capability_dqo_rda) | \
+        GVE_CAP1(gve_driver_capability_alt_miss_compl))
 
 #define GVE_DRIVER_CAPABILITY_FLAGS2 0x0
 #define GVE_DRIVER_CAPABILITY_FLAGS3 0x0
index e8fe9adef7f24c8f29dcaa272db8c40b72ee5571..f79cd059111038929ebc5764e1f1de26e634aec8 100644 (file)
@@ -176,6 +176,11 @@ static_assert(sizeof(struct gve_tx_compl_desc) == 8);
 #define GVE_COMPL_TYPE_DQO_MISS 0x1 /* Miss path completion */
 #define GVE_COMPL_TYPE_DQO_REINJECTION 0x3 /* Re-injection completion */
 
+/* The most significant bit in the completion tag can change the completion
+ * type from packet completion to miss path completion.
+ */
+#define GVE_ALT_MISS_COMPL_BIT BIT(15)
+
 /* Descriptor to post buffers to HW on buffer queue. */
 struct gve_rx_desc_dqo {
        __le16 buf_id; /* ID returned in Rx completion descriptor */
index 588d64819ed5ed29ffc20f2ac813c15bc8cb9161..b76143bfd594793bbdd4a5eae4aa9a7fc4632e91 100644 (file)
@@ -953,12 +953,18 @@ int gve_clean_tx_done_dqo(struct gve_priv *priv, struct gve_tx_ring *tx,
                        atomic_set_release(&tx->dqo_compl.hw_tx_head, tx_head);
                } else if (type == GVE_COMPL_TYPE_DQO_PKT) {
                        u16 compl_tag = le16_to_cpu(compl_desc->completion_tag);
-
-                       gve_handle_packet_completion(priv, tx, !!napi,
-                                                    compl_tag,
-                                                    &pkt_compl_bytes,
-                                                    &pkt_compl_pkts,
-                                                    /*is_reinjection=*/false);
+                       if (compl_tag & GVE_ALT_MISS_COMPL_BIT) {
+                               compl_tag &= ~GVE_ALT_MISS_COMPL_BIT;
+                               gve_handle_miss_completion(priv, tx, compl_tag,
+                                                          &miss_compl_bytes,
+                                                          &miss_compl_pkts);
+                       } else {
+                               gve_handle_packet_completion(priv, tx, !!napi,
+                                                            compl_tag,
+                                                            &pkt_compl_bytes,
+                                                            &pkt_compl_pkts,
+                                                            false);
+                       }
                } else if (type == GVE_COMPL_TYPE_DQO_MISS) {
                        u16 compl_tag = le16_to_cpu(compl_desc->completion_tag);
 
@@ -972,7 +978,7 @@ int gve_clean_tx_done_dqo(struct gve_priv *priv, struct gve_tx_ring *tx,
                                                     compl_tag,
                                                     &reinject_compl_bytes,
                                                     &reinject_compl_pkts,
-                                                    /*is_reinjection=*/true);
+                                                    true);
                }
 
                tx->dqo_compl.head =