From 8fedfc04d95cc372b3835c52139e80ef51665f53 Mon Sep 17 00:00:00 2001
From: Ye Ting <ting.ye@intel.com>
Date: Fri, 14 Jun 2013 08:18:34 +0000
Subject: [PATCH] =?UTF-8?q?Fixed=20the=20issue=20that=20IPv6=20network=20s?=
 =?UTF-8?q?tack=20assert=20infrequently=20=E2=80=93=20assert=20once=20ever?=
 =?UTF-8?q?y=20300~400=20PXE=20boots.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Ye Ting <ting.ye@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ouyang Qian <qian.ouyang@intel.com>



git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@14422 6f19259b-4bc3-4df7-8a09-765794883524
---
 NetworkPkg/Ip6Dxe/Ip6Driver.c | 87 ++++++++++++++++++++---------------
 1 file changed, 49 insertions(+), 38 deletions(-)

diff --git a/NetworkPkg/Ip6Dxe/Ip6Driver.c b/NetworkPkg/Ip6Dxe/Ip6Driver.c
index 8095ed1ae2..b9a64a80f8 100644
--- a/NetworkPkg/Ip6Dxe/Ip6Driver.c
+++ b/NetworkPkg/Ip6Dxe/Ip6Driver.c
@@ -1,7 +1,7 @@
 /** @file
   The driver binding and service binding protocol for IP6 driver.
 
-  Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved.<BR>
 
   This program and the accompanying materials
   are licensed and made available under the terms and conditions of the BSD License
@@ -412,39 +412,6 @@ Ip6CreateService (
     goto ON_ERROR;
   }
 
-  //
-  // The timer expires every 100 (IP6_TIMER_INTERVAL_IN_MS) milliseconds.
-  //
-  Status = gBS->SetTimer (IpSb->FasterTimer, TimerPeriodic, TICKS_PER_MS * IP6_TIMER_INTERVAL_IN_MS);
-  if (EFI_ERROR (Status)) {
-    goto ON_ERROR;
-  }
-
-  //
-  // The timer expires every 1000 (IP6_ONE_SECOND_IN_MS) milliseconds.
-  //
-  Status = gBS->SetTimer (IpSb->Timer, TimerPeriodic, TICKS_PER_MS * IP6_ONE_SECOND_IN_MS);
-  if (EFI_ERROR (Status)) {
-    goto ON_ERROR;
-  }
-
-
-  Status = gBS->CreateEvent (
-                  EVT_NOTIFY_SIGNAL,
-                  TPL_NOTIFY,
-                  Ip6OnFrameReceived,
-                  &IpSb->RecvRequest,
-                  &MnpToken->Event
-                  );
-  if (EFI_ERROR (Status)) {
-    goto ON_ERROR;
-  }
-
-  Status = Ip6ReceiveFrame (Ip6AcceptFrame, IpSb);
-  if (EFI_ERROR (Status)) {
-    goto ON_ERROR;
-  }
-
   Status = NetLibGetMacString (IpSb->Controller, IpSb->Image, &IpSb->MacString);
   if (EFI_ERROR (Status)) {
     goto ON_ERROR;
@@ -461,6 +428,17 @@ Ip6CreateService (
     goto ON_ERROR;
   }
 
+  Status = gBS->CreateEvent (
+                  EVT_NOTIFY_SIGNAL,
+                  TPL_NOTIFY,
+                  Ip6OnFrameReceived,
+                  &IpSb->RecvRequest,
+                  &MnpToken->Event
+                  );
+  if (EFI_ERROR (Status)) {
+    goto ON_ERROR;
+  }
+
   //
   // If there is any manual address, set it.
   //
@@ -557,19 +535,52 @@ Ip6DriverBindingStart (
                   NULL
                   );
 
-  if (EFI_ERROR (Status)) {
+  if (!EFI_ERROR (Status)) {
+    //
+    // ready to go: start the receiving and timer
+    //
+    Status = Ip6ReceiveFrame (Ip6AcceptFrame, IpSb);
+    if (EFI_ERROR (Status)) {
+      goto ON_ERROR;
+    }
+
+    //
+    // The timer expires every 100 (IP6_TIMER_INTERVAL_IN_MS) milliseconds.
+    //
+    Status = gBS->SetTimer (
+                    IpSb->FasterTimer,
+                    TimerPeriodic,
+                    TICKS_PER_MS * IP6_TIMER_INTERVAL_IN_MS
+                    );
+    if (EFI_ERROR (Status)) {
+      goto ON_ERROR;
+    }
+
+    //
+    // The timer expires every 1000 (IP6_ONE_SECOND_IN_MS) milliseconds.
+    //
+    Status = gBS->SetTimer (
+                    IpSb->Timer,
+                    TimerPeriodic,
+                    TICKS_PER_MS * IP6_ONE_SECOND_IN_MS
+                    );
+    if (EFI_ERROR (Status)) {
+      goto ON_ERROR;
+    }    
 
-    Ip6CleanService (IpSb);
-    FreePool (IpSb);
-  } else {
     //
     // Initialize the IP6 ID
     //
     mIp6Id = NET_RANDOM (NetRandomInitSeed ());
 
     Ip6SetVariableData (IpSb);
+
+    return EFI_SUCCESS;
   }
 
+ON_ERROR:
+  Ip6CleanService (IpSb);
+  FreePool (IpSb);
   return Status;
 }