From 11bbc25789c65514005c1ed7eaca38c095e0187a Mon Sep 17 00:00:00 2001 From: Ronald Cron Date: Mon, 27 Oct 2014 10:42:13 +0000 Subject: EmbeddedPkg/Lan9118Dxe: Rework filter init, enabling and disabling Correct the setting of the hardware filters according to what it asked to the driver through the ReceiveFilters() interface function. Keep track of the hardware settings in the "ReceiveFilterSetting" field of the EFI_SIMPLE_NETWORK_MODE structure. From now, after initialization, all filters are disabled and thus no packet received. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ronald Cron Reviewed-By: Olivier Martin git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16243 6f19259b-4bc3-4df7-8a09-765794883524 --- EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c | 130 ++++++++++++++---------- EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118DxeUtil.c | 3 +- 2 files changed, 77 insertions(+), 56 deletions(-) diff --git a/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c b/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c index 507e55f..60b8c96 100644 --- a/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c +++ b/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c @@ -99,16 +99,25 @@ Lan9118DxeEntry ( SnpMode->NvRamSize = 0; // No NVRAM with this device SnpMode->NvRamAccessSize = 0; // No NVRAM with this device - // Update network mode information - SnpMode->ReceiveFilterMask = EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST | - EFI_SIMPLE_NETWORK_RECEIVE_UNICAST | - EFI_SIMPLE_NETWORK_RECEIVE_BROADCAST | - EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS;/* | - EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS_MULTICAST;*/ - // Current allowed settings - SnpMode->ReceiveFilterSetting = EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST | - EFI_SIMPLE_NETWORK_RECEIVE_UNICAST | - EFI_SIMPLE_NETWORK_RECEIVE_BROADCAST; + // + // Claim that all receive filter settings are supported, though the MULTICAST mode + // is not completely supported. The LAN9118 Ethernet controller is only able to + // do a "hash filtering" and not a perfect filtering on multicast addresses. The + // controller does not filter the multicast addresses directly but a hash value + // of them. The hash value of a multicast address is derived from its CRC and + // ranges from 0 to 63 included. + // We claim that the perfect MULTICAST filtering mode is supported because + // we do not want the user to switch directly to the PROMISCOUS_MULTICAST mode + // and thus not being able to take advantage of the hash filtering. + // + SnpMode->ReceiveFilterMask = EFI_SIMPLE_NETWORK_RECEIVE_UNICAST | + EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST | + EFI_SIMPLE_NETWORK_RECEIVE_BROADCAST | + EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS | + EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS_MULTICAST; + + // We do not intend to receive anything for the time being. + SnpMode->ReceiveFilterSetting = 0; // LAN9118 has 64bit hash table, can filter 64 MCast MAC Addresses SnpMode->MaxMCastFilterCount = MAX_MCAST_FILTER_CNT; @@ -344,12 +353,7 @@ SnpInitialize ( return Status; } - // Enable the receiver and transmitter - Status = StartRx (0, Snp); - if (EFI_ERROR(Status)) { - return Status; - } - + // Enable the transmitter Status = StartTx (START_TX_MAC | START_TX_CFG, Snp); if (EFI_ERROR(Status)) { return Status; @@ -553,7 +557,9 @@ SnpReceiveFilters ( UINT32 Crc; UINT8 HashValue; UINT32 MacCSRValue; + UINT32 ReceiveFilterSetting; EFI_MAC_ADDRESS *Mac; + EFI_MAC_ADDRESS ZeroMac; // Check Snp Instance if (Snp == NULL) { @@ -632,70 +638,74 @@ SnpReceiveFilters ( } // + // Before to change anything, stop and reset the reception of + // packets. + // + StopRx (STOP_RX_CLEAR, Snp); + + // // Write the mask of the selected hash values for the multicast filtering. // The two masks are set to zero if the multicast filtering is not enabled. // IndirectMACWrite32 (INDIRECT_MAC_INDEX_HASHL, MultHashTableLow); IndirectMACWrite32 (INDIRECT_MAC_INDEX_HASHH, MultHashTableHigh); + ReceiveFilterSetting = (Mode->ReceiveFilterSetting | Enable) & (~Disable); + + // // Read MAC controller - MacCSRValue = IndirectMACRead32 (INDIRECT_MAC_INDEX_CR); + // + MacCSRValue = IndirectMACRead32 (INDIRECT_MAC_INDEX_CR); + MacCSRValue &= ~(MACCR_HPFILT | MACCR_BCAST | MACCR_PRMS | MACCR_MCPAS); - // Set the options for the MAC_CSR - if (Enable & EFI_SIMPLE_NETWORK_RECEIVE_UNICAST) { - StartRx (0, Snp); + if (ReceiveFilterSetting & EFI_SIMPLE_NETWORK_RECEIVE_UNICAST) { + Lan9118SetMacAddress (&Mode->CurrentAddress, Snp); DEBUG ((DEBUG_NET, "Allowing Unicast Frame Reception\n")); + } else { + // + // The Unicast packets do not have to be listen to, set the MAC + // address of the LAN9118 to be the "not configured" all zeroes + // ethernet MAC address. + // + ZeroMem (&ZeroMac, NET_ETHER_ADDR_LEN); + Lan9118SetMacAddress (&ZeroMac, Snp); } - if (Disable & EFI_SIMPLE_NETWORK_RECEIVE_UNICAST) { - StopRx (0, Snp); - DEBUG ((DEBUG_NET, "Disabling Unicast Frame Reception\n")); - } - - if (Enable & EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST) { + if (ReceiveFilterSetting & EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST) { MacCSRValue |= MACCR_HPFILT; DEBUG ((DEBUG_NET, "Allowing Multicast Frame Reception\n")); } - if (Disable & EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST) { - MacCSRValue &= ~MACCR_HPFILT; - DEBUG ((DEBUG_NET, "Disabling Multicast Frame Reception\n")); - } - - if (Enable & EFI_SIMPLE_NETWORK_RECEIVE_BROADCAST) { - MacCSRValue &= ~(MACCR_BCAST); - DEBUG ((DEBUG_NET, "Allowing Broadcast Frame Reception\n")); + if (ReceiveFilterSetting & EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS_MULTICAST) { + MacCSRValue |= MACCR_MCPAS; + DEBUG ((DEBUG_NET, "Enabling Promiscuous Multicast Mode\n")); } - if (Disable & EFI_SIMPLE_NETWORK_RECEIVE_BROADCAST) { + if ((ReceiveFilterSetting & EFI_SIMPLE_NETWORK_RECEIVE_BROADCAST) == 0) { MacCSRValue |= MACCR_BCAST; - DEBUG ((DEBUG_NET, "Disabling Broadcast Frame Reception\n")); + } else { + DEBUG ((DEBUG_NET, "Allowing Broadcast Frame Reception\n")); } - if (Enable & EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS) { + if (ReceiveFilterSetting & EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS) { MacCSRValue |= MACCR_PRMS; DEBUG ((DEBUG_NET, "Enabling Promiscuous Mode\n")); } - if (Disable & EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS) { - MacCSRValue &= ~MACCR_PRMS; - DEBUG ((DEBUG_NET, "Disabling Promiscuous Mode\n")); - } - - if (Enable & EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS_MULTICAST) { - MacCSRValue |= (MACCR_HPFILT | MACCR_PRMS); - DEBUG ((DEBUG_NET, "Enabling Promiscuous Multicast Mode\n")); - } - - if (Disable & EFI_SIMPLE_NETWORK_RECEIVE_PROMISCUOUS_MULTICAST) { - MacCSRValue &= ~(MACCR_HPFILT | MACCR_PRMS); - DEBUG ((DEBUG_NET, "Disabling Promiscuous Multicast Mode\n")); - } - + // // Write the options to the MAC_CSR + // IndirectMACWrite32 (INDIRECT_MAC_INDEX_CR, MacCSRValue); gBS->Stall (LAN9118_STALL); + // + // If we have to retrieve something, start packet reception. + // + Mode->ReceiveFilterSetting = ReceiveFilterSetting; + if (ReceiveFilterSetting != 0) { + StartRx (0, Snp); + } + return EFI_SUCCESS; } @@ -769,8 +779,20 @@ SnpStationAddress ( } } - // Write address - Lan9118SetMacAddress (New, Snp); + CopyMem (&Snp->Mode->CurrentAddress, New, NET_ETHER_ADDR_LEN); + + // + // If packet reception is currently activated, stop and reset it, + // set the new ethernet address and restart the packet reception. + // Otherwise, nothing to do, the MAC address will be updated in + // SnpReceiveFilters() when the UNICAST packet reception will be + // activated. + // + if (Snp->Mode->ReceiveFilterSetting != 0) { + StopRx (STOP_RX_CLEAR, Snp); + Lan9118SetMacAddress (New, Snp); + StartRx (0, Snp); + } return EFI_SUCCESS; } diff --git a/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118DxeUtil.c b/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118DxeUtil.c index 20fb110..301c42f 100644 --- a/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118DxeUtil.c +++ b/EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118DxeUtil.c @@ -320,8 +320,6 @@ Lan9118SetMacAddress ( (UINT32)(Mac->Addr[4] & 0xFF) | ((Mac->Addr[5] & 0xFF) << 8) ); - - CopyMem (&Snp->Mode->CurrentAddress, &Mac, NET_ETHER_ADDR_LEN); } VOID @@ -398,6 +396,7 @@ Lan9118Initialize ( DEBUG ((EFI_D_WARN, "Warning: using driver-default MAC address\n")); DefaultMacAddress = FixedPcdGet64 (PcdLan9118DefaultMacAddress); Lan9118SetMacAddress((EFI_MAC_ADDRESS *) &DefaultMacAddress, Snp); + CopyMem (&Snp->Mode->CurrentAddress, &DefaultMacAddress, NET_ETHER_ADDR_LEN); } } else { // Store the MAC address that was loaded from EEPROM -- cgit v1.1