* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-01-09 12:52 Jeff Kirsher 2014-01-09 12:52 ` [net-next 01/15] i40e: Re-enable interrupt on ICR0 Jeff Kirsher ` (15 more replies) 0 siblings, 16 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only. Anjali provides a fix where interrupts were not being re-enabled on ICR0 even though they were auto masked by hardware. Then provides a fix to cleanup RSS initialization because it was doing some extra work, so remove the extra work and any bugs it created when managing number of queues. Since hardware requires a full packet template to be pointed to when adding hardware flow filters, add the template and use it for programming filters. Jesse provides a fix to replace the use of driver specific defines with kernel ETH_ALEN defines. Then disables packet split because with the use of GRO, we do not need the extra bus overhead. Fixes spelling error in code comment. Kamil provides a fix for the driver where the hardware expects the MAC address in a very specific format and the driver was filing the data incorrectly. Mitch provides a fix to resolve a panic on reset by adding checks to VSI->rx_rings. Then shortens alloc_rx_buff_failed and alloc_rx_page_failed variables since both part of an RX specific structure so just remove the _rx part of the name. Then fixes badly formatted lines, long lines and mis-formatted lines. Shannon provides a fix to call AQ to release any reservation held by this PF on the NVM resource lock on startup, in order to clear anything that might have been left over from a previous run. Then removes interrupt on AQ error since nearly everything we do is synchronous, using the interrupt-on-error bit is unnecessary and causing unneeded interrupts. Adds code to handle the ability to send messages among the physical function interfaces by the admin queue. Catherine sets the MFP flag earlier in software init and uses that flag to decide if other hardware work-arounds are required which turns off flow director in MFP mode. The following are changes since commit 54b553e2c16001d13e0186cad2531764065f9a1b: Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (2): i40e: Re-enable interrupt on ICR0 i40e: Cleanup reconfig rss path Catherine Sullivan (1): i40e: Turn flow director off in MFP mode Greg Rose (2): i40e: Fix GPL header i40e: add a comment on barrier and fix panic on reset Jesse Brandeburg (4): i40e: use kernel specific defines i40e: disable packet split i40e: fix spelling errors i40e: Add a dummy packet template Kamil Krawczyk (1): i40e: Fix MAC format in Write MAC address AQ cmd Mitch Williams (2): i40e: shorten wordy fields i40e: trivial: formatting and checkpatch fixes Shannon Nelson (3): i40e: release NVM resource reservation on startup i40e: remove interrupt on AQ error i40e: accept pf to pf adminq messages drivers/net/ethernet/intel/i40e/Makefile | 7 +-- drivers/net/ethernet/intel/i40e/i40e.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_adminq.c | 12 ++-- drivers/net/ethernet/intel/i40e/i40e_adminq.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_alloc.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_common.c | 14 +++-- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 13 ++--- drivers/net/ethernet/intel/i40e/i40e_diag.c | 7 +-- drivers/net/ethernet/intel/i40e/i40e_diag.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 33 ++++++++--- drivers/net/ethernet/intel/i40e/i40e_hmc.c | 7 +-- drivers/net/ethernet/intel/i40e/i40e_hmc.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c | 7 +-- drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_main.c | 68 +++++++++++----------- drivers/net/ethernet/intel/i40e/i40e_nvm.c | 7 +-- drivers/net/ethernet/intel/i40e/i40e_osdep.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_register.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_status.h | 7 +-- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 44 +++++++------- drivers/net/ethernet/intel/i40e/i40e_txrx.h | 11 ++-- drivers/net/ethernet/intel/i40e/i40e_type.h | 15 ++--- drivers/net/ethernet/intel/i40e/i40e_virtchnl.h | 11 ++-- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 10 ++-- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h | 7 +-- 27 files changed, 176 insertions(+), 174 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 01/15] i40e: Re-enable interrupt on ICR0 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 02/15] i40e: use kernel specific defines Jeff Kirsher ` (14 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Anjali Singhai Jain, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Anjali Singhai Jain <anjali.singhai@intel.com> The hardware can occasionally give an interrupt on the misc queue for which there is no driver work to do. In that case the driver was not re-enabling interrupts even though they were auto masked by hardware. This left interrupts disabled on this queue. Re-enable the interrupt whenever leaving this function. Signed-off-by: Anjali Singhai Jain <anjali.singhai@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index f736c44..38ec66f 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -2758,16 +2758,16 @@ static irqreturn_t i40e_intr(int irq, void *data) { struct i40e_pf *pf = (struct i40e_pf *)data; struct i40e_hw *hw = &pf->hw; + irqreturn_t ret = IRQ_NONE; u32 icr0, icr0_remaining; u32 val, ena_mask; icr0 = rd32(hw, I40E_PFINT_ICR0); + ena_mask = rd32(hw, I40E_PFINT_ICR0_ENA); /* if sharing a legacy IRQ, we might get called w/o an intr pending */ if ((icr0 & I40E_PFINT_ICR0_INTEVENT_MASK) == 0) - return IRQ_NONE; - - ena_mask = rd32(hw, I40E_PFINT_ICR0_ENA); + goto enable_intr; /* if interrupt but no bits showing, must be SWINT */ if (((icr0 & ~I40E_PFINT_ICR0_INTEVENT_MASK) == 0) || @@ -2843,7 +2843,9 @@ static irqreturn_t i40e_intr(int irq, void *data) } ena_mask &= ~icr0_remaining; } + ret = IRQ_HANDLED; +enable_intr: /* re-enable interrupt causes */ wr32(hw, I40E_PFINT_ICR0_ENA, ena_mask); if (!test_bit(__I40E_DOWN, &pf->state)) { @@ -2851,7 +2853,7 @@ static irqreturn_t i40e_intr(int irq, void *data) i40e_irq_dynamic_enable_icr0(pf); } - return IRQ_HANDLED; + return ret; } /** -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 02/15] i40e: use kernel specific defines 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher 2014-01-09 12:52 ` [net-next 01/15] i40e: Re-enable interrupt on ICR0 Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 03/15] i40e: Fix GPL header Jeff Kirsher ` (13 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem; +Cc: Jesse Brandeburg, netdev, gospo, sassmann, Jeff Kirsher From: Jesse Brandeburg <jesse.brandeburg@intel.com> Replace uses of I40E_LENGTH_OF_ADDRESS with ETH_ALEN. Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_type.h | 8 +++----- drivers/net/ethernet/intel/i40e/i40e_virtchnl.h | 4 ++-- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_type.h b/drivers/net/ethernet/intel/i40e/i40e_type.h index bcf0317..53c40cb 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_type.h +++ b/drivers/net/ethernet/intel/i40e/i40e_type.h @@ -77,8 +77,6 @@ struct i40e_hw; typedef void (*I40E_ADMINQ_CALLBACK)(struct i40e_hw *, struct i40e_aq_desc *); -#define I40E_ETH_LENGTH_OF_ADDRESS 6 - /* Data type manipulation macros. */ #define I40E_DESC_UNUSED(R) \ @@ -240,9 +238,9 @@ struct i40e_hw_capabilities { struct i40e_mac_info { enum i40e_mac_type type; - u8 addr[I40E_ETH_LENGTH_OF_ADDRESS]; - u8 perm_addr[I40E_ETH_LENGTH_OF_ADDRESS]; - u8 san_addr[I40E_ETH_LENGTH_OF_ADDRESS]; + u8 addr[ETH_ALEN]; + u8 perm_addr[ETH_ALEN]; + u8 san_addr[ETH_ALEN]; u16 max_fcoeq; }; diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h b/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h index cc6654f..6bcfcef 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h @@ -142,7 +142,7 @@ struct i40e_virtchnl_vsi_resource { u16 num_queue_pairs; enum i40e_vsi_type vsi_type; u16 qset_handle; - u8 default_mac_addr[I40E_ETH_LENGTH_OF_ADDRESS]; + u8 default_mac_addr[ETH_ALEN]; }; /* VF offload flags */ #define I40E_VIRTCHNL_VF_OFFLOAD_L2 0x00000001 @@ -265,7 +265,7 @@ struct i40e_virtchnl_queue_select { */ struct i40e_virtchnl_ether_addr { - u8 addr[I40E_ETH_LENGTH_OF_ADDRESS]; + u8 addr[ETH_ALEN]; u8 pad[2]; }; -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 03/15] i40e: Fix GPL header 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher 2014-01-09 12:52 ` [net-next 01/15] i40e: Re-enable interrupt on ICR0 Jeff Kirsher 2014-01-09 12:52 ` [net-next 02/15] i40e: use kernel specific defines Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 04/15] i40e: Fix MAC format in Write MAC address AQ cmd Jeff Kirsher ` (12 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem; +Cc: Greg Rose, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Greg Rose <gregory.v.rose@intel.com> The GPL header included in each file in the i40e driver doesn't need to include the "this program" text since this driver is already part of the larger kernel. Signed-off-by: Greg Rose <gregory.v.rose@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/Makefile | 7 +++---- drivers/net/ethernet/intel/i40e/i40e.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_adminq.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_adminq.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_alloc.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_common.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_diag.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_diag.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_hmc.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_hmc.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_main.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_nvm.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_osdep.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_register.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_status.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_txrx.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_type.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_virtchnl.h | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 7 +++---- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h | 7 +++---- 27 files changed, 81 insertions(+), 108 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/Makefile b/drivers/net/ethernet/intel/i40e/Makefile index 479b2c4..6ec1a79 100644 --- a/drivers/net/ethernet/intel/i40e/Makefile +++ b/drivers/net/ethernet/intel/i40e/Makefile @@ -1,7 +1,7 @@ ################################################################################ # # Intel Ethernet Controller XL710 Family Linux Driver -# Copyright(c) 2013 Intel Corporation. +# Copyright(c) 2013 - 2014 Intel Corporation. # # This program is free software; you can redistribute it and/or modify it # under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ # FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for # more details. # -# You should have received a copy of the GNU General Public License along with -# this program; if not, write to the Free Software Foundation, Inc., -# 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. +# You should have received a copy of the GNU General Public License along +# with this program. If not, see <http://www.gnu.org/licenses/>. # # The full GNU General Public License is included in this distribution in # the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e.h b/drivers/net/ethernet/intel/i40e/i40e.h index 09d92d0..4d4cdbf 100644 --- a/drivers/net/ethernet/intel/i40e/i40e.h +++ b/drivers/net/ethernet/intel/i40e/i40e.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq.c b/drivers/net/ethernet/intel/i40e/i40e_adminq.c index 2b32084..eb014bb 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_adminq.c +++ b/drivers/net/ethernet/intel/i40e/i40e_adminq.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq.h b/drivers/net/ethernet/intel/i40e/i40e_adminq.h index 21801c3..993f768 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_adminq.h +++ b/drivers/net/ethernet/intel/i40e/i40e_adminq.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h b/drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h index 0075087..c009eb4 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h +++ b/drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_alloc.h b/drivers/net/ethernet/intel/i40e/i40e_alloc.h index 3b1cc21..926811a 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_alloc.h +++ b/drivers/net/ethernet/intel/i40e/i40e_alloc.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_common.c b/drivers/net/ethernet/intel/i40e/i40e_common.c index d564910..8f28aee 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_common.c +++ b/drivers/net/ethernet/intel/i40e/i40e_common.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_debugfs.c b/drivers/net/ethernet/intel/i40e/i40e_debugfs.c index 125f758..e227d7cb 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_debugfs.c +++ b/drivers/net/ethernet/intel/i40e/i40e_debugfs.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_diag.c b/drivers/net/ethernet/intel/i40e/i40e_diag.c index 6a1657e..b2380da 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_diag.c +++ b/drivers/net/ethernet/intel/i40e/i40e_diag.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_diag.h b/drivers/net/ethernet/intel/i40e/i40e_diag.h index d1fc68c..0b59116 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_diag.h +++ b/drivers/net/ethernet/intel/i40e/i40e_diag.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c index 42c36d1..2016ca0 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c +++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_hmc.c b/drivers/net/ethernet/intel/i40e/i40e_hmc.c index bcedf3f..76dfef3 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_hmc.c +++ b/drivers/net/ethernet/intel/i40e/i40e_hmc.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_hmc.h b/drivers/net/ethernet/intel/i40e/i40e_hmc.h index aacd42a..72bf30a 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_hmc.h +++ b/drivers/net/ethernet/intel/i40e/i40e_hmc.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c index a695b91..101ed41 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c +++ b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h index f8afbcb0..341de92 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h +++ b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index 38ec66f..ea76134 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_nvm.c b/drivers/net/ethernet/intel/i40e/i40e_nvm.c index e2da0a2..37d66c8 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_nvm.c +++ b/drivers/net/ethernet/intel/i40e/i40e_nvm.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_osdep.h b/drivers/net/ethernet/intel/i40e/i40e_osdep.h index 702c81b..ecd0f0b 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_osdep.h +++ b/drivers/net/ethernet/intel/i40e/i40e_osdep.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_prototype.h b/drivers/net/ethernet/intel/i40e/i40e_prototype.h index e05d303..c7c3d82 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_prototype.h +++ b/drivers/net/ethernet/intel/i40e/i40e_prototype.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_register.h b/drivers/net/ethernet/intel/i40e/i40e_register.h index d188ec0..1d40f42 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_register.h +++ b/drivers/net/ethernet/intel/i40e/i40e_register.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_status.h b/drivers/net/ethernet/intel/i40e/i40e_status.h index 5e5bcdd..5f9cac5 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_status.h +++ b/drivers/net/ethernet/intel/i40e/i40e_status.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_txrx.c b/drivers/net/ethernet/intel/i40e/i40e_txrx.c index fac4fb3..e81e5bf 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_txrx.c +++ b/drivers/net/ethernet/intel/i40e/i40e_txrx.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_txrx.h b/drivers/net/ethernet/intel/i40e/i40e_txrx.h index 92f5cf5..7ec744b 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_txrx.h +++ b/drivers/net/ethernet/intel/i40e/i40e_txrx.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_type.h b/drivers/net/ethernet/intel/i40e/i40e_type.h index 53c40cb..12473ad 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_type.h +++ b/drivers/net/ethernet/intel/i40e/i40e_type.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h b/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h index 6bcfcef..22a1b69 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c index efb9a24..66fffbd 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h index 8d0f4dd..cc1feee 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h @@ -1,7 +1,7 @@ /******************************************************************************* * * Intel Ethernet Controller XL710 Family Linux Driver - * Copyright(c) 2013 Intel Corporation. + * Copyright(c) 2013 - 2014 Intel Corporation. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -12,9 +12,8 @@ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for * more details. * - * You should have received a copy of the GNU General Public License along with - * this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + * You should have received a copy of the GNU General Public License along + * with this program. If not, see <http://www.gnu.org/licenses/>. * * The full GNU General Public License is included in this distribution in * the file called "COPYING". -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 04/15] i40e: Fix MAC format in Write MAC address AQ cmd 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (2 preceding siblings ...) 2014-01-09 12:52 ` [net-next 03/15] i40e: Fix GPL header Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 05/15] i40e: add a comment on barrier and fix panic on reset Jeff Kirsher ` (11 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Kamil Krawczyk, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Kamil Krawczyk <kamil.krawczyk@intel.com> The MAC address format expected by the hardware is in a very specific format, and the driver was filling in the data incorrectly. Change-ID: I7bc66505ef459ee347dd3bda68051004c141c689 Signed-off-by: Kamil Krawczyk <kamil.krawczyk@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_common.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_common.c b/drivers/net/ethernet/intel/i40e/i40e_common.c index 8f28aee..807312b 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_common.c +++ b/drivers/net/ethernet/intel/i40e/i40e_common.c @@ -254,8 +254,11 @@ i40e_status i40e_aq_mac_address_write(struct i40e_hw *hw, i40e_fill_default_direct_cmd_desc(&desc, i40e_aqc_opc_mac_address_write); cmd_data->command_flags = cpu_to_le16(flags); - memcpy(&cmd_data->mac_sal, &mac_addr[0], 4); - memcpy(&cmd_data->mac_sah, &mac_addr[4], 2); + cmd_data->mac_sah = cpu_to_le16((u16)mac_addr[0] << 8 | mac_addr[1]); + cmd_data->mac_sal = cpu_to_le32(((u32)mac_addr[2] << 24) | + ((u32)mac_addr[3] << 16) | + ((u32)mac_addr[4] << 8) | + mac_addr[5]); status = i40e_asq_send_command(hw, &desc, NULL, 0, cmd_details); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (3 preceding siblings ...) 2014-01-09 12:52 ` [net-next 04/15] i40e: Fix MAC format in Write MAC address AQ cmd Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 14:42 ` Sergei Shtylyov 2014-01-09 22:16 ` Scott Feldman 2014-01-09 12:52 ` [net-next 06/15] i40e: disable packet split Jeff Kirsher ` (10 subsequent siblings) 15 siblings, 2 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Greg Rose, netdev, gospo, sassmann, Mitch Williams, Jesse Brandeburg, Jeff Kirsher From: Greg Rose <gregory.v.rose@intel.com> The memory barrier used in maybe_stop_tx can use a comment. Also add checks to VSI->rx_rings to ensure a kernel panic is not induced. Change-ID: I48cc1bf1d6cf301818155b737edeef77c0d790c7 Change-ID: I1363a8445fbf521a26267849966296ed55f43ad8 Signed-off-by: Greg Rose <gregory.v.rose@intel.com> Signed-off-by: Mitch Williams <mitch.a.williams@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 15 +++++++++++---- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 1 + 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index ea76134..5cdc67c 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -422,7 +422,7 @@ void i40e_vsi_reset_stats(struct i40e_vsi *vsi) memset(&vsi->net_stats_offsets, 0, sizeof(vsi->net_stats_offsets)); memset(&vsi->eth_stats, 0, sizeof(vsi->eth_stats)); memset(&vsi->eth_stats_offsets, 0, sizeof(vsi->eth_stats_offsets)); - if (vsi->rx_rings) + if (vsi->rx_rings && vsi->rx_rings[0]) { for (i = 0; i < vsi->num_queue_pairs; i++) { memset(&vsi->rx_rings[i]->stats, 0 , sizeof(vsi->rx_rings[i]->stats)); @@ -433,6 +433,7 @@ void i40e_vsi_reset_stats(struct i40e_vsi *vsi) memset(&vsi->tx_rings[i]->tx_stats, 0, sizeof(vsi->tx_rings[i]->tx_stats)); } + } vsi->stat_offsets_loaded = false; } @@ -2067,8 +2068,11 @@ static void i40e_vsi_free_tx_resources(struct i40e_vsi *vsi) { int i; + if (!vsi->tx_rings) + return; + for (i = 0; i < vsi->num_queue_pairs; i++) - if (vsi->tx_rings[i]->desc) + if (vsi->tx_rings[i] && vsi->tx_rings[i]->desc) i40e_free_tx_resources(vsi->tx_rings[i]); } @@ -2101,8 +2105,11 @@ static void i40e_vsi_free_rx_resources(struct i40e_vsi *vsi) { int i; + if (!vsi->rx_rings) + return; + for (i = 0; i < vsi->num_queue_pairs; i++) - if (vsi->rx_rings[i]->desc) + if (vsi->rx_rings[i] && vsi->rx_rings[i]->desc) i40e_free_rx_resources(vsi->rx_rings[i]); } @@ -5349,7 +5356,7 @@ static void i40e_vsi_clear_rings(struct i40e_vsi *vsi) { int i; - if (vsi->tx_rings[0]) { + if (vsi->tx_rings && vsi->tx_rings[0]) { for (i = 0; i < vsi->alloc_queue_pairs; i++) { kfree_rcu(vsi->tx_rings[i], rcu); vsi->tx_rings[i] = NULL; diff --git a/drivers/net/ethernet/intel/i40e/i40e_txrx.c b/drivers/net/ethernet/intel/i40e/i40e_txrx.c index e81e5bf..bc72f4e 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_txrx.c +++ b/drivers/net/ethernet/intel/i40e/i40e_txrx.c @@ -1715,6 +1715,7 @@ dma_error: static inline int __i40e_maybe_stop_tx(struct i40e_ring *tx_ring, int size) { netif_stop_subqueue(tx_ring->netdev, tx_ring->queue_index); + /* Memory barrier before checking head and tail */ smp_mb(); /* Check again in a case another CPU has just made room available. */ -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 12:52 ` [net-next 05/15] i40e: add a comment on barrier and fix panic on reset Jeff Kirsher @ 2014-01-09 14:42 ` Sergei Shtylyov 2014-01-09 17:17 ` Greg Rose 2014-01-09 22:16 ` Scott Feldman 1 sibling, 1 reply; 63+ messages in thread From: Sergei Shtylyov @ 2014-01-09 14:42 UTC (permalink / raw) To: Jeff Kirsher, davem Cc: Greg Rose, netdev, gospo, sassmann, Mitch Williams, Jesse Brandeburg Hello. On 09-01-2014 16:52, Jeff Kirsher wrote: > From: Greg Rose <gregory.v.rose@intel.com> > The memory barrier used in maybe_stop_tx can use a comment. > Also add checks to VSI->rx_rings to ensure a kernel panic is not induced. Don't see why this is made in one patch instead of two since the two things look completely unrelated and even modifying different files. > Change-ID: I48cc1bf1d6cf301818155b737edeef77c0d790c7 > Change-ID: I1363a8445fbf521a26267849966296ed55f43ad8 Why even 2 of them? > Signed-off-by: Greg Rose <gregory.v.rose@intel.com> > Signed-off-by: Mitch Williams <mitch.a.williams@intel.com> > Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> > Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> WBR, Sergei ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 14:42 ` Sergei Shtylyov @ 2014-01-09 17:17 ` Greg Rose 2014-01-09 20:12 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Greg Rose @ 2014-01-09 17:17 UTC (permalink / raw) To: Sergei Shtylyov Cc: Jeff Kirsher, davem, netdev, gospo, sassmann, Mitch Williams, Jesse Brandeburg On Thu, 9 Jan 2014 18:42:00 +0400 Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> wrote: > Hello. > > On 09-01-2014 16:52, Jeff Kirsher wrote: > > > From: Greg Rose <gregory.v.rose@intel.com> > > > The memory barrier used in maybe_stop_tx can use a comment. > > > Also add checks to VSI->rx_rings to ensure a kernel panic is not > > induced. > > Don't see why this is made in one patch instead of two since the > two things look completely unrelated and even modifying different > files. > > > Change-ID: I48cc1bf1d6cf301818155b737edeef77c0d790c7 > > Change-ID: I1363a8445fbf521a26267849966296ed55f43ad8 > > Why even 2 of them? > > > Signed-off-by: Greg Rose <gregory.v.rose@intel.com> > > Signed-off-by: Mitch Williams <mitch.a.williams@intel.com> > > Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> > > Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > WBR, Sergei > Yes, it's apparent that two different internal patches were compressed together. If its unacceptable to do this then I'll speak to Jeff about splitting them. - Greg ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 17:17 ` Greg Rose @ 2014-01-09 20:12 ` David Miller 2014-01-09 20:14 ` Rose, Gregory V 2014-01-09 21:21 ` Ben Hutchings 0 siblings, 2 replies; 63+ messages in thread From: David Miller @ 2014-01-09 20:12 UTC (permalink / raw) To: gregory.v.rose Cc: sergei.shtylyov, jeffrey.t.kirsher, netdev, gospo, sassmann, mitch.a.williams, jesse.brandeburg From: Greg Rose <gregory.v.rose@intel.com> Date: Thu, 9 Jan 2014 09:17:35 -0800 > Yes, it's apparent that two different internal patches were compressed > together. If its unacceptable to do this then I'll speak to Jeff about > splitting them. I won't reject the entire pull request on account of this, but please don't do this in the future. ^ permalink raw reply [flat|nested] 63+ messages in thread
* RE: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 20:12 ` David Miller @ 2014-01-09 20:14 ` Rose, Gregory V 2014-01-09 21:21 ` Ben Hutchings 1 sibling, 0 replies; 63+ messages in thread From: Rose, Gregory V @ 2014-01-09 20:14 UTC (permalink / raw) To: David Miller Cc: sergei.shtylyov@cogentembedded.com, Kirsher, Jeffrey T, netdev@vger.kernel.org, gospo@redhat.com, sassmann@redhat.com, Williams, Mitch A, Brandeburg, Jesse > -----Original Message----- > From: David Miller [mailto:davem@davemloft.net] > Sent: Thursday, January 09, 2014 12:12 PM > To: Rose, Gregory V > Cc: sergei.shtylyov@cogentembedded.com; Kirsher, Jeffrey T; > netdev@vger.kernel.org; gospo@redhat.com; sassmann@redhat.com; Williams, > Mitch A; Brandeburg, Jesse > Subject: Re: [net-next 05/15] i40e: add a comment on barrier and fix panic > on reset > > From: Greg Rose <gregory.v.rose@intel.com> > Date: Thu, 9 Jan 2014 09:17:35 -0800 > > > Yes, it's apparent that two different internal patches were compressed > > together. If its unacceptable to do this then I'll speak to Jeff > > about splitting them. > > I won't reject the entire pull request on account of this, but please > don't do this in the future. Thanks Dave, We've discussed this internally and ahem... mistakes were made. We'll look out for it in the future. - Greg ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 20:12 ` David Miller 2014-01-09 20:14 ` Rose, Gregory V @ 2014-01-09 21:21 ` Ben Hutchings 2014-01-09 21:23 ` Ben Hutchings 1 sibling, 1 reply; 63+ messages in thread From: Ben Hutchings @ 2014-01-09 21:21 UTC (permalink / raw) To: David Miller Cc: gregory.v.rose, sergei.shtylyov, jeffrey.t.kirsher, netdev, gospo, sassmann, mitch.a.williams, jesse.brandeburg On Thu, 2014-01-09 at 15:12 -0500, David Miller wrote: > From: Greg Rose <gregory.v.rose@intel.com> > Date: Thu, 9 Jan 2014 09:17:35 -0800 > > > Yes, it's apparent that two different internal patches were compressed > > together. If its unacceptable to do this then I'll speak to Jeff about > > splitting them. > > I won't reject the entire pull request on account of this, but please > don't do this in the future. I don't understand this. You don't want to see patches that are known to introduce regressions, and you expect people to squash together known buggy patches with their subsequent fix-ups before submitting. So, if a Change-Id is supposed to refer back to an 'original' version of a patch/commit, shouldn't there sometimes be more than one of them in the version that goes upstream? You do something very similar yourself sometimes, with upstream references in stable backports. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 21:21 ` Ben Hutchings @ 2014-01-09 21:23 ` Ben Hutchings 2014-01-09 21:56 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Ben Hutchings @ 2014-01-09 21:23 UTC (permalink / raw) To: David Miller Cc: gregory.v.rose, sergei.shtylyov, jeffrey.t.kirsher, netdev, gospo, sassmann, mitch.a.williams, jesse.brandeburg On Thu, 2014-01-09 at 21:21 +0000, Ben Hutchings wrote: > On Thu, 2014-01-09 at 15:12 -0500, David Miller wrote: > > From: Greg Rose <gregory.v.rose@intel.com> > > Date: Thu, 9 Jan 2014 09:17:35 -0800 > > > > > Yes, it's apparent that two different internal patches were compressed > > > together. If its unacceptable to do this then I'll speak to Jeff about > > > splitting them. > > > > I won't reject the entire pull request on account of this, but please > > don't do this in the future. > > I don't understand this. You don't want to see patches that are known > to introduce regressions, and you expect people to squash together known > buggy patches with their subsequent fix-ups before submitting. [...] Hmm, looking at this again, I suppose it is just this particular combination of two unrelated changes you're objecting to? Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 21:23 ` Ben Hutchings @ 2014-01-09 21:56 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-01-09 21:56 UTC (permalink / raw) To: bhutchings Cc: gregory.v.rose, sergei.shtylyov, jeffrey.t.kirsher, netdev, gospo, sassmann, mitch.a.williams, jesse.brandeburg From: Ben Hutchings <bhutchings@solarflare.com> Date: Thu, 9 Jan 2014 21:23:10 +0000 > On Thu, 2014-01-09 at 21:21 +0000, Ben Hutchings wrote: >> On Thu, 2014-01-09 at 15:12 -0500, David Miller wrote: >> > From: Greg Rose <gregory.v.rose@intel.com> >> > Date: Thu, 9 Jan 2014 09:17:35 -0800 >> > >> > > Yes, it's apparent that two different internal patches were compressed >> > > together. If its unacceptable to do this then I'll speak to Jeff about >> > > splitting them. >> > >> > I won't reject the entire pull request on account of this, but please >> > don't do this in the future. >> >> I don't understand this. You don't want to see patches that are known >> to introduce regressions, and you expect people to squash together known >> buggy patches with their subsequent fix-ups before submitting. > [...] > > Hmm, looking at this again, I suppose it is just this particular > combination of two unrelated changes you're objecting to? Yes. ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 12:52 ` [net-next 05/15] i40e: add a comment on barrier and fix panic on reset Jeff Kirsher 2014-01-09 14:42 ` Sergei Shtylyov @ 2014-01-09 22:16 ` Scott Feldman 2014-01-10 17:34 ` Williams, Mitch A 1 sibling, 1 reply; 63+ messages in thread From: Scott Feldman @ 2014-01-09 22:16 UTC (permalink / raw) To: Jeff Kirsher Cc: David Miller, Greg Rose, Netdev, gospo, sassmann, Mitch Williams, Jesse Brandeburg On Jan 9, 2014, at 4:52 AM, Jeff Kirsher <jeffrey.t.kirsher@intel.com> wrote: > From: Greg Rose <gregory.v.rose@intel.com> > > diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c > index ea76134..5cdc67c 100644 > --- a/drivers/net/ethernet/intel/i40e/i40e_main.c > +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c > @@ -422,7 +422,7 @@ void i40e_vsi_reset_stats(struct i40e_vsi *vsi) > memset(&vsi->net_stats_offsets, 0, sizeof(vsi->net_stats_offsets)); > memset(&vsi->eth_stats, 0, sizeof(vsi->eth_stats)); > memset(&vsi->eth_stats_offsets, 0, sizeof(vsi->eth_stats_offsets)); > - if (vsi->rx_rings) > + if (vsi->rx_rings && vsi->rx_rings[0]) { Any reason why just [0] is checked for !NULL here... > for (i = 0; i < vsi->num_queue_pairs; i++) { > memset(&vsi->rx_rings[i]->stats, 0 , > sizeof(vsi->rx_rings[i]->stats)); > @@ -433,6 +433,7 @@ void i40e_vsi_reset_stats(struct i40e_vsi *vsi) > memset(&vsi->tx_rings[i]->tx_stats, 0, > sizeof(vsi->tx_rings[i]->tx_stats)); > } > + } > vsi->stat_offsets_loaded = false; > } > > @@ -2101,8 +2105,11 @@ static void i40e_vsi_free_rx_resources(struct i40e_vsi *vsi) > { > int i; > > + if (!vsi->rx_rings) > + return; > + > for (i = 0; i < vsi->num_queue_pairs; i++) > - if (vsi->rx_rings[i]->desc) > + if (vsi->rx_rings[i] && vsi->rx_rings[i]->desc) but here every [i] is checked for !NULL here? > i40e_free_rx_resources(vsi->rx_rings[i]); > } > If [0] check is sufficient to know if array members are allocated, maybe an wrapper func would help document intent: static bool i40e_vsi_rings_allocated(struct i40e_ring *ring) { return (ring && ring[0]); } -scott ^ permalink raw reply [flat|nested] 63+ messages in thread
* RE: [net-next 05/15] i40e: add a comment on barrier and fix panic on reset 2014-01-09 22:16 ` Scott Feldman @ 2014-01-10 17:34 ` Williams, Mitch A 0 siblings, 0 replies; 63+ messages in thread From: Williams, Mitch A @ 2014-01-10 17:34 UTC (permalink / raw) To: Scott Feldman, Kirsher, Jeffrey T Cc: David Miller, Rose, Gregory V, Netdev, gospo@redhat.com, sassmann@redhat.com, Brandeburg, Jesse > -----Original Message----- > From: Scott Feldman [mailto:sfeldma@cumulusnetworks.com] > Sent: Thursday, January 09, 2014 2:17 PM > To: Kirsher, Jeffrey T > Cc: David Miller; Rose, Gregory V; Netdev; gospo@redhat.com; > sassmann@redhat.com; Williams, Mitch A; Brandeburg, Jesse > Subject: Re: [net-next 05/15] i40e: add a comment on barrier and fix panic > on reset > > > On Jan 9, 2014, at 4:52 AM, Jeff Kirsher <jeffrey.t.kirsher@intel.com> > wrote: > > > From: Greg Rose <gregory.v.rose@intel.com> > > > > diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c > b/drivers/net/ethernet/intel/i40e/i40e_main.c > > index ea76134..5cdc67c 100644 > > --- a/drivers/net/ethernet/intel/i40e/i40e_main.c > > +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c > > @@ -422,7 +422,7 @@ void i40e_vsi_reset_stats(struct i40e_vsi *vsi) > > memset(&vsi->net_stats_offsets, 0, sizeof(vsi->net_stats_offsets)); > > memset(&vsi->eth_stats, 0, sizeof(vsi->eth_stats)); > > memset(&vsi->eth_stats_offsets, 0, sizeof(vsi->eth_stats_offsets)); > > - if (vsi->rx_rings) > > + if (vsi->rx_rings && vsi->rx_rings[0]) { > > Any reason why just [0] is checked for !NULL here... > > > for (i = 0; i < vsi->num_queue_pairs; i++) { > > memset(&vsi->rx_rings[i]->stats, 0 , > > sizeof(vsi->rx_rings[i]->stats)); > > @@ -433,6 +433,7 @@ void i40e_vsi_reset_stats(struct i40e_vsi *vsi) > > memset(&vsi->tx_rings[i]->tx_stats, 0, > > sizeof(vsi->tx_rings[i]->tx_stats)); > > } > > + } > > vsi->stat_offsets_loaded = false; > > } > > > > @@ -2101,8 +2105,11 @@ static void i40e_vsi_free_rx_resources(struct > i40e_vsi *vsi) > > { > > int i; > > > > + if (!vsi->rx_rings) > > + return; > > + > > for (i = 0; i < vsi->num_queue_pairs; i++) > > - if (vsi->rx_rings[i]->desc) > > + if (vsi->rx_rings[i] && vsi->rx_rings[i]->desc) > > but here every [i] is checked for !NULL here? > > > i40e_free_rx_resources(vsi->rx_rings[i]); > > } > > > > If [0] check is sufficient to know if array members are allocated, maybe an > wrapper func would help document intent: > > static bool i40e_vsi_rings_allocated(struct i40e_ring *ring) > { > return (ring && ring[0]); > } > > -scott The assumption in vsi_reset_stats() is that the device is up and running normally, in which case all of the rings will be allocated, so we only need to check the first one. OTOH, in free_rx_resources can be called in case of an allocation failure, in which case you could conceivably have some rings allocated and some not. -Mitch ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 06/15] i40e: disable packet split 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (4 preceding siblings ...) 2014-01-09 12:52 ` [net-next 05/15] i40e: add a comment on barrier and fix panic on reset Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 07/15] i40e: Cleanup reconfig rss path Jeff Kirsher ` (9 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem; +Cc: Jesse Brandeburg, netdev, gospo, sassmann, Jeff Kirsher From: Jesse Brandeburg <jesse.brandeburg@intel.com> The driver and hardware support splitting packets on headers but with the use of GRO we don't need the extra bus overhead, so make this driver more like igb and ixgbe and disable packet split. Change-ID: Id42f2c3736baa9d5bdfe1f72d64226e7d8ebd737 Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index 5cdc67c..beb90e9 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -5817,7 +5817,6 @@ static int i40e_sw_init(struct i40e_pf *pf) pf->flags = I40E_FLAG_RX_CSUM_ENABLED | I40E_FLAG_MSI_ENABLED | I40E_FLAG_MSIX_ENABLED | - I40E_FLAG_RX_PS_ENABLED | I40E_FLAG_RX_1BUF_ENABLED; /* Depending on PF configurations, it is possible that the RSS -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 07/15] i40e: Cleanup reconfig rss path 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (5 preceding siblings ...) 2014-01-09 12:52 ` [net-next 06/15] i40e: disable packet split Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 08/15] i40e: release NVM resource reservation on startup Jeff Kirsher ` (8 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Anjali Singhai Jain, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Anjali Singhai Jain <anjali.singhai@intel.com> RSS initialization was doing some extra work, remove the extra work and any bugs it created when managing number of queues. Change-ID: Iea75b04a70d73ce76947b6a177ce89ab4899d4c6 Signed-off-by: Anjali Singhai Jain <anjali.singhai@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index beb90e9..e782db9 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -5771,16 +5771,8 @@ int i40e_reconfig_rss_queues(struct i40e_pf *pf, int queue_count) queue_count = rounddown_pow_of_two(queue_count); if (queue_count != pf->rss_size) { - if (pf->queues_left < (queue_count - pf->rss_size)) { - dev_info(&pf->pdev->dev, - "Not enough queues to do RSS on %d queues: remaining queues %d\n", - queue_count, pf->queues_left); - return pf->rss_size; - } i40e_prep_for_reset(pf); - pf->num_lan_qps += (queue_count - pf->rss_size); - pf->queues_left -= (queue_count - pf->rss_size); pf->rss_size = queue_count; i40e_reset_and_rebuild(pf, true); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 08/15] i40e: release NVM resource reservation on startup 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (6 preceding siblings ...) 2014-01-09 12:52 ` [net-next 07/15] i40e: Cleanup reconfig rss path Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 09/15] i40e: remove interrupt on AQ error Jeff Kirsher ` (7 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Shannon Nelson, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Shannon Nelson <shannon.nelson@intel.com> Call AQ to release any reservation held by this PF on the NVM resource lock on startup, in order to clear anything that might have been left over from a previous run. The lock is only cleared by the requestor calling for it to be cleared, on power-on, or firmware reset. This should help limit the need for rebooting a customer machine if something goes wrong on a firmware update or some other action. Change-ID: I8c8473e601d4ef512dda7baa77a6e75f2e5fea49 Signed-off-by: Shannon Nelson <shannon.nelson@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_adminq.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq.c b/drivers/net/ethernet/intel/i40e/i40e_adminq.c index eb014bb..2b76aba 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_adminq.c +++ b/drivers/net/ethernet/intel/i40e/i40e_adminq.c @@ -583,6 +583,9 @@ i40e_status i40e_init_adminq(struct i40e_hw *hw) goto init_adminq_free_arq; } + /* pre-emptive resource lock release */ + i40e_aq_release_resource(hw, I40E_NVM_RESOURCE_ID, 0, NULL); + ret_code = i40e_aq_set_hmc_resource_profile(hw, I40E_HMC_PROFILE_DEFAULT, 0, -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 09/15] i40e: remove interrupt on AQ error 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (7 preceding siblings ...) 2014-01-09 12:52 ` [net-next 08/15] i40e: release NVM resource reservation on startup Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 10/15] i40e: accept pf to pf adminq messages Jeff Kirsher ` (6 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Shannon Nelson, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Shannon Nelson <shannon.nelson@intel.com> Since nearly everything we do is synchronous, using the interrupt-on-error bit is unnecessary and causing unneeded interrupts. If anyone wants to use the bit they can turn it on in individual AQ requests using the cmd_details parameter. Change-ID: I4690a9c561d3e0836aeadb4f88f8a8702b1d1366 Signed-off-by: Shannon Nelson <shannon.nelson@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_adminq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq.c b/drivers/net/ethernet/intel/i40e/i40e_adminq.c index 2b76aba..c75aa2d 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_adminq.c +++ b/drivers/net/ethernet/intel/i40e/i40e_adminq.c @@ -864,7 +864,7 @@ void i40e_fill_default_direct_cmd_desc(struct i40e_aq_desc *desc, /* zero out the desc */ memset((void *)desc, 0, sizeof(struct i40e_aq_desc)); desc->opcode = cpu_to_le16(opcode); - desc->flags = cpu_to_le16(I40E_AQ_FLAG_EI | I40E_AQ_FLAG_SI); + desc->flags = cpu_to_le16(I40E_AQ_FLAG_SI); } /** -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 10/15] i40e: accept pf to pf adminq messages 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (8 preceding siblings ...) 2014-01-09 12:52 ` [net-next 09/15] i40e: remove interrupt on AQ error Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 11/15] i40e: shorten wordy fields Jeff Kirsher ` (5 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Shannon Nelson, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Shannon Nelson <shannon.nelson@intel.com> The admin queue can be used to send messages among the physical function interfaces. This adds the code to handle that case. Change-ID: I0700fcc47e41433131a381f0eb72fc7b01b6bd87 Signed-off-by: Shannon Nelson <shannon.nelson@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index e782db9..4b75de8 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -4522,10 +4522,13 @@ static void i40e_clean_adminq_subtask(struct i40e_pf *pf) dev_info(&pf->pdev->dev, "ARQ LAN queue overflow event received\n"); i40e_handle_lan_overflow_event(pf, &event); break; + case i40e_aqc_opc_send_msg_to_peer: + dev_info(&pf->pdev->dev, "ARQ: Msg from other pf\n"); + break; default: dev_info(&pf->pdev->dev, - "ARQ Error: Unknown event %d received\n", - event.desc.opcode); + "ARQ Error: Unknown event 0x%04x received\n", + opcode); break; } } while (pending && (i++ < pf->adminq_work_limit)); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 11/15] i40e: shorten wordy fields 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (9 preceding siblings ...) 2014-01-09 12:52 ` [net-next 10/15] i40e: accept pf to pf adminq messages Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 12/15] i40e: trivial: formatting and checkpatch fixes Jeff Kirsher ` (4 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Mitch Williams, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Mitch Williams <mitch.a.williams@intel.com> The alloc_rx_buff_failed and alloc_rx_page_failed variables are both part of an rx specific structure so just remove the _rx part of the name. No functional changes. Change-ID: Icffa2f5d13c6f2b1e09cf45b9472b83c9dae8fc6 Signed-off-by: Mitch Williams <mitch.a.williams@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 6 +++--- drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 8 ++++---- drivers/net/ethernet/intel/i40e/i40e_txrx.h | 4 ++-- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_debugfs.c b/drivers/net/ethernet/intel/i40e/i40e_debugfs.c index e227d7cb..0220b18 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_debugfs.c +++ b/drivers/net/ethernet/intel/i40e/i40e_debugfs.c @@ -515,10 +515,10 @@ static void i40e_dbg_dump_vsi_seid(struct i40e_pf *pf, int seid) rx_ring->stats.bytes, rx_ring->rx_stats.non_eop_descs); dev_info(&pf->pdev->dev, - " rx_rings[%i]: rx_stats: alloc_rx_page_failed = %lld, alloc_rx_buff_failed = %lld\n", + " rx_rings[%i]: rx_stats: alloc_page_failed = %lld, alloc_buff_failed = %lld\n", i, - rx_ring->rx_stats.alloc_rx_page_failed, - rx_ring->rx_stats.alloc_rx_buff_failed); + rx_ring->rx_stats.alloc_page_failed, + rx_ring->rx_stats.alloc_buff_failed); dev_info(&pf->pdev->dev, " rx_rings[%i]: size = %i, dma = 0x%08lx\n", i, rx_ring->size, diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index 4b75de8..e270b61 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -786,8 +786,8 @@ void i40e_update_stats(struct i40e_vsi *vsi) } while (u64_stats_fetch_retry_bh(&p->syncp, start)); rx_b += bytes; rx_p += packets; - rx_buf += p->rx_stats.alloc_rx_buff_failed; - rx_page += p->rx_stats.alloc_rx_page_failed; + rx_buf += p->rx_stats.alloc_buff_failed; + rx_page += p->rx_stats.alloc_page_failed; } rcu_read_unlock(); vsi->tx_restart = tx_restart; diff --git a/drivers/net/ethernet/intel/i40e/i40e_txrx.c b/drivers/net/ethernet/intel/i40e/i40e_txrx.c index bc72f4e..5245ebb 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_txrx.c +++ b/drivers/net/ethernet/intel/i40e/i40e_txrx.c @@ -774,7 +774,7 @@ void i40e_alloc_rx_buffers(struct i40e_ring *rx_ring, u16 cleaned_count) skb = netdev_alloc_skb_ip_align(rx_ring->netdev, rx_ring->rx_buf_len); if (!skb) { - rx_ring->rx_stats.alloc_rx_buff_failed++; + rx_ring->rx_stats.alloc_buff_failed++; goto no_buffers; } /* initialize queue mapping */ @@ -788,7 +788,7 @@ void i40e_alloc_rx_buffers(struct i40e_ring *rx_ring, u16 cleaned_count) rx_ring->rx_buf_len, DMA_FROM_DEVICE); if (dma_mapping_error(rx_ring->dev, bi->dma)) { - rx_ring->rx_stats.alloc_rx_buff_failed++; + rx_ring->rx_stats.alloc_buff_failed++; bi->dma = 0; goto no_buffers; } @@ -798,7 +798,7 @@ void i40e_alloc_rx_buffers(struct i40e_ring *rx_ring, u16 cleaned_count) if (!bi->page) { bi->page = alloc_page(GFP_ATOMIC); if (!bi->page) { - rx_ring->rx_stats.alloc_rx_page_failed++; + rx_ring->rx_stats.alloc_page_failed++; goto no_buffers; } } @@ -813,7 +813,7 @@ void i40e_alloc_rx_buffers(struct i40e_ring *rx_ring, u16 cleaned_count) DMA_FROM_DEVICE); if (dma_mapping_error(rx_ring->dev, bi->page_dma)) { - rx_ring->rx_stats.alloc_rx_page_failed++; + rx_ring->rx_stats.alloc_page_failed++; bi->page_dma = 0; goto no_buffers; } diff --git a/drivers/net/ethernet/intel/i40e/i40e_txrx.h b/drivers/net/ethernet/intel/i40e/i40e_txrx.h index 7ec744b..6f8506c 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_txrx.h +++ b/drivers/net/ethernet/intel/i40e/i40e_txrx.h @@ -174,8 +174,8 @@ struct i40e_tx_queue_stats { struct i40e_rx_queue_stats { u64 non_eop_descs; - u64 alloc_rx_page_failed; - u64 alloc_rx_buff_failed; + u64 alloc_page_failed; + u64 alloc_buff_failed; }; enum i40e_ring_state_t { -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 12/15] i40e: trivial: formatting and checkpatch fixes 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (10 preceding siblings ...) 2014-01-09 12:52 ` [net-next 11/15] i40e: shorten wordy fields Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 13/15] i40e: fix spelling errors Jeff Kirsher ` (3 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Mitch Williams, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Mitch Williams <mitch.a.williams@intel.com> Fix some badly formatted lines, long lines and a mis-formatted else. Change-ID: Iac2eef064ae27c55a0c3d9c15c525bf8fed8ab6f Signed-off-by: Mitch Williams <mitch.a.williams@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 30 +++++++++++----------- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 3 ++- 2 files changed, 17 insertions(+), 16 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_txrx.c b/drivers/net/ethernet/intel/i40e/i40e_txrx.c index 5245ebb..43d88dd 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_txrx.c +++ b/drivers/net/ethernet/intel/i40e/i40e_txrx.c @@ -990,15 +990,15 @@ static int i40e_clean_rx_irq(struct i40e_ring *rx_ring, int budget) skb = rx_bi->skb; prefetch(skb->data); - rx_packet_len = (qword & I40E_RXD_QW1_LENGTH_PBUF_MASK) - >> I40E_RXD_QW1_LENGTH_PBUF_SHIFT; - rx_header_len = (qword & I40E_RXD_QW1_LENGTH_HBUF_MASK) - >> I40E_RXD_QW1_LENGTH_HBUF_SHIFT; - rx_sph = (qword & I40E_RXD_QW1_LENGTH_SPH_MASK) - >> I40E_RXD_QW1_LENGTH_SPH_SHIFT; - - rx_error = (qword & I40E_RXD_QW1_ERROR_MASK) - >> I40E_RXD_QW1_ERROR_SHIFT; + rx_packet_len = (qword & I40E_RXD_QW1_LENGTH_PBUF_MASK) >> + I40E_RXD_QW1_LENGTH_PBUF_SHIFT; + rx_header_len = (qword & I40E_RXD_QW1_LENGTH_HBUF_MASK) >> + I40E_RXD_QW1_LENGTH_HBUF_SHIFT; + rx_sph = (qword & I40E_RXD_QW1_LENGTH_SPH_MASK) >> + I40E_RXD_QW1_LENGTH_SPH_SHIFT; + + rx_error = (qword & I40E_RXD_QW1_ERROR_MASK) >> + I40E_RXD_QW1_ERROR_SHIFT; rx_hbo = rx_error & (1 << I40E_RX_DESC_ERROR_HBO_SHIFT); rx_error &= ~(1 << I40E_RX_DESC_ERROR_HBO_SHIFT); @@ -1114,8 +1114,8 @@ next_desc: /* use prefetched values */ rx_desc = next_rxd; qword = le64_to_cpu(rx_desc->wb.qword1.status_error_len); - rx_status = (qword & I40E_RXD_QW1_STATUS_MASK) - >> I40E_RXD_QW1_STATUS_SHIFT; + rx_status = (qword & I40E_RXD_QW1_STATUS_MASK) >> + I40E_RXD_QW1_STATUS_SHIFT; } rx_ring->next_to_clean = i; @@ -1414,10 +1414,10 @@ static int i40e_tso(struct i40e_ring *tx_ring, struct sk_buff *skb, cd_cmd = I40E_TX_CTX_DESC_TSO; cd_tso_len = skb->len - *hdr_len; cd_mss = skb_shinfo(skb)->gso_size; - *cd_type_cmd_tso_mss |= ((u64)cd_cmd << I40E_TXD_CTX_QW1_CMD_SHIFT) - | ((u64)cd_tso_len - << I40E_TXD_CTX_QW1_TSO_LEN_SHIFT) - | ((u64)cd_mss << I40E_TXD_CTX_QW1_MSS_SHIFT); + *cd_type_cmd_tso_mss |= ((u64)cd_cmd << I40E_TXD_CTX_QW1_CMD_SHIFT) | + ((u64)cd_tso_len << + I40E_TXD_CTX_QW1_TSO_LEN_SHIFT) | + ((u64)cd_mss << I40E_TXD_CTX_QW1_MSS_SHIFT); return 1; } diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c index 66fffbd..51a4f61 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c @@ -176,7 +176,8 @@ static void i40e_config_irq_link_list(struct i40e_vf *vf, u16 vsi_idx, (I40E_MAX_VSI_QP * I40E_VIRTCHNL_SUPPORTED_QTYPES), next_q + 1); - if (next_q < (I40E_MAX_VSI_QP * I40E_VIRTCHNL_SUPPORTED_QTYPES)) { + if (next_q < + (I40E_MAX_VSI_QP * I40E_VIRTCHNL_SUPPORTED_QTYPES)) { vsi_queue_id = next_q / I40E_VIRTCHNL_SUPPORTED_QTYPES; qtype = next_q % I40E_VIRTCHNL_SUPPORTED_QTYPES; pf_queue_id = i40e_vc_get_pf_queue_id(vf, vsi_idx, -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 13/15] i40e: fix spelling errors 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (11 preceding siblings ...) 2014-01-09 12:52 ` [net-next 12/15] i40e: trivial: formatting and checkpatch fixes Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 14/15] i40e: Add a dummy packet template Jeff Kirsher ` (2 subsequent siblings) 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem; +Cc: Jesse Brandeburg, netdev, gospo, sassmann, Jeff Kirsher From: Jesse Brandeburg <jesse.brandeburg@intel.com> This patch does s/rebuid/rebuild/. Change-ID: I77ff0c26db3d23ca5dc971b82d71b4ed8c6bcc14 Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index e270b61..63ea703 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -4795,7 +4795,7 @@ static int i40e_prep_for_reset(struct i40e_pf *pf) } /** - * i40e_reset_and_rebuild - reset and rebuid using a saved config + * i40e_reset_and_rebuild - reset and rebuild using a saved config * @pf: board private structure * @reinit: if the Main VSI needs to re-initialized. **/ -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 14/15] i40e: Add a dummy packet template 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (12 preceding siblings ...) 2014-01-09 12:52 ` [net-next 13/15] i40e: fix spelling errors Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 12:52 ` [net-next 15/15] i40e: Turn flow director off in MFP mode Jeff Kirsher 2014-01-09 20:13 ` [net-next 00/15][pull request] Intel Wired LAN Driver Updates David Miller 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Jesse Brandeburg, netdev, gospo, sassmann, Anjali Singhai Jain, Jeff Kirsher From: Jesse Brandeburg <jesse.brandeburg@intel.com> The hardware requires a full packet template to be pointed to when adding hardware flow filters. This patch adds the template and uses it for programming filters. Change-ID: I09db9f4ab0207ca9c520ae36596d74e1a0663ae5 Signed-off-by: Anjali Singhai Jain <anjali.singhai@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 26 +++++++++++++++++++++++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c index 2016ca0..b886ee5 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c +++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c @@ -1243,6 +1243,7 @@ static int i40e_set_rss_hash_opt(struct i40e_pf *pf, struct ethtool_rxnfc *nfc) } #define IP_HEADER_OFFSET 14 +#define I40E_UDPIP_DUMMY_PACKET_LEN 42 /** * i40e_add_del_fdir_udpv4 - Add/Remove UDPv4 Flow Director filters for * a specific flow spec @@ -1263,6 +1264,12 @@ static int i40e_add_del_fdir_udpv4(struct i40e_vsi *vsi, bool err = false; int ret; int i; + char packet[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x08, 0, + 0x45, 0, 0, 0x1c, 0, 0, 0x40, 0, 0x40, 0x11, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0}; + + memcpy(fd_data->raw_packet, packet, I40E_UDPIP_DUMMY_PACKET_LEN); ip = (struct iphdr *)(fd_data->raw_packet + IP_HEADER_OFFSET); udp = (struct udphdr *)(fd_data->raw_packet + IP_HEADER_OFFSET @@ -1293,6 +1300,7 @@ static int i40e_add_del_fdir_udpv4(struct i40e_vsi *vsi, return err ? -EOPNOTSUPP : 0; } +#define I40E_TCPIP_DUMMY_PACKET_LEN 54 /** * i40e_add_del_fdir_tcpv4 - Add/Remove TCPv4 Flow Director filters for * a specific flow spec @@ -1312,6 +1320,14 @@ static int i40e_add_del_fdir_tcpv4(struct i40e_vsi *vsi, struct iphdr *ip; bool err = false; int ret; + /* Dummy packet */ + char packet[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x08, 0, + 0x45, 0, 0, 0x28, 0, 0, 0x40, 0, 0x40, 0x6, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0x80, 0x11, 0x0, 0x72, 0, 0, 0, 0}; + + memcpy(fd_data->raw_packet, packet, I40E_TCPIP_DUMMY_PACKET_LEN); ip = (struct iphdr *)(fd_data->raw_packet + IP_HEADER_OFFSET); tcp = (struct tcphdr *)(fd_data->raw_packet + IP_HEADER_OFFSET @@ -1319,6 +1335,8 @@ static int i40e_add_del_fdir_tcpv4(struct i40e_vsi *vsi, ip->daddr = fsp->h_u.tcp_ip4_spec.ip4dst; tcp->dest = fsp->h_u.tcp_ip4_spec.pdst; + ip->saddr = fsp->h_u.tcp_ip4_spec.ip4src; + tcp->source = fsp->h_u.tcp_ip4_spec.psrc; fd_data->pctype = I40E_FILTER_PCTYPE_NONF_IPV4_TCP_SYN; ret = i40e_program_fdir_filter(fd_data, pf, add); @@ -1333,9 +1351,6 @@ static int i40e_add_del_fdir_tcpv4(struct i40e_vsi *vsi, fd_data->pctype, ret); } - ip->saddr = fsp->h_u.tcp_ip4_spec.ip4src; - tcp->source = fsp->h_u.tcp_ip4_spec.psrc; - fd_data->pctype = I40E_FILTER_PCTYPE_NONF_IPV4_TCP; ret = i40e_program_fdir_filter(fd_data, pf, add); @@ -1369,6 +1384,7 @@ static int i40e_add_del_fdir_sctpv4(struct i40e_vsi *vsi, return -EOPNOTSUPP; } +#define I40E_IP_DUMMY_PACKET_LEN 34 /** * i40e_add_del_fdir_ipv4 - Add/Remove IPv4 Flow Director filters for * a specific flow spec @@ -1388,7 +1404,11 @@ static int i40e_add_del_fdir_ipv4(struct i40e_vsi *vsi, bool err = false; int ret; int i; + char packet[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x08, 0, + 0x45, 0, 0, 0x14, 0, 0, 0x40, 0, 0x40, 0x10, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}; + memcpy(fd_data->raw_packet, packet, I40E_IP_DUMMY_PACKET_LEN); ip = (struct iphdr *)(fd_data->raw_packet + IP_HEADER_OFFSET); ip->saddr = fsp->h_u.usr_ip4_spec.ip4src; -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* [net-next 15/15] i40e: Turn flow director off in MFP mode 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (13 preceding siblings ...) 2014-01-09 12:52 ` [net-next 14/15] i40e: Add a dummy packet template Jeff Kirsher @ 2014-01-09 12:52 ` Jeff Kirsher 2014-01-09 20:13 ` [net-next 00/15][pull request] Intel Wired LAN Driver Updates David Miller 15 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-09 12:52 UTC (permalink / raw) To: davem Cc: Catherine Sullivan, netdev, gospo, sassmann, Jesse Brandeburg, Jeff Kirsher From: Catherine Sullivan <catherine.sullivan@intel.com> The driver needs to set the MFP flag earlier in i40e_sw_init and then can use that flag to decide if other hardware work-arouds are required. Change-ID: Ib17ad1e3485f57b28845ab4722294a99f203bd48 Signed-off-by: Catherine Sullivan <catherine.sullivan@intel.com> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- drivers/net/ethernet/intel/i40e/i40e_main.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index 63ea703..1b792ee 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -4658,7 +4658,7 @@ static int i40e_get_capabilities(struct i40e_pf *pf) } } while (err); - if (pf->hw.revision_id == 0 && pf->hw.func_caps.npar_enable) { + if (pf->hw.revision_id == 0 && (pf->flags & I40E_FLAG_MFP_ENABLED)) { pf->hw.func_caps.num_msix_vectors += 1; pf->hw.func_caps.num_tx_qp = min_t(int, pf->hw.func_caps.num_tx_qp, @@ -5827,6 +5827,12 @@ static int i40e_sw_init(struct i40e_pf *pf) pf->rss_size = 1; } + /* MFP mode enabled */ + if (pf->hw.func_caps.npar_enable || pf->hw.func_caps.mfp_mode_1) { + pf->flags |= I40E_FLAG_MFP_ENABLED; + dev_info(&pf->pdev->dev, "MFP mode Enabled\n"); + } + if (pf->hw.func_caps.dcb) pf->num_tc_qps = I40E_DEFAULT_QUEUES_PER_TC; else @@ -5855,12 +5861,6 @@ static int i40e_sw_init(struct i40e_pf *pf) pf->num_vmdq_qps = I40E_DEFAULT_QUEUES_PER_VMDQ; } - /* MFP mode enabled */ - if (pf->hw.func_caps.npar_enable || pf->hw.func_caps.mfp_mode_1) { - pf->flags |= I40E_FLAG_MFP_ENABLED; - dev_info(&pf->pdev->dev, "MFP mode Enabled\n"); - } - #ifdef CONFIG_PCI_IOV if (pf->hw.func_caps.num_vfs) { pf->num_vf_qps = I40E_DEFAULT_QUEUES_PER_VF; -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher ` (14 preceding siblings ...) 2014-01-09 12:52 ` [net-next 15/15] i40e: Turn flow director off in MFP mode Jeff Kirsher @ 2014-01-09 20:13 ` David Miller 15 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-01-09 20:13 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Thu, 9 Jan 2014 04:52:09 -0800 > This series contains updates to i40e only. ... > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-05-29 15:10 Jeff Kirsher 2014-05-31 0:53 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-05-29 15:10 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e and i40evf. Kevin updates the i40e and i40evf driver i40e_check_asq_alive() to ensure the length register offset is non-zero which indicates that the software has initialized the admin queue. Also removes PCTYPE definitions which are now reserved. Mitch enables descriptor prefetch for rings belonging to the virtual function. Also configures the VF minimum transmit rate to 50 Mbps rather than 0 which was be interpreted as no limit at all. Mitch found in order for the VF to achieve its programmed transmit rate, we need to set the max credit value to 4. Lastly fixes a Tx hang and firmware crash that happens after setting the MTU on a VF by not using the RESETTING state during reinit, this is because the RESETTING state means that a catastrophic hardware bad thing is happening and the driver needs to tiptoe around and not use the admin queue or registers. A reinit is no big deal and we can use the admin queue (and we should) so do not set the state to RESETTING during reinit to resolve the bug. Akeem changes the declaration of the transmit and receive rings inside several loops to eliminate declaring the same ring every time for the duration of the loop and declares them just once before the loop. Also fixes the driver to clear the recovery pending bit if pf_reset fails instead of falling through the setup process. Anjali makes a change based on feedback from Ben Hutchings that cmd->data needs to be reported in ETHTOOL_GRXCLSRLCNT and use a helper function to calculate the total filter count. Jesse removes storm control since the storm control features are not apart of the hardware and were mistakenly left in the code. Greg changes tx_lpi_status and rx_lpi_status from bool to u32 to avoid sparse errors. Shannon adds the clear_pxe AdminQ API call to tell the firmware that the driver is taking over from PXE. In addition, relaxes the firmware API check to allow more flexibility in handling newer NICs and NVMs in the field. Vasu ensures that FCoE is disabled for MFP modes since it is not supported by overriding the hardware FCoE capability. The following are changes since commit 6623b4194459c07859d3e3196c3994fa7be5b88e: Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Akeem G Abodunrin (2): i40e: Tx/Rx rings declaration i40e: Clear recovery pending, if reset failed Anjali Singhai Jain (2): i40e: Report cmd->data in ETHTOOL_GRXCLSRLCNT instead of ETHTOOL_GRXCLSRULE i40e: Use the new i40e_get_fd_cnt_all function in other places Greg Rose (1): i40e/i40evf: Change type to u32 to avoid sparse error Jesse Brandeburg (1): i40e/i40evf: remove storm control Kevin Scott (2): i40e/i40evf: Update check for AQ aliveness i40e/i40evf: Remove reserved PCTYPE defines Mitch Williams (4): i40e: enable descriptor prefetch for VFs i40evf: don't round Tx rate down to 0 i40evf: tweak Tx rate params and de-magic-ify i40evf: don't use RESETTING state during reinit Shannon Nelson (2): i40e: add clear_pxe AdminQ request i40e: relax the firmware API version check Vasu Dev (1): i40: disable FCoE for MFP modes drivers/net/ethernet/intel/i40e/i40e.h | 9 +++++ drivers/net/ethernet/intel/i40e/i40e_adminq.c | 3 +- drivers/net/ethernet/intel/i40e/i40e_common.c | 42 +++++++++++++++++++++- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 37 +++++++------------ drivers/net/ethernet/intel/i40e/i40e_main.c | 19 +++++----- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 2 ++ drivers/net/ethernet/intel/i40e/i40e_register.h | 4 --- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 33 +++++++---------- drivers/net/ethernet/intel/i40e/i40e_txrx.h | 7 +--- drivers/net/ethernet/intel/i40e/i40e_type.h | 14 +++----- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 13 +++++-- drivers/net/ethernet/intel/i40evf/i40e_common.c | 6 +++- drivers/net/ethernet/intel/i40evf/i40e_register.h | 4 --- drivers/net/ethernet/intel/i40evf/i40e_txrx.h | 7 +--- drivers/net/ethernet/intel/i40evf/i40e_type.h | 14 +++----- drivers/net/ethernet/intel/i40evf/i40evf_main.c | 2 -- 16 files changed, 116 insertions(+), 100 deletions(-) -- 1.9.3 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-05-29 15:10 Jeff Kirsher @ 2014-05-31 0:53 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-05-31 0:53 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Thu, 29 May 2014 08:10:01 -0700 > This series contains updates to i40e and i40evf. ... > The following are changes since commit 6623b4194459c07859d3e3196c3994fa7be5b88e: > Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next > and are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks Jeff. And, as always, thanks for the detailed cover letter. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-05-27 9:12 Jeff Kirsher 2014-05-28 16:42 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-05-27 9:12 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to e1000, e1000e, ixgbe and i40evf. Emil provides a fix for ixgbe so that non-fiber devices with MNG FW enabled are able to link at 100Mbps. Jacob provides several changes for ixgbe, most of which are PTP related. Renames ixgbe_ptp_enable() to ixgbe_ptp_feature_enable() to better reflect the functions purpose. Extracts the hardware setup logic for the PTP hardware bits from the ixgbe_ptp_set_ts_config() to enable future work for the ixgbe_ptp_reset(). Maintain the hwstamp configuration through a reset and extracts the creation of the PTP clock device from ptp_init() in order to properly handle a suspend/resume cycle and only calls it if we don't already have a ptp_clock pointer. David provides a patch to expend the e1000e driver to turn on unicast PROMISC when there is failure to write to a shared receive address register. The fix update_phy_task() for 82579 is expanded to include newer PHYs as well so that the dev_spec->eee_lp_ability has the correct value when going into SX states. Todd provides a e1000e fix an errata for 82574/82583 where it is possible bad bits are read from SYSTIMH/L so check to see that the time is incrementing at a reasonable rate and is a multiple of the time incremental value. Removes a redundant igb PHY power down register write. Andi Kleen out of lines two write functions for e1000e to save 30k text size. Tobias Klauser converts the e1000 and i40evf drivers to use the is_broadcast_ether_addr() and is_multicast_ether_addr(). The following are changes since commit 3bea8eddfebe1c61ea824dd2880d4368791edc1f: Merge branch 'net-sysfs-docs' and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Andi Kleen (1): e1000e: Out of line __ew32_prepare/__ew32 David Ertman (3): e1000e: Failure to write SHRA turns on PROMISC mode e1000e: Cleanup parenthesis around return value e1000e: Fix expand setting EEE link info to all affected parts Emil Tantilov (1): ixgbe: fix linking at 100Mbps on copper devices with MNG FW enabled Jacob Keller (6): ixgbe: rename ixgbe_ptp_enable to ixgbe_ptp_feature_enable ixgbe: extract the hardware setup from the ixgbe_ptp_set_ts_config ixgbe: allow ixgbe_ptp_reset to maintain current hwtstamp config ixgbe: extract PTP clock device from ptp_init ixgbe: separate the PTP suspend and stop actions ixgbe: avoid duplicate code in suspend and stop paths Tobias Klauser (2): e1000: Use is_broadcast_ether_addr/is_multicast_ether_addr helpers i40evf: Use is_multicast_ether_addr helper Todd Fujinaka (2): e1000e: 82574/82583 TimeSync errata for SYSTIM read igb: remove redundant PHY power down register write drivers/net/ethernet/intel/e1000/e1000_hw.c | 4 +- drivers/net/ethernet/intel/e1000e/80003es2lan.c | 1 + drivers/net/ethernet/intel/e1000e/82571.c | 1 + drivers/net/ethernet/intel/e1000e/e1000.h | 33 +---- drivers/net/ethernet/intel/e1000e/hw.h | 3 +- drivers/net/ethernet/intel/e1000e/ich8lan.c | 61 +++++++-- drivers/net/ethernet/intel/e1000e/mac.c | 9 +- drivers/net/ethernet/intel/e1000e/mac.h | 3 +- drivers/net/ethernet/intel/e1000e/netdev.c | 69 +++++++++- drivers/net/ethernet/intel/i40evf/i40evf_main.c | 2 +- drivers/net/ethernet/intel/igb/e1000_phy.c | 14 -- drivers/net/ethernet/intel/igb/e1000_phy.h | 1 - drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 29 +++-- drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c | 163 ++++++++++++++++++------ 15 files changed, 277 insertions(+), 117 deletions(-) -- 1.9.3 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-05-27 9:12 Jeff Kirsher @ 2014-05-28 16:42 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-05-28 16:42 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Tue, 27 May 2014 02:12:16 -0700 > This series contains updates to e1000, e1000e, ixgbe and i40evf. Pulled, thanks a lot Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-05-21 12:00 Jeff Kirsher 2014-05-22 16:05 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-05-21 12:00 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e and i40evf. Shannon makes minor changes to the AdminQ interface to bring it up to date. Removes the hard coding of stats struct size in ethtool, in prep for adding data fields which are configuration dependent. Catherine removes some unused and unneeded PCI bus defines. Jesse fixes the copyright headers and finishes up the removal of the PTP Tx work functionality which allows us to rely on the Tx timesync interrupt. Mitch provides a number of fixes and cleanups for i40e/i40evf based on suggestions from Ben Hutchings. First is to use a macro parameter for ethtool stats instead of just assuming that a valid netdev variable exists. Second is not to tell ethtool that the VF can do 10GbaseT, when it really has no idea what its link speed is, so set the supported value to 0 instead. Make the ethtool_ops structure constant since it is extremely unlikely to change at runtime. Ethtool consistently reports 0 values for our ITR settings because we never actually use them, so fix this by setting the default values to the specified default values. Greg avoids a compile error by wrapping the call to i40e_alloc_vfs() in CONFIG_PCI_IOV because the function itself is wrapped in the same conditional compile block. Alexander Gordeev updates the driver to use the new pci_enable_msi_range() and pci_enable_msix_range() or pci_enable_msi_exact() and pci_enable_msix_exact(). Jean Sacren provides a fix where the wrong error code was being passed to i40e_open(). The following are changes since commit b6052af61a9e0dee236bcf4c69843126c0d28e4f: Merge tag 'batman-adv-for-davem' of git://git.open-mesh.org/linux-merge and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Alexander Gordeev (1): i40evf: Use pci_enable_msix_range() instead of pci_enable_msix() Catherine Sullivan (1): i40evf: Remove unused defines Greg Rose (1): i40e: Check PCI_IOV config to avoid compile error Jean Sacren (1): i40e: fix passing wrong error code to i40e_open() Jesse Brandeburg (2): i40evf: Fix the headers and update copyright year. i40e: remove Tx work for ptp Mitch Williams (7): i40e: add required include i40evf: Use macro param for ethtool stats i40evf: don't lie to ethtool i40evf: make ethtool_ops const i40e/i40evf: set proper default for ITR registers i40e/i40evf: control auto ITR through ethtool i40e: Don't disable SR-IOV when VFs are assigned Shannon Nelson (2): i40evf: Update AdminQ interface i40e: remove hardcode of stats struct size in ethtool drivers/net/ethernet/intel/i40e/i40e.h | 4 -- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 63 +++++++--------- drivers/net/ethernet/intel/i40e/i40e_main.c | 8 ++- drivers/net/ethernet/intel/i40e/i40e_ptp.c | 1 - drivers/net/ethernet/intel/i40e/i40e_txrx.c | 1 + drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 11 ++- drivers/net/ethernet/intel/i40evf/Makefile | 5 +- drivers/net/ethernet/intel/i40evf/i40e_adminq.c | 5 +- drivers/net/ethernet/intel/i40evf/i40e_adminq.h | 5 +- .../net/ethernet/intel/i40evf/i40e_adminq_cmd.h | 27 ++++--- drivers/net/ethernet/intel/i40evf/i40e_alloc.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_common.c | 5 +- drivers/net/ethernet/intel/i40evf/i40e_hmc.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_lan_hmc.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_osdep.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_prototype.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_register.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_status.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_txrx.c | 3 + drivers/net/ethernet/intel/i40evf/i40e_txrx.h | 5 +- drivers/net/ethernet/intel/i40evf/i40e_type.h | 12 +--- drivers/net/ethernet/intel/i40evf/i40e_virtchnl.h | 5 +- drivers/net/ethernet/intel/i40evf/i40evf.h | 3 + drivers/net/ethernet/intel/i40evf/i40evf_ethtool.c | 84 +++++++++++----------- drivers/net/ethernet/intel/i40evf/i40evf_main.c | 40 +++++------ .../net/ethernet/intel/i40evf/i40evf_virtchnl.c | 3 + 26 files changed, 179 insertions(+), 146 deletions(-) -- 1.9.0 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-05-21 12:00 Jeff Kirsher @ 2014-05-22 16:05 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-05-22 16:05 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Wed, 21 May 2014 05:00:58 -0700 > This series contains updates to i40e and i40evf. Pulled, thanks Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-04-28 13:52 Jeff Kirsher 2014-04-28 18:53 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-04-28 13:52 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e and i40evf. Jacob provides a i40e patch to remove ptp_tx_work() because it conflicts with the interrupt-based handler and will cause spurious warning messages to be displayed in the kernel log. Since the hardware can properly trigger an interrupt, we do not need to poll for an available timestamp in a work queue any more. Greg provides two patches for i40e, first to retain the MAC filter list when changing a port VLAN. Second fixes an issue where the function to set the VF MAC address was not taking the port VLAN filter into account when setting/clearing/resetting the VF's host administered MAC address. Mitch provides three patches for i40evf, first adds support for getting and setting RSS hashing options, RSS LUT entries, and getting the number of device channels from ethtool. Fixes a panic that woudl occur in the VF if the PF driver failed or was removed from the host kernel. Shannon adds driver version string to the driver version command so that can be passed down through the firmware to low-level NC-SI functions. Neerav provides several patches, first redistributes queue vectors after DCB reconfiguration because the total number of queues that are configured for a VSI may change based on a change in the total number of TCs enabled as a result of a change in the DCB configuration based on DCBX. Second, allows the driver probe to continue when querying DCB config fails. Lastly, provides changes to retrieve and store missing link config information that was not being cached in the struct i40e_link_status. Kevin provides a couple of patches, first to change a int variable type to u16 to avoid a typecheck failure. Second to update a function parameters to remove an argument which is no longer used. The following are changes since commit a49eb42a341f1df8fa0f9dc4449f9dd4a3234a2f: sched, act: allow to clear all actions as well and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (2): i40e: Do not enable NTUPLE feature control in MFP mode i40e: Do not expose fd-sb commands from debugfs Catherine Sullivan (1): i40e/i40evf: Bump build version Greg Rose (2): i40e: Retain MAC filters when changing port VLAN i40e: Use port VLAN in MAC/VLAN filter configuration Jacob Keller (1): i40e: remove ptp_tx_work timestamp work item Kevin Scott (2): i40e: Change variable type to avoid typecheck failure i40e: Update function formal parameters Mitch A Williams (1): i40evf: support ethtool RSS options Mitch Williams (2): i40evf: fix panic on PF driver fail i40evf: remove debugging message Neerav Parikh (3): i40e: Redistribute queue vectors after DCB reconfiguration i40e: Don't stop driver probe when querying DCB config fails i40e/i40evf: Retrieve and store missing link config information Shannon Nelson (1): i40e/i40evf: add driver version string to driver version command drivers/net/ethernet/intel/i40e/i40e.h | 2 - drivers/net/ethernet/intel/i40e/i40e_common.c | 21 +- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 6 - drivers/net/ethernet/intel/i40e/i40e_main.c | 19 +- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 4 +- drivers/net/ethernet/intel/i40e/i40e_ptp.c | 36 --- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 3 - drivers/net/ethernet/intel/i40e/i40e_type.h | 4 + drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 11 +- drivers/net/ethernet/intel/i40evf/i40e_type.h | 4 + drivers/net/ethernet/intel/i40evf/i40evf_ethtool.c | 316 +++++++++++++++++++++ drivers/net/ethernet/intel/i40evf/i40evf_main.c | 15 +- 12 files changed, 373 insertions(+), 68 deletions(-) -- 1.9.0 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-04-28 13:52 Jeff Kirsher @ 2014-04-28 18:53 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-04-28 18:53 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Mon, 28 Apr 2014 06:52:23 -0700 > This series contains updates to i40e and i40evf. Pulled, thanks a lot Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-03-26 18:03 Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-03-26 18:03 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to e1000e, igb, i40e and i40evf Anjali provides i40e fix to remove the ATR filter on RST as well as FIN packets. Cleans up add_del_fdir() because it was used and implemented only for the add, so change the name and drop a parameter. Adds the ability to drop a flow if we wanted to and adds a flow director message level to be used for flow director specific messages. Provide a fallback debug flow for i40e if the reset fails or the firmware is unresponsive. Mitch fixes an issue on i40evf where the Tx watchdog handler was causing an oops when sending an admin queue message to request a reset because the admin queue functions use spinlocks. Greg provides a change to i40e to make the alloc and free queue vector calls orthogonal. Shannon fixes i40e to verify the eeprom checksum and firmware CRC status bits, and shutdown the driver if they fail. This change stops the processing of traffic, but does not kill the PF netdev so that the NVMUpdate process still has a chance at fixing the image. Also provides a fix to make sure the VSI has a netdev before trying to use it in the debugfs netdev_ops commands. Jakub Kicinski provides patches for e1000e and igb to fix a number issues found in the PTP code. The following are changes since commit e74dbb73271c5c3e822af3aa66c9a28bf7dbfbfd: Merge branch 'bnx2x-next' and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (6): i40e: Delete ATR filter on RST i40e: Cleanup in FDIR SB ethtool code i40e: Add functionality for FD SB to drop packets i40e/i40evf: Add an FD message level i40e: Use DEBUG_FD message level for an FD message i40e: Add a fallback debug flow for the driver Greg Rose (1): i40e: Make the alloc and free queue vector calls orthogonal Jakub Kicinski (4): e1000e: add timeout for TX HW time stamping work e1000e: remove redundant if clause from PTP work igb: never generate both software and hardware timestamps igb: fix race conditions on queuing skb for HW time stamp Jesse Brandeburg (1): i40evf: remove double space after return Mitch Williams (1): i40evf: fix oops in watchdog handler Shannon Nelson (2): i40e: eeprom integrity check on load and empr i40e: check for netdev before debugfs use drivers/net/ethernet/intel/e1000e/e1000.h | 2 + drivers/net/ethernet/intel/e1000e/ethtool.c | 1 + drivers/net/ethernet/intel/e1000e/netdev.c | 10 +- drivers/net/ethernet/intel/i40e/i40e.h | 2 + drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 120 ++++++++++++--------- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 39 ++++--- drivers/net/ethernet/intel/i40e/i40e_main.c | 72 ++++++++++--- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 14 ++- drivers/net/ethernet/intel/i40e/i40e_type.h | 1 + drivers/net/ethernet/intel/i40evf/i40e_prototype.h | 2 +- drivers/net/ethernet/intel/i40evf/i40e_type.h | 1 + drivers/net/ethernet/intel/i40evf/i40evf.h | 1 + drivers/net/ethernet/intel/i40evf/i40evf_main.c | 10 +- drivers/net/ethernet/intel/igb/igb.h | 3 +- drivers/net/ethernet/intel/igb/igb_main.c | 7 +- drivers/net/ethernet/intel/igb/igb_ptp.c | 3 + 16 files changed, 186 insertions(+), 102 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-03-20 8:32 Jeff Kirsher 2014-03-20 18:20 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-03-20 8:32 UTC (permalink / raw) To: davem Cc: Jeff Kirsher, netdev, gospo, sassmann, emil.s.tantilov, asharma, donald.c.skidmore This series contains updates to e100, igb, igbvf, ixgbe and ixgbevf. Stefan adds a igb patch to enable the ability strip VLAN header information for packets bound for a VM on i350 hardware. Joe Perches provides patches for e100, igb, igbvf, ixgbe and ixgbevf to convert the use of __constant_<foo> to just <foo> to align with the rest of the kernel. Don provides two fixes for ixgbe, first resolves a link issue with DA cables where we were not always freeing the firmware/software semaphore after grabbing it. Second stops caching whether the management firmware was enabled, however since this is not static, we really need to verify with each check. Jacob provides six fixes/cleanups for ixgbe, most notably, correct the stop_mac_link_on d3() to check the Core Clock Disable bit before stopping link and to fully check to see if manage firmware is running or could be enabled before bringing down the link. Fix flow control auto-negation for KR/KX/K4 interfaces, since setting up MAC link, the cached autoc value and current autoc value were being incorrectly used to determine whether link reset is required. Emil provides a fix for ixgbe where there was a chance for aggressive start_ndo_zmit() callers to sneak packets between enabling the Tx queues and the link coming up. To resolve this, move the call to enable Tx queues to after the link is established. The following are changes since commit a9baf10aafe7001501189099df4b91f90739e182: qeth: Fix IP version detection for VLAN traffic and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Don Skidmore (2): ixgbe: fix errors related to protected AUTOC calls ixgbe: Stop cacheing if the MNG FW enabled Emil Tantilov (1): ixgbe: enable tx queues after link up Jacob Keller (6): ixgbe: check Core Clock Disable bit ixgbe: fix ixgbe_stop_mac_link_on_d3_82599 to check mng correctly ixgbe: fix ixgbe_setup_mac_link_82599 autoc variables ixgbe: remove unused media type ixgbe: use ixgbe_read_pci_cfg_word ixgbe: clean up ixgbe_atr_compute_perfect_hash_82599 Joe Perches (5): e100: Convert uses of __constant_<foo> to <foo> igb: Convert uses of __constant_<foo> to <foo> igbvf: Convert uses of __constant_<foo> to <foo> ixgbe: Convert uses of __constant_<foo> to <foo> ixgbevf: Convert uses of __constant_<foo> to <foo> Stefan Assmann (1): igb: enable VLAN stripping for VMs with i350 drivers/net/ethernet/intel/e100.c | 4 +- drivers/net/ethernet/intel/igb/e1000_82575.h | 4 + drivers/net/ethernet/intel/igb/e1000_regs.h | 1 + drivers/net/ethernet/intel/igb/igb_main.c | 19 ++- drivers/net/ethernet/intel/igbvf/netdev.c | 4 +- drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 1 - drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c | 170 +++++----------------- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 24 +-- drivers/net/ethernet/intel/ixgbe/ixgbe_common.h | 1 + drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c | 8 +- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 68 +++++---- drivers/net/ethernet/intel/ixgbe/ixgbe_phy.h | 4 - drivers/net/ethernet/intel/ixgbe/ixgbe_type.h | 6 +- drivers/net/ethernet/intel/ixgbe/ixgbe_x540.c | 1 - drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 4 +- 15 files changed, 113 insertions(+), 206 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-03-20 8:32 Jeff Kirsher @ 2014-03-20 18:20 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-03-20 18:20 UTC (permalink / raw) To: jeffrey.t.kirsher Cc: netdev, gospo, sassmann, emil.s.tantilov, asharma, donald.c.skidmore From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Thu, 20 Mar 2014 01:32:50 -0700 > This series contains updates to e100, igb, igbvf, ixgbe and ixgbevf. ... > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-03-08 9:25 Jeff Kirsher 2014-03-08 23:50 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-03-08 9:25 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to e1000e, ixgbevf and igb. Majority of this series contains fixes and cleanups to e1000e, most notably are: Todd provides a fix to PTP in e1000e which adds a lock in e1000e_phc_adjfreq to prevent concurrent changes to TIMINCA and SYSTIMH/L. Then provides an igb fix to use ARRAY_SIZE for array size calculation. David provides the remaining e1000e which contain: - cleanup of pointer references that are no longer used - fix an issue on systems with Management Engine enabled with the ethernet cable unplugged - fix an issue on 82579 where enabling EEE LPI sooner than one second after link up causes link issues on some switches - refactor the power management flows to prevent the suspend path from being executed twice when hibernating - refactor the runtime power management to fix interfering with the functionality of Energy Efficient Ethernet when enabled and to fix the device from repeatedly flip between suspend and resume with the interface administratively downed - enable the feature PHY Ultra Low Power Mode which is a power saving feature that reduces the power consumption of the PHY when a cable is not connected - fix the ethtool offline tests for 82579 parts - fix SHRA register access for 82579 parts which was introduced by previous commit c3a0dce35af0 "e1000e: fix overrun of PHY RAR array" Florian provides a fix for ixgbevf where skb->pkt_type was being checked like a bitmask, but it is not a bitmask. Fix an issue reported by Stephen Hemminger where there was a warning about code defined but never used if IGB_HWMON is not defined. The following are changes since commit 37147652cfaa20a87ead9bb04aec1834b40c5c97: 6lowpan: reassembly: fix return of init function and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master David Ertman (11): e1000e: Cleanup unecessary references e1000e: Resolve issues with Management Engine (ME) briefly blocking PHY resets e1000e: Fix 82579 sets LPI too early. e1000e: Cleanup - Update GPL header and Copyright e1000e: Add missing branding strings in ich8lan.c e1000e: Refactor PM flows e1000e Refactor of Runtime Power Management e1000e: Feature Enable PHY Ultra Low Power Mode (ULP) e1000e: Fix not generating an error on invalid load parameter e1000e: Fix ethtool offline tests for 82579 parts e1000e: Fix SHRA register access for 82579 Florian Fainelli (1): ixgbevf: fix skb->pkt_type checks Jeff Kirsher (1): igb: fix warning if !CONFIG_IGB_HWMON Todd Fujinaka (2): e1000e: PTP lock in e1000e_phc_adjustfreq igb: fix array size calculation drivers/net/ethernet/intel/e1000e/80003es2lan.c | 47 +-- drivers/net/ethernet/intel/e1000e/80003es2lan.h | 47 +-- drivers/net/ethernet/intel/e1000e/82571.c | 47 +-- drivers/net/ethernet/intel/e1000e/82571.h | 47 +-- drivers/net/ethernet/intel/e1000e/Makefile | 7 +- drivers/net/ethernet/intel/e1000e/defines.h | 55 ++- drivers/net/ethernet/intel/e1000e/e1000.h | 50 ++- drivers/net/ethernet/intel/e1000e/ethtool.c | 63 ++-- drivers/net/ethernet/intel/e1000e/hw.h | 55 +-- drivers/net/ethernet/intel/e1000e/ich8lan.c | 427 ++++++++++++++++++---- drivers/net/ethernet/intel/e1000e/ich8lan.h | 72 ++-- drivers/net/ethernet/intel/e1000e/mac.c | 47 +-- drivers/net/ethernet/intel/e1000e/mac.h | 47 +-- drivers/net/ethernet/intel/e1000e/manage.c | 47 +-- drivers/net/ethernet/intel/e1000e/manage.h | 47 +-- drivers/net/ethernet/intel/e1000e/netdev.c | 286 +++++++++------ drivers/net/ethernet/intel/e1000e/nvm.c | 47 +-- drivers/net/ethernet/intel/e1000e/nvm.h | 47 +-- drivers/net/ethernet/intel/e1000e/param.c | 53 ++- drivers/net/ethernet/intel/e1000e/phy.c | 47 +-- drivers/net/ethernet/intel/e1000e/phy.h | 47 +-- drivers/net/ethernet/intel/e1000e/ptp.c | 52 ++- drivers/net/ethernet/intel/e1000e/regs.h | 48 ++- drivers/net/ethernet/intel/igb/e1000_82575.c | 6 +- drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 3 +- 25 files changed, 1003 insertions(+), 738 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-03-08 9:25 Jeff Kirsher @ 2014-03-08 23:50 ` David Miller 2014-03-09 0:10 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: David Miller @ 2014-03-08 23:50 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Sat, 8 Mar 2014 01:25:26 -0800 > This series contains updates to e1000e, ixgbevf and igb. ... > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks a lot Jeff, and again welcome back. ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-03-08 23:50 ` David Miller @ 2014-03-09 0:10 ` David Miller 2014-03-09 1:12 ` Jeff Kirsher 0 siblings, 1 reply; 63+ messages in thread From: David Miller @ 2014-03-09 0:10 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: David Miller <davem@davemloft.net> Date: Sat, 08 Mar 2014 18:50:08 -0500 (EST) > From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > Date: Sat, 8 Mar 2014 01:25:26 -0800 > >> This series contains updates to e1000e, ixgbevf and igb. > ... >> git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master > > Pulled, thanks a lot Jeff, and again welcome back. Jeff, please get this new warning fixed, thanks: drivers/net/ethernet/intel/i40e/i40e_ethtool.c: In function ‘i40e_del_fdir_entry’: drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1441:18: warning: unused variable ‘pf’ [-Wunused-variable] struct i40e_pf *pf = vsi->back; ^ ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-03-09 0:10 ` David Miller @ 2014-03-09 1:12 ` Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-03-09 1:12 UTC (permalink / raw) To: David Miller; +Cc: netdev, gospo, sassmann [-- Attachment #1: Type: text/plain, Size: 913 bytes --] On Sat, 2014-03-08 at 19:10 -0500, David Miller wrote: > From: David Miller <davem@davemloft.net> > Date: Sat, 08 Mar 2014 18:50:08 -0500 (EST) > > > From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > Date: Sat, 8 Mar 2014 01:25:26 -0800 > > > >> This series contains updates to e1000e, ixgbevf and igb. > > ... > >> git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next > master > > > > Pulled, thanks a lot Jeff, and again welcome back. > > Jeff, please get this new warning fixed, thanks: > > drivers/net/ethernet/intel/i40e/i40e_ethtool.c: In function > ‘i40e_del_fdir_entry’: > drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1441:18: warning: > unused variable ‘pf’ [-Wunused-variable] > struct i40e_pf *pf = vsi->back; Yeah, I noticed that as well. I have a patch in my queue already to resolve that, should be in my next pull request. Thanks Dave! [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-01-07 9:32 Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-07 9:32 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only. Anjali adds more functionality to debugfs to assist development and testing of admin queue commands. Greg makes sure broadcast promiscuous is disabled by default, otherwise VLAN tagged packets out of the assigned VLAN domain are received. Also provides a fix when the 8021q driver is loaded, so that VLAN 0 tagged packets are accepted so that upper layers can interpret the priority bits. Then provides a fix to let the VF to request the PF to set its already assigned MAC address without generating an error. Greg also adds helper functions to enable or disable internal switch loopback when VFs are created or destroyed via the sysfs interface. Shannon provides most of the changes, where he adds code to ensure that the hardware waits to make sure that the firmware is ready as well after reset. Also updates the code to use the new features in the firmware. Provides a fix while in MFP mode where resources are reduced, so use a smaller range of test registers than when in SFP mode. Moves the PF ID initialization code to earlier in the driver initialization function since a few operations need the information before the first PF reset is called. Shannon adds a check for MAC type before reading anything from the registers to ensure we dealing with the correct MAC type. Then reworks Shadow RAM read word/buffer functions as to not block whole NVM resources for SR read operations. Mitch lastly provides a fix to correctly setup ARQ descriptors in the cleanup code. Catherine bumps the driver version due to all the recent changes. The following are changes since commit 83111e7fe85f2dc8b61e463c71b971f4998d8e53: netfilter: Fix build failure in nfnetlink_queue_core.c. and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (1): i40e: Expose AQ debugfs hooks Catherine Sullivan (1): i40e: Bump version Greg Rose (4): i40e: Do not enable broadcast promiscuous by default i40e: Stop accepting any VLAN tag on VLAN 0 filter set i40e: Allow VF to set already assigned MAC address i40e: Enable/Disable PF switch LB on SR-IOV configure changes Kamil Krawczyk (1): i40e: remove redundant AQ enable Mitch Williams (1): i40e: correctly setup ARQ descriptors Shannon Nelson (7): i40e: Add code to wait for FW to complete in reset path i40e: update firmware api to 1.1 i40e: Reduce range of interrupt reg in reg test i40e: move PF ID init from PF reset to SC init i40e: check MAC type before any REG access i40e: rework shadow ram read functions i40e: whitespace paren and comment tweaks drivers/net/ethernet/intel/i40e/i40e_adminq.c | 27 ++--- drivers/net/ethernet/intel/i40e/i40e_adminq_cmd.h | 126 +++++++++++++++------ drivers/net/ethernet/intel/i40e/i40e_common.c | 51 ++++++--- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 112 ++++++++++++++++++ drivers/net/ethernet/intel/i40e/i40e_diag.c | 6 +- drivers/net/ethernet/intel/i40e/i40e_main.c | 29 ++--- drivers/net/ethernet/intel/i40e/i40e_nvm.c | 67 +++-------- drivers/net/ethernet/intel/i40e/i40e_register.h | 22 ++++ drivers/net/ethernet/intel/i40e/i40e_type.h | 11 +- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 100 +++++++++++++--- 10 files changed, 395 insertions(+), 156 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-01-06 12:30 Jeff Kirsher 2014-01-06 18:26 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-01-06 12:30 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only. Majority of this series contains patches from Greg and Mitch to fix up or add functionality to the PF/VF driver interactions. Notably, a fix for SR-IOV VF port VLAN which resolved the problem of port VLAN configurations not being persistent across VF driver loads and unloads and enable/disable of the feature. Also do not enable the default port on the VEB, which is designed only to bridge the PF to an Open vSwitch or bridge. Another fix to resolve a possible memory corruption condition where ARQ messages are written to random memory locations. Fix a problem where the 'ip link show' command would display stale link address information after the link address was set via the 'ip link set' command. Anjali provides several patches, one which saves information that can be used while cleaning the Tx ring and useful in detecting Tx hangs. Then provides a fixes to the admin queue shutdown function to ensure we are shutting down the queue in the shutdown path and ensure ASQ is alive before issuing the admin queue command. Shannon provides a fix for get/update vsi params where the incorrect struct was being used. The following are changes since commit b912b2f8fc71df4c3ffa7a9fe2c2227e8bcdaa07: net/mlx4_core: Warn if device doesn't have enough PCI bandwidth and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (4): i40e: Record dma buffer info for dummy packets i40e: Admin queue shutdown fixes i40e: check asq alive before notify i40e: Do not allow AQ calls from ndo-ops Greg Rose (4): i40e: Fix SR-IOV VF port VLAN i40e: Do not enable default port on the VEB i40e: Fix VF driver MAC address configuration i40e: Hide the Port VLAN VLAN ID Jesse Brandeburg (1): i40e: fix whitespace Mitch Williams (5): i40e: avoid unnecessary register read i40e: use struct assign instead of memcpy i40e: don't allocate zero size i40e: acknowledge VFLR when disabling SR-IOV i40e: support VFs on PFs other than 0 Shannon Nelson (1): i40e: use correct struct for get and update vsi params drivers/net/ethernet/intel/i40e/i40e_adminq.c | 3 + drivers/net/ethernet/intel/i40e/i40e_common.c | 75 ++++++++------ drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 1 + drivers/net/ethernet/intel/i40e/i40e_main.c | 45 ++++---- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 5 +- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 9 +- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 114 ++++++++++++++++----- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h | 2 + 8 files changed, 169 insertions(+), 85 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-01-06 12:30 Jeff Kirsher @ 2014-01-06 18:26 ` David Miller 2014-01-06 18:49 ` Jeff Kirsher 2014-01-06 20:14 ` Jeff Kirsher 0 siblings, 2 replies; 63+ messages in thread From: David Miller @ 2014-01-06 18:26 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Mon, 6 Jan 2014 04:30:21 -0800 > This series contains updates to i40e only. ... > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, please make sure the coding style feedback does actually get addressed. Thanks Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-01-06 18:26 ` David Miller @ 2014-01-06 18:49 ` Jeff Kirsher 2014-01-06 20:14 ` Jeff Kirsher 1 sibling, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-06 18:49 UTC (permalink / raw) To: David Miller; +Cc: netdev, gospo, sassmann [-- Attachment #1: Type: text/plain, Size: 459 bytes --] On Mon, 2014-01-06 at 13:26 -0500, David Miller wrote: > From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > Date: Mon, 6 Jan 2014 04:30:21 -0800 > > > This series contains updates to i40e only. > ... > > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master > > Pulled, please make sure the coding style feedback does actually > get addressed. > > Thanks Jeff. Yes, I will make sure it is in one of the upcoming series. [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-01-06 18:26 ` David Miller 2014-01-06 18:49 ` Jeff Kirsher @ 2014-01-06 20:14 ` Jeff Kirsher 1 sibling, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2014-01-06 20:14 UTC (permalink / raw) To: David Miller, Sergei Shtylyov; +Cc: netdev, gospo, sassmann [-- Attachment #1: Type: text/plain, Size: 500 bytes --] On Mon, 2014-01-06 at 13:26 -0500, David Miller wrote: > From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > Date: Mon, 6 Jan 2014 04:30:21 -0800 > > > This series contains updates to i40e only. > ... > > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master > > Pulled, please make sure the coding style feedback does actually > get addressed. > > Thanks Jeff. Just to be clear, this coding style will be fixed in patch 13 of the next series I am sending out. [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2014-01-05 10:46 Jeff Kirsher 2014-01-06 1:31 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2014-01-05 10:46 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only. Anjali provides two cleanups to remove unnecessary code and a fix to resolve debugfs dumping only half the NVM. Then provides a fix to ethtool NVM reads where shadow RAM was used instead of actual NVM reads. Jesse provides a couple of fixes, one removes custom i40e functions which duplicate existing kernel functionality. Second fixes constant cast issues by replacing __constant_htons with htons. Mitch provides a couple of fixes for the VF interfaces in i40e. First provides a fix to guard against VF message races with can cause a panic. Second fix reinitializes the buffer size each time we clean the ARQ, because subsequent messages can be truncated. Lastly adds functionality to enable/disable ICR 0 dynamically. Vasu adds a simple guard against multiple includes of the i40e_txrx.h file. Shannon provides a couple of fixes, first fix swaps a couple of lines around in the error handling if the allocation for the VSI array fails. Second fixes an issue where we try to free the q_vector that has not been setup which can panic the kernel. David provides a patch to save off the point to memory and the length of 2 structs used in the admin queue in order to store all info about allocated kernel memory. Neerav fixes ring allocation where allocation and clearing of rings for a VSI should be using the alloc_queue_pairs and not num_queue_pairs. Then removes the unused define for multi-queue enabled. The following are changes since commit a1d4b03a076d95edc88d070f7627a73ab80abddc: Merge branch 'bgmac' and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (4): i40e: Dump the whole NVM, not half i40e: Change the ethtool NVM read method to use AQ i40e: Remove unnecessary prototypes i40e: remove un-necessary io-write David Cassard (1): i40e: keep allocated memory in structs Jesse Brandeburg (2): i40e: fix mac address checking i40e: fix constant cast issues Mitch Williams (3): i40e: guard against vf message races i40e: use functions to enable and disable icr 0 i40e: reinit buffer size each time Neerav Parikh (2): i40e: Fix ring allocation i40e: I40E_FLAG_MQ_ENABLED is not used Shannon Nelson (2): i40e: fix error handling when alloc of vsi array fails i40e: catch unset q_vector Vasu Dev (1): i40e: add header file flag _I40E_TXRX_H_ drivers/net/ethernet/intel/i40e/i40e.h | 40 +++----- drivers/net/ethernet/intel/i40e/i40e_adminq.c | 103 +++++++++------------ drivers/net/ethernet/intel/i40e/i40e_adminq.h | 13 +-- drivers/net/ethernet/intel/i40e/i40e_common.c | 27 ------ drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 8 +- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 68 +++++++++----- drivers/net/ethernet/intel/i40e/i40e_main.c | 47 +++++----- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 1 - drivers/net/ethernet/intel/i40e/i40e_txrx.c | 8 +- drivers/net/ethernet/intel/i40e/i40e_txrx.h | 4 + drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 28 +++--- 11 files changed, 162 insertions(+), 185 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2014-01-05 10:46 Jeff Kirsher @ 2014-01-06 1:31 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2014-01-06 1:31 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Sun, 5 Jan 2014 02:46:52 -0800 > This series contains updates to i40e only. ... > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Looks fine, pulled, thanks Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-12-19 6:34 Jeff Kirsher 2013-12-19 19:20 ` Jeff Kirsher 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2013-12-19 6:34 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only. Anjali provides a patch to prevent messages from stray HMC events, except at interrupt message level, and refactors the HMC error handling. Catherine adds routines in probe to populate/check PCI bus speed and width, then verify we are in a 8GT/s x8 PCIe slot and warn when we are not. Shannon adds Wake-on-LAN support for i40e, fixes curly brace use as well as return type for i40e_vsi_clear_rings(). Lastly implements the set_settings and set_pauseparam ethtool functionality. Joseph implements receive offload for VXLAN for i40e, where the hardware supports checksum offload/verification of the inner/outer header. Mitch provides the bulk of the changes, where he refactors the VF reset code so that it works on real hardware. Then does code cleanup by calling existing functions to enable and disable queues for VFs and remove unused functions. Removes a unnecessary log messages that are seen at every VF reset, for example complaining about disabling queues that are already disabled. Fixes an error return when the VF asks to add an invalid MAC address and if the VF sends a bad message, make it more informative about what is actually going on. Jesse refactors the LED function to flash LED lights correctly. The following are changes since commit 020e867c755610fa5a2a08a69707e636cf368b55: isdn: use strlcpy() instead strcpy() and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (1): i40e: Suppress HMC error to Interrupt message level Catherine Sullivan (1): i40e: Populate and check pci bus speed and width Jesse Brandeburg (1): i40e: update led set args Joseph Gasparakis (1): i40e: Rx checksum offload for VXLAN Mitch Williams (8): i40e: move i40e_reset_vf i40e: refactor VF reset flow i40e: remove redundant code i40e: remove chatty log messages i40e: fix error return i40e: be more informative i40e: make a define from a large constant i40e: report VF MAC addresses correctly Shannon Nelson (3): i40e: add wake-on-lan support i40e: fix curly brace use and return type i40e: add set settings and pauseparam drivers/net/ethernet/intel/i40e/i40e.h | 3 + drivers/net/ethernet/intel/i40e/i40e_common.c | 136 ++++- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 145 +++++- drivers/net/ethernet/intel/i40e/i40e_main.c | 105 ++-- drivers/net/ethernet/intel/i40e/i40e_prototype.h | 3 +- drivers/net/ethernet/intel/i40e/i40e_txrx.c | 56 ++- drivers/net/ethernet/intel/i40e/i40e_txrx.h | 20 + drivers/net/ethernet/intel/i40e/i40e_type.h | 61 ++- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 559 ++++++--------------- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h | 2 +- 10 files changed, 572 insertions(+), 518 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2013-12-19 6:34 Jeff Kirsher @ 2013-12-19 19:20 ` Jeff Kirsher 2013-12-19 19:47 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2013-12-19 19:20 UTC (permalink / raw) To: davem; +Cc: netdev, gospo, sassmann [-- Attachment #1: Type: text/plain, Size: 1897 bytes --] On Wed, 2013-12-18 at 22:34 -0800, Jeff Kirsher wrote: > This series contains updates to i40e only. > > Anjali provides a patch to prevent messages from stray HMC events, > except > at interrupt message level, and refactors the HMC error handling. > > Catherine adds routines in probe to populate/check PCI bus speed and > width, > then verify we are in a 8GT/s x8 PCIe slot and warn when we are not. > > Shannon adds Wake-on-LAN support for i40e, fixes curly brace use as > well as > return type for i40e_vsi_clear_rings(). Lastly implements the > set_settings > and set_pauseparam ethtool functionality. > > Joseph implements receive offload for VXLAN for i40e, where the > hardware > supports checksum offload/verification of the inner/outer header. > > Mitch provides the bulk of the changes, where he refactors the VF > reset > code so that it works on real hardware. Then does code cleanup by > calling existing functions to enable and disable queues for VFs and > remove unused functions. Removes a unnecessary log messages that are > seen at every VF reset, for example complaining about disabling queues > that are already disabled. Fixes an error return when the VF asks to > add an invalid MAC address and if the VF sends a bad message, make it > more informative about what is actually going on. > > Jesse refactors the LED function to flash LED lights correctly. > > The following are changes since commit > 020e867c755610fa5a2a08a69707e636cf368b55: > isdn: use strlcpy() instead strcpy() > and are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next > master Dave- Disregard this pull request. I overlooked a patch that needs to either precede this series or get added to the series for VXLAN. I will be sending out a v2 with the necessary patch added. Cheers, Jeff [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2013-12-19 19:20 ` Jeff Kirsher @ 2013-12-19 19:47 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2013-12-19 19:47 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Thu, 19 Dec 2013 11:20:36 -0800 > Disregard this pull request. I overlooked a patch that needs to either > precede this series or get added to the series for VXLAN. > > I will be sending out a v2 with the necessary patch added. Ok. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-12-12 12:40 Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2013-12-12 12:40 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only (again). Jesse provides a fix for when tx_rings structure is NULL and we do not want to panic. Then refactors the flow control set up and disables L2 flow control by default. Provides some trivial fixes as well as prevent compiler warnings. Then to align to similar behaviour in ixgbe, use the total number of CPUs in the system to suggest the number of transmit and receive queue pairs. Shannon provides a i40e ethtool fix to get some more reasonable information reports back out to the ethtool. In addition, fixes PF reset after offline test, where it reorders the test to put the register test last as it is the only one that needs a reset, and we wait to trigger the reset until after we clear the testing bit. Lastly provides basic support for handling suspend and resume for now, later on Wake-On-LAN support will be added. Elizabeth Kappler adds netdev ops helper function. Anjali provides changes to tell the stack about our actual number of queues in order for RFS/RPS/XFS to work correctly. Then provides several patches to implement dynamically changing the queue count for the main VSI. Adds basic support for get/set channels for RSS so that the number of receive and transmit queue pair can be changed via ethtool. Cleans up the use of rtnl_lock in the reset patch since it runs from a work time. Neerav Parikh cleans up the VF interface to remove FCoE code as this feature will not be supported on VF interfaces. The following are changes since commit c45f812f0280c13f1b7992be5e0de512312a9e8f: 8390 : Replace ei_debug with msg_enable/NETIF_MSG_* feature and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (6): i40e: Tell the stack about our actual number of queues i40e: reinit flow for the main VSI i40e: function to reconfigure RSS queues and rebuild i40e: debugfs command to change RSS queue count i40e: Add basic support for get/set channels for RSS i40e: rtnl_lock in reset path fixes Elizabeth Kappler (1): i40e: add netdev ops helper function Jesse Brandeburg (5): i40e: fix null dereference i40e: fix up some of the ethtool connection reporting i40e: init flow control settings to disabled i40e: trivial fixes i40e: use same number of queues as CPUs Neerav Parikh (1): i40e: Remove FCoE in i40e_virtchnl_pf.c code Shannon Nelson (2): i40e: fix pf reset after offline test i40e: support for suspend and resume drivers/net/ethernet/intel/i40e/i40e.h | 6 +- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 22 +- drivers/net/ethernet/intel/i40e/i40e_diag.c | 2 +- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 146 +++++++- drivers/net/ethernet/intel/i40e/i40e_hmc.c | 2 +- drivers/net/ethernet/intel/i40e/i40e_main.c | 380 ++++++++++++++++++--- drivers/net/ethernet/intel/i40e/i40e_type.h | 4 + drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 29 +- 8 files changed, 499 insertions(+), 92 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-12-06 8:06 Jeff Kirsher 2013-12-06 19:49 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2013-12-06 8:06 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to i40e only. Christopher Pau provides a patch to set pf_id based on device and function numbers since NICs with ARI enabled can have function numbers larger than 8. Anjali provides 3 i40e patches to update hardware defines to keep in sync with hardware updates. Shannon provides the majority of i40e patches, with 7. First patch clears the admin queue head and tail registers during admin queue shutdown. Then simplifies the admin queue head-tail-len setups to use more virtual registers. Provides several patches to cleanup and fix driver load and reset procedures to make more robust. Lastly, provides an ethtool test for interrupts using the software interrupt. Mitch provides some i40e patches which fixes up VF code in the PF driver, specifically the number of vectors per VF are reported by the hardware does not include vector 0, so we need to account for this when checking. In addition, cleans up debugging messages. Kamil provides an i40e patch to fix the diagnostics test by restricting the diagnostic test length. The following are changes since commit d70f2cf520cf8c86c6b467fc5dcea524ec51dc2f: macvlan: Support creating macvtaps from macvlans and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Anjali Singhai Jain (3): i40e: sync header files with hardware i40e: register file updates i40e: firmware version fields offsets update Christopher Pau (1): i40e: set pf_id based on device and function numbers Kamil Krawczyk (1): i40e: restrict diag test length Mitch Williams (3): i40e: allow one more vector for VFs i40e: properly add VF MAC addresses i40e: fix debugging messages Shannon Nelson (7): i40e: clear AQ head and tail registers i40e: simplify aq head-tail-len setups i40e: select reset counters correctly i40e: retry call on timeout i40e: default debug mask setting i40e: add interrupt test i40e: add support for triggering EMPR drivers/net/ethernet/intel/i40e/i40e.h | 19 +-- drivers/net/ethernet/intel/i40e/i40e_adminq.c | 66 +++++----- drivers/net/ethernet/intel/i40e/i40e_adminq.h | 1 + drivers/net/ethernet/intel/i40e/i40e_common.c | 6 +- drivers/net/ethernet/intel/i40e/i40e_debugfs.c | 4 + drivers/net/ethernet/intel/i40e/i40e_diag.c | 8 +- drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 10 +- drivers/net/ethernet/intel/i40e/i40e_lan_hmc.h | 4 +- drivers/net/ethernet/intel/i40e/i40e_main.c | 37 +++++- drivers/net/ethernet/intel/i40e/i40e_register.h | 134 +++++++-------------- drivers/net/ethernet/intel/i40e/i40e_type.h | 11 +- drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 8 +- 12 files changed, 155 insertions(+), 153 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2013-12-06 8:06 Jeff Kirsher @ 2013-12-06 19:49 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2013-12-06 19:49 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Fri, 6 Dec 2013 00:06:13 -0800 > This series contains updates to i40e only. > > Christopher Pau provides a patch to set pf_id based on device and > function numbers since NICs with ARI enabled can have function > numbers larger than 8. > > Anjali provides 3 i40e patches to update hardware defines to keep > in sync with hardware updates. > > Shannon provides the majority of i40e patches, with 7. First patch > clears the admin queue head and tail registers during admin queue > shutdown. Then simplifies the admin queue head-tail-len setups to > use more virtual registers. Provides several patches to cleanup > and fix driver load and reset procedures to make more robust. Lastly, > provides an ethtool test for interrupts using the software interrupt. > > Mitch provides some i40e patches which fixes up VF code in the PF > driver, specifically the number of vectors per VF are reported by the > hardware does not include vector 0, so we need to account for this > when checking. In addition, cleans up debugging messages. > > Kamil provides an i40e patch to fix the diagnostics test by restricting > the diagnostic test length. Pulled, thanks a lot Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-08-21 8:23 Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2013-08-21 8:23 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to igb and e1000e. Akeem provides 3 igb patches, the first resets the link when EEE is enabled or disabled if the link is up. His second patch changes a register read which normally stores of the read value to "just-read" so that hardware can accurately latch the register read. Lastly, he adds rcu_lock to avoid a possible race condition with igb_update_stats function. Mitch provides a fix for SR-IOV, where MSI-X interrupts are required, so make sure that MSI-X is enabled before allowing the user to turn on SR-IOV. Alex's igb patch make it so that we limit the lower bound for max_frame_size to the size of a standard Ethernet frame. This allows for feature parity with other Intel based drivers such as ixgbe. Carolyn adds a SKU for a flashless i210 device and a fix for get_fw_version() so that it works for all parts for igb. In addition, she has 2 igb patches to refactor NVM code to accommodate devices with no flash. Lastly, she adds code to check for the failure of pci_disable_link_state() to attempt to work around a problem found with some systems. Laura provides the remaining 2 igb patches. One removing the hard-coded value for the size of the RETA indirection table, and creates a macro instead for the RETA indirection table. The second adds the ethtool callbacks necessary to change the RETA indirection table from userspace. Bruce fixes a whitespace issue in a recent commit and resolves a jiffies comparison warning by using time_after(). Li provides a fix for e1000e to avoid a kernel crash on shutdown by adding one more check in e1000e_shutdown(). This is due to e1000e_shutdown() trying to clear correctable errors on the upstream P2P bridge, when under some cases we do not have the upstream P2P bridge. The following are changes since commit 84ce22df92f5cf9ecb76d53f4bf5f6d504684540: net: davinci_mdio: use platform_{get,set}_drvdata() and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Akeem G Abodunrin (3): igb: Reset the link when EEE setting changed igb: Read register for latch_on without return value igb: Added rcu_lock to avoid race Alexander Duyck (1): igb: Update MTU so that it is always at least a standard frame size Bruce Allan (2): e1000e: cleanup whitespace in recent commit e1000e: resolve checkpatch JIFFIES_COMPARISON warning Carolyn Wyborny (5): igb: Refactor of init_nvm_params igb: Refactor NVM read functions to accommodate devices with no flash igb: Add device support for flashless SKU of i210 device igb: Fix get_fw_version function for all parts e1000e: Add code to check for failure of pci_disable_link_state call Laura Mihaela Vasilescu (2): igb: Add macro for size of RETA indirection table igb: Expose RSS indirection table for ethtool Li Zhang (1): e1000e: Avoid kernel crash during shutdown Mitch A Williams (1): igb: don't allow SR-IOV without MSI-X drivers/net/ethernet/intel/e1000e/ethtool.c | 2 +- drivers/net/ethernet/intel/e1000e/hw.h | 3 +- drivers/net/ethernet/intel/e1000e/netdev.c | 92 ++++++++++----- drivers/net/ethernet/intel/igb/e1000_82575.c | 117 ++++++++----------- drivers/net/ethernet/intel/igb/e1000_defines.h | 34 ++++-- drivers/net/ethernet/intel/igb/e1000_hw.h | 3 + drivers/net/ethernet/intel/igb/e1000_i210.c | 155 ++++++++++++++++--------- drivers/net/ethernet/intel/igb/e1000_i210.h | 5 +- drivers/net/ethernet/intel/igb/e1000_nvm.c | 80 +++++++++---- drivers/net/ethernet/intel/igb/e1000_nvm.h | 1 + drivers/net/ethernet/intel/igb/igb.h | 5 + drivers/net/ethernet/intel/igb/igb_ethtool.c | 106 ++++++++++++++++- drivers/net/ethernet/intel/igb/igb_main.c | 105 ++++++++++------- drivers/net/ethernet/intel/igb/igb_ptp.c | 8 +- 14 files changed, 476 insertions(+), 240 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-07-29 12:51 Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2013-07-29 12:51 UTC (permalink / raw) To: davem, bhelgaas; +Cc: Jeff Kirsher, netdev, gospo, sassmann, linux-pci This series contains updates to ixgbe and pci. The first patch for ixgbe from Greg Rose is the second submission. The first submission of "ixgbe: Retain VLAN filtering in promiscuous + VT mode" had a typo, which Joe Perches pointed out and is fixed in this submission. Alex updates the ixgbe driver to use the generic helper pci_vfs_assigned instead of the driver specific function ixgbe_vfs_are_assigned. Don Skidmore provides 4 patches for ixgbe, the first being a fix for flow control ethtool reporting. Originally ixgbe_device_supports_autoneg_fc() was expected to be called by only copper devices, which lead to false information being displayed via ethtool. Two other patches add support for fixed fiber for SFP+ devices and the addition of a quad-port x520 adapter. The last patch simply bumps the driver version. Emil Tantilov provides 3 fixes for ixgbe, two of which resolve semaphore lock issues. The third fix resolves several issues in the previous implementation of the SFF data dumps of SFP+ modules. The remaining ixgbe and pci patches are from Jacob Keller. The pci patches exposes bus speed, link speed and bus width so that drivers can take advantage of this information. In addition, adds a pci function which obtains minimum link width and speed. Jacob also provides the ixgbe patch to incorporate the pci function. He provides a patch that fixes a lockdep issue created due to ixgbe_ptp_stop always running cancel_work_sync even if the work item had not been created properly with INIT_WORK. This issue was found and reported by Stephen Hemminger. The following are changes since commit fe6f700d6cbb7e8a61711e325f53d9c9e0a42a4c: net/mlx4_core: Respond to operation request by firmware and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Alexander Duyck (1): ixgbe: Use pci_vfs_assigned instead of ixgbe_vfs_are_assigned Don Skidmore (4): ixgbe: fix fc autoneg ethtool reporting. ixgbe: add new media type ixgbe: bump version number ixgbe: add support for quad-port x520 adapter Emil Tantilov (3): ixgbe: fix semaphore lock for I2C read/writes on 82598 ixgbe: fix SFF data dumps of SFP+ modules ixgbe: clear semaphore bits on timeouts Greg Rose (1): ixgbe: Retain VLAN filtering in promiscuous + VT mode Jacob Keller (6): pci: expose pcie_link_speed and pcix_bus_speed arrays pci: move enum pcie_link_width into pci.h pci: Add function to obtain minimum link width and speed ixgbe: call pcie_get_mimimum_link to check if device has enough bandwidth ixgbe: fix lockdep annotation issue for ptp's work item ixgbe: rename LL_EXTENDED_STATS to use queue instead of q drivers/net/ethernet/intel/ixgbe/ixgbe.h | 7 +- drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 34 +- drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c | 231 ++++++++++- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 88 +++-- drivers/net/ethernet/intel/ixgbe/ixgbe_ethtool.c | 94 ++--- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 158 +++++++- drivers/net/ethernet/intel/ixgbe/ixgbe_phy.c | 470 ++++++++++++++++------- drivers/net/ethernet/intel/ixgbe/ixgbe_phy.h | 41 +- drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c | 12 +- drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 101 +++-- drivers/net/ethernet/intel/ixgbe/ixgbe_type.h | 12 + drivers/pci/pci.c | 43 +++ drivers/pci/probe.c | 4 +- include/linux/pci.h | 18 + include/linux/pci_hotplug.h | 13 - 15 files changed, 983 insertions(+), 343 deletions(-) -- 1.7.11.7 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-04-18 23:30 Jeff Kirsher 2013-04-19 18:19 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2013-04-18 23:30 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to ixgbe and igb. The ixgbe changes contains 2 patches from the community, one which is a fix from akepner to fix a issue where netif_running() in shutdown was not done under rtnl_lock. The other community fix from Joe Perches cleans up #ifdef CONFIG_DEBUG_FS which is no longer necessary. The last ixgbe patch, from Jacob Keller, adds support for WoL on 82559 SFP+ LOM. The remaining patches are against igb, 10 of which were previously submitted in a pull request where changes were requested. The following igb patches: igb: Support for 100base-fx SFP igb: Support to read and export SFF-8472/8079 data are v2 based on feedback from Dan Carpenter and Ben Hutchings in the previous pull request. The largest set of changes are in my patch to cleanup code comments and whitespace to align the igb driver with the networking style of code comments. While cleaning up the code comments, fixed several other whitespace/checkpatch.pl code formatting issues. Other notable igb patches are EEE capable devices query the PHY to determine what the link partner is advertising, added support for i354 devices and added support for spoofchk config. The following are changes since commit c1cb0d3b569b1d21ada8666efb42dfd437617a09: Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Akeem G. Abodunrin (5): igb: Support for 100base-fx SFP igb: Support to read and export SFF-8472/8079 data igb: Implement support to power sfp cage and turn on I2C igb: random code and comments fix igb: Fix sparse warnings on function pointers Alexander Duyck (3): igb: Mask off check of frag_off as we only want fragment offset igb: Pull adapter out of main path in igb_xmit_frame_ring igb: Use rx/tx_itr_setting when setting up initial value of itr Carolyn Wyborny (1): igb: Add support for i354 devices Jacob Keller (1): ixgbe: Add support for WoL on 82599 SFP+ LOM Jeff Kirsher (1): igb: Fix code comments and whitespace Joe Perches (1): ixgbe: Remove unnecessary #ifdef CONFIG_DEBUG_FS tests Lior Levy (1): igb: add support for spoofchk config Matthew Vick (1): igb: Enable EEE LP advertisement akepner (1): ixgbe: in shutdown, do netif_running() under rtnl_lock drivers/net/ethernet/intel/igb/e1000_82575.c | 252 +++-- drivers/net/ethernet/intel/igb/e1000_82575.h | 2 + drivers/net/ethernet/intel/igb/e1000_defines.h | 50 +- drivers/net/ethernet/intel/igb/e1000_hw.h | 57 +- drivers/net/ethernet/intel/igb/e1000_i210.c | 93 +- drivers/net/ethernet/intel/igb/e1000_i210.h | 4 + drivers/net/ethernet/intel/igb/e1000_mac.c | 124 +-- drivers/net/ethernet/intel/igb/e1000_mac.h | 17 +- drivers/net/ethernet/intel/igb/e1000_mbx.c | 11 +- drivers/net/ethernet/intel/igb/e1000_mbx.h | 52 +- drivers/net/ethernet/intel/igb/e1000_nvm.c | 27 +- drivers/net/ethernet/intel/igb/e1000_phy.c | 259 ++--- drivers/net/ethernet/intel/igb/e1000_regs.h | 53 +- drivers/net/ethernet/intel/igb/igb.h | 133 +-- drivers/net/ethernet/intel/igb/igb_ethtool.c | 345 +++++-- drivers/net/ethernet/intel/igb/igb_hwmon.c | 29 +- drivers/net/ethernet/intel/igb/igb_main.c | 1270 +++++++++++++----------- drivers/net/ethernet/intel/igb/igb_ptp.c | 61 +- drivers/net/ethernet/intel/ixgbe/ixgbe.h | 5 + drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 15 +- drivers/net/ethernet/intel/ixgbe/ixgbe_type.h | 1 + 21 files changed, 1604 insertions(+), 1256 deletions(-) -- 1.7.11.7 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2013-04-18 23:30 Jeff Kirsher @ 2013-04-19 18:19 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2013-04-19 18:19 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Thu, 18 Apr 2013 16:30:07 -0700 > This series contains updates to ixgbe and igb. > > The ixgbe changes contains 2 patches from the community, one which is a > fix from akepner to fix a issue where netif_running() in shutdown was > not done under rtnl_lock. The other community fix from Joe Perches > cleans up #ifdef CONFIG_DEBUG_FS which is no longer necessary. The > last ixgbe patch, from Jacob Keller, adds support for WoL on 82559 > SFP+ LOM. > > The remaining patches are against igb, 10 of which were previously > submitted in a pull request where changes were requested. > > The following igb patches: > igb: Support for 100base-fx SFP > igb: Support to read and export SFF-8472/8079 data > are v2 based on feedback from Dan Carpenter and Ben Hutchings in > the previous pull request. > > The largest set of changes are in my patch to cleanup code comments > and whitespace to align the igb driver with the networking style of > code comments. While cleaning up the code comments, fixed several > other whitespace/checkpatch.pl code formatting issues. > > Other notable igb patches are EEE capable devices query the PHY to > determine what the link partner is advertising, added support for > i354 devices and added support for spoofchk config. > > The following are changes since commit c1cb0d3b569b1d21ada8666efb42dfd437617a09: > Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next > and are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks a lot Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-03-08 10:07 Jeff Kirsher 2013-03-08 17:25 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2013-03-08 10:07 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to ixgbevf and e1000e. Alex's ixgbevf patch is meant to address several race issues that become possible because next_to_watch could possibly be set to a value that shows that the descriptor is done when it is not. In order to correct that we instead make next_to_watch a pointer that is set to NULL during cleanup, and set to the eop_desc after the descriptor rings have been written. Stephen's ixgbevf patch makes the PCI id table a const and reformats the table to match what the ixgbe driver does. The remaining 13 patches from Bruce are cleanup patches for e1000e to resolve checkpatch.pl warnings/errors, removing blank lines where necessary and fix code formatting. The following are changes since commit 7f0e44ac9f7f12a2519bfed9ea4df3c1471bd8bb: ipv6 flowlabel: add __rcu annotations and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Alexander Duyck (1): ixgbevf: Make next_to_watch a pointer and adjust memory barriers to avoid races Bruce Allan (13): e1000e: cleanup CODE_INDENT checkpatch errors e1000e: cleanup SPACING checkpatch errors and warnings e1000e: cleanup LONG_LINE checkpatch warnings e1000e: cleanup LEADING_SPACE checkpatch warnings e1000e: cleanup PARENTHESIS_ALIGNMENT checkpatch checks e1000e: cleanup SPACING checkpatch checks e1000e: cleanup (add/remove) blank lines where appropriate e1000e: cleanup unusually placed comments e1000e: cleanup formatting of static structs e1000e: cleanup unnecessary line breaks e1000e: cleanup USLEEP_RANGE checkpatch checks e1000e: cleanup format of struct e1000_opt_list struct e1000e: cleanup - move defines to appropriate header file Stephen Hemminger (1): ixgbevf: use PCI_DEVICE_TABLE macro drivers/net/ethernet/intel/e1000e/80003es2lan.c | 131 ++++++----- drivers/net/ethernet/intel/e1000e/82571.c | 38 ++- drivers/net/ethernet/intel/e1000e/82571.h | 2 + drivers/net/ethernet/intel/e1000e/defines.h | 25 +- drivers/net/ethernet/intel/e1000e/e1000.h | 18 +- drivers/net/ethernet/intel/e1000e/ethtool.c | 177 +++++++------- drivers/net/ethernet/intel/e1000e/hw.h | 4 +- drivers/net/ethernet/intel/e1000e/ich8lan.c | 195 ++++++++-------- drivers/net/ethernet/intel/e1000e/mac.c | 10 +- drivers/net/ethernet/intel/e1000e/netdev.c | 269 ++++++++++------------ drivers/net/ethernet/intel/e1000e/nvm.c | 2 +- drivers/net/ethernet/intel/e1000e/param.c | 62 +++-- drivers/net/ethernet/intel/e1000e/phy.c | 130 +++++------ drivers/net/ethernet/intel/ixgbevf/ixgbevf.h | 2 +- drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 80 ++++--- 15 files changed, 574 insertions(+), 571 deletions(-) -- 1.7.11.7 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2013-03-08 10:07 Jeff Kirsher @ 2013-03-08 17:25 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2013-03-08 17:25 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Fri, 8 Mar 2013 02:07:02 -0800 > This series contains updates to ixgbevf and e1000e. > > Alex's ixgbevf patch is meant to address several race issues that become > possible because next_to_watch could possibly be set to a value that shows > that the descriptor is done when it is not. In order to correct that we > instead make next_to_watch a pointer that is set to NULL during cleanup, > and set to the eop_desc after the descriptor rings have been written. > > Stephen's ixgbevf patch makes the PCI id table a const and reformats the > table to match what the ixgbe driver does. > > The remaining 13 patches from Bruce are cleanup patches for e1000e to > resolve checkpatch.pl warnings/errors, removing blank lines where > necessary and fix code formatting. > > The following are changes since commit 7f0e44ac9f7f12a2519bfed9ea4df3c1471bd8bb: > ipv6 flowlabel: add __rcu annotations > and are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks Jeff. ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver Updates @ 2013-01-11 10:23 Jeff Kirsher 0 siblings, 0 replies; 63+ messages in thread From: Jeff Kirsher @ 2013-01-11 10:23 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann This series contains updates to e1000e only. The following are changes since commit 1def9238d4aa2146924994aa4b7dc861f03b9362: net_sched: more precise pkt_len computation and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Bruce Allan (15): e1000e: SerDes autoneg flow control e1000e: Acquire/release semaphore when writing each EEPROM page e1000e: Invalid Image CSUM bit changed for I217 e1000e: helper functions for accessing EMI registers e1000e: 82577: workaround for link drop issue e1000e: fix enabling of EEE on 82579 and I217 e1000e: unexpected "Reset adapter" message when cable pulled e1000e: add missing bailout on error e1000e: resolve checkpatch PREFER_PR_LEVEL warning e1000e: cleanup redundant statistics counter e1000e: cleanup unusual comment placement e1000e: cleanup unnecessary line wrap e1000e: cleanup magic number e1000e: cleanup code duplication e1000e: merge multiple conditional statements into one drivers/net/ethernet/intel/e1000e/defines.h | 10 +- drivers/net/ethernet/intel/e1000e/e1000.h | 2 +- drivers/net/ethernet/intel/e1000e/ethtool.c | 1 - drivers/net/ethernet/intel/e1000e/hw.h | 4 + drivers/net/ethernet/intel/e1000e/ich8lan.c | 213 +++++++++++++++++++++------- drivers/net/ethernet/intel/e1000e/mac.c | 125 ++++++++++++++++ drivers/net/ethernet/intel/e1000e/netdev.c | 67 ++++----- drivers/net/ethernet/intel/e1000e/nvm.c | 26 ++-- drivers/net/ethernet/intel/e1000e/param.c | 3 +- 9 files changed, 340 insertions(+), 111 deletions(-) -- 1.7.11.7 ^ permalink raw reply [flat|nested] 63+ messages in thread
* [net-next 00/15][pull request] Intel Wired LAN Driver updates @ 2012-02-13 21:47 Jeff Kirsher 2012-02-13 23:48 ` David Miller 0 siblings, 1 reply; 63+ messages in thread From: Jeff Kirsher @ 2012-02-13 21:47 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann The following series contains updates to e1000e only. All of the patches are cleanup of the e1000e driver. The following are changes since commit fafdbf84d7eb2cd7aee435cc381a0796e1a5ccb0: trivial: typo hnalder -> handler and are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Bruce Allan (15): e1000e: cleanup: use goto for common work needed by multiple exit points e1000e: cleanup: rename goto labels to be more meaningful e1000e: potentially incorrect return for e1000_cfg_kmrn_10_100_80003es2lan e1000e: potential incorrect return for e1000_setup_copper_link_80003es2lan e1000e: cleanup: remove unnecessary assignments just before returning e1000e: cleanup: always return 0 e1000e: potentially incorrect return for e1000_set_d3_lplu_state_ich8lan e1000e: cleanup: remove unreachable statement e1000e: cleanup: remove unnecessary variable ret_val e1000e: cleanup: remove unnecessary test and return e1000e: cleanup: remove unnecessary variable initializations e1000e: cleanup: minor whitespace addition (insert blank line separator) e1000e: potentially incorrect return for e1000_init_hw_ich8lan e1000e: potentially incorrect return for e1000e_setup_fiber_serdes_link e1000e: cleanup goto statements to exit points without common work drivers/net/ethernet/intel/e1000e/80003es2lan.c | 36 +--- drivers/net/ethernet/intel/e1000e/82571.c | 36 ++--- drivers/net/ethernet/intel/e1000e/ich8lan.c | 220 +++++++++------------ drivers/net/ethernet/intel/e1000e/mac.c | 25 +-- drivers/net/ethernet/intel/e1000e/manage.c | 32 +-- drivers/net/ethernet/intel/e1000e/netdev.c | 16 +- drivers/net/ethernet/intel/e1000e/nvm.c | 32 ++-- drivers/net/ethernet/intel/e1000e/phy.c | 241 ++++++++++------------- 8 files changed, 265 insertions(+), 373 deletions(-) -- 1.7.7.6 ^ permalink raw reply [flat|nested] 63+ messages in thread
* Re: [net-next 00/15][pull request] Intel Wired LAN Driver updates 2012-02-13 21:47 [net-next 00/15][pull request] Intel Wired LAN Driver updates Jeff Kirsher @ 2012-02-13 23:48 ` David Miller 0 siblings, 0 replies; 63+ messages in thread From: David Miller @ 2012-02-13 23:48 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Mon, 13 Feb 2012 13:47:35 -0800 > The following series contains updates to e1000e only. All of the > patches are cleanup of the e1000e driver. > > The following are changes since commit fafdbf84d7eb2cd7aee435cc381a0796e1a5ccb0: > trivial: typo hnalder -> handler > and are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next master Pulled, thanks. ^ permalink raw reply [flat|nested] 63+ messages in thread
end of thread, other threads:[~2014-05-31 0:53 UTC | newest] Thread overview: 63+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-01-09 12:52 [net-next 00/15][pull request] Intel Wired LAN Driver Updates Jeff Kirsher 2014-01-09 12:52 ` [net-next 01/15] i40e: Re-enable interrupt on ICR0 Jeff Kirsher 2014-01-09 12:52 ` [net-next 02/15] i40e: use kernel specific defines Jeff Kirsher 2014-01-09 12:52 ` [net-next 03/15] i40e: Fix GPL header Jeff Kirsher 2014-01-09 12:52 ` [net-next 04/15] i40e: Fix MAC format in Write MAC address AQ cmd Jeff Kirsher 2014-01-09 12:52 ` [net-next 05/15] i40e: add a comment on barrier and fix panic on reset Jeff Kirsher 2014-01-09 14:42 ` Sergei Shtylyov 2014-01-09 17:17 ` Greg Rose 2014-01-09 20:12 ` David Miller 2014-01-09 20:14 ` Rose, Gregory V 2014-01-09 21:21 ` Ben Hutchings 2014-01-09 21:23 ` Ben Hutchings 2014-01-09 21:56 ` David Miller 2014-01-09 22:16 ` Scott Feldman 2014-01-10 17:34 ` Williams, Mitch A 2014-01-09 12:52 ` [net-next 06/15] i40e: disable packet split Jeff Kirsher 2014-01-09 12:52 ` [net-next 07/15] i40e: Cleanup reconfig rss path Jeff Kirsher 2014-01-09 12:52 ` [net-next 08/15] i40e: release NVM resource reservation on startup Jeff Kirsher 2014-01-09 12:52 ` [net-next 09/15] i40e: remove interrupt on AQ error Jeff Kirsher 2014-01-09 12:52 ` [net-next 10/15] i40e: accept pf to pf adminq messages Jeff Kirsher 2014-01-09 12:52 ` [net-next 11/15] i40e: shorten wordy fields Jeff Kirsher 2014-01-09 12:52 ` [net-next 12/15] i40e: trivial: formatting and checkpatch fixes Jeff Kirsher 2014-01-09 12:52 ` [net-next 13/15] i40e: fix spelling errors Jeff Kirsher 2014-01-09 12:52 ` [net-next 14/15] i40e: Add a dummy packet template Jeff Kirsher 2014-01-09 12:52 ` [net-next 15/15] i40e: Turn flow director off in MFP mode Jeff Kirsher 2014-01-09 20:13 ` [net-next 00/15][pull request] Intel Wired LAN Driver Updates David Miller -- strict thread matches above, loose matches on Subject: below -- 2014-05-29 15:10 Jeff Kirsher 2014-05-31 0:53 ` David Miller 2014-05-27 9:12 Jeff Kirsher 2014-05-28 16:42 ` David Miller 2014-05-21 12:00 Jeff Kirsher 2014-05-22 16:05 ` David Miller 2014-04-28 13:52 Jeff Kirsher 2014-04-28 18:53 ` David Miller 2014-03-26 18:03 Jeff Kirsher 2014-03-20 8:32 Jeff Kirsher 2014-03-20 18:20 ` David Miller 2014-03-08 9:25 Jeff Kirsher 2014-03-08 23:50 ` David Miller 2014-03-09 0:10 ` David Miller 2014-03-09 1:12 ` Jeff Kirsher 2014-01-07 9:32 Jeff Kirsher 2014-01-06 12:30 Jeff Kirsher 2014-01-06 18:26 ` David Miller 2014-01-06 18:49 ` Jeff Kirsher 2014-01-06 20:14 ` Jeff Kirsher 2014-01-05 10:46 Jeff Kirsher 2014-01-06 1:31 ` David Miller 2013-12-19 6:34 Jeff Kirsher 2013-12-19 19:20 ` Jeff Kirsher 2013-12-19 19:47 ` David Miller 2013-12-12 12:40 Jeff Kirsher 2013-12-06 8:06 Jeff Kirsher 2013-12-06 19:49 ` David Miller 2013-08-21 8:23 Jeff Kirsher 2013-07-29 12:51 Jeff Kirsher 2013-04-18 23:30 Jeff Kirsher 2013-04-19 18:19 ` David Miller 2013-03-08 10:07 Jeff Kirsher 2013-03-08 17:25 ` David Miller 2013-01-11 10:23 Jeff Kirsher 2012-02-13 21:47 [net-next 00/15][pull request] Intel Wired LAN Driver updates Jeff Kirsher 2012-02-13 23:48 ` David Miller
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).