Re: [PATCH bpf-next V13 4/7] bpf: add BPF-helper for MTU checking

2021-01-28 Thread Jesper Dangaard Brouer
On Thu, 28 Jan 2021 22:51:23 -0800 John Fastabend wrote: > Jesper Dangaard Brouer wrote: > > This BPF-helper bpf_check_mtu() works for both XDP and TC-BPF programs. > > > > The SKB object is complex and the skb->len value (accessible from > > BPF-prog) also include the length of any extra GRO/GS

Re: [patch net-next RFC 00/10] introduce line card support for modular switch

2021-01-28 Thread Jiri Pirko
Thu, Jan 28, 2021 at 03:17:13PM CET, and...@lunn.ch wrote: >On Thu, Jan 28, 2021 at 09:14:34AM +0100, Jiri Pirko wrote: >> Wed, Jan 27, 2021 at 03:14:34PM CET, and...@lunn.ch wrote: >> >> >There are Linux standard APIs for controlling the power to devices, >> >> >the regulator API. So i assume mlxr

RE: [PATCH bpf-next V13 6/7] selftests/bpf: use bpf_check_mtu in selftest test_cls_redirect

2021-01-28 Thread John Fastabend
Jesper Dangaard Brouer wrote: > This demonstrate how bpf_check_mtu() helper can easily be used together > with bpf_skb_adjust_room() helper, prior to doing size adjustment, as > delta argument is already setup. > > Hint: This specific test can be selected like this: > ./test_progs -t cls_redirect

RE: [PATCH bpf-next V13 5/7] bpf: drop MTU check when doing TC-BPF redirect to ingress

2021-01-28 Thread John Fastabend
Jesper Dangaard Brouer wrote: > The use-case for dropping the MTU check when TC-BPF does redirect to > ingress, is described by Eyal Birger in email[0]. The summary is the > ability to increase packet size (e.g. with IPv6 headers for NAT64) and > ingress redirect packet and let normal netstack frag

RE: [PATCH bpf-next V13 4/7] bpf: add BPF-helper for MTU checking

2021-01-28 Thread John Fastabend
Jesper Dangaard Brouer wrote: > This BPF-helper bpf_check_mtu() works for both XDP and TC-BPF programs. > > The SKB object is complex and the skb->len value (accessible from > BPF-prog) also include the length of any extra GRO/GSO segments, but > without taking into account that these GRO/GSO segm

Re: [net-next PATCH v4 01/15] Documentation: ACPI: DSD: Document MDIO PHY

2021-01-28 Thread Calvin Johnson
On Thu, Jan 28, 2021 at 02:27:00PM +0100, Rafael J. Wysocki wrote: > On Thu, Jan 28, 2021 at 2:12 PM Calvin Johnson > wrote: > > > > On Thu, Jan 28, 2021 at 01:00:40PM +0100, Rafael J. Wysocki wrote: > > > On Thu, Jan 28, 2021 at 12:27 PM Calvin Johnson > > > wrote: > > > > > > > > Hi Rafael, > >

[PATCH] staging: qlge/qlge_ethtool.c: strlcpy -> strscpy

2021-01-28 Thread Kumar Kartikeya Dwivedi
Fixes checkpatch warnings for usage of strlcpy. Signed-off-by: Kumar Kartikeya Dwivedi --- drivers/staging/qlge/qlge_ethtool.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/qlge/qlge_ethtool.c b/drivers/staging/qlge/qlge_ethtool.c index a28f0254c..635

Re: [RFC PATCH v3 00/13] virtio/vsock: introduce SOCK_SEQPACKET support

2021-01-28 Thread Arseny Krasnov
On 28.01.2021 20:19, Stefano Garzarella wrote: > Hi Arseny, > I reviewed a part, tomorrow I hope to finish the other patches. > > Just a couple of comments in the TODOs below. > > On Mon, Jan 25, 2021 at 02:09:00PM +0300, Arseny Krasnov wrote: >> This patchset impelements support of SOCK_SEQ

RE: [PATCH iproute2-next RESEND] devlink: Extend man page for port function set command

2021-01-28 Thread Parav Pandit
Hi David, Stephen, > From: Parav Pandit > Sent: Tuesday, January 26, 2021 6:54 PM > > Hi Stephen, > > > From: Parav Pandit > > Sent: Friday, January 22, 2021 10:32 AM > > > > Extended devlink-port man page for synopsis, description and example > > for setting devlink port function attribute. >

Re: [RFC PATCH v3 03/13] af_vsock: implement SEQPACKET rx loop

2021-01-28 Thread Arseny Krasnov
On 28.01.2021 19:55, Stefano Garzarella wrote: > On Mon, Jan 25, 2021 at 02:12:36PM +0300, Arseny Krasnov wrote: >> This adds receive loop for SEQPACKET. It looks like receive loop for >> SEQPACKET, but there is a little bit difference: >> 1) It doesn't call notify callbacks. >> 2) It doesn't car

Re: [Patch bpf-next v5 1/3] bpf: introduce timeout hash map

2021-01-28 Thread Yonghong Song
On 1/28/21 9:57 PM, Cong Wang wrote: On Thu, Jan 28, 2021 at 6:54 PM Alexei Starovoitov wrote: I meant it would look like: noinline per_elem_callback(map, key, value, ...) { if (value->foo > ...) bpf_delete_map_elem(map, key); } noinline timer_callback(timer, ctx) { map = ctx->

[PATCH] rtl8xxxu: assign value when defining variables

2021-01-28 Thread samirweng1979
From: wengjianfeng define ret and then assign value to it, which we should do one time. Signed-off-by: wengjianfeng --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8

Re: [PATCH net-next v4] net: psample: Introduce stubs to remove NIC driver dependency

2021-01-28 Thread Chris Mi
On 1/29/2021 1:14 PM, Cong Wang wrote: On Wed, Jan 27, 2021 at 5:48 PM Chris Mi wrote: In order to send sampled packets to userspace, NIC driver calls psample api directly. But it creates a hard dependency on module psample. Introduce psample_ops to remove the hard dependency. It is initialized

RE: [PATCH bpf-next V13 3/7] bpf: bpf_fib_lookup return MTU value as output when looked up

2021-01-28 Thread John Fastabend
Jesper Dangaard Brouer wrote: > The BPF-helpers for FIB lookup (bpf_xdp_fib_lookup and bpf_skb_fib_lookup) > can perform MTU check and return BPF_FIB_LKUP_RET_FRAG_NEEDED. The BPF-prog > don't know the MTU value that caused this rejection. > > If the BPF-prog wants to implement PMTU (Path MTU Disc

RE: [PATCH bpf-next V13 2/7] bpf: fix bpf_fib_lookup helper MTU check for SKB ctx

2021-01-28 Thread John Fastabend
Jesper Dangaard Brouer wrote: > BPF end-user on Cilium slack-channel (Carlo Carraro) wants to use > bpf_fib_lookup for doing MTU-check, but *prior* to extending packet size, > by adjusting fib_params 'tot_len' with the packet length plus the expected > encap size. (Just like the bpf_check_mtu helpe

Re: [PATCH net] net: hdlc_x25: Use qdisc to queue outgoing LAPB frames

2021-01-28 Thread Martin Schiller
On 2021-01-28 23:06, Xie He wrote: On Thu, Jan 28, 2021 at 11:47 AM Jakub Kicinski wrote: Noob question - could you point at or provide a quick guide to layering here? I take there is only one netdev, and something maintains an internal queue which is not stopped when HW driver stops the qdi

Re: [Patch bpf-next v5 1/3] bpf: introduce timeout hash map

2021-01-28 Thread Cong Wang
On Thu, Jan 28, 2021 at 6:54 PM Alexei Starovoitov wrote: > > I meant it would look like: > > noinline per_elem_callback(map, key, value, ...) > { > if (value->foo > ...) > bpf_delete_map_elem(map, key); > } > > noinline timer_callback(timer, ctx) > { > map = ctx->map; > bpf_for_each_map

Re: [Patch net] net: fix dev_ifsioc_locked() race condition

2021-01-28 Thread Cong Wang
On Thu, Jan 28, 2021 at 9:21 PM Jakub Kicinski wrote: > > On Thu, 28 Jan 2021 21:08:05 -0800 Cong Wang wrote: > > On Thu, Jan 28, 2021 at 12:55 PM Jakub Kicinski wrote: > > > > > > On Sat, 23 Jan 2021 17:30:49 -0800 Cong Wang wrote: > > > > From: Cong Wang > > > > > > > > dev_ifsioc_locked() is

Re: [PATCH] linux/qed: fix spelling typo in qed_chain.h

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Wed, 27 Jan 2021 10:28:01 +0800 you wrote: > From: dingsenjie > > allocted -> allocated > > Signed-off-by: dingsenjie > --- > include/linux/qed/qed_chain.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) He

Re: [PATCH v2] octeontx2-af: Fix 'physical' typos

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Wed, 27 Jan 2021 12:13:59 -0600 you wrote: > From: Bjorn Helgaas > > Fix misspellings of "physical". > > Signed-off-by: Bjorn Helgaas > --- > Thanks, Willem! > > [...] Here is the summary with links: - [v2] octeo

Re: [Patch net] net: fix dev_ifsioc_locked() race condition

2021-01-28 Thread Jakub Kicinski
On Thu, 28 Jan 2021 21:08:05 -0800 Cong Wang wrote: > On Thu, Jan 28, 2021 at 12:55 PM Jakub Kicinski wrote: > > > > On Sat, 23 Jan 2021 17:30:49 -0800 Cong Wang wrote: > > > From: Cong Wang > > > > > > dev_ifsioc_locked() is called with only RCU read lock, so when > > > there is a parallel wri

Re: [PATCH net-next v4] net: psample: Introduce stubs to remove NIC driver dependency

2021-01-28 Thread Cong Wang
On Wed, Jan 27, 2021 at 5:48 PM Chris Mi wrote: > > In order to send sampled packets to userspace, NIC driver calls > psample api directly. But it creates a hard dependency on module > psample. Introduce psample_ops to remove the hard dependency. > It is initialized when psample module is loaded a

Re: [PATCH net-next 00/12] nexthop: Preparations for resilient next-hop groups

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Thu, 28 Jan 2021 13:49:12 +0100 you wrote: > At this moment, there is only one type of next-hop group: an mpath group. > Mpath groups implement the hash-threshold algorithm, described in RFC > 2992[1]. > > To select a

Re: [Patch net] net: fix dev_ifsioc_locked() race condition

2021-01-28 Thread Cong Wang
On Thu, Jan 28, 2021 at 12:55 PM Jakub Kicinski wrote: > > On Sat, 23 Jan 2021 17:30:49 -0800 Cong Wang wrote: > > From: Cong Wang > > > > dev_ifsioc_locked() is called with only RCU read lock, so when > > there is a parallel writer changing the mac address, it could > > get a partially updated m

Re: WARNING in cfg80211_change_iface

2021-01-28 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:d03154e8 Add linux-next specific files for 20210128 git tree: linux-next console output: https://syzkaller.appspot.com/x/log.txt?x=1243cbc8d0 kernel config: https://syzkaller.appspot.com/x/.config?x

Re: [PATCH net-next 0/5] s390/qeth: updates 2021-01-28

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Thu, 28 Jan 2021 12:25:46 +0100 you wrote: > Hi Dave & Jakub, > > please apply the following patch series for qeth to netdev's net-next tree. > > Nothing special, mostly fine-tuning and follow-on cleanups for earlier

Re: [PATCH net-next 0/5] net/iucv: updates 2021-01-28

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Thu, 28 Jan 2021 12:41:03 +0100 you wrote: > Hi Dave & Jakub, > > please apply the following patch series for iucv to netdev's net-next tree. > > This reworks & simplifies the TX notification path in af_iucv, so that

Re: [PATCH net-next v2 0/6] net: ipa: hardware pipeline cleanup fixes

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Tue, 26 Jan 2021 12:56:57 -0600 you wrote: > Version 2 of this series fixes a "restricted __le16 degrades to > integer" warning from sparse in the third patch. The normal host > architecture is little-endian, so the pr

Re: [PATCH net-next V1] net: adjust net_device layout for cacheline usage

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Tue, 26 Jan 2021 18:39:39 +0100 you wrote: > The current layout of net_device is not optimal for cacheline usage. > > The member adj_list.lower linked list is split between cacheline 2 and 3. > The ifindex is placed tog

[PATCH net v2] ibmvnic: device remove has higher precedence over reset

2021-01-28 Thread Lijun Pan
Returning -EBUSY in ibmvnic_remove() does not actually hold the removal procedure since driver core doesn't care for the return value (see __device_release_driver() in drivers/base/dd.c calling dev->bus->remove()) though vio_bus_remove (in arch/powerpc/platforms/pseries/vio.c) records the return va

Re: [PATCH net-next 05/10] net: ipv4: Emit notification when fib hardware flags are changed

2021-01-28 Thread Jakub Kicinski
On Thu, 28 Jan 2021 20:33:22 -0700 David Ahern wrote: > On 1/28/21 8:04 PM, Jakub Kicinski wrote: > > On Tue, 26 Jan 2021 15:23:06 +0200 Ido Schimmel wrote: > >> Emit RTM_NEWROUTE notifications whenever RTM_F_OFFLOAD/RTM_F_TRAP flags > >> are changed. The aim is to provide an indication to user-s

Re: [PATCH net-next V1] net: adjust net_device layout for cacheline usage

2021-01-28 Thread David Ahern
On 1/26/21 10:39 AM, Jesper Dangaard Brouer wrote: > The current layout of net_device is not optimal for cacheline usage. > > The member adj_list.lower linked list is split between cacheline 2 and 3. > The ifindex is placed together with stats (struct net_device_stats), > although most modern driv

Re: [PATCH 2/2] vdpa/mlx5: Restore the hardware used index after change map

2021-01-28 Thread Jason Wang
On 2021/1/28 下午9:41, Eli Cohen wrote: When a change of memory map occurs, the hardware resources are destroyed and then re-created again with the new memory map. In such case, we need to restore the hardware available and used indices. The driver failed to restore the used index which is added

Re: [PATCH 1/2] vdpa/mlx5: Avoid unnecessary query virtqueue

2021-01-28 Thread Jason Wang
On 2021/1/28 下午9:41, Eli Cohen wrote: suspend_vq should only suspend the VQ on not save the current available index. This is done when a change of map occurs when the driver calls save_channel_info(). Signed-off-by: Eli Cohen Acked-by: Jason Wang --- drivers/vdpa/mlx5/net/mlx5_vnet.c

[PATCH net 1/2] ibmvnic: fix a race between open and reset

2021-01-28 Thread Sukadev Bhattiprolu
__ibmvnic_reset() currently reads the adapter->state before getting the rtnl and saves that state as the "target state" for the reset. If this read occurs when adapter is in PROBED state, the target state would be PROBED. Just after the target state is saved, and before the actual reset process is

[PATCH net 2/2] ibmvnic: fix race with multiple open/close

2021-01-28 Thread Sukadev Bhattiprolu
If two or more instances of 'ip link set' commands race and first one already brings the interface up (or down), the subsequent instances can simply return without redoing the up/down operation. Fixes: ed651a10875f ("ibmvnic: Updated reset handling") Reported-by: Abdul Haleem Tested-by: Abdul Hal

Re: [PATCH net-next 05/10] net: ipv4: Emit notification when fib hardware flags are changed

2021-01-28 Thread David Ahern
On 1/28/21 8:04 PM, Jakub Kicinski wrote: > On Tue, 26 Jan 2021 15:23:06 +0200 Ido Schimmel wrote: >> Emit RTM_NEWROUTE notifications whenever RTM_F_OFFLOAD/RTM_F_TRAP flags >> are changed. The aim is to provide an indication to user-space >> (e.g., routing daemons) about the state of the route in

RE: [PATCH iproute2-next v2 2/2] vdpa: Add vdpa tool

2021-01-28 Thread Parav Pandit
Hi David, > From: Parav Pandit > Sent: Friday, January 29, 2021 12:13 AM [..] > --- > changelog: > v1->v2: > - added and used library helpers for socket communication > - added and used library functions for string processing helpers > - added and used library functions indent processing hel

Re: [PATCH net-next 00/12] nexthop: Preparations for resilient next-hop groups

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > At this moment, there is only one type of next-hop group: an mpath group. > Mpath groups implement the hash-threshold algorithm, described in RFC > 2992[1]. > > To select a next hop, hash-threshold algorithm first assigns a range of > hashes to each next h

Re: [PATCH net-next 12/12] nexthop: Extract a helper for validation of get/del RTNL requests

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > Validation of messages for get / del of a next hop is the same as will be > validation of messages for get of a resilient next hop group bucket. The > difference is that policy for resilient next hop group buckets is a > superset of that used for next-hop g

Re: [PATCH net-next 11/12] nexthop: Add a callback parameter to rtm_dump_walk_nexthops()

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > In order to allow different handling for next-hop tree dumper and for > bucket dumper, parameterize the next-hop tree walker with a callback. Add > rtm_dump_nexthop_cb() with just the bits relevant for next-hop tree > dumping. > > Signed-off-by: Petr Macha

Re: [PATCH net-next 10/12] nexthop: Extract a helper for walking the next-hop tree

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > Extract from rtm_dump_nexthop() a helper to walk the next hop tree. A > separate function for this will be reusable from the bucket dumper. > > Signed-off-by: Petr Machata > Reviewed-by: Ido Schimmel > --- > net/ipv4/nexthop.c | 52 +

Re: [PATCH net-next 09/12] nexthop: Strongly-type context of rtm_dump_nexthop()

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > The dump operations need to keep state from one invocation to another. A > scratch area is dedicated for this purpose in the passed-in argument, cb, > namely via two aliased arrays, struct netlink_callback.args and .ctx. > > Dumping of buckets will end up

Re: [PATCH net-next 07/12] nexthop: Extract dump filtering parameters into a single structure

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > Requests to dump nexthops have many attributes in common with those that > requests to dump buckets of resilient NH groups will have. In order to make > reuse of this code simpler, convert the code to use a single structure with > filtering configuration in

Re: [PATCH net-next 08/12] nexthop: Extract a common helper for parsing dump attributes

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > Requests to dump nexthops have many attributes in common with those that > requests to dump buckets of resilient NH groups will have. However, they > have different policies. To allow reuse of this code, extract a > policy-agnostic wrapper out of nh_valid_d

Re: [PATCH net-next 06/12] nexthop: Dispatch notifier init()/fini() by group type

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > After there are several next-hop group types, initialization and > finalization of notifier type needs to reflect the actual type. Transform > nh_notifier_grp_info_init() and _fini() to make extending them easier. > > Signed-off-by: Petr Machata > Reviewe

Re: [PATCH net-next 05/12] nexthop: Use enum to encode notification type

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > From: Ido Schimmel > > Currently there are only two types of in-kernel nexthop notification. > The two are distinguished by the 'is_grp' boolean field in 'struct > nh_notifier_info'. > > As more notification types are introduced for more next-hop group t

Re: [PATCH net-next 04/12] nexthop: Assert the invariant that a NH group is of only one type

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > Most of the code that deals with nexthop groups relies on the fact that the > group is of exactly one well-known type. Currently there is only one type, > "mpath", but as more next-hop group types come, it becomes desirable to > have a central place where t

Re: [PATCH net-next 03/12] nexthop: Introduce to struct nh_grp_entry a per-type union

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > The values that a next-hop group needs to keep track of depend on the group > type. Introduce a union to separate fields specific to the mpath groups > from fields specific to other group types. > > Signed-off-by: Petr Machata > Reviewed-by: Ido Schimmel

Re: [PATCH net-next 02/12] nexthop: Dispatch nexthop_select_path() by group type

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > The logic for selecting path depends on the next-hop group type. Adapt the > nexthop_select_path() to dispatch according to the group type. > > Signed-off-by: Petr Machata > Reviewed-by: Ido Schimmel > --- > net/ipv4/nexthop.c | 22 -

Re: [PATCH net-next 01/12] nexthop: Rename nexthop_free_mpath

2021-01-28 Thread David Ahern
On 1/28/21 5:49 AM, Petr Machata wrote: > From: David Ahern > > nexthop_free_mpath really should be nexthop_free_group. Rename it. > > Signed-off-by: David Ahern > Reviewed-by: Ido Schimmel > Signed-off-by: Petr Machata > --- > net/ipv4/nexthop.c | 4 ++-- > 1 file changed, 2 insertions(+),

Re: [PATCH net-next 05/10] net: ipv4: Emit notification when fib hardware flags are changed

2021-01-28 Thread Jakub Kicinski
On Tue, 26 Jan 2021 15:23:06 +0200 Ido Schimmel wrote: > Emit RTM_NEWROUTE notifications whenever RTM_F_OFFLOAD/RTM_F_TRAP flags > are changed. The aim is to provide an indication to user-space > (e.g., routing daemons) about the state of the route in hardware. What does the daemon in the user spa

Re: [Patch bpf-next v5 1/3] bpf: introduce timeout hash map

2021-01-28 Thread Alexei Starovoitov
On Wed, Jan 27, 2021 at 10:28:15PM -0800, Cong Wang wrote: > On Wed, Jan 27, 2021 at 10:00 AM Alexei Starovoitov > wrote: > > > > On Tue, Jan 26, 2021 at 11:00 PM Cong Wang wrote: > > > > > ret = PTR_ERR(l_new); > > > > > + if (ret == -EAGAIN) { > > > > > +

Re: [PATCH v2] lan743x: fix endianness when accessing descriptors

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Thu, 28 Jan 2021 09:48:59 +0500 you wrote: > TX/RX descriptor ring fields are always little-endian, but conversion > wasn't performed for big-endian CPUs, so the driver failed to work. > > This patch makes the driver wo

Re: [net v2] net: ip_tunnel: fix mtu calculation

2021-01-28 Thread Willem de Bruijn
On Thu, Jan 28, 2021 at 9:21 PM Willem de Bruijn wrote: > > On Thu, Jan 28, 2021 at 8:02 PM Vadim Fedorenko wrote: > > > > dev->hard_header_len for tunnel interface is set only when header_ops > > are set too and already contains full overhead of any tunnel encapsulation. > > That's why there is

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

2021-01-28 Thread Jakub Kicinski
On Thu, 28 Jan 2021 21:45:02 + Nguyen, Anthony L wrote: > > drivers/net/ethernet/intel/e100.c | 92 - > > My apologies, this patch slipped through the cracks for me. I will send > it in my next net-next 1GbE series or Jakub you can take it directly if > you'd like.

Re: [PATCH v5 net-next 00/18] net: mvpp2: Add TX Flow Control support

2021-01-28 Thread Jakub Kicinski
On Thu, 28 Jan 2021 20:31:04 +0200 stef...@marvell.com wrote: > From: Stefan Chulski > > Armada hardware has a pause generation mechanism in GOP (MAC). > The GOP generate flow control frames based on an indication programmed in > Ports Control 0 Register. There is a bit per port. > However asser

Re: [net v2] net: ip_tunnel: fix mtu calculation

2021-01-28 Thread Willem de Bruijn
On Thu, Jan 28, 2021 at 8:02 PM Vadim Fedorenko wrote: > > dev->hard_header_len for tunnel interface is set only when header_ops > are set too and already contains full overhead of any tunnel encapsulation. > That's why there is not need to use this overhead twice in mtu calc. > > Fixes: fdafed459

Re: [PATCH] net: phy: remove h from printk format specifier

2021-01-28 Thread Jakub Kicinski
On Mon, 25 Jan 2021 08:45:28 -0800 t...@redhat.com wrote: > From: Tom Rix > > This change fixes the checkpatch warning described in this commit > commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouraging use of > unnecessary %h[xudi] and %hh[xudi]") > > Standard integer promotion is already

Re: [PATCH v7 net-next 08/11] net: dsa: allow changing the tag protocol via the "tagging" device attribute

2021-01-28 Thread Jakub Kicinski
On Thu, 28 Jan 2021 22:07:50 +0200 Vladimir Oltean wrote: > On Wed, Jan 27, 2021 at 05:30:44PM -0800, Jakub Kicinski wrote: > > > +const struct dsa_device_ops *dsa_find_tagger_by_name(const char *buf) > > > +{ > > > + const struct dsa_device_ops *ops = NULL; > > > + struct dsa_tag_driver *dsa_tag_d

Re: [PATCH net-next v2] net: qmi_wwan: Add pass through mode

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Mon, 25 Jan 2021 00:33:35 -0700 you wrote: > Pass through mode is to allow packets in MAP format to be passed > on to the stack. rmnet driver can be used to process and demultiplex > these packets. > > Pass through mode

Re: [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Wed, 27 Jan 2021 16:34:31 +0100 you wrote: > Hello, > > this patch series add a sysfs file to let userspace know which mux > id has been used to create a qmimux network interface. > > I'm aware that adding new sysfs f

Re: [PATCH] ipvlan: remove h from printk format specifier

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Sun, 24 Jan 2021 11:08:04 -0800 you wrote: > From: Tom Rix > > This change fixes the checkpatch warning described in this commit > commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouraging use of > unnecessary %h[

RE: [PATCH net] stmmac: intel: Configure EHL PSE0 GbE and PSE1 GbE to 32 bits DMA addressing

2021-01-28 Thread Ismail, Mohammad Athari
Hi, Thank you. Regards, Athari -Original Message- From: patchwork-bot+netdev...@kernel.org Sent: Friday, January 29, 2021 5:20 AM To: Ismail, Mohammad Athari Cc: peppe.cavall...@st.com; alexandre.tor...@st.com; joab...@synopsys.com; da...@davemloft.net; k...@kernel.org; mcoquelin.st.

Re: [PATCH net-next 1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces

2021-01-28 Thread Jakub Kicinski
On Wed, 27 Jan 2021 08:26:13 +0100 Bjørn Mork wrote: > Jakub Kicinski writes: > > We got two patches adding new sysfs files for QMI in close succession - > > is there a sense of how much this interface will grow over time? > > The honest answer is no. > > I do not expect this interface to grow

Re: [PATCHv17 bpf-next 6/6] selftests/bpf: add xdp_redirect_multi test

2021-01-28 Thread Hangbin Liu
Hi John, On Thu, Jan 28, 2021 at 03:37:09PM -0800, John Fastabend wrote: > Otherwise, its not the most elegant, but testing XDP at the moment > doesn't fit into the normal test framework very well either. Thanks a lot for your help in reviewing the patches. I will add updating XDP test in my todo

Re: [net-next V10 01/14] devlink: Prepare code to fill multiple port function attributes

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Fri, 22 Jan 2021 11:36:45 -0800 you wrote: > From: Parav Pandit > > Prepare code to fill zero or more port function optional attributes. > Subsequent patch makes use of this to fill more port function > attributes. >

Re: [net 01/12] net/mlx5: Fix memory leak on flow table creation error flow

2021-01-28 Thread Jakub Kicinski
On Thu, 28 Jan 2021 00:33:20 -0800 Saeed Mahameed wrote: > On Thu, 2021-01-28 at 03:30 +, patchwork-bot+netdev...@kernel.org > wrote: > > Hello: > > > > This series was applied to netdev/net.git (refs/heads/master): > > > > > ... > > >     https://git.kernel.org/netdev/net/c/89e394675818

Re: [PATCH bpf-next] bpf: simplify cases in bpf_base_func_proto

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to bpf/bpf-next.git (refs/heads/master): On Wed, 27 Jan 2021 18:46:15 +0100 you wrote: > !perfmon_capable() is checked before the last switch(func_id) in > bpf_base_func_proto. Thus, the cases BPF_FUNC_trace_printk and > BPF_FUNC_snprintf_btf can be moved to that las

[PATCH net-next 15/16] selftests: mptcp: add testcases for ADD_ADDR with port

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds testcases for ADD_ADDR with port and the related MIB counters check in chk_add_nr. The output looks like this: 24 signal address with port syn[ ok ] - synack[ ok ] - ack[ ok ] add[ ok ] - echo [ ok ] - pt [ ok

[PATCH net-next 16/16] selftests: mptcp: add command line arguments for mptcp_join.sh

2021-01-28 Thread Mat Martineau
From: Geliang Tang Since the mptcp_join script is becoming too big, this patch splits it into several smaller chunks, each of them has been defined in a function as a individual test group for several related testcases. Using bash getopts function to parse command line arguments, and invoke each

Re: [PATCH bpf-next v2 0/4] bpf: expose bpf_{g,s}etsockopt to more bpf_sock_addr hooks

2021-01-28 Thread patchwork-bot+netdevbpf
Hello: This series was applied to bpf/bpf-next.git (refs/heads/master): On Wed, 27 Jan 2021 15:28:49 -0800 you wrote: > We'd like to use the SENDMSG ones, Daniel suggested to > expose to more hooks while are here. > > Stanislav Fomichev (4): > bpf: enable bpf_{g,s}etsockopt in BPF_CGROUP_UDP{4

[PATCH net-next 14/16] mptcp: add the mibs for ADD_ADDR with port

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds the mibs for ADD_ADDR with port: MPTCP_MIB_PORTADD for received ADD_ADDR suboption with a port number. MPTCP_MIB_PORTSYNRX, MPTCP_MIB_PORTSYNACKRX, MPTCP_MIB_PORTACKRX, for received MP_JOIN's SYN or SYN/ACK or ACK with a port number which is different from the

[PATCH net-next 13/16] selftests: mptcp: add port argument for pm_nl_ctl

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds a new argument for pm_nl_ctl tool. We can use it like this: # pm_nl_ctl add 10.0.2.1 flags signal port 10100 # pm_nl_ctl dump id 1 flags signal 10.0.2.1 10100 Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- tools/testing/selftests/net/mptcp/

[PATCH net-next 11/16] mptcp: enable use_port when invoke addresses_equal

2021-01-28 Thread Mat Martineau
From: Geliang Tang When dealing with the addresses list local_addr_list or anno_list, we should enable the function addresses_equal's parameter use_port. And enable it in address_zero too. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- net/mptcp/pm_netlink.c | 8 1 file

[PATCH net-next 12/16] mptcp: deal with MPTCP_PM_ADDR_ATTR_PORT in PM netlink

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds MPTCP_PM_ADDR_ATTR_PORT filling and parsing in PM netlink. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- net/mptcp/pm_netlink.c | 5 + 1 file changed, 5 insertions(+) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 5b04

[PATCH net-next 10/16] mptcp: add port number check for MP_JOIN

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds two new helpers, subflow_use_different_sport and subflow_use_different_dport, to check whether the subflow's source or destination port number is different from the msk's port number. When receiving the MP_JOIN's SYN/SYNACK/ACK, we do these port number checks an

[PATCH net-next 06/16] selftests: mptcp: add testcases for newly added addresses

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds testcases to create subflows or signal addresses for the newly added IPv4 or IPv6 addresses. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- .../testing/selftests/net/mptcp/mptcp_join.sh | 73 ++- 1 file changed, 71 insertions(+)

[PATCH net-next 07/16] mptcp: create the listening socket for new port

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch creates a listening socket when an address with a port-number is added by PM netlink. Then binds the new port to the socket, and listens for new connections. When the address is removed or the addresses are flushed by PM netlink, release the listening socket. Signe

[PATCH net-next 09/16] mptcp: add a new helper subflow_req_create_thmac

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch adds a new helper named subflow_req_create_thmac, which is extracted from subflow_token_join_request. It initializes subflow_req's local_nonce and thmac fields, those are the more expensive to populate. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau ---

[PATCH net-next 08/16] mptcp: drop unused skb in subflow_token_join_request

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch drops the unused parameter skb in subflow_token_join_request. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- net/mptcp/subflow.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/net/mptcp/subflow.c b/net/mptcp/subflow.c index

[PATCH net-next 03/16] mptcp: create subflow or signal addr for newly added address

2021-01-28 Thread Mat Martineau
From: Geliang Tang Currently, when a new MPTCP endpoint is added, the existing MPTCP sockets are not affected. This patch implements a new function mptcp_nl_add_subflow_or_signal_addr, invoked when an address is added from PM netlink. This function traverses the MPTCP sockets list and invokes mp

[PATCH net-next 05/16] selftests: mptcp: use minus values for removing address numbers

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch changes the removing addresses numbers to minus values, left the plus values for the adding addresses numbers. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- .../testing/selftests/net/mptcp/mptcp_join.sh | 32 ++- 1 file changed, 17

[PATCH net-next 04/16] mptcp: send ack for every add_addr

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch changes the sending ACK conditions for the ADD_ADDR, send an ACK packet for any ADD_ADDR, not just when ipv6 addresses or port numbers are included. Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/139 Acked-by: Paolo Abeni Signed-off-by: Geliang Tang

[PATCH net-next 00/16] mptcp: ADD_ADDR enhancements

2021-01-28 Thread Mat Martineau
This patch series from the MPTCP tree contains enhancements and associated tests for the ADD_ADDR ("add address") MPTCP option. This option allows already-connected MPTCP peers to share additional IP addresses with each other, which can then be used to create additional subflows within those MPTCP

[PATCH net-next 02/16] mptcp: drop *_max fields in mptcp_pm_data

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch drops the per-msk values add_addr_signal_max, add_addr_accept_max, local_addr_max and subflows_max fields in struct mptcp_pm_data, uses the pernet *_max values instead. And adds four new helpers to get the pernet *_max values separately. Co-developed-by: Matthieu Ba

[PATCH net-next 01/16] mptcp: use WRITE_ONCE/READ_ONCE for the pernet *_max

2021-01-28 Thread Mat Martineau
From: Geliang Tang This patch uses WRITE_ONCE() and READ_ONCE for all the pernet add_addr_signal_max, add_addr_accept_max, local_addr_max and subflows_max fields in struct pm_nl_pernet to avoid concurrency issues. Signed-off-by: Geliang Tang Signed-off-by: Mat Martineau --- net/mptcp/pm_netli

Re: [PATCH net-next 4/8] net: dsa: microchip: add support for phylink management

2021-01-28 Thread Andrew Lunn
> + /* For T1 PHY */ > + if (lan937x_is_internal_t1_phy_port(dev, port)) { > + phylink_set(mask, 100baseT_Full); > + phylink_set_port_modes(mask); Since this is a T1 PHY, you should be using 100baseT1_Full. This might be the first user of this for phylink, so pleas

Re: [PATCH net-next 3/8] net: dsa: microchip: add DSA support for microchip lan937x

2021-01-28 Thread Andrew Lunn
> +bool lan937x_is_internal_phy_port(struct ksz_device *dev, int port) > +{ > + /* Check if the port is RGMII */ > + if (port == LAN937X_RGMII_1_PORT || port == LAN937X_RGMII_2_PORT) > + return false; > + > + /* Check if the port is SGMII */ > + if (port == LAN937X_SGMII

Re: [PATCH bpf-next v2 4/4] bpf: enable bpf_{g,s}etsockopt in BPF_CGROUP_UDP{4,6}_RECVMSG

2021-01-28 Thread Daniel Borkmann
On 1/29/21 1:59 AM, Stanislav Fomichev wrote: On Thu, Jan 28, 2021 at 4:52 PM Daniel Borkmann wrote: On 1/28/21 12:28 AM, Stanislav Fomichev wrote: Those hooks run as BPF_CGROUP_RUN_SA_PROG_LOCK and operate on a locked socket. Signed-off-by: Stanislav Fomichev --- net/core/filter.c

[PATCH v8 net-next 10/11] net: dsa: add a second tagger for Ocelot switches based on tag_8021q

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean There are use cases for which the existing tagger, based on the NPI (Node Processor Interface) functionality, is insufficient. Namely: - Frames injected through the NPI port bypass the frame analyzer, so no source address learning is performed, no TSN stream classificatio

[PATCH v8 net-next 11/11] net: dsa: felix: perform switch setup for tag_8021q

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean Unlike sja1105, the only other user of the software-defined tag_8021q.c tagger format, the implementation we choose for the Felix DSA switch driver preserves full functionality under a vlan_filtering bridge (i.e. IP termination works through the DSA user ports under all circ

[PATCH v8 net-next 09/11] net: dsa: felix: convert to the new .change_tag_protocol DSA API

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean In expectation of the new tag_ocelot_8021q tagger implementation, we need to be able to do runtime switchover between one tagger and another. So we must structure the existing code for the current NPI-based tagger in a certain way. We move the felix_npi_port_init function i

[PATCH v8 net-next 04/11] net: mscc: ocelot: reapply bridge forwarding mask on bonding join/leave

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean Applying the bridge forwarding mask currently is done only on the STP state changes for any port. But it depends on both STP state changes, and bonding interface state changes. Export the bit that recalculates the forwarding mask so that it could be reused, and call it when

[PATCH v8 net-next 03/11] net: mscc: ocelot: store a namespaced VCAP filter ID

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean We will be adding some private VCAP filters that should not interfere in any way with the filters added using tc-flower. So we need to allocate some IDs which will not be used by tc. Currently ocelot uses an u32 id derived from the flow cookie, which in itself is an unsigne

[PATCH v8 net-next 06/11] net: dsa: document the existing switch tree notifiers and add a new one

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean The existence of dsa_broadcast has generated some confusion in the past: https://www.mail-archive.com/netdev@vger.kernel.org/msg365042.html So let's document the existing dsa_port_notify and dsa_broadcast functions and explain when each of them should be used. Also, in fac

[PATCH v8 net-next 08/11] net: dsa: allow changing the tag protocol via the "tagging" device attribute

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean Currently DSA exposes the following sysfs: $ cat /sys/class/net/eno2/dsa/tagging ocelot which is a read-only device attribute, introduced in the kernel as commit 98cdb4807123 ("net: dsa: Expose tagging protocol to user-space"), and used by libpcap since its commit 993db3800

[PATCH v8 net-next 05/11] net: mscc: ocelot: don't use NPI tag prefix for the CPU port module

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean Context: Ocelot switches put the injection/extraction frame header in front of the Ethernet header. When used in NPI mode, a DSA master would see junk instead of the destination MAC address, and it would most likely drop the packets. So the Ocelot frame header can have an op

[PATCH v8 net-next 07/11] net: dsa: keep a copy of the tagging protocol in the DSA switch tree

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean Cascading DSA switches can be done multiple ways. There is the brute force approach / tag stacking, where one upstream switch, located between leaf switches and the host Ethernet controller, will just happily transport the DSA header of those leaf switches as payload. For th

[PATCH v8 net-next 01/11] net: dsa: tag_8021q: add helpers to deduce whether a VLAN ID is RX or TX VLAN

2021-01-28 Thread Vladimir Oltean
From: Vladimir Oltean The sja1105 implementation can be blind about this, but the felix driver doesn't do exactly what it's being told, so it needs to know whether it is a TX or an RX VLAN, so it can install the appropriate type of TCAM rule. Signed-off-by: Vladimir Oltean Reviewed-by: Florian

  1   2   3   4   >