Re: [PATCH net-next 09/11] devlink: convert occ_get op to separate registration

Thu, Apr 05, 2018 at 01:00:18AM CEST, dsah...@gmail.com wrote: >On 4/4/18 4:59 PM, Jakub Kicinski wrote: >> On Wed, 4 Apr 2018 08:25:11 +0200, Jiri Pirko wrote: Jiri, I am not aware of any other API where a driver registers with it yet doesn't want the handler to be called so either

Re: [PATCH v2 bpf-next 0/3] bpf/verifier: subprog/func_call simplifications

On Thu, Apr 05, 2018 at 12:58:46AM +0100, Edward Cree wrote: > On 04/04/18 00:37, Alexei Starovoitov wrote: > > hmm. that doesn't fail for me and any other bots didn't complain. > > Are you sure you're running the latest kernel and tests? > Ah, test_progs isn't actually rebuilding because __NR_bpf

Re: [PATCH net-next RFC 0/5] ipv6: sr: introduce seg6local End.BPF action

On Wed, Apr 4, 2018 at 2:34 AM, Mathieu Xhonneux wrote: > 2018-04-03 16:25 GMT+02:00 David Lebrun : >> Actually I'm wrong here. dst_input() will call either ip6_input() or >> ip6_forward(), not ipv6_rcv(). Both functions expect IP6CB() to be set, >> so

Re: [PATCH v3 2/4] bus: fsl-mc: add restool userspace support

On Wed, Apr 4, 2018 at 7:42 AM, Andrew Lunn wrote: >> I hear you. It is more complicated this way...having all these individual >> objects vs just a single "bundle" of them that represents a NIC. But, that's >> the way the DPAA2 hardware is, and we're implementing kernel support

Re: [PATCH net-next] vxlan: add ttl inherit support

On Wed, Apr 04, 2018 at 12:03:39PM -0400, David Miller wrote: > The net-next tree is closed, please resubmit this when the net-next tree opens > back up. Sorry, I didn't pay much attention on the net-next open/close cycle before. After re-read netdev-FAQ.txt. Now I understand the steps and I need

marvell switch

Hello, I am trying to use marvell switch in linux, Is it that the kernel drivers from marvell switch are used just to enable all ports, or do they also provide APIs to userspace to enable specific ports only. I have not find examples or wiki for marvell switch, so I am not too sure as what are

Re: [PATCH net] net: dsa: Discard frames from unused ports

On Wed, Apr 04, 2018 at 05:49:10PM -0700, Florian Fainelli wrote: > On 04/04/2018 04:56 PM, Andrew Lunn wrote: > > The Marvell switches under some conditions will pass a frame to the > > host with the port being the CPU port. Such frames are invalid, and > > should be dropped. Not dropping them

Re: [rtlwifi-btcoex] Suspicious code in halbtc8821a1ant driver

On Thu, 2018-04-05 at 01:25 +, Gustavo A. R. Silva wrote: > Hi all, > > While doing some static analysis I came across the following piece of code at > drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8821a1ant.c:1581: > > 1581 static void btc8821a1ant_act_bt_sco_hid_only_busy(struct

[PULL] fwcfg, vhost: features and fixes

The following changes since commit 0c8efd610b58cb23cefdfa12015799079aef94ae: Linux 4.16-rc5 (2018-03-11 17:25:09 -0700) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git tags/for_linus for you to fetch changes up to

[rtlwifi-btcoex] Suspicious code in halbtc8821a1ant driver

Hi all, While doing some static analysis I came across the following piece of code at drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8821a1ant.c:1581: 1581 static void btc8821a1ant_act_bt_sco_hid_only_busy(struct btc_coexist *btcoexist, 1582

RE: [Intel-wired-lan] [next-queue PATCH v6 08/10] igb: Add MAC address support for ethtool nftuple filters

> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@osuosl.org] On > Behalf Of Vinicius Costa Gomes > Sent: Thursday, March 29, 2018 2:08 PM > To: intel-wired-...@lists.osuosl.org > Cc: netdev@vger.kernel.org; Sanchez-Palencia, Jesus palen...@intel.com> > Subject: [Intel-wired-lan]

Re: linux-next: manual merge of the net-next tree with the pci tree

Hi all, On Tue, 3 Apr 2018 13:14:54 +1000 Stephen Rothwell wrote: > > Today's linux-next merge of the net-next tree got a conflict in: > > drivers/net/ethernet/mellanox/mlx5/core/en_main.c > > between commit: > > 2907938d2375 ("net/mlx5e: Use

Re: [PATCH net] route: check sysctl_fib_multipath_use_neigh earlier than hash

Hi Xin Long. [This is an automated email] This commit has been processed because it contains a "Fixes:" tag. fixing commit: a6db4494d218 net: ipv4: Consider failed nexthops in multipath routes. The bot has also determined it's probably a bug fixing patch. (score: 58.0140) The bot has tested

Re: [PATCH v2 net-next 01/10] mlxsw: spectrum_acl: Fix flex actions header ifndef define construct

Hi Ido Schimmel Jiri Pirko. [This is an automated email] This commit has been processed by the -stable helper bot and determined to be a high probability candidate for -stable trees. (score: 2.6781) The bot has tested the following trees: v4.15.15, v4.14.32, v4.9.92, v4.4.126, v4.15.15: Build

Re: [PATCH v2 net-next 06/10] mlxsw: core: Fix arg name of MLXSW_CORE_RES_VALID and MLXSW_CORE_RES_GET

Hi Ido Schimmel Jiri Pirko. [This is an automated email] This commit has been processed by the -stable helper bot and determined to be a high probability candidate for -stable trees. (score: 1.5151) The bot has tested the following trees: v4.15.15, v4.14.32, v4.9.92, v4.4.126, v4.15.15: Build

Re: [PATCH net] netns: filter uevents correctly

On Wed, Apr 04, 2018 at 05:38:02PM -0500, Eric W. Biederman wrote: > Christian Brauner writes: > > > On Wed, Apr 04, 2018 at 09:48:57PM +0200, Christian Brauner wrote: > >> commit 07e98962fa77 ("kobject: Send hotplug events in all network > >> namespaces") > >>

Re: [PATCH net 2/2] net: bgmac: Fix endian access in bgmac_dma_tx_ring_free()

Hi Florian Fainelli. [This is an automated email] This commit has been processed because it contains a "Fixes:" tag. fixing commit: 9cde94506eac bgmac: implement scatter/gather support. The bot has also determined it's probably a bug fixing patch. (score: 45.9160) The bot has tested the

Re: [PATCH net v5 3/3] ipv6: udp6: set dst cache for a connected sk if current not valid

Hi Alexey Kodanev. [This is an automated email] This commit has been processed because it contains a "Fixes:" tag. fixing commit: 33c162a980fe ipv6: datagram: Update dst cache of a connected datagram sk during pmtu update. The bot has tested the following trees: v4.15.15, v4.14.32, v4.9.92,

Re: [PATCH net 1/2] net: bgmac: Correctly annotate register space

Hi Florian Fainelli. [This is an automated email] This commit has been processed because it contains a "Fixes:" tag. fixing commit: f6a95a24957a net: ethernet: bgmac: Add platform device support. The bot has also determined it's probably a bug fixing patch. (score: 67.8237) The bot has tested

Re: [PATCH net] net: dsa: Discard frames from unused ports

On 04/04/2018 04:56 PM, Andrew Lunn wrote: > The Marvell switches under some conditions will pass a frame to the > host with the port being the CPU port. Such frames are invalid, and > should be dropped. Not dropping them can result in a crash when > incrementing the receive statistics for an

[PATCH] brcm80211: brcmsmac: phy_lcn: remove duplicate code

Remove and refactor some code in order to avoid having identical code for different branches. Notice that this piece of code hasn't been modified since 2011. Addresses-Coverity-ID: 1226756 ("Identical code for different branches") Signed-off-by: Gustavo A. R. Silva ---

Re: [RFC] net: bump the default number of RSS queues

On Tue, 3 Apr 2018 17:20:49 -0700, Eric Dumazet wrote: > On 04/03/2018 05:14 PM, Jakub Kicinski wrote: > > Some popular NIC vendors are not adhering to > > netif_get_num_default_rss_queues() which leads to users being > > surprised and filing bugs :) Bump the number of default RX > > queues to

Re: [PATCH v2 bpf-next 0/3] bpf/verifier: subprog/func_call simplifications

On 04/04/18 00:37, Alexei Starovoitov wrote: > hmm. that doesn't fail for me and any other bots didn't complain. > Are you sure you're running the latest kernel and tests? Ah, test_progs isn't actually rebuilding because __NR_bpf is undeclared;  something must be going wrong with header files.

[PATCH net] net: dsa: Discard frames from unused ports

The Marvell switches under some conditions will pass a frame to the host with the port being the CPU port. Such frames are invalid, and should be dropped. Not dropping them can result in a crash when incrementing the receive statistics for an invalid port. Reported-by: Chris Healy

Re: [PATCH iproute2] ip/l2tp: remove offset and peer-offset options

On Tue, 3 Apr 2018 17:39:54 +0200 Guillaume Nault wrote: > Ignore options "peer-offset" and "offset" when creating sessions. Keep > them when dumping sessions in order to avoid breaking external scripts. > > "peer-offset" has always been a noop in iproute2. "offset" is now

Re: [PATCH iproute2 rdma: Ignore unknown netlink attributes

On Tue, 3 Apr 2018 10:28:42 +0300 Leon Romanovsky wrote: > From: Leon Romanovsky > > The check if netlink attributes supplied more than maximum supported > is to strict and may lead to backward compatibility issues with old > application with a newer

Re: [PATCH iproute2-next] tc: Correct json output for actions

On Wed, 04 Apr 2018 17:09:04 -0400 Roman Mashak wrote: > Yuval Mintz writes: > > > Commit 9fd3f0b255d9 ("tc: enable json output for actions") added JSON > > support for tc-actions at the expense of breaking other use cases that > > reach

Re: [PATCH net-next 09/11] devlink: convert occ_get op to separate registration

On 4/4/18 4:59 PM, Jakub Kicinski wrote: > On Wed, 4 Apr 2018 08:25:11 +0200, Jiri Pirko wrote: >>> Jiri, I am not aware of any other API where a driver registers with it >>> yet doesn't want the handler to be called so either waits to register >> >> Again, the thing is, this is kind of unusual

Re: [PATCH net-next 09/11] devlink: convert occ_get op to separate registration

On Wed, 4 Apr 2018 08:25:11 +0200, Jiri Pirko wrote: > >Jiri, I am not aware of any other API where a driver registers with it > >yet doesn't want the handler to be called so either waits to register > > Again, the thing is, this is kind of unusual because of the reload > thing. FWIW my knee

Re: [PATCH net] netns: filter uevents correctly

Christian Brauner writes: > On Wed, Apr 04, 2018 at 09:48:57PM +0200, Christian Brauner wrote: >> commit 07e98962fa77 ("kobject: Send hotplug events in all network >> namespaces") >> >> enabled sending hotplug events into all network namespaces back in 2010. >>

RE: [RFC PATCH] packet: mark ring entry as in-use inside spin_lock to prevent RX ring overrun

> >> >One issue with the above proposed change to use TP_STATUS_IN_PROGRESS > >> >is that the documentation of the tp_status field is somewhat > >> >inconsistent. In some places it's described as TP_STATUS_KERNEL(0) > >> >meaning the entry is owned by the kernel and

Re: [RFC PATCH] packet: mark ring entry as in-use inside spin_lock to prevent RX ring overrun

>> >One issue with the above proposed change to use TP_STATUS_IN_PROGRESS >> >is that the documentation of the tp_status field is somewhat >> >inconsistent. In some places it's described as TP_STATUS_KERNEL(0) >> >meaning the entry is owned by the kernel and !TP_STATUS_KERNEL(0)

Re: [PATCH iproute2-next] tc: Correct json output for actions

Yuval Mintz writes: > Commit 9fd3f0b255d9 ("tc: enable json output for actions") added JSON > support for tc-actions at the expense of breaking other use cases that > reach tc_print_action(), as the latter don't expect the 'actions' array > to be a new object. > > Consider

Re: [PATCH v15 ] net/veth/XDP: Line-rate packet forwarding in kernel

On Wed, Apr 4, 2018 at 2:16 AM, Jesper Dangaard Brouer wrote: > > On Sun, 1 Apr 2018 20:47:28 -0400 Md. Islam" wrote: > >> [...] More specifically, header parsing and fib >> lookup only takes around 82 ns. This shows that this could be used to >> implement

Re: [PATCH net] netns: filter uevents correctly

On Wed, Apr 04, 2018 at 09:48:57PM +0200, Christian Brauner wrote: > commit 07e98962fa77 ("kobject: Send hotplug events in all network namespaces") > > enabled sending hotplug events into all network namespaces back in 2010. > Over time the set of uevents that get sent into all network namespaces

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

> Networking vendors have out of tree kernel modules. Those modules use a > netdev (call it a master netdev, a control netdev, cpu port, whatever) > to pull packets from the ASIC and deliver to virtual netdevices > representing physical ports. Sounds a lot like DSA. Please ask the vendor to

[jkirsher/next-queue, RFC PATCH 3/3] net-sysfs: Add interface for Rx queue map per Tx queue

Extend transmit queue sysfs attribute to configure Rx queue map per Tx queue. By default no receive queues are configured for the Tx queue. - /sys/class/net/eth0/queues/tx-*/xps_rxqs Signed-off-by: Amritha Nambiar --- net/core/net-sysfs.c | 81

[jkirsher/next-queue, RFC PATCH 0/3] Symmetric queue selection using XPS for Rx queues

This patch series implements support for Tx queue selection based on Rx queue map. This is done by configuring Rx queue map per Tx-queue using sysfs attribute. If the user configuration for Rx queues does not apply, then the Tx queue selection falls back to XPS using CPUs and finally to hashing.

[jkirsher/next-queue, RFC PATCH 2/3] net: Enable Tx queue selection based on Rx queues

This patch adds support to pick Tx queue based on the Rx queue map configuration set by the admin through the sysfs attribute for each Tx queue. If the user configuration for receive queue map does not apply, then the Tx queue selection falls back to CPU map based selection and finally to hashing.

[jkirsher/next-queue, RFC PATCH 1/3] net: Refactor XPS for CPUs and Rx queues

Refactor XPS code to support Tx queue selection based on CPU map or Rx queue map. Signed-off-by: Amritha Nambiar --- include/linux/netdevice.h | 82 +- net/core/dev.c| 208 ++---

Re: [PATCH 00/15] ARM: pxa: switch to DMA slave maps

On Wed, 04 Apr 2018 21:49:26 +0200 Robert Jarzmik wrote: > Ulf Hansson writes: > > > On 2 April 2018 at 16:26, Robert Jarzmik wrote: > >> Hi, > >> > >> This serie is aimed at removing the dmaengine slave compat use, and

Re: [PATCH 00/15] ARM: pxa: switch to DMA slave maps

Ulf Hansson writes: > On 2 April 2018 at 16:26, Robert Jarzmik wrote: >> Hi, >> >> This serie is aimed at removing the dmaengine slave compat use, and transfer >> knowledge of the DMA requestors into architecture code. >> As this looks like a

[PATCH net-next] netns: filter uevents correctly

commit 07e98962fa77 ("kobject: Send hotplug events in all network namespaces") enabled sending hotplug events into all network namespaces back in 2010. Over time the set of uevents that get sent into all network namespaces has shrunk. We have now reached the point where hotplug events for all

Re: possible deadlock in skb_queue_tail

On Wed, Apr 4, 2018 at 7:08 AM, Cong Wang wrote: > On Tue, Apr 3, 2018 at 4:42 AM, Kirill Tkhai wrote: >> On 03.04.2018 14:25, Dmitry Vyukov wrote: >>> On Tue, Apr 3, 2018 at 11:50 AM, Kirill Tkhai wrote:

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

Wed, Apr 04, 2018 at 07:37:49PM CEST, da...@davemloft.net wrote: >From: David Ahern >Date: Wed, 4 Apr 2018 11:21:54 -0600 > >> It is a netdev so there is no reason to have a separate ip command to >> inspect it. 'ip link' is the right place. > >I agree on this. > >What I really

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

On Wed, Apr 4, 2018 at 10:21 AM, David Ahern wrote: > On 4/4/18 1:36 AM, Siwei Liu wrote: >> On Tue, Apr 3, 2018 at 6:04 PM, David Ahern wrote: >>> On 4/3/18 9:42 AM, Jiri Pirko wrote: > > There are other use cases that want to hide a device from

[GIT] Networking

This fixes some fallout from the net-next merge the other day, plus some non-merge-window-related bug fixes: 1) Fix sparse warnings in bcmgenet,systemport, b53, and mt7530, from Florian Fainelli. 2) pptp does a bogus dst_release() on a route we have a single refcount on, and attached to a

Re: [PATCH v3 net 2/5] tcp: prevent bogus FRTO undos with non-SACK flows

On Wed, Apr 4, 2018 at 1:41 PM Yuchung Cheng wrote: > On Wed, Apr 4, 2018 at 10:22 AM, Neal Cardwell wrote: > > n Wed, Apr 4, 2018 at 1:13 PM Yuchung Cheng wrote: > >> Agreed. That's a good point. And I would much preferred to rename

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

On Wed, 4 Apr 2018 11:37:52 -0600 David Ahern wrote: > Networking vendors have out of tree kernel modules. Those modules use a > netdev (call it a master netdev, a control netdev, cpu port, whatever) > to pull packets from the ASIC and deliver to virtual netdevices >

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

From: David Ahern Date: Wed, 4 Apr 2018 11:37:52 -0600 > Networking vendors have out of tree kernel modules. Those modules use a > netdev (call it a master netdev, a control netdev, cpu port, whatever) > to pull packets from the ASIC and deliver to virtual netdevices >

Re: [PATCH v3 net 2/5] tcp: prevent bogus FRTO undos with non-SACK flows

On Wed, Apr 4, 2018 at 10:22 AM, Neal Cardwell wrote: > n Wed, Apr 4, 2018 at 1:13 PM Yuchung Cheng wrote: >> Agreed. That's a good point. And I would much preferred to rename that >> to FLAG_ORIG_PROGRESS (w/ updated comment). > >> so I think we're in

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

[ dropping virtio-...@lists.oasis-open.org since it is a closed list and I am tired of deleting bounces ] On 4/4/18 2:28 AM, Siwei Liu wrote: > On Tue, Apr 3, 2018 at 6:04 PM, David Ahern wrote: >> On 4/3/18 9:42 AM, Jiri Pirko wrote: There are other use cases that

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

From: David Ahern Date: Wed, 4 Apr 2018 11:21:54 -0600 > It is a netdev so there is no reason to have a separate ip command to > inspect it. 'ip link' is the right place. I agree on this. What I really don't understand still is the use case... really. So there are control

Re: [Intel-wired-lan] [iwl next-queue PATCH 02/10] macvlan: Rename fwd_priv to accel_priv and add accessor function

On Wed, Apr 4, 2018 at 9:53 AM, Shannon Nelson wrote: > On 4/3/2018 2:16 PM, Alexander Duyck wrote: > > [...] >> >> +static inline void *macvlan_accel_priv(struct net_device *dev) >> +{ >> + struct macvlan_dev *macvlan = netdev_priv(dev); >> + >> + return

Re: [PATCH v3 net 2/5] tcp: prevent bogus FRTO undos with non-SACK flows

n Wed, Apr 4, 2018 at 1:13 PM Yuchung Cheng wrote: > Agreed. That's a good point. And I would much preferred to rename that > to FLAG_ORIG_PROGRESS (w/ updated comment). > so I think we're in agreement to use existing patch w/ the new name > FLAG_ORIG_PROGRESS Yes, SGTM. I

Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice

On 4/4/18 1:36 AM, Siwei Liu wrote: > On Tue, Apr 3, 2018 at 6:04 PM, David Ahern wrote: >> On 4/3/18 9:42 AM, Jiri Pirko wrote: There are other use cases that want to hide a device from userspace. I >>> >>> What usecases do you have in mind? >> >> As mentioned in a

[PATCH iproute2-next 1/1] tc: jsonify tunnel_key action

Signed-off-by: Roman Mashak --- tc/m_tunnel_key.c | 36 +--- 1 file changed, 25 insertions(+), 11 deletions(-) diff --git a/tc/m_tunnel_key.c b/tc/m_tunnel_key.c index bac3c07fa90b..0fa461549ad9 100644 --- a/tc/m_tunnel_key.c +++

Re: [PATCH v3 net 1/5] tcp: feed correct number of pkts acked to cc modules also in recovery

On Wed, Apr 4, 2018 at 6:42 AM Ilpo Järvinen wrote: > On Wed, 28 Mar 2018, Yuchung Cheng wrote: ... > > Your patch looks good. Some questions / comments: The patch looks good to me as well (I read the Mar 28 version). Thanks for this fix, Ilpo. > Just to be sure that

Re: [PATCH iproute2-next] tc: Correct json output for actions

On 4/4/18 6:24 AM, Yuval Mintz wrote: > Commit 9fd3f0b255d9 ("tc: enable json output for actions") added JSON > support for tc-actions at the expense of breaking other use cases that > reach tc_print_action(), as the latter don't expect the 'actions' array > to be a new object. > > Consider the

Re: [PATCH v3 net 2/5] tcp: prevent bogus FRTO undos with non-SACK flows

On Wed, Apr 4, 2018 at 9:33 AM, Neal Cardwell wrote: > > On Wed, Apr 4, 2018 at 6:35 AM Ilpo Järvinen > wrote: > > > On Wed, 28 Mar 2018, Yuchung Cheng wrote: > > > > On Tue, Mar 13, 2018 at 3:25 AM, Ilpo Järvinen > > >

Re: [PATCH net] arp: fix arp_filter on l3slave devices

On 4/4/18 4:13 AM, Miguel Fadon Perlines wrote: > arp_filter performs an ip_route_output search for arp source address and > > checks if output device is the same where the arp request was received, > > if it is not, the arp request is not answered. > >   > > This route lookup is always done

Re: [Intel-wired-lan] [iwl next-queue PATCH 03/10] macvlan: Use software path for offloaded local, broadcast, and multicast traffic

On Wed, Apr 4, 2018 at 9:53 AM, Shannon Nelson wrote: > On 4/3/2018 2:16 PM, Alexander Duyck wrote: >> >> This change makes it so that we use a software path for packets that are >> going to be locally switched between two macvlan interfaces on the same >> device. In

Re: [PATCH] netdevsim: remove incorrect __net_initdata annotations

From: Arnd Bergmann Date: Wed, 4 Apr 2018 18:03:41 +0200 > On Wed, Apr 4, 2018 at 5:52 PM, David Miller wrote: >> From: Arnd Bergmann >> Date: Wed, 4 Apr 2018 14:12:39 +0200 >> >>> The __net_initdata section cannot currently be used for

Re: [Intel-wired-lan] [iwl next-queue PATCH 02/10] macvlan: Rename fwd_priv to accel_priv and add accessor function

On 4/3/2018 2:16 PM, Alexander Duyck wrote: [...] +static inline void *macvlan_accel_priv(struct net_device *dev) +{ + struct macvlan_dev *macvlan = netdev_priv(dev); + + return macvlan->accel_priv; Perhaps a check for (macvlan == NULL) before using it? sln +} #endif /*

Re: [Intel-wired-lan] [iwl next-queue PATCH 03/10] macvlan: Use software path for offloaded local, broadcast, and multicast traffic

On 4/3/2018 2:16 PM, Alexander Duyck wrote: This change makes it so that we use a software path for packets that are going to be locally switched between two macvlan interfaces on the same device. In addition we resort to software replication of broadcast and multicast packets instead of

Re: [PATCH v3 net 2/5] tcp: prevent bogus FRTO undos with non-SACK flows

On Wed, Apr 4, 2018 at 6:35 AM Ilpo Järvinen wrote: > On Wed, 28 Mar 2018, Yuchung Cheng wrote: > > On Tue, Mar 13, 2018 at 3:25 AM, Ilpo Järvinen > > wrote: > > > > > > If SACK is not enabled and the first cumulative ACK after the RTO > >

Re: [PATCH net] sfc: remove ctpio_dmabuf_start from stats

From: Bert Kenward Date: Wed, 4 Apr 2018 16:40:30 +0100 > The ctpio_dmabuf_start entry is not actually a stat and shouldn't > be exposed to ethtool. > > Fixes: 2c0b6ee837db ("sfc: expose CTPIO stats on NICs that support > them") Please don't break up a long Fixes tag

Commits going to plain 'net' tree now.

Therefore, please submit bug fixes relative to 'net', thank you.

Re: [PATCH] netdevsim: remove incorrect __net_initdata annotations

On Wed, Apr 4, 2018 at 5:52 PM, David Miller wrote: > From: Arnd Bergmann > Date: Wed, 4 Apr 2018 14:12:39 +0200 > >> The __net_initdata section cannot currently be used for structures that >> get cleaned up in an exitcall using unregister_pernet_operations:

Re: [PATCH net-next 2/2] inet: frags: fix ip6frag_low_thresh boundary

From: Eric Dumazet Date: Wed, 4 Apr 2018 08:35:10 -0700 > Giving an integer to proc_doulongvec_minmax() is dangerous on 64bit arches, > since linker might place next to it a non zero value preventing a change > to ip6frag_low_thresh. > > ip6frag_low_thresh is not used

Re: [PATCH net-next] vxlan: add ttl inherit support

From: Hangbin Liu Date: Wed, 4 Apr 2018 21:26:21 +0800 > Like tos inherit, ttl inherit should also means inherit the inner protocol's > ttl values, which actually not implemented in vxlan yet. > > But we could not treat ttl == 0 as "use the inner TTL", because that would

Re: WARNING: refcount bug in should_fail

Al Viro writes: > On Mon, Apr 02, 2018 at 10:59:34PM +0100, Al Viro wrote: > >> FWIW, I'm going through the ->kill_sb() instances, fixing that sort >> of bugs (most of them preexisting, but I should've checked instead >> of assuming that everything's fine). Will push

Re: [net-next 1/1] tipc: Fix namespace violation in tipc_sk_fill_sock_diag

From: GhantaKrishnamurthy MohanKrishna Date: Wed, 4 Apr 2018 14:49:47 +0200 > To fetch UID info for socket diagnostics, we determine the > namespace of user context using tipc socket instance. This > may cause namespace violation, as the kernel

Re: [PATCH net-next] net: avoid unneeded atomic operation in ip*_append_data()

From: Paolo Abeni Date: Wed, 4 Apr 2018 14:30:01 +0200 > After commit 694aba690de0 ("ipv4: factorize sk_wmem_alloc updates > done by __ip_append_data()") and commit 1f4c6eb24029 ("ipv6: > factorize sk_wmem_alloc updates done by __ip6_append_data()"), > when transmitting sub

Re: [PATCH] netdevsim: remove incorrect __net_initdata annotations

From: Arnd Bergmann Date: Wed, 4 Apr 2018 14:12:39 +0200 > The __net_initdata section cannot currently be used for structures that > get cleaned up in an exitcall using unregister_pernet_operations: > > WARNING: vmlinux.o(.text+0x868c34): Section mismatch in reference from the

Re: [PATCH 40/47] netfilter: nf_tables: build-in filter chain type

On Wed, Apr 4, 2018 at 5:46 PM, Pablo Neira Ayuso wrote: > On Wed, Apr 04, 2018 at 05:38:31PM +0200, Arnd Bergmann wrote: >> On Fri, Mar 30, 2018 at 1:46 PM, Pablo Neira Ayuso >> wrote: >> > One module per supported filter chain family type takes too

Re: [Cluster-devel] [PATCH v2 0/2] gfs2: Stop using rhashtable_walk_peek

On Wed, Apr 04, 2018 at 11:46:28AM -0400, Bob Peterson wrote: > > The patches look good. The big question is whether to add them to this > merge window while it's still open. Opinions? We're still hashing out the rhashtable interface so I don't think now is the time to rush things. Thanks, --

Re: [PATCH] mac80211: Fix bad line warning

From: Masanari Iida Date: Wed, 4 Apr 2018 20:53:33 +0900 > After 03fe2debbb2771fb90881e merged during 4.17 marge window, > I start to see following warning during "make xmldocs" > > ./include/net/mac80211.h:2083: warning: bad line: > > > Replace ">" with "*" fix the

Re: [PATCH] [net] nvmem: disallow modular CONFIG_NVMEM

From: Arnd Bergmann Date: Wed, 4 Apr 2018 12:38:40 +0200 > The new of_get_nvmem_mac_address() helper function causes a link error > with CONFIG_NVMEM=m: > > drivers/of/of_net.o: In function `of_get_nvmem_mac_address': > of_net.c:(.text+0x168): undefined reference to

Re: [PATCH] net: hns3: fix length overflow when CONFIG_ARM64_64K_PAGES

From: Tan Xiaojun Date: Wed, 4 Apr 2018 17:40:48 +0800 > When enable the config item "CONFIG_ARM64_64K_PAGES", the size of PAGE_SIZE > is 65536(64K). But the type of length is u16, it will overflow. So change it > to u32. > > Signed-off-by: Tan Xiaojun

Re: [PATCH 40/47] netfilter: nf_tables: build-in filter chain type

On Wed, Apr 04, 2018 at 05:38:31PM +0200, Arnd Bergmann wrote: > On Fri, Mar 30, 2018 at 1:46 PM, Pablo Neira Ayuso > wrote: > > One module per supported filter chain family type takes too much memory > > for very little code - too much modularization - place all chain

Re: [PATCH net] nfp: use full 40 bits of the NSP buffer address

From: Jakub Kicinski Date: Tue, 3 Apr 2018 17:24:23 -0700 > From: Dirk van der Merwe > > The NSP default buffer is a piece of NFP memory where additional > command data can be placed. Its format has been copied from > host buffer,

Re: [Cluster-devel] [PATCH v2 0/2] gfs2: Stop using rhashtable_walk_peek

- Original Message - > Here's a second version of the patch (now a patch set) to eliminate > rhashtable_walk_peek in gfs2. > > The first patch introduces lockref_put_not_zero, the inverse of > lockref_get_not_zero. > > The second patch eliminates rhashtable_walk_peek in gfs2. In >

Re: [PATCH v2] lan78xx: Connect phy early

From: Alexander Graf Date: Wed, 4 Apr 2018 00:19:35 +0200 > When using wicked with a lan78xx device attached to the system, we > end up with ethtool commands issued on the device before an ifup > got issued. That lead to the following crash: > ... > The culprit is quite simple:

Re: [PATCH] netdevsim: remove incorrect __net_initdata annotations

On Wed, Apr 4, 2018 at 5:33 PM, David Ahern wrote: > On 4/4/18 6:12 AM, Arnd Bergmann wrote: >> The __net_initdata section cannot currently be used for structures that >> get cleaned up in an exitcall using unregister_pernet_operations: >> > > I am confused ... I do the

[PATCH net] sfc: remove ctpio_dmabuf_start from stats

The ctpio_dmabuf_start entry is not actually a stat and shouldn't be exposed to ethtool. Fixes: 2c0b6ee837db ("sfc: expose CTPIO stats on NICs that support them") Signed-off-by: Bert Kenward --- drivers/net/ethernet/sfc/ef10.c | 2 -- drivers/net/ethernet/sfc/nic.h | 1

Re: [PATCH net-next 2/2] inet: frags: fix ip6frag_low_thresh boundary

On 04/04/2018 08:35 AM, Eric Dumazet wrote: > Giving an integer to proc_doulongvec_minmax() is dangerous on 64bit arches, > since linker might place next to it a non zero value preventing a change > to ip6frag_low_thresh. > > ip6frag_low_thresh is not used anymore in the kernel, but we do not >

Re: [PATCH 40/47] netfilter: nf_tables: build-in filter chain type

On Fri, Mar 30, 2018 at 1:46 PM, Pablo Neira Ayuso wrote: > One module per supported filter chain family type takes too much memory > for very little code - too much modularization - place all chain filter > definitions in one single file. > > Signed-off-by: Pablo Neira Ayuso

Re: [PATCH net-next] nfp: add a separate counter for packets with CHECKSUM_COMPLETE

From: Jakub Kicinski Date: Mon, 2 Apr 2018 13:31:20 -0700 > We are currently counting packets with CHECKSUM_COMPLETE as > "hw_rx_csum_ok". This is confusing. Add a new counter. > To make sure it fits in the same cacheline move the less used > error counter to a

[PATCH net-next 2/2] inet: frags: fix ip6frag_low_thresh boundary

Giving an integer to proc_doulongvec_minmax() is dangerous on 64bit arches, since linker might place next to it a non zero value preventing a change to ip6frag_low_thresh. ip6frag_low_thresh is not used anymore in the kernel, but we do not want to prematuraly break user scripts wanting to change

Re: [PATCH] netdevsim: remove incorrect __net_initdata annotations

On 4/4/18 6:12 AM, Arnd Bergmann wrote: > The __net_initdata section cannot currently be used for structures that > get cleaned up in an exitcall using unregister_pernet_operations: > > WARNING: vmlinux.o(.text+0x868c34): Section mismatch in reference from the > function nsim_devlink_exit() to

Re: [net-next 1/1] tipc: Fix missing list initializations in struct tipc_subscription

From: Jon Maloy Date: Tue, 3 Apr 2018 19:11:19 +0200 > When an item of struct tipc_subscription is created, we fail to > initialize the two lists aggregated into the struct. This has so far > never been a problem, since the items are just added to a root > object by

Re: [PATCH net v6 0/4] ipv6: udp: set dst cache for a connected sk if current not valid

From: Alexey Kodanev Date: Tue, 3 Apr 2018 15:00:06 +0300 > A new RTF_CACHE route can be created with the socket's dst cache > update between the below calls in udpv6_sendmsg(), when datagram > sending results to ICMPV6_PKT_TOOBIG error: > >dst =

Re: [PATCH net-next 5/5] macvlan/macvtap: Add support for SCTP checksum offload.

On Mon, 2018-04-02 at 09:40 -0400, Vladislav Yasevich wrote: > Since we now have support for software CRC32c offload, turn it on > for macvlan and macvtap devices so that guests can take advantage > of offload SCTP checksums to the host or host hardware. > > Signed-off-by: Vladislav Yasevich

Re: [PATCH v2] net: phy: marvell10g: add thermal hwmon device

From: Russell King Date: Tue, 03 Apr 2018 10:31:45 +0100 > Add a thermal monitoring device for the Marvell 88x3310, which updates > once a second. We also need to hook into the suspend/resume mechanism > to ensure that the thermal monitoring is reconfigured when we

Re: [PATCH net-next] pptp: remove a buggy dst release in pptp_connect()

From: Eric Dumazet Date: Mon, 2 Apr 2018 18:48:37 -0700 > Once dst has been cached in socket via sk_setup_caps(), > it is illegal to call ip_rt_put() (or dst_release()), > since sk_setup_caps() did not change dst refcount. > > We can still dereference it since we hold

Re: [PATCH net] net: dsa: b53: Fix sparse warnings in b53_mmap.c

From: Florian Fainelli Date: Mon, 2 Apr 2018 16:17:01 -0700 > sparse complains about the following warnings: > > drivers/net/dsa/b53/b53_mmap.c:33:31: warning: incorrect type in > initializer (different address spaces) > drivers/net/dsa/b53/b53_mmap.c:33:31:expected

Re: [PATCH net] net: dsa: mt7530: Use NULL instead of plain integer

From: Florian Fainelli Date: Mon, 2 Apr 2018 16:24:14 -0700 > We would be passing 0 instead of NULL as the rsp argument to > mt7530_fdb_cmd(), fix that. > > Fixes: b8f126a8d543 ("net-next: dsa: add dsa support for Mediatek MT7530 > switch") > Signed-off-by: Florian

Re: [Patch net] af_unix: remove redundant lockdep class

From: Cong Wang Date: Mon, 2 Apr 2018 11:01:27 -0700 > After commit 581319c58600 ("net/socket: use per af lockdep classes for sk > queues") > sock queue locks now have per-af lockdep classes, including unix socket. > It is no longer necessary to workaround it. > > I

Re: [PATCH net 0/2] net: Broadcom drivers sparse fixes

From: Florian Fainelli Date: Mon, 2 Apr 2018 15:58:54 -0700 > This patch series fixes the same warning reported by sparse in bcmsysport and > bcmgenet in the code that deals with inserting the TX checksum pointers: > > drivers/net/ethernet/broadcom/bcmsysport.c:1155:26:

  1   2   >