Blame SOURCES/0165-efinet-handle-get_status-on-buggy-firmware-properly.patch

28f7f8
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
ecb9bb
From: Josef Bacik <jbacik@fb.com>
ecb9bb
Date: Thu, 6 Aug 2015 10:49:46 -0700
28f7f8
Subject: [PATCH] efinet: handle get_status() on buggy firmware properly
ecb9bb
ecb9bb
The EFI spec indicates that get_status() should return the address of the buffer
ecb9bb
we passed into transmit to indicate the the buffer was transmitted.  However we
ecb9bb
have boxes where the firmware returns some arbitrary address instead, which
ecb9bb
makes grub think that we've not sent anything.  So since we have the SNP stuff
ecb9bb
opened in exclusive mode just assume any non-NULL txbuf means that our transmit
ecb9bb
occurred properly.  This makes grub able to do its networking stuff properly on
ecb9bb
our broken firmware.  Thanks,
ecb9bb
ecb9bb
cc: Peter Jones <pjones@redhat.com>
ecb9bb
Signed-off-by: Josef Bacik <jbacik@fb.com>
ecb9bb
---
ecb9bb
 grub-core/net/drivers/efi/efinet.c | 21 +++++++++++----------
ecb9bb
 1 file changed, 11 insertions(+), 10 deletions(-)
ecb9bb
ecb9bb
diff --git a/grub-core/net/drivers/efi/efinet.c b/grub-core/net/drivers/efi/efinet.c
28f7f8
index 7b8c4a59d10..ea0e0ca360e 100644
ecb9bb
--- a/grub-core/net/drivers/efi/efinet.c
ecb9bb
+++ b/grub-core/net/drivers/efi/efinet.c
ecb9bb
@@ -47,19 +47,19 @@ send_card_buffer (struct grub_net_card *dev,
ecb9bb
 	if (st != GRUB_EFI_SUCCESS)
ecb9bb
 	  return grub_error (GRUB_ERR_IO,
ecb9bb
 			     N_("couldn't send network packet"));
ecb9bb
-	if (txbuf == dev->txbuf)
ecb9bb
+	/*
ecb9bb
+	   Some buggy firmware could return an arbitrary address instead of the
ecb9bb
+	   txbuf address we trasmitted, so just check that txbuf is non NULL
ecb9bb
+	   for success.  This is ok because we open the SNP protocol in
ecb9bb
+	   exclusive mode so we know we're the only ones transmitting on this
ecb9bb
+	   box and since we only transmit one packet at a time we know our
ecb9bb
+	   transmit was successfull.
ecb9bb
+	 */
ecb9bb
+	if (txbuf)
ecb9bb
 	  {
ecb9bb
 	    dev->txbusy = 0;
ecb9bb
 	    break;
ecb9bb
 	  }
ecb9bb
-	if (txbuf)
ecb9bb
-	  {
ecb9bb
-	    st = efi_call_7 (net->transmit, net, 0, dev->last_pkt_size,
ecb9bb
-			     dev->txbuf, NULL, NULL, NULL);
ecb9bb
-	    if (st != GRUB_EFI_SUCCESS)
ecb9bb
-	      return grub_error (GRUB_ERR_IO,
ecb9bb
-				 N_("couldn't send network packet"));
ecb9bb
-	  }
ecb9bb
 	if (limit_time < grub_get_time_ms ())
ecb9bb
 	  return grub_error (GRUB_ERR_TIMEOUT,
ecb9bb
 			     N_("couldn't send network packet"));
ecb9bb
@@ -84,8 +84,9 @@ send_card_buffer (struct grub_net_card *dev,
ecb9bb
      we run in the GRUB_ERR_TIMEOUT case above.
ecb9bb
      Perhaps a timeout in the FW has discarded the recycle buffer.
ecb9bb
    */
ecb9bb
+  txbuf = NULL;
ecb9bb
   st = efi_call_3 (net->get_status, net, 0, &txbuf);
ecb9bb
-  dev->txbusy = !(st == GRUB_EFI_SUCCESS && txbuf == dev->txbuf);
ecb9bb
+  dev->txbusy = !(st == GRUB_EFI_SUCCESS && txbuf);
ecb9bb
 
ecb9bb
   return GRUB_ERR_NONE;
ecb9bb
 }