Attachment 171652 Details for Bug 150249 – just a bunch of printf code to see what path the driver is taking.

[patch] just a bunch of printf code to see what path the driver is taking.

ixgbe_sfp_debug.diff (text/plain), 3.22 KB, created by Sean Bruno on 2016-06-21 16:03:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Sean Bruno

Created: 2016-06-21 16:03:54 UTC

Size: 3.22 KB

patch

obsolete

>Index: sys/dev/ixgbe/ixgbe_82599.c
>===================================================================
>--- sys/dev/ixgbe/ixgbe_82599.c	(revision 301929)
>+++ sys/dev/ixgbe/ixgbe_82599.c	(working copy)
>@@ -2183,6 +2183,7 @@
> 			physical_layer |= IXGBE_PHYSICAL_LAYER_1000BASE_T;
> 		if (ext_ability & IXGBE_MDIO_PHY_100BASETX_ABILITY)
> 			physical_layer |= IXGBE_PHYSICAL_LAYER_100BASE_TX;
>+		printf("phy.type set physical layer %u\n", physical_layer);
> 		goto out;
> 	default:
> 		break;
>@@ -2194,6 +2195,7 @@
> 		if (pma_pmd_1g == IXGBE_AUTOC_1G_KX_BX) {
> 			physical_layer = IXGBE_PHYSICAL_LAYER_1000BASE_KX |
> 			    IXGBE_PHYSICAL_LAYER_1000BASE_BX;
>+			printf("autoc_1g set physical layer %u\n", physical_layer);
> 			goto out;
> 		} else
> 			/* SFI mode so read SFP module */
>@@ -2206,11 +2208,13 @@
> 			physical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_KX4;
> 		else if (pma_pmd_10g_parallel == IXGBE_AUTOC_10G_XAUI)
> 			physical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_XAUI;
>+			printf("autoc_10g set physical layer %u\n", physical_layer);
> 		goto out;
> 		break;
> 	case IXGBE_AUTOC_LMS_10G_SERIAL:
> 		if (pma_pmd_10g_serial == IXGBE_AUTOC2_10G_KR) {
> 			physical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_KR;
>+			printf("autoc_10g_serial set physical layer %u\n", physical_layer);
> 			goto out;
> 		} else if (pma_pmd_10g_serial == IXGBE_AUTOC2_10G_SFI)
> 			goto sfp_check;
>@@ -2223,9 +2227,11 @@
> 			physical_layer |= IXGBE_PHYSICAL_LAYER_10GBASE_KX4;
> 		if (autoc & IXGBE_AUTOC_KR_SUPP)
> 			physical_layer |= IXGBE_PHYSICAL_LAYER_10GBASE_KR;
>+			printf("autoc_LMS set physical layer %u\n", physical_layer);
> 		goto out;
> 		break;
> 	default:
>+		printf("Unable to set physical layer %u\n", physical_layer);
> 		goto out;
> 		break;
> 	}
>@@ -2235,6 +2241,7 @@
> 	 * test KR mode -  we need to id KR mode correctly before SFP module.
> 	 * Call identify_sfp because the pluggable module may have changed */
> 	physical_layer = ixgbe_get_supported_phy_sfp_layer_generic(hw);
>+	printf("phy_sfp_layer set physical layer %u\n", physical_layer);
> out:
> 	return physical_layer;
> }
>Index: sys/dev/ixgbe/ixgbe_phy.c
>===================================================================
>--- sys/dev/ixgbe/ixgbe_phy.c	(revision 301929)
>+++ sys/dev/ixgbe/ixgbe_phy.c	(working copy)
>@@ -1638,6 +1638,7 @@
> 	if (hw->phy.sfp_type == ixgbe_sfp_type_not_present)
> 		return physical_layer;
> 
>+	printf("hw->phy.type %d\n", hw->phy.type);
> 	switch (hw->phy.type) {
> 	case ixgbe_phy_sfp_passive_tyco:
> 	case ixgbe_phy_sfp_passive_unknown:
>@@ -1657,6 +1658,8 @@
> 		      IXGBE_SFF_1GBE_COMP_CODES, &comp_codes_1g);
> 		hw->phy.ops.read_i2c_eeprom(hw,
> 		      IXGBE_SFF_10GBE_COMP_CODES, &comp_codes_10g);
>+		printf("SFP comp_codes_1g %d\n", comp_codes_1g);
>+		printf("SFP comp_codes_10g %d\n", comp_codes_10g);
> 		if (comp_codes_10g & IXGBE_SFF_10GBASESR_CAPABLE)
> 			physical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_SR;
> 		else if (comp_codes_10g & IXGBE_SFF_10GBASELR_CAPABLE)
>@@ -1670,6 +1673,7 @@
> 	case ixgbe_phy_qsfp_unknown:
> 		hw->phy.ops.read_i2c_eeprom(hw,
> 		      IXGBE_SFF_QSFP_10GBE_COMP, &comp_codes_10g);
>+		printf("QSFP comp_codes_10g %d\n", comp_codes_10g);
> 		if (comp_codes_10g & IXGBE_SFF_10GBASESR_CAPABLE)
> 			physical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_SR;
> 		else if (comp_codes_10g & IXGBE_SFF_10GBASELR_CAPABLE)

Actions: View | Diff

Attachments on bug 150249: 171652 | 171751 | 171752 | 171974