e1000e: Alert the user that C-states will be disabled by enabling jumbo frames

Matt Turner · Jeff Kirsher · commit 8299b006d743 · 2018-01-24T12:27:48.000-08:00
I personally spent a long time trying to decypher why my CPU would not
reach deeper C-states. Let's just tell the next user what's going on.

Signed-off-by: Matt Turner &lt;matt.turner@intel.com&gt;
Acked-by: Shannon Nelson &lt;shannon.nelson@oracle.com&gt;
Tested-by: Aaron Brown &lt;aaron.f.brown@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c b/drivers/net/ethernet/intel/e1000e/netdev.c
@@ -3306,6 +3306,8 @@ static void e1000_configure_rx(struct e1000_adapter *adapter)
 			ew32(RXDCTL(0), rxdctl | 0x3 | BIT(8));
 		}
 
+		dev_info(&adapter->pdev->dev,
+			 "Some CPU C-states have been disabled in order to enable jumbo frames\n");
 		pm_qos_update_request(&adapter->pm_qos_req, lat);
 	} else {
 		pm_qos_update_request(&adapter->pm_qos_req,

Original file line number	Diff line number	Diff line change
`@@ -3306,6 +3306,8 @@ static void e1000_configure_rx(struct e1000_adapter *adapter)`
`3306`	`3306`	`ew32(RXDCTL(0), rxdctl \| 0x3 \| BIT(8));`
`3307`	`3307`	`}`
`3308`	`3308`
	`3309`	`+ dev_info(&adapter->pdev->dev,`
	`3310`	`+ "Some CPU C-states have been disabled in order to enable jumbo frames\n");`
`3309`	`3311`	`pm_qos_update_request(&adapter->pm_qos_req, lat);`
`3310`	`3312`	`} else {`
`3311`	`3313`	`pm_qos_update_request(&adapter->pm_qos_req,`