[PATCH v3] isdn: eicon: fix a missing-check bug

2018-05-20 Thread Wenwen Wang
In divasmain.c, the function divas_write() firstly invokes the function diva_xdi_open_adapter() to open the adapter that matches with the adapter number provided by the user, and then invokes the function diva_xdi_write() to perform the write operation using the matched adapter. The two functions d

[PACTH net-next] cxgb4: copy the length of cpl_tx_pkt_core to fw_wr

2018-05-20 Thread Ganesh Goudar
immdlen field of FW_ETH_TX_PKT_WR is filled in a wrong way, we must copy the length of all the cpls encapsulated in fw work request. In the xmit path we missed adding the length of CPL_TX_PKT_CORE but we added the length of WR_HDR and it worked because WR_HDR and CPL_TX_PKT_CORE are of same length.

Re: [PATCH v2] isdn: eicon: fix a missing-check bug

2018-05-20 Thread Wenwen Wang
On Sun, May 20, 2018 at 5:37 PM, David Miller wrote: > From: Wenwen Wang > Date: Fri, 18 May 2018 16:33:47 -0500 > >> In divasmain.c, the function divas_write() firstly invokes the function >> diva_xdi_open_adapter() to open the adapter that matches with the adapter >> number provided by the user

[lkp-robot] [net] 3416099d53: net/core/dev.c:#suspicious_rcu_dereference_protected()usage

2018-05-20 Thread kernel test robot
FYI, we noticed the following commit (built with gcc-6): commit: 3416099d53d9593ab1d374180e9cd2a71fa2f4d5 ("net: Refactor XPS for CPUs and Rx queues") url: https://github.com/0day-ci/linux/commits/Amritha-Nambiar/net-Refactor-XPS-for-CPUs-and-Rx-queues/20180518-053915 in testcase: boot on te

Re: [PATCH bpf-next 0/5] fix test_sockmap

2018-05-20 Thread Prashant Bhole
On 5/19/2018 1:54 AM, Shuah Khan wrote: On 05/18/2018 01:17 AM, Prashant Bhole wrote: This series fixes bugs in test_sockmap code. They weren't caught previously because failure in RX/TX thread was not notified to the main thread. Also fixed data verification logic and slightly improved test

Re: [PATCH bpf-next 2/5] selftests/bpf: test_sockmap, join cgroup in selftest mode

2018-05-20 Thread Prashant Bhole
On 5/19/2018 1:45 AM, John Fastabend wrote: On 05/18/2018 12:17 AM, Prashant Bhole wrote: In case of selftest mode, temporary cgroup environment is created but cgroup is not joined. It causes test failures. Fixed by joining the cgroup Fixes: 16962b2404ac ("bpf: sockmap, add selftests") Signed

Re: [PATCH bpf-next 0/5] fix test_sockmap

2018-05-20 Thread Prashant Bhole
On 5/19/2018 1:42 AM, John Fastabend wrote: On 05/18/2018 12:17 AM, Prashant Bhole wrote: This series fixes bugs in test_sockmap code. They weren't caught previously because failure in RX/TX thread was not notified to the main thread. Also fixed data verification logic and slightly improved t

Re: [PATCH bpf-next 3/5] selftests/bpf: test_sockmap, fix test timeout

2018-05-20 Thread Prashant Bhole
On 5/19/2018 1:47 AM, John Fastabend wrote: On 05/18/2018 12:17 AM, Prashant Bhole wrote: In order to reduce runtime of tests, recently timout for select() call was reduced from 1sec to 10usec. This was causing many tests failures. It was caught with failure handling commits in this series. R

Re: [PATCH net-next] sctp: add support for SCTP_REUSE_PORT sockopt

2018-05-20 Thread Tom Herbert
On Sun, May 20, 2018 at 6:54 PM, Marcelo Ricardo Leitner wrote: > On Sun, May 20, 2018 at 08:50:59PM -0400, Neil Horman wrote: >> On Sat, May 19, 2018 at 03:44:40PM +0800, Xin Long wrote: >> > This feature is actually already supported by sk->sk_reuse which can be >> > set by SO_REUSEADDR. But it'

[PATCH net-next] net: ethernet: Sort Kconfig sourcing alphabetically

2018-05-20 Thread Florian Fainelli
A number of entries were not alphabetically sorted, remedy that. Signed-off-by: Florian Fainelli --- drivers/net/ethernet/Kconfig | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/net/ethernet/Kconfig b/drivers/net/ethernet/Kconfig index 54d71e1c48d5..a

[PATCH net-next] net: phy: phylink: Don't release NULL GPIO

2018-05-20 Thread Florian Fainelli
If CONFIG_GPIOLIB is disabled, gpiod_put() becomes a stub that produces a warning, this helped identify that we could be attempting to release a NULL pl->link_gpio GPIO descriptor, so guard against that. Fixes: daab3349ad1a ("net: phy: phylink: Release link GPIO") Signed-off-by: Florian Fainelli

System hung for reg_check_changs_work()-> rtnl_lock()->mutex_lock()

2018-05-20 Thread Shawn Lin
Hi, I found this hung for several times these days, and seems syzbot already reported a similar problem. Is there any patch(es) for that? Successfully initialized wpa_supplicant [ 240.091941] INFO: task kworker/u8:1:39 blocked for more than 120 seconds. [ 240.092004] Not tainted 4.4.126

Re: [PATCH v2] Revert "alx: remove WoL support"

2018-05-20 Thread David Miller
From: AceLan Kao Date: Mon, 21 May 2018 11:14:00 +0800 > We are willing to fix the issue, but we don't have a machine to > reproduce it, and the WoL feature has been removed 5 years ago, it's > hard to find those buggy machines. Have you bothered to ask the person who did the revert? > WoL is a

Re: [PATCH v2] Revert "alx: remove WoL support"

2018-05-20 Thread AceLan Kao
Hi David, We are willing to fix the issue, but we don't have a machine to reproduce it, and the WoL feature has been removed 5 years ago, it's hard to find those buggy machines. WoL is a feature that is only used by a very small group of people, and the wake up issue looks like only happens on so

Re: [RFC v4 3/5] virtio_ring: add packed ring support

2018-05-20 Thread Tiwei Bie
On Mon, May 21, 2018 at 10:30:51AM +0800, Jason Wang wrote: > On 2018年05月19日 10:29, Tiwei Bie wrote: > > > I don't hope so. > > > > > > > I agreed driver should track the DMA addrs or some > > > > other necessary things from the very beginning. And > > > > I also repeated the spec to emphasize tha

Re: KASAN: use-after-free Read in vhost_chr_write_iter

2018-05-20 Thread Jason Wang
On 2018年05月18日 17:24, Jason Wang wrote: On 2018年05月17日 21:45, DaeRyong Jeong wrote: We report the crash: KASAN: use-after-free Read in vhost_chr_write_iter This crash has been found in v4.17-rc1 using RaceFuzzer (a modified version of Syzkaller), which we describe more at the end of this re

Re: [RFC V4 PATCH 0/8] Packed ring layout for vhost

2018-05-20 Thread Jason Wang
On 2018年05月21日 00:25, Wei Xu wrote: On Wed, May 16, 2018 at 08:32:13PM +0800, Jason Wang wrote: Hi all: This RFC implement packed ring layout. The code were tested with Tiwei's RFC V3 ahttps://lkml.org/lkml/2018/4/25/34. Some fixups and tweaks were needed on top of Tiwei's code to make it run

Re: [RFC v4 3/5] virtio_ring: add packed ring support

2018-05-20 Thread Jason Wang
On 2018年05月19日 10:29, Tiwei Bie wrote: I don't hope so. I agreed driver should track the DMA addrs or some other necessary things from the very beginning. And I also repeated the spec to emphasize that it does make sense. And I'd like to do that. What I was saying is that, to support OOO, we

Re: [PATCH net-next] sctp: add support for SCTP_REUSE_PORT sockopt

2018-05-20 Thread Marcelo Ricardo Leitner
On Sun, May 20, 2018 at 08:50:59PM -0400, Neil Horman wrote: > On Sat, May 19, 2018 at 03:44:40PM +0800, Xin Long wrote: > > This feature is actually already supported by sk->sk_reuse which can be > > set by SO_REUSEADDR. But it's not working exactly as RFC6458 demands in > > section 8.1.27, like:

RE: [PATCH 1/2] net: fec: ptp: Switch to SPDX identifier

2018-05-20 Thread Andy Duan
From: Fabio Estevam Sent: 2018年5月21日 0:55 > Adopt the SPDX license identifier headers to ease license compliance > management. > > Signed-off-by: Fabio Estevam Acked-by: Fugang Duan > --- > drivers/net/ethernet/freescale/fec_ptp.c | 14 +- > 1 file changed, 1 insertion(+), 13 del

RE: [PATCH 2/2] net: fec: Add a SPDX identifier

2018-05-20 Thread Andy Duan
From: Fabio Estevam Sent: 2018年5月21日 0:55 > From: Fabio Estevam > > Currently there is no license information in the header of this file. > > The MODULE_LICENSE field contains ("GPL"), which means GNU Public > License v2 or later, so add a corresponding SPDX license identifier. > > Signed-off

Re: [PATCH net-next] sctp: add support for SCTP_REUSE_PORT sockopt

2018-05-20 Thread Neil Horman
On Sat, May 19, 2018 at 03:44:40PM +0800, Xin Long wrote: > This feature is actually already supported by sk->sk_reuse which can be > set by SO_REUSEADDR. But it's not working exactly as RFC6458 demands in > section 8.1.27, like: > > - This option only supports one-to-one style SCTP sockets >

Re: [PATCH v2] isdn: eicon: fix a missing-check bug

2018-05-20 Thread kbuild test robot
Hi Wenwen, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v4.17-rc5 next-20180517] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/c

[GIT] Networking

2018-05-20 Thread David Miller
Sorry, no drug fuelled pull request this time. As for the commits themselves, I can't say for sure :-) 1) Fix refcounting bug for connections in on-packet scheduling mode of IPVS, from Julian Anastasov. 2) Set network header properly in AF_PACKET's packet_snd, from Willem de Bruijn.

Re: [PATCH v2] packet: track ring entry use using a shadow ring to prevent RX ring overrun

2018-05-20 Thread Willem de Bruijn
On Sun, May 20, 2018 at 6:51 PM, Willem de Bruijn wrote: > On Sat, May 19, 2018 at 8:07 AM, Jon Rosen wrote: >> Fix PACKET_RX_RING bug for versions TPACKET_V1 and TPACKET_V2 which >> casues the ring to get corrupted by allowing multiple kernel threads >> to claim ownership of the same ring entry.

Re: WARNING in ip_recv_error

2018-05-20 Thread Willem de Bruijn
On Fri, May 18, 2018 at 2:59 PM, Willem de Bruijn wrote: > On Fri, May 18, 2018 at 2:46 PM, Willem de Bruijn > wrote: >> On Fri, May 18, 2018 at 2:44 PM, Willem de Bruijn >> wrote: >>> On Fri, May 18, 2018 at 1:09 PM, Willem de Bruijn >>> wrote: On Fri, May 18, 2018 at 11:44 AM, David Mill

Re: WARNING in __static_key_slow_dec

2018-05-20 Thread Willem de Bruijn
> > -static void sock_disable_timestamp(struct sock *sk, unsigned long flags) > +static void sock_disable_timestamp(struct sock *sk, unsigned long flag) > { > - if (sk->sk_flags & flags) { > - sk->sk_flags &= ~flags; > - if (sock_needs_netstamp(sk) && > -

Re: WARNING in __static_key_slow_dec

2018-05-20 Thread Willem de Bruijn
On Fri, May 18, 2018 at 4:30 PM, Willem de Bruijn wrote: > On Fri, May 18, 2018 at 4:03 AM, DaeRyong Jeong wrote: >> We report the crash: WARNING in __static_key_slow_dec >> >> This crash has been found in v4.8 using RaceFuzzer (a modified >> version of Syzkaller), which we describe more at the e

[PATCH net-next] mv88e6xxx: Fix uninitialized variable warning.

2018-05-20 Thread David Miller
In mv88e6xxx_probe(), ("np" or "pdata") might be an invariant but GCC can't see that, therefore: drivers/net/dsa/mv88e6xxx/chip.c: In function ‘mv88e6xxx_probe’: drivers/net/dsa/mv88e6xxx/chip.c:4420:13: warning: ‘compat_info’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Re: [PATCH net-next v2] net: dsa: b53: Extend platform data to include DSA ports

2018-05-20 Thread David Miller
From: Florian Fainelli Date: Sun, 20 May 2018 08:56:30 -0700 > The b53 driver already defines and internally uses platform data to let the > glue drivers specify parameters such as the chip id. What we were missing was > a way to tell the core DSA layer about the ports and their type. > > Place

Re: [PATCH net-next 0/3] Platform data support for mv88exxx

2018-05-20 Thread David Miller
From: Andrew Lunn Date: Sat, 19 May 2018 22:31:32 +0200 > There are a few Intel based platforms making use of the mv88exxx. > These don't easily have access to device tree in order to instantiate > the switch driver. These patches allow the use of platform data to > hold the configuration. Serie

Re: [PATCH 0/3] sh_eth: fix typos/grammar

2018-05-20 Thread David Miller
From: Sergei Shtylyov Date: Sat, 19 May 2018 23:57:50 +0300 > Here's a set of 3 patches against DaveM's 'net-next.git' repo plus the > R8A77980 > support patches posted earlier. They fix the comments typos/grammar and > another > typo in the EESR bit... Series applied.

Re: [PATCH net-next 0/9] Misc. bug fixes and cleanup for HNS3 driver

2018-05-20 Thread David Miller
From: Salil Mehta Date: Sat, 19 May 2018 16:53:14 +0100 > This patch-set presents miscellaneous bug fixes and cleanups found > during internal review, system testing and cleanup. Series applied, thank you.

Re: [PATCH v2] packet: track ring entry use using a shadow ring to prevent RX ring overrun

2018-05-20 Thread Willem de Bruijn
On Sat, May 19, 2018 at 8:07 AM, Jon Rosen wrote: > Fix PACKET_RX_RING bug for versions TPACKET_V1 and TPACKET_V2 which > casues the ring to get corrupted by allowing multiple kernel threads > to claim ownership of the same ring entry. Track ownership in a shadow > ring structure to prevent other

Re: [PATCH 13/14] net: sched: use unique idr insert function in unlocked actions

2018-05-20 Thread Marcelo Ricardo Leitner
On Mon, May 21, 2018 at 12:40:54AM +0300, Or Gerlitz wrote: > On Mon, May 21, 2018 at 12:33 AM, Marcelo Ricardo Leitner > wrote: > > On Mon, May 21, 2018 at 12:13:06AM +0300, Or Gerlitz wrote: > >> On Sun, May 20, 2018 at 1:20 AM, Marcelo Ricardo Leitner > >> wrote: > >> > On Mon, May 14, 2018 at

Re: [PATCH net-next] r8169: fix network error on resume from suspend

2018-05-20 Thread David Miller
From: Heiner Kallweit Date: Sat, 19 May 2018 10:29:33 +0200 > This commit removed calls to rtl_set_rx_mode(). This is ok for the > standard path if the link is brought up, however it breaks system > resume from suspend. Link comes up but no network traffic. > > Meanwhile common code from rtl_hw_

Re: [PATCH v2] isdn: eicon: fix a missing-check bug

2018-05-20 Thread David Miller
From: Wenwen Wang Date: Fri, 18 May 2018 16:33:47 -0500 > In divasmain.c, the function divas_write() firstly invokes the function > diva_xdi_open_adapter() to open the adapter that matches with the adapter > number provided by the user, and then invokes the function diva_xdi_write() > to perform

Re: [PATCHv2 net-next] erspan: set bso bit based on mirrored packet's len

2018-05-20 Thread David Miller
From: William Tu Date: Fri, 18 May 2018 19:41:01 -0700 > Before the patch, the erspan BSO bit (Bad/Short/Oversized) is not > handled. BSO has 4 possible values: > 00 --> Good frame with no error, or unknown integrity > 11 --> Payload is a Bad Frame with CRC or Alignment Error > 01 --> Payl

Re: pull request: bluetooth-next 2018-05-18

2018-05-20 Thread David Miller
From: Johan Hedberg Date: Fri, 18 May 2018 22:51:05 +0300 > Here's the first bluetooth-next pull request for the 4.18 kernel: > > - Refactoring of the btbcm driver > - New USB IDs for QCA_ROME and LiteOn controllers > - Buffer overflow fix if the controller sends invalid advertising data leng

Re: [net-next] Revert "ixgbe: release lock for the duration of ixgbe_suspend_close()"

2018-05-20 Thread David Miller
From: Jeff Kirsher Date: Fri, 18 May 2018 11:58:30 -0700 > This reverts commit 6710f970d9979d8f03f6e292bb729b2ee1526d0e. > > Gotta love when developers have offline discussions, thinking everyone > is reading their responses/dialog. > > The change had the potential for a number of race conditio

Re: [PATCH v2] net: qcom/emac: Allocate buffers from local node

2018-05-20 Thread David Miller
From: Hemanth Puranik Date: Fri, 18 May 2018 08:59:29 +0530 > Currently we use non-NUMA aware allocation for TPD and RRD buffers, > this patch modifies to use NUMA friendly allocation. > > Signed-off-by: Hemanth Puranik > --- > Change since v1: > - Addressed comments related to ordering Applie

Re: [PATCH 13/14] net: sched: use unique idr insert function in unlocked actions

2018-05-20 Thread Or Gerlitz
On Mon, May 21, 2018 at 12:33 AM, Marcelo Ricardo Leitner wrote: > On Mon, May 21, 2018 at 12:13:06AM +0300, Or Gerlitz wrote: >> On Sun, May 20, 2018 at 1:20 AM, Marcelo Ricardo Leitner >> wrote: >> > On Mon, May 14, 2018 at 05:27:14PM +0300, Vlad Buslov wrote: >> >> Substitute calls to action i

Re: [PATCH 13/14] net: sched: use unique idr insert function in unlocked actions

2018-05-20 Thread Marcelo Ricardo Leitner
On Mon, May 21, 2018 at 12:13:06AM +0300, Or Gerlitz wrote: > On Sun, May 20, 2018 at 1:20 AM, Marcelo Ricardo Leitner > wrote: > > On Mon, May 14, 2018 at 05:27:14PM +0300, Vlad Buslov wrote: > >> Substitute calls to action insert function with calls to action insert > >> unique function that war

Re: [PATCH 13/14] net: sched: use unique idr insert function in unlocked actions

2018-05-20 Thread Or Gerlitz
On Sun, May 20, 2018 at 1:20 AM, Marcelo Ricardo Leitner wrote: > On Mon, May 14, 2018 at 05:27:14PM +0300, Vlad Buslov wrote: >> Substitute calls to action insert function with calls to action insert >> unique function that warns if insertion overwrites index in idr. > > I know this patch may be

Re: [PATCH net-next] sctp: add support for SCTP_REUSE_PORT sockopt

2018-05-20 Thread Tom Herbert
On Sat, May 19, 2018 at 12:44 AM, Xin Long wrote: > This feature is actually already supported by sk->sk_reuse which can be > set by SO_REUSEADDR. But it's not working exactly as RFC6458 demands in > section 8.1.27, like: > > - This option only supports one-to-one style SCTP sockets > - This s

Re: [PATCH 09/14] net: sched: don't release reference on action overwrite

2018-05-20 Thread Vlad Buslov
On Sat 19 May 2018 at 21:52, Marcelo Ricardo Leitner wrote: > On Mon, May 14, 2018 at 05:27:10PM +0300, Vlad Buslov wrote: >> Return from action init function with reference to action taken, >> even when overwriting existing action. > > Isn't this patch necessary before patch 7, to not break thin

Re: [PATCH 10/32] aio: implement IOCB_CMD_POLL

2018-05-20 Thread Christoph Hellwig
On Sun, May 20, 2018 at 08:33:39AM +0100, Al Viro wrote: > > ... get buggered on attempt to dereference a pointer fetched from freed and > > reused object. > > FWIW, how painful would it be to pull the following trick: > * insert into wait queue under ->ctx_lock > * have wakeup do sche

[PATCH 2/2] net: fec: Add a SPDX identifier

2018-05-20 Thread Fabio Estevam
From: Fabio Estevam Currently there is no license information in the header of this file. The MODULE_LICENSE field contains ("GPL"), which means GNU Public License v2 or later, so add a corresponding SPDX license identifier. Signed-off-by: Fabio Estevam --- drivers/net/ethernet/freescale/fec

[PATCH 1/2] net: fec: ptp: Switch to SPDX identifier

2018-05-20 Thread Fabio Estevam
From: Fabio Estevam Adopt the SPDX license identifier headers to ease license compliance management. Signed-off-by: Fabio Estevam --- drivers/net/ethernet/freescale/fec_ptp.c | 14 +- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/drivers/net/ethernet/freescale/fec_p

Re: [RFC V4 PATCH 0/8] Packed ring layout for vhost

2018-05-20 Thread Wei Xu
On Wed, May 16, 2018 at 08:32:13PM +0800, Jason Wang wrote: > Hi all: > > This RFC implement packed ring layout. The code were tested with > Tiwei's RFC V3 ahttps://lkml.org/lkml/2018/4/25/34. Some fixups and > tweaks were needed on top of Tiwei's code to make it run for event > index. Could you

[PATCH net-next v2] net: dsa: b53: Extend platform data to include DSA ports

2018-05-20 Thread Florian Fainelli
The b53 driver already defines and internally uses platform data to let the glue drivers specify parameters such as the chip id. What we were missing was a way to tell the core DSA layer about the ports and their type. Place a dsa_chip_data structure at the beginning of b53_platform_data for dsa_

Re: [PATCH net-next] net: dsa: b53: Extend platform data to include DSA ports

2018-05-20 Thread Florian Fainelli
On 05/20/2018 07:06 AM, Andrew Lunn wrote: > On Sat, May 19, 2018 at 08:14:41PM -0700, Florian Fainelli wrote: >> On May 19, 2018 6:42:50 PM PDT, Andrew Lunn wrote: >>> On Sat, May 19, 2018 at 03:55:10PM -0700, Florian Fainelli wrote: Provide a means for !OF platforms to supply their DSA pl

Re: [PATCH net-next] net: dsa: b53: Extend platform data to include DSA ports

2018-05-20 Thread Andrew Lunn
On Sat, May 19, 2018 at 08:14:41PM -0700, Florian Fainelli wrote: > On May 19, 2018 6:42:50 PM PDT, Andrew Lunn wrote: > >On Sat, May 19, 2018 at 03:55:10PM -0700, Florian Fainelli wrote: > >> Provide a means for !OF platforms to supply their DSA platform data > >> configuration using the dsa_plat

[PATCH bpf-next v7 2/6] ipv6: sr: export function lookup_nexthop

2018-05-20 Thread Mathieu Xhonneux
The function lookup_nexthop is essential to implement most of the seg6local actions. As we want to provide a BPF helper allowing to apply some of these actions on the packet being processed, the helper should be able to call this function, hence the need to make it public. Moreover, if one argumen

[PATCH bpf-next v7 5/6] ipv6: sr: Add seg6local action End.BPF

2018-05-20 Thread Mathieu Xhonneux
This patch adds the End.BPF action to the LWT seg6local infrastructure. This action works like any other seg6local End action, meaning that an IPv6 header with SRH is needed, whose DA has to be equal to the SID of the action. It will also advance the SRH to the next segment, the BPF program does no

[PATCH bpf-next v7 6/6] selftests/bpf: test for seg6local End.BPF action

2018-05-20 Thread Mathieu Xhonneux
Add a new test for the seg6local End.BPF action. The following helpers are also tested: - bpf_lwt_push_encap within the LWT BPF IN hook - bpf_lwt_seg6_action - bpf_lwt_seg6_adjust_srh - bpf_lwt_seg6_store_bytes A chain of End.BPF actions is built. The SRH is injected through a LWT BPF IN hook bef

[PATCH bpf-next v7 4/6] bpf: Split lwt inout verifier structures

2018-05-20 Thread Mathieu Xhonneux
The new bpf_lwt_push_encap helper should only be accessible within the LWT BPF IN hook, and not the OUT one, as this may lead to a skb under panic. At the moment, both LWT BPF IN and OUT share the same list of helpers, whose calls are authorized by the verifier. This patch separates the verifier o

[PATCH bpf-next v7 3/6] bpf: Add IPv6 Segment Routing helpers

2018-05-20 Thread Mathieu Xhonneux
The BPF seg6local hook should be powerful enough to enable users to implement most of the use-cases one could think of. After some thinking, we figured out that the following actions should be possible on a SRv6 packet, requiring 3 specific helpers : - bpf_lwt_seg6_store_bytes: Modify non-sensi

[PATCH bpf-next v7 1/6] ipv6: sr: make seg6.h includable without IPv6

2018-05-20 Thread Mathieu Xhonneux
include/net/seg6.h cannot be included in a source file if CONFIG_IPV6 is not enabled: include/net/seg6.h: In function 'seg6_pernet': >> include/net/seg6.h:52:14: error: 'struct net' has no member named 'ipv6'; did you mean 'ipv4'? return net->ipv6.seg

[PATCH bpf-next v7 0/6] ipv6: sr: introduce seg6local End.BPF action

2018-05-20 Thread Mathieu Xhonneux
As of Linux 4.14, it is possible to define advanced local processing for IPv6 packets with a Segment Routing Header through the seg6local LWT infrastructure. This LWT implements the network programming principles defined in the IETF “SRv6 Network Programming” draft. The implemented operations are

Re: [PATCH bpf-next v6 5/6] ipv6: sr: Add seg6local action End.BPF

2018-05-20 Thread Mathieu Xhonneux
2018-05-18 21:24 GMT+01:00 Daniel Borkmann : >> +#define MAX_PROG_NAME 256 >> +static const struct nla_policy bpf_prog_policy[LWT_BPF_PROG_MAX + 1] = { >> + [LWT_BPF_PROG_FD] = { .type = NLA_U32, }, > > From UAPI point of view, I wouldn't name it LWT_BPF_PROG_FD but rather > something like >

Re: [PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-20 Thread kbuild test robot
Hi David, I love your patch! Yet something to improve: [auto build test ERROR on bpf-next/master] url: https://github.com/0day-ci/linux/commits/David-Ahern/bpf-Add-MTU-check-to-fib-lookup-helper/20180520-103417 base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master

[PATCH] bpf: fix mem leak in error path of lwt bpf setup

2018-05-20 Thread Mathieu Xhonneux
In bpf_parse_prog, if bpf_prog_get_type fails, the function is immediately terminated without freeing the previously allocated prog->name. This patch adds a kfree before the return. Signed-off-by: Mathieu Xhonneux --- net/core/lwt_bpf.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) d

[PATCH net-next v1] netfilter: provide input interface for route lookup for rpfilter

2018-05-20 Thread Vincent Bernat
In commit 47b7e7f82802, this bit was removed at the same time the RT6_LOOKUP_F_IFACE flag was removed. However, it is needed when link-local addresses are used, which is a very common case: when packets are routed, neighbor solicitations are done using link-local addresses. For example, the followi

Re: [PATCH 06/14] net: sched: implement reference counted action release

2018-05-20 Thread Vlad Buslov
On Sun 20 May 2018 at 06:22, Jiri Pirko wrote: > Sat, May 19, 2018 at 11:43:27PM CEST, marcelo.leit...@gmail.com wrote: >>On Mon, May 14, 2018 at 05:27:07PM +0300, Vlad Buslov wrote: >>... >>> @@ -1052,6 +1088,36 @@ static int tca_action_flush(struct net *net, struct >>> nlattr *nla, >>> ret

Re: [PATCH 02/14] net: sched: change type of reference and bind counters

2018-05-20 Thread Vlad Buslov
On Sat 19 May 2018 at 21:04, Marcelo Ricardo Leitner wrote: > On Mon, May 14, 2018 at 05:27:03PM +0300, Vlad Buslov wrote: >> Change type of action reference counter to refcount_t. >> >> Change type of action bind counter to atomic_t. >> This type is used to allow decrementing bind counter witho

[PATCH net] sctp: fix the issue that flags are ignored when using kernel_connect

2018-05-20 Thread Xin Long
Now sctp uses inet_dgram_connect as its proto_ops .connect, and the flags param can't be passed into its proto .connect where this flags is really needed. sctp works around it by getting flags from socket file in __sctp_connect. It works for connecting from userspace, as inherently the user sock h

Re: INFO: rcu detected stall in is_bpf_text_address

2018-05-20 Thread Xin Long
On Sat, May 19, 2018 at 11:57 PM, Eric Dumazet wrote: > SCTP experts, please take a look. > > On 05/19/2018 08:55 AM, syzbot wrote: >> Hello, >> >> syzbot found the following crash on: >> >> HEAD commit:73fcb1a370c7 Merge branch 'akpm' (patches from Andrew) >> git tree: upstream >> conso

[patch iproute2/net-next 2/2] devlink: introduce support for showing port number and split subport number

2018-05-20 Thread Jiri Pirko
From: Jiri Pirko Signed-off-by: Jiri Pirko --- devlink/devlink.c| 6 ++ include/uapi/linux/devlink.h | 2 ++ 2 files changed, 8 insertions(+) diff --git a/devlink/devlink.c b/devlink/devlink.c index df2c66dac1c7..b0ae17767dab 100644 --- a/devlink/devlink.c +++ b/devlink/devlink

[patch iproute2/net-next 1/2] devlink: introduce support for showing port flavours

2018-05-20 Thread Jiri Pirko
From: Jiri Pirko Signed-off-by: Jiri Pirko --- devlink/devlink.c| 20 include/uapi/linux/devlink.h | 12 2 files changed, 32 insertions(+) diff --git a/devlink/devlink.c b/devlink/devlink.c index fa33684cb20a..df2c66dac1c7 100644 --- a/devlink/devl

[patch iproute2/net-next 0/2] devlink: add port flavours/number/split_subport

2018-05-20 Thread Jiri Pirko
From: Jiri Pirko Jiri Pirko (2): devlink: introduce support for showing port flavours devlink: introduce support for showing port number and split subport number devlink/devlink.c| 26 ++ include/uapi/linux/devlink.h | 14 ++ 2 files chang

Wohltätigkeitsspende in Höhe von € 2.000.000,00

2018-05-20 Thread vicente . ortiz
Lieber Freund, Ich bin Herr Richard Wahl der Mega-Gewinner von $ 533M In Mega Millions Jackpot spende ich an 5 zufällige Personen, wenn Sie diese E-Mail erhalten, dann wurde Ihre E-Mail nach einem Spinball ausgewählt. Ich habe den größten Teil meines Vermögens auf eine Reihe von Wohltätigkeits

Re: [PATCH 10/32] aio: implement IOCB_CMD_POLL

2018-05-20 Thread Al Viro
On Sun, May 20, 2018 at 06:32:25AM +0100, Al Viro wrote: > > + spin_lock_irqsave(&ctx->ctx_lock, flags); > > + list_add_tail(&aiocb->ki_list, &ctx->delayed_cancel_reqs); > > + spin_unlock(&ctx->ctx_lock); > > ... and io_cancel(2) comes, finds it and inhume^Wcompletes it, leaving us > to...