Nt32Pkg/SnpNt32Dxe: Fix hang issue when multiple network interfaces existed

Currently all the network interfaces share the one recycled transmit buffer
array, which is used to store the recycled buffer address. However, those
recycled buffers are allocated by the different MNP interface if the multiple
network interfaces existed. Then, SNP GetStatus may return one recycled transmit
buffer address to the another MNP interface, which may result in the MNP driver
hang after 'reconnect -r' operation.

Cc: Ye Ting <ting.ye@intel.com>
Cc: Fu Siyuan <siyuan.fu@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>

1 files changed, 16 insertions, 15 deletions

diff --git a/Nt32Pkg/SnpNt32Dxe/SnpNt32.h b/Nt32Pkg/SnpNt32Dxe/SnpNt32.h
index cb95c5711c..56251342e6 100644
--- a/Nt32Pkg/SnpNt32Dxe/SnpNt32.h
+++ b/Nt32Pkg/SnpNt32Dxe/SnpNt32.h
@@ -1,6 +1,6 @@
 /** @file

 

-Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2006 - 2017, 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

 which accompanies this distribution.  The full text of the license may be found at

@@ -161,20 +161,6 @@ struct _SNPNT32_GLOBAL_DATA {
   EFI_LOCK                          Lock;

 

   //

-  // Array of the recycled transmit buffer address.

-  //

-  UINT64                            *RecycledTxBuf;

-

-  //

-  // Current number of recycled buffer pointers in RecycledTxBuf.

-  //

-  UINT32                             RecycledTxBufCount;

-

-  // The maximum number of recycled buffer pointers in RecycledTxBuf.

-  //

-  UINT32                             MaxRecycledTxBuf;

-

-  //

   //  Private functions

   //

   SNPNT32_INITIALIZE_GLOBAL_DATA    InitializeGlobalData;

@@ -195,6 +181,21 @@ struct _SNPNT32_INSTANCE_DATA {
   //

   LIST_ENTRY                  Entry;

 

+  //

+  // Array of the recycled transmit buffer address.

+  //

+  UINT64                      *RecycledTxBuf;

+

+  //

+  // Current number of recycled buffer pointers in RecycledTxBuf.

+  //

+  UINT32                      RecycledTxBufCount;

+

+  //

+  // The maximum number of recycled buffer pointers in RecycledTxBuf.

+  //

+  UINT32                      MaxRecycledTxBuf;

+

   SNPNT32_GLOBAL_DATA         *GlobalData;

 

   EFI_HANDLE                  DeviceHandle;