Re: [PATCH] ice: set the value of global config lock timeout longer

2021-04-20 Thread Nguyen, Anthony L
On Mon, 2021-04-19 at 17:31 +0800, Liwei Song wrote: > It may need hold Global Config Lock a longer time when download DDP > package file, extend the timeout value to 5000ms to ensure that > download can be finished before other AQ command got time to run, > this will fix the issue below when probe

Re: [PATCH net v2] igb: Fix XDP with PTP enabled

2021-04-20 Thread Nguyen, Anthony L
On Mon, 2021-04-19 at 09:23 +0200, Kurt Kanzenbach wrote: > When using native XDP with the igb driver, the XDP frame data doesn't > point to > the beginning of the packet. It's off by 16 bytes. Everything works > as expected > with XDP skb mode. > > Actually these 16 bytes are used to store the pa

Re: [PATCH net-next 2/6] igb: Add double-check MTA_REGISTER for i210 and i211

2021-04-20 Thread Nguyen, Anthony L
On Fri, 2021-04-16 at 14:12 -0700, Jakub Kicinski wrote: > On Fri, 16 Apr 2021 13:44:56 -0700 Tony Nguyen wrote: > > + bool is_failed; > > + int i; > > + > > + do { > > + is_failed = false; > > + for (i = hw->mac.mta_reg_count - 1; i >= 0; i--) { > > + if

Re: [PATCH net-next 2/4] ixgbe: aggregate of all receive errors through netdev's rx_errors

2021-04-13 Thread Nguyen, Anthony L
On Fri, 2021-04-09 at 18:00 -0700, Jakub Kicinski wrote: > On Fri, 9 Apr 2021 12:03:12 -0700 Tony Nguyen wrote: > > From: Radoslaw Tyl > > > > The global rx error does not take into account all the error > > counters > > that are counted by device. > > > > Extend rx error with the following cou

Re: [PATCH net] ice: Re-organizes reqstd/avail {R,T}XQ check/code for efficiency+readability

2021-04-12 Thread Nguyen, Anthony L
On Sun, 2021-04-11 at 02:45 +0100, Salil Mehta wrote: > If user has explicitly requested the number of {R,T}XQs, then it is > unnecessary > to get the count of already available {R,T}XQs from the PF > avail_{r,t}xqs > bitmap. This value will get overriden by user specified value in any s/override

Re: [PATCH V2 net] ice: fix memory leak of aRFS after resuming from suspend

2021-04-01 Thread Nguyen, Anthony L
On Wed, 2021-03-31 at 02:28 +, Liu, Yongxin wrote: > Hello Brett, > > Could you please help to review this V2? > Hi Yongxin, I have this applied to the Intel-wired-lan tree to go through some testing. Also, adding the Intel-wired-lan list for reviews. Thanks, Tony > Thanks, > Yongxin > >

Re: [PATCH net 1/3] net: ethernet: ixgbe: don't propagate -ENODEV from ixgbe_mii_bus_init()

2021-03-04 Thread Nguyen, Anthony L
On Thu, 2021-03-04 at 10:50 -0800, Jakub Kicinski wrote: > On Wed, 3 Mar 2021 17:06:47 -0800 Tony Nguyen wrote: > > From: Bartosz Golaszewski > > > > It's a valid use-case for ixgbe_mii_bus_init() to return -ENODEV - > > we > > still want to finalize the registration of the ixgbe device. Check >

Re: [PATCH] e1000e: use proper #include guard name in hw.h

2021-03-02 Thread Nguyen, Anthony L
On Sat, 2021-02-27 at 10:58 +0100, Greg Kroah-Hartman wrote: > The include guard for the e1000e and e1000 hw.h files are the same, > so > add the proper "E" term to the hw.h file for the e1000e driver. There's a patch in process that addresses this issue [1]. > This resolves some static analyzer

Re: [PATCH net 7/8] i40e: Fix add TC filter for IPv6

2021-02-19 Thread Nguyen, Anthony L
On Fri, 2021-02-19 at 09:31 -0800, Jakub Kicinski wrote: > On Thu, 18 Feb 2021 15:25:03 -0800 Tony Nguyen wrote: > > From: Mateusz Palczewski > > > > Fix insufficient distinction between IPv4 and IPv6 addresses > > when creating a filter. > > IPv4 and IPv6 are kept in the same memory area. If IPv

Re: [5.10] i40e/udp_tunnel: RTNL: assertion failed at net/ipv4/udp_tunnel_nic.c

2021-02-02 Thread Nguyen, Anthony L
On Tue, 2021-02-02 at 08:30 -0800, Jakub Kicinski wrote: > On Tue, 2 Feb 2021 09:59:56 + Pierre Cheynier wrote: > > On Sat, 30 Jan 2021 04:27:00 +0100 Jakub Kicinski wrote: > > > > > I must have missed that i40e_setup_pf_switch() is called from the > > > probe > > > path. > > > > Do you wan

Re: [PATCH net] igb: Enable RSS for Intel I211 Ethernet Controller

2021-02-01 Thread Nguyen, Anthony L
On Mon, 2021-02-01 at 08:47 -0800, Jakub Kicinski wrote: > On Sun, 31 Jan 2021 18:17:11 -0500 Matt Corallo wrote: > > Given this fixes a major (albeit ancient) performance regression, > > is > > it not a candidate for backport? It landed on Tony's dev-queue > > branch > > with a Fixes tag but no st

Re: [PATCH resend] e100: switch from 'pci_' to 'dma_' API

2021-01-28 Thread Nguyen, Anthony L
On Thu, 2021-01-28 at 22:07 +0100, Christophe JAILLET wrote: > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has > been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > > Whe

Re: [Intel-wired-lan] [PATCH net-next] ice: remove redundant null check on pf

2021-01-27 Thread Nguyen, Anthony L
On Wed, 2021-01-27 at 19:50 +0800, wangyunjian wrote: > From: Yunjian Wang > > Before calling ice_set_dflt_mib(), the 'pf' has been dereferenced. > So the additional check is unnecessary, just remove it. > > Addresses-Coverity: ("Dereference before null check") > Fixes: 7d9c9b791f9e ("ice: Imple

Re: [PATCH net v2 7/7] igc: fix link speed advertising

2021-01-27 Thread Nguyen, Anthony L
On Wed, 2021-01-27 at 08:04 +0100, Heiner Kallweit wrote: > On 26.01.2021 23:10, Tony Nguyen wrote: > > From: Corinna Vinschen > > > > Link speed advertising in igc has two problems: > > > > - When setting the advertisement via ethtool, the link speed is > > converted > > to the legacy 32 bit

Re: [Intel-wired-lan] igc: fix link speed advertising

2021-01-27 Thread Nguyen, Anthony L
On Tue, 2021-01-26 at 12:00 -0800, Jakub Kicinski wrote: > On Tue, 26 Jan 2021 11:30:37 +0100 Corinna Vinschen wrote: > > Ping? > > > > It looks like this patch got lost somehow. Without this patch, > > setting link speed advertising is broken. > > Adding Intel maintainers. I talked to the team

Re: [PATCH net 4/7] ice: use correct xdp_ring with XDP_TX action

2021-01-27 Thread Nguyen, Anthony L
On Sat, 2021-01-23 at 19:52 -0800, Jakub Kicinski wrote: > On Fri, 22 Jan 2021 15:57:31 -0800 Tony Nguyen wrote: > > From: Piotr Raczynski > > > > XDP queue number for XDP_TX action is used inconsistently > > and may result with no packets transmitted. Fix queue number > > used by the driver when

Re: [Intel-wired-lan] [PATCH net v2] ixgbe: add NULL pointer check before calling xdp_rxq_info_reg

2021-01-22 Thread Nguyen, Anthony L
On Fri, 2021-01-22 at 09:54 -0800, Alexander Duyck wrote: > On Fri, Jan 22, 2021 at 5:29 AM wangyunjian > wrote: > > > > From: Yunjian Wang > > > > The rx_ring->q_vector could be NULL, so it needs to be checked > > before > > calling xdp_rxq_info_reg. > > > > Fixes: b02e5a0ebb172 ("xsk: Propag

Re: [net-next v3 05/15] ice: create flow profile

2020-12-09 Thread Nguyen, Anthony L
On Tue, 2020-12-08 at 14:22 -0800, Alexander Duyck wrote: > On Tue, Dec 8, 2020 at 2:01 PM Nguyen, Anthony L > wrote: > > > > On Tue, 2020-12-08 at 11:00 -0800, Alexander Duyck wrote: > > > On Tue, Dec 8, 2020 at 8:58 AM Nguyen, Anthony L > > > wrote: >

Re: [net-next v3 05/15] ice: create flow profile

2020-12-08 Thread Nguyen, Anthony L
On Tue, 2020-12-08 at 11:00 -0800, Alexander Duyck wrote: > On Tue, Dec 8, 2020 at 8:58 AM Nguyen, Anthony L > wrote: > > > > On Mon, 2020-11-23 at 17:11 -0800, Alexander Duyck wrote: > > > On Mon, Nov 23, 2020 at 3:21 PM Jesse Brandeburg > > > wrote: &g

Re: [net-next v3 05/15] ice: create flow profile

2020-12-08 Thread Nguyen, Anthony L
On Mon, 2020-11-23 at 17:11 -0800, Alexander Duyck wrote: > On Mon, Nov 23, 2020 at 3:21 PM Jesse Brandeburg > wrote: > > > > Alexander Duyck wrote: > > > > > > > I'm not sure this logic is correct. Can the flow director > > > > > rules > > > > > handle > > > > > a field that is removed? Last I

Re: [PATCH net-next 0/3] i40e, ice, ixgbe: optimize for XDP_REDIRECT in xsk path

2020-12-03 Thread Nguyen, Anthony L
On Thu, 2020-12-03 at 07:58 +0100, Magnus Karlsson wrote: > On Wed, Dec 2, 2020 at 9:49 PM Maciej Fijalkowski > wrote: > > > > On Wed, Dec 02, 2020 at 04:07:21PM +0100, Magnus Karlsson wrote: > > > Optimize run_xdp_zc() for the XDP program verdict being > > > XDP_REDIRECT > > > in the zsk zero-co

Re: [PATCH net] iavf: fix double-release of rtnl_lock

2020-12-03 Thread Nguyen, Anthony L
On Wed, 2020-12-02 at 18:18 -0800, Jakub Kicinski wrote: > This code does not jump to exit on an error in iavf_lan_add_device(), > so the rtnl_unlock() from the normal path will follow. > > Fixes: b66c7bc1cd4d ("iavf: Refactor init state machine") > Signed-off-by: Jakub Kicinski > --- > drivers/

Re: [net-next v3 08/15] ice: don't always return an error for Get PHY Abilities AQ command

2020-11-20 Thread Nguyen, Anthony L
On Fri, 2020-11-13 at 17:25 -0800, Alexander Duyck wrote: > On Fri, Nov 13, 2020 at 1:49 PM Tony Nguyen < > anthony.l.ngu...@intel.com> wrote: > > > > From: Paul M Stillwell Jr > > > > There are times when the driver shouldn't return an error when the > > Get > > PHY abilities AQ command (0x0600

Re: [net-next v3 05/15] ice: create flow profile

2020-11-20 Thread Nguyen, Anthony L
On Fri, 2020-11-13 at 15:56 -0800, Alexander Duyck wrote: > On Fri, Nov 13, 2020 at 1:46 PM Tony Nguyen < > anthony.l.ngu...@intel.com> wrote: > > > > From: Real Valiquette > > > > Implement the initial steps for creating an ACL filter to support > > ntuple > > masks. Create a flow profile based

Re: [net-next v2 03/15] ice: initialize ACL table

2020-11-20 Thread Nguyen, Anthony L
On Fri, 2020-11-13 at 14:59 -0800, Alexander Duyck wrote: > On Fri, Nov 13, 2020 at 1:36 PM Tony Nguyen < > anthony.l.ngu...@intel.com> wrote: > > > > From: Real Valiquette > > > > ACL filtering can be utilized to expand support of ntuple rules by > > allowing > > mask values to be specified for

Re: [net-next v2 03/15] ice: initialize ACL table

2020-11-13 Thread Nguyen, Anthony L
On Fri, 2020-11-13 at 14:59 -0800, Alexander Duyck wrote: > On Fri, Nov 13, 2020 at 1:36 PM Tony Nguyen < > anthony.l.ngu...@intel.com> wrote: > > > > From: Real Valiquette > > > > ACL filtering can be utilized to expand support of ntuple rules by > > allowing > > mask values to be specified for

Re: [net 0/4][pull request] Intel Wired LAN Driver Updates 2020-11-10

2020-11-12 Thread Nguyen, Anthony L
On Thu, 2020-11-12 at 08:55 -0800, Jakub Kicinski wrote: > On Tue, 10 Nov 2020 16:19:51 -0800 Tony Nguyen wrote: > > This series contains updates to i40e and igc drivers and the > > MAINTAINERS > > file. > > > > Slawomir fixes updating VF MAC addresses to fix various issues > > related > > to repo

Re: [net-next 1/3] i40e: Allow changing FEC settings on X722 if supported by FW

2020-10-12 Thread Nguyen, Anthony L
On Fri, 2020-10-09 at 16:36 -0700, Jakub Kicinski wrote: > On Wed, 7 Oct 2020 16:10:48 -0700 Tony Nguyen wrote: > > + if (hw->mac.type == I40E_MAC_X722 && > > + !(hw->flags & I40E_HW_FLAG_X722_FEC_REQUEST_CAPABLE)) { > > + netdev_err(netdev, "Setting FEC encoding not supported >

Re: [net-next 2/3] i40e: Fix MAC address setting for a VF via Host/VM

2020-10-12 Thread Nguyen, Anthony L
On Fri, 2020-10-09 at 13:46 -0400, Willem de Bruijn wrote: > On Wed, Oct 7, 2020 at 7:11 PM Tony Nguyen < > anthony.l.ngu...@intel.com> wrote: > > > > From: Aleksandr Loktionov > > > > Fix MAC setting flow for the PF driver. > > > > Without this change the MAC address setting was interpreted >

Re: [Intel-wired-lan] [PATCH] net: ethernet: ixgbe: don't propagate -ENODEV from ixgbe_mii_bus_init()

2020-10-12 Thread Nguyen, Anthony L
On Mon, 2020-10-12 at 08:16 -0700, Jakub Kicinski wrote: > On Mon, 12 Oct 2020 14:20:16 +0200 Bartosz Golaszewski wrote: > > On Mon, Sep 28, 2020 at 9:17 AM Bartosz Golaszewski > > wrote: > > > > > > From: Bartosz Golaszewski > > > > > > It's a valid use-case for ixgbe_mii_bus_init() to return

Re: [PATCH] net: intel: ice: Use uintptr_t for casting data

2020-10-09 Thread Nguyen, Anthony L
On Fri, 2020-10-09 at 14:18 +0800, Pujin Shi wrote: > Fix up a compiler error on 64bit architectures where pointers. > > In file included from > drivers/net/ethernet/intel/ice/ice_flex_pipe.c:6:0: > drivers/net/ethernet/intel/ice/ice_flex_pipe.c: In function > 'ice_free_flow_profs': > drivers/net/

Re: [PATCH net-next v2 00/10] udp_tunnel: convert Intel drivers with shared tables

2020-09-28 Thread Nguyen, Anthony L
On Fri, 2020-09-25 at 17:56 -0700, Jakub Kicinski wrote: > This set converts Intel drivers which have the ability to spawn > multiple netdevs, but have only one UDP tunnel port table. > > Appropriate support is added to the core infra in patch 1, > followed by netdevsim support and a selftest. >

Re: [Intel-wired-lan] [PATCH net-next v1 4/7] selftests: net: add a test for shared UDP tunnel info tables

2020-09-24 Thread Nguyen, Anthony L
On Mon, 2020-09-21 at 14:44 -0700, Jakub Kicinski wrote: > On Sat, 19 Sep 2020 07:23:58 + Brown, Aaron F wrote: > > > From: Intel-wired-lan On > > > Behalf Of Jakub > > > Kicinski > > > Sent: Tuesday, July 21, 2020 6:27 PM > > > To: da...@davemloft.net > > > Cc: netdev@vger.kernel.org; intel-w

Re: i4oe/XXV710 suspect tx_dropped stats

2020-09-17 Thread Nguyen, Anthony L
On Tue, 2020-09-15 at 09:22 -0400, Jamal Hadi Salim wrote: > Ethtool reporting zero. ip -s reporting non-zero. > Which is right? See attached for more details. > I've asked the team responsible for the driver to look into this issue. I will let you know what I hear back. Thanks, Tony

Re: [PATCH] i40e: Fix use of uninitialized variable

2020-09-17 Thread Nguyen, Anthony L
On Wed, 2020-09-16 at 21:49 +0100, Alex Dewar wrote: > In i40e_clean_rx_irq_zc(), the variable failure is only set when a > condition is met, but then its value is used unconditionally. Fix > this. > > Addresses-Coverity: 1496986 ("Uninitialized value") > Fixes: 8cbf74149903 ("i40e, xsk: move buff

Re: [net 0/4][pull request] Intel Wired LAN Driver Updates 2020-09-09

2020-09-11 Thread Nguyen, Anthony L
On Thu, 2020-09-10 at 07:48 -0700, Jakub Kicinski wrote: > On Wed, 9 Sep 2020 17:04:07 -0700 Tony Nguyen wrote: > > This series contains updates to i40e and igc drivers. > > > > Stefan Assmann changes num_vlans to u16 to fix may be used > > uninitialized > > error and propagates error in i40_set_

Re: [PATCH net-next v1 0/7] udp_tunnel: convert Intel drivers with shared tables

2020-09-04 Thread Nguyen, Anthony L
On Thu, 2020-09-03 at 16:22 -0700, Jakub Kicinski wrote: > On Thu, 23 Jul 2020 20:06:15 +0000 Nguyen, Anthony L wrote: > > On Wed, 2020-07-22 at 14:22 -0700, Tony Nguyen wrote: > > > On Tue, 2020-07-21 at 18:27 -0700, Jakub Kicinski wrote: > > > > This set convert

Re: [net 2/3] i40e: Set RX_ONLY mode for unicast promiscuous on VLAN

2020-08-14 Thread Nguyen, Anthony L
On Fri, 2020-08-14 at 14:19 -0700, Jakub Kicinski wrote: > On Fri, 14 Aug 2020 13:36:42 -0700 Tony Nguyen wrote: > > + * @hw: pointer to the hw struct > > + * @maj: api major value > > + * @min: api minor value > > +static bool i40e_is_aq_api_ver_ge(struct i40e_adminq_info *aq, u16 > > maj, > > +

Re: [Intel-wired-lan] [PATCH v1] ice: devlink: use %*phD to print small buffer

2020-07-31 Thread Nguyen, Anthony L
On Fri, 2020-07-31 at 16:49 -0700, David Miller wrote: > From: Andy Shevchenko > Date: Thu, 30 Jul 2020 19:04:51 +0300 > > > Use %*phD format to print small buffer as hex string. > > > > Signed-off-by: Andy Shevchenko > > I am assuming the Intel folks will integrate this into their tree. Yes,

Re: [net 0/2][pull request] Intel Wired LAN Driver Updates 2020-07-30

2020-07-31 Thread Nguyen, Anthony L
On Thu, 2020-07-30 at 16:27 -0700, Jakub Kicinski wrote: > On Thu, 30 Jul 2020 10:09:36 -0700 Tony Nguyen wrote: > > This series contains updates to the e1000e and igb drivers. > > > > Aaron Ma allows PHY initialization to continue if ULP disable > > failed for > > e1000e. > > > > Francesco Rugge

Re: [PATCH net-next v1 0/7] udp_tunnel: convert Intel drivers with shared tables

2020-07-23 Thread Nguyen, Anthony L
On Wed, 2020-07-22 at 14:22 -0700, Tony Nguyen wrote: > On Tue, 2020-07-21 at 18:27 -0700, Jakub Kicinski wrote: > > This set converts Intel drivers which have the ability to spawn > > multiple netdevs, but have only one UDP tunnel port table. > > > > Appropriate support is added to the core infra

Re: [net-next 1/8] igc: Fix double definition

2020-07-23 Thread Nguyen, Anthony L
On Wed, 2020-07-22 at 15:48 -0700, Jakub Kicinski wrote: > On Wed, 22 Jul 2020 14:31:43 -0700 Tony Nguyen wrote: > > Accordance to the i225 specification address 0x4118 used for > > Host Good Packet Transmitted Count and defined as read on clear. > > IGC_ICTXQEC not in use and could be removed. >

Re: [PATCH net-next v1 0/7] udp_tunnel: convert Intel drivers with shared tables

2020-07-22 Thread Nguyen, Anthony L
On Tue, 2020-07-21 at 18:27 -0700, Jakub Kicinski wrote: > This set converts Intel drivers which have the ability to spawn > multiple netdevs, but have only one UDP tunnel port table. > > Appropriate support is added to the core infra in patch 1, > followed by netdevsim support and a selftest. >

Re: [Intel-wired-lan] [net-next, v7 5/5] ice: add switch rule management for DCF

2020-07-08 Thread Nguyen, Anthony L
On Wed, 2020-07-01 at 09:25 +0800, Haiyue Wang wrote: > The PF shall track all the outstanding switch filters (filter IDs to > be > precise) added by the DCF. > > Upon a VF reset event, the PF shall clear all outstanding switch > filters > for the given VF. Upon completion of either VF or PF reset

Re: [RFC 01/20] ice: Initialize and register multi-function device to provide RDMA

2019-09-26 Thread Nguyen, Anthony L
On Thu, 2019-09-26 at 20:05 +0200, Greg KH wrote: > On Thu, Sep 26, 2019 at 09:45:00AM -0700, Jeff Kirsher wrote: > > From: Tony Nguyen > > > > The RDMA block does not advertise on the PCI bus or any other bus. > > Huh? How do you "know" where it is then? Isn't is usually assigned > to > a PCI

Re: [net-next 01/15] ice: Implement ethtool ops for channels

2019-08-16 Thread Nguyen, Anthony L
On Mon, 2019-08-12 at 15:24 -0700, Jakub Kicinski wrote: > On Mon, 12 Aug 2019 15:07:09 +0000, Nguyen, Anthony L wrote: > > On Fri, 2019-08-09 at 14:15 -0700, Jakub Kicinski wrote: > > > On Fri, 9 Aug 2019 11:31:25 -0700, Jeff Kirsher wrote: > > > > From: Henry Tie

Re: [net-next 01/15] ice: Implement ethtool ops for channels

2019-08-12 Thread Nguyen, Anthony L
On Fri, 2019-08-09 at 14:15 -0700, Jakub Kicinski wrote: > On Fri, 9 Aug 2019 11:31:25 -0700, Jeff Kirsher wrote: > > From: Henry Tieman > > > > Add code to query and set the number of queues on the primary > > VSI for a PF. This is accessed from the 'ethtool -l' and 'ethtool > > -L' > > command

RE: [net-next 1/6] ixgbe: Do not allow LRO or MTU change with XDP

2018-07-26 Thread Nguyen, Anthony L
> From: Jakub Kicinski [mailto:jakub.kicin...@netronome.com] > Sent: Thursday, July 26, 2018 2:47 PM > On Thu, 26 Jul 2018 14:21:08 -0700, Jakub Kicinski wrote: > > On Thu, 26 Jul 2018 10:40:45 -0700, Jeff Kirsher wrote: > > > From: Tony Nguyen > > > > > > XDP does not support jumbo frames or LRO.

RE: [net-next v3 1/5] ixgbe: Ensure MAC filter was added before setting MACVLAN

2017-07-19 Thread Nguyen, Anthony L
> -Original Message- > From: Joe Perches [mailto:j...@perches.com] > Sent: Wednesday, July 19, 2017 3:55 AM > Subject: Re: [net-next v3 1/5] ixgbe: Ensure MAC filter was added before > setting > MACVLAN > > On Tue, 2017-07-18 at 18:23 -0700, Jeff Kirsher wrote: > > This patch adds a che