d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sfxge(4): update external port mapping for Medford

Browse Source 
Extend the mapping table for external port numbering to support port modes
which output to the second external port only. Where supported, map from
the current port mode rather than inferring from all the available modes.
Updated comments for clarity.

Submitted by:   Richard Houldsworth <rhouldsworth at solarflare.com>
Sponsored by:   Solarflare Communications, Inc.
MFC after:      2 days
Differential Revision:  https://reviews.freebsd.org/D8210
This commit is contained in:
Andrew Rybchenko 2016-10-11 13:08:48 +00:00
parent 6886c70100
commit b36a7ad2c9
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=307038
1 changed files with 54 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

76
sys/dev/sfxge/common/ef10_nic.c
View File

@ -1099,41 +1099,50 @@ ef10_get_privilege_mask(
/*
* The external port mapping is a one-based numbering of the external
* connectors on the board. It does not distinguish off-board separated
* outputs such as multi-headed cables.
* The number of ports that map to each external port connector
* on the board is determined by the chip family and the port modes to
* which the NIC can be configured. The mapping table lists modes with
* port numbering requirements in increasing order.
* Table of mapping schemes from port number to the number of the external
* connector on the board. The external numbering does not distinguish
* off-board separated outputs such as from multi-headed cables.
*
* The count of adjacent port numbers that map to each external port
* and the offset in the numbering, is determined by the chip family and
* current port mode.
*
* For the Huntington family, the current port mode cannot be discovered,
* so the mapping used is instead the last match in the table to the full
* set of port modes to which the NIC can be configured. Therefore the
* ordering of entries in the the mapping table is significant.
*/
static struct {
efx_family_t family;
uint32_t modes_mask;
uint32_t stride;
int32_t count;
int32_t offset;
} __ef10_external_port_mappings[] = {
/* Supported modes requiring 1 output per port */
/* Supported modes with 1 output per external port */
{
EFX_FAMILY_HUNTINGTON,
(1 << TLV_PORT_MODE_10G) |
(1 << TLV_PORT_MODE_10G_10G) |
(1 << TLV_PORT_MODE_10G_10G_10G_10G),
1,
1
},
{
EFX_FAMILY_MEDFORD,
(1 << TLV_PORT_MODE_10G) |
(1 << TLV_PORT_MODE_10G_10G),
1,
1
},
/* Supported modes requiring 2 outputs per port */
/* Supported modes with 2 outputs per external port */
{
EFX_FAMILY_HUNTINGTON,
(1 << TLV_PORT_MODE_40G) |
(1 << TLV_PORT_MODE_40G_40G) |
(1 << TLV_PORT_MODE_40G_10G_10G) |
(1 << TLV_PORT_MODE_10G_10G_40G),
2
2,
1
},
{
EFX_FAMILY_MEDFORD,
@ -1142,15 +1151,22 @@ static struct {
(1 << TLV_PORT_MODE_40G_10G_10G) |
(1 << TLV_PORT_MODE_10G_10G_40G) |
(1 << TLV_PORT_MODE_10G_10G_10G_10G_Q1_Q2),
2
2,
1
},
/* Supported modes requiring 4 outputs per port */
/* Supported modes with 4 outputs per external port */
{
EFX_FAMILY_MEDFORD,
(1 << TLV_PORT_MODE_10G_10G_10G_10G_Q) |
(1 << TLV_PORT_MODE_10G_10G_10G_10G_Q1) |
(1 << TLV_PORT_MODE_10G_10G_10G_10G_Q1),
4,
1,
},
{
EFX_FAMILY_MEDFORD,
(1 << TLV_PORT_MODE_10G_10G_10G_10G_Q2),
4
4,
2
},
};
@ -1164,11 +1180,26 @@ ef10_external_port_mapping(
int i;
uint32_t port_modes;
uint32_t matches;
uint32_t stride = 1; /* default 1-1 mapping */
uint32_t current;
int32_t count = 1; /* Default 1-1 mapping */
int32_t offset = 1; /* Default starting external port number */
if ((rc = efx_mcdi_get_port_modes(enp, &port_modes, NULL)) != 0) {
/* No port mode information available - use default mapping */
goto out;
if ((rc = efx_mcdi_get_port_modes(enp, &port_modes, &current)) != 0) {
/*
* No current port mode information
* - infer mapping from available modes
*/
if ((rc = efx_mcdi_get_port_modes(enp,
&port_modes, NULL)) != 0) {
/*
* No port mode information available
* - use default mapping
*/
goto out;
}
} else {
/* Only need to scan the current mode */
port_modes = 1 << current;
}
/*
@ -1182,7 +1213,8 @@ ef10_external_port_mapping(
matches = (__ef10_external_port_mappings[i].modes_mask &
port_modes);
if (matches != 0) {
stride = __ef10_external_port_mappings[i].stride;
count = __ef10_external_port_mappings[i].count;
offset = __ef10_external_port_mappings[i].offset;
port_modes &= ~matches;
}
}
@ -1196,9 +1228,9 @@ ef10_external_port_mapping(
out:
/*
* Scale as required by last matched mode and then convert to
* one-based numbering
* correctly offset numbering
*/
*external_portp = (uint8_t)(port / stride) + 1;
*external_portp = (uint8_t)((port / count) + offset);
return (0);
fail1: