[patch net-next 03/12] ipmr: Add FIB notification access functions

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Make the ipmr module register as a FIB notifier. To do that, implement both the ipmr_seq_read and ipmr_dump ops. The ipmr_seq_read op returns a sequence counter that is incremented on every notification related operation done by the ipmr. To implement that, add a sequence counte

[patch net-next 02/12] ipmr: Add reference count to MFC entries

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Next commits will introduce MFC notifications through the atomic fib_notification chain, thus allowing modules to be aware of MFC entries. Due to the fact that modules may need to hold a reference to an MFC entry, add reference count to MFC entries to prevent them from being fre

[patch net-next 10/12] mlxsw: spectrum_router: Add multicast routes notification handling functionality

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Add functionality for calling the multicast routing offloading logic upon MFC and VIF add and delete notifications. In addition, call the multicast routing upon RIF addition and deletion events. As the multicast routing offload logic may sleep, the actual calls are done in a def

[patch net-next 01/12] fib: notifier: Add VIF add and delete event types

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi In order for an interface to forward packets according to the kernel multicast routing table, it must be configured with a VIF index according to the mroute user API. The VIF index is then used to refer to that interface in the mroute user API, for example, to set the iif and oif

[patch net-next 05/12] net: ipmr: Add MFC offload indication

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Allow drivers, registered to the fib notification chain indicate whether a multicast MFC route is offloaded or not, similarly to unicast routes. The indication of whether a route is offloaded is done using the mfc_flags field on an mfc_cache struct, and the information is sent to

[patch net-next 04/12] ipmr: Send FIB notifications on MFC and VIF entries

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Use the newly introduced notification chain to send events upon VIF and MFC addition and deletion. The MFC notifications are sent only on resolved MFC entries, as unresolved cannot be offloaded. Signed-off-by: Yotam Gigi Reviewed-by: Ido Schimmel Signed-off-by: Jiri Pirko ---

[patch net-next 09/12] mlxsw: spectrum: router: Squash the default route table to main

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Currently, the mlxsw Spectrum driver offloads only either the RT_TABLE_MAIN FIB table or the VRF tables, so the RT_TABLE_LOCAL table is squashed to the RT_TABLE_MAIN table to allow local routes to be offloaded too. By default, multicast MFC routes which are not assigned to any u

[patch net-next 00/12] mlxsw: Add support for offloading IPv4 multicast routes

2017-09-20 Thread Jiri Pirko
From: Jiri Pirko Yotam says: This patchset introduces offloading of the kernel IPv4 multicast router logic in the Spectrum driver. Patches 1-4 update ipmr to use the FIB notification chain for both MFC and VIF notifications, and patches 7-11 update the Spectrum driver to register to these notif

[patch net-next 11/12] mlxsw: spectrum: Notify multicast router on RIF MTU changes

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Due to the fact that multicast routes hold the minimum MTU of all the egress RIFs and trap packets that don't meet it, notify the mulitcast router code on RIF MTU changes. Signed-off-by: Yotam Gigi Reviewed-by: Ido Schimmel Signed-off-by: Jiri Pirko --- drivers/net/ethernet/

[patch net-next 12/12] mlxsw: spectrum: router: Don't ignore IPMR notifications

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Make the Spectrum router logic not ignore the RTNL_FAMILY_IPMR FIB notifications. Past commits added the IPMR VIF and MFC add/del notifications via the fib_notifier chain. In addition, a code for handling these notifications in the Spectrum router logic was added. Make the Spect

[patch net-next 08/12] mlxsw: spectrum: Add the multicast routing hardware logic

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Implement the multicast routing hardware API introduced in previous patch for the specific spectrum hardware. The spectrum hardware multicast routes are written using the RMFT2 register and point to an ACL flexible action set. The actions used for multicast routes are: - Counte

[patch net-next 06/12] net: mroute: Check if rule is a default rule

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi When the ipmr starts, it adds one default FIB rule that matches all packets and sends them to the DEFAULT (multicast) FIB table. A more complex rule can be added by user to specify that for a specific interface, a packet should be look up at either an arbitrary table or according

[patch net-next 07/12] mlxsw: spectrum: Add the multicast routing offloading logic

2017-09-20 Thread Jiri Pirko
From: Yotam Gigi Add the multicast router offloading logic, which is in charge of handling the VIF and MFC notifications and translating it to the hardware logic API. The offloading logic has to overcome several obstacles in order to safely comply with the kernel multicast router user API: - It

[PATCH 3/3] net: stmmac: Cocci spatch "of_table"

2017-09-20 Thread Thomas Meyer
Make sure (of/i2c/platform)_device_id tables are NULL terminated. Found by coccinelle spatch "misc/of_table.cocci" Signed-off-by: Thomas Meyer --- diff -u -p a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c --- a/drivers/net/etherne

[PATCH 2/6] e100: Cocci spatch "pool_zalloc-simple"

2017-09-20 Thread Thomas Meyer
Use *_pool_zalloc rather than *_pool_alloc followed by memset with 0. Found by coccinelle spatch "api/alloc/pool_zalloc-simple.cocci" Signed-off-by: Thomas Meyer --- diff -u -p a/drivers/net/ethernet/intel/e100.c b/drivers/net/ethernet/intel/e100.c --- a/drivers/net/ethernet/intel/e100.c +++ b/

Re: [PATCH net] bpf: one perf event close won't free bpf program attached by another perf event

2017-09-20 Thread Yonghong Song
On 9/20/17 10:17 PM, Yonghong Song wrote: On 9/20/17 6:41 PM, Steven Rostedt wrote: On Mon, 18 Sep 2017 16:38:36 -0700 Yonghong Song wrote: This patch fixes a bug exhibited by the following scenario:    1. fd1 = perf_event_open with attr.config = ID1    2. attach bpf program prog1 to fd1

Re: [PATCH net] bpf: one perf event close won't free bpf program attached by another perf event

2017-09-20 Thread Yonghong Song
On 9/20/17 6:41 PM, Steven Rostedt wrote: On Mon, 18 Sep 2017 16:38:36 -0700 Yonghong Song wrote: This patch fixes a bug exhibited by the following scenario: 1. fd1 = perf_event_open with attr.config = ID1 2. attach bpf program prog1 to fd1 3. fd2 = perf_event_open with attr.config

Re:Re: [PATCH net-next ] net: Remove useless function skb_header_release

2017-09-20 Thread Gao Feng
>On Thu, 2017-09-21 at 12:39 +0800, gfree.w...@vip.163.com wrote: >> From: Gao Feng >> >> There is no one which would invokes the function skb_header_release. >> So just remove it now. > >This in incomplete. >There are other references to this function. > >$ git grep -w skb_header_release >driver

Re: [PATCH net-next] net: dsa: use dedicated CPU port

2017-09-20 Thread Florian Fainelli
On 09/20/2017 09:28 AM, Vivien Didelot wrote: > Each port in DSA has its own dedicated CPU port currently available in > its parent switch's ds->ports[port].cpu_dp. Use it instead of getting > the unique tree CPU port, which will be deprecated soon. > > Signed-off-by: Vivien Didelot > --- > dr

Re: [PATCH net-next] net: dsa: add port fdb dump

2017-09-20 Thread Florian Fainelli
On 09/20/2017 04:32 PM, Vivien Didelot wrote: > Dumping a DSA port's FDB entries is not specific to a DSA slave, so add > a dsa_port_fdb_dump function, similarly to dsa_port_fdb_add and > dsa_port_fdb_del. > > Signed-off-by: Vivien Didelot Reviewed-by: Florian Fainelli -- Florian

Re: [PATCH net-next] net: dsa: better scoping of slave functions

2017-09-20 Thread Florian Fainelli
On 09/20/2017 04:31 PM, Vivien Didelot wrote: > A few DSA slave functions take a dsa_slave_priv pointer as first > argument, whereas the scope of the slave.c functions is the slave > net_device structure. Fix this and rename dsa_netpoll_send_skb to > dsa_slave_netpoll_send_skb. > > Signed-off-by

Re: [PATCH net-next 12/14] gtp: Configuration for zero UDP checksum

2017-09-20 Thread Harald Welte
Hi Tom, On Wed, Sep 20, 2017 at 11:09:29AM -0700, Tom Herbert wrote: > On Mon, Sep 18, 2017 at 9:24 PM, David Miller wrote: > > From: Tom Herbert > >> Add configuration to control use of zero checksums on transmit for both > >> IPv4 and IPv6, and control over accepting zero IPv6 checksums on > >

Re: [PATCH net-next ] net: Remove useless function skb_header_release

2017-09-20 Thread Joe Perches
On Thu, 2017-09-21 at 12:39 +0800, gfree.w...@vip.163.com wrote: > From: Gao Feng > > There is no one which would invokes the function skb_header_release. > So just remove it now. This in incomplete. There are other references to this function. $ git grep -w skb_header_release drivers/net/usb/a

[PATCH net-next ] net: Remove useless function skb_header_release

2017-09-20 Thread gfree . wind
From: Gao Feng There is no one which would invokes the function skb_header_release. So just remove it now. Signed-off-by: Gao Feng --- include/linux/skbuff.h | 16 1 file changed, 16 deletions(-) diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 72299ef..ce63

Re: [PATCH net-next 0/5] net: introduce noref sk

2017-09-20 Thread David Miller
From: Paolo Abeni Date: Wed, 20 Sep 2017 18:54:00 +0200 > This series introduce the infrastructure to store inside the skb a socket > pointer without carrying a refcount to the socket. > > Such infrastructure is then used in the network receive path - and > specifically the early demux operation

Re: [PATCH iproute2 v3] ip: ip_print: if no json obj is initialized default fp is stdout

2017-09-20 Thread Stephen Hemminger
On Wed, 20 Sep 2017 18:23:56 -0700 Julien Fortin wrote: > From: Julien Fortin > > The ip monitor didn't call `new_json_obj` (even for in non json context), > so the static FILE* _fp variable wasn't initialized, thus raising a > SIGSEGV in ipaddress.c. This patch should fix this issue for good,

Re: [Patch v3 1/3] ipv4: Namespaceify tcp_fastopen knob

2017-09-20 Thread 严海双
> On 2017年9月21日, at 上午5:22, David Miller wrote: > > From: Haishuang Yan > Date: Tue, 19 Sep 2017 17:38:14 +0800 > >> -if ((sysctl_tcp_fastopen & TFO_SERVER_WO_SOCKOPT1) && >> -(sysctl_tcp_fastopen & TFO_SERVER_ENABLE) && >> +tcp_fastopen = sock_net(sk)

Re: [REGRESSION] Warning in tcp_fastretrans_alert() of net/ipv4/tcp_input.c

2017-09-20 Thread Roman Gushchin
> Hello. > > Since, IIRC, v4.11, there is some regression in TCP stack resulting in the > warning shown below. Most of the time it is harmless, but rarely it just > causes either freeze or (I believe, this is related too) panic in > tcp_sacktag_walk() (because sk_buff passed to this function is

Re: [PATCH net] bpf: one perf event close won't free bpf program attached by another perf event

2017-09-20 Thread Steven Rostedt
On Mon, 18 Sep 2017 16:38:36 -0700 Yonghong Song wrote: > This patch fixes a bug exhibited by the following scenario: > 1. fd1 = perf_event_open with attr.config = ID1 > 2. attach bpf program prog1 to fd1 > 3. fd2 = perf_event_open with attr.config = ID1 > > 4. user program closes f

[PATCH iproute2 v3] ip: ip_print: if no json obj is initialized default fp is stdout

2017-09-20 Thread Julien Fortin
From: Julien Fortin The ip monitor didn't call `new_json_obj` (even for in non json context), so the static FILE* _fp variable wasn't initialized, thus raising a SIGSEGV in ipaddress.c. This patch should fix this issue for good, new paths won't have to call `new_json_obj`. $ ip -t mon label link

Re: [PATCH iproute2 v2] ip: ip_print: if no json obj is initialized default fp is stdout

2017-09-20 Thread Julien Fortin
looks like git-mail ate all the traceback lines starting with #, will re-submit v3 On Wed, Sep 20, 2017 at 6:16 PM, Julien Fortin wrote: > From: Julien Fortin > > The ip monitor didn't call `new_json_obj` (even for in non json context), > so the static FILE* _fp variable wasn't initialized, thus

Re: Latest net-next from GIT panic

2017-09-20 Thread Eric Dumazet
On Wed, 2017-09-20 at 18:09 -0700, Wei Wang wrote: > > Thanks very much Pawel for the feedback. > > > > I was looking into the code (specifically IPv4 part) and found that in > > free_fib_info_rcu(), we call free_nh_exceptions() without holding the > > fnhe_lock. I am wondering if that could cause

[PATCH iproute2 v2] ip: ip_print: if no json obj is initialized default fp is stdout

2017-09-20 Thread Julien Fortin
From: Julien Fortin The ip monitor didn't call `new_json_obj` (even for in non json context), so the static FILE* _fp variable wasn't initialized, thus raising a SIGSEGV in ipaddress.c. This patch should fix this issue for good, new paths won't have to call `new_json_obj`. $ ip -t mon label link

Re: [PATCH iproute2/net-next] tc: flower: support for matching MPLS labels

2017-09-20 Thread Stephen Hemminger
On Tue, 12 Sep 2017 16:06:15 +0200 Simon Horman wrote: > From: Benjamin LaHaise > > This patch adds support to the iproute2 tc filter command for matching MPLS > labels in the flower classifier. The ability to match the Time To Live, > Bottom Of Stack, Traffic Control and Label fields are adde

Re: Latest net-next from GIT panic

2017-09-20 Thread Wei Wang
> Thanks very much Pawel for the feedback. > > I was looking into the code (specifically IPv4 part) and found that in > free_fib_info_rcu(), we call free_nh_exceptions() without holding the > fnhe_lock. I am wondering if that could cause some race condition on > fnhe->fnhe_rth_input/output so a dou

Re: [PATCH iproute2 v2] ip: ipaddress: fix missing space after prefixlen

2017-09-20 Thread Stephen Hemminger
On Wed, 20 Sep 2017 13:26:51 -0700 Julien Fortin wrote: > From: Julien Fortin > > Fixes: d0e720111aad2 ("ip: ipaddress.c: add support for json output") > Reported-by: Sabrina Dubroca > Reviewed-by: Roopa Prabhu > Signed-off-by: Julien Fortin Applied, thanks for spotting this.

Re: [patch net-next 00/16] mlxsw: Multicast flood update

2017-09-20 Thread David Miller
From: Jiri Pirko Date: Wed, 20 Sep 2017 16:15:00 +0200 > From: Jiri Pirko > > Nogah says: > > Currently, there are four erroneous flows in MC flood: > 1. When MC is disabled it affects only the flood table for unregistered >MC packets, but packets that match an entry in the MDB are unaffec

Re: [PATCH 1/1] ip: ip_print: if no json obj is initialized default fp is stdout

2017-09-20 Thread Stephen Hemminger
On Wed, 20 Sep 2017 15:19:14 -0700 Julien Fortin wrote: > From: Julien Fortin > > The ip monitor didn't call `new_json_obj` (even for in non json context), > so the static FILE* _fp variable wasn't initialized, thus raising a > SIGSEGV in ipaddress.c. This patch should fix this issue for good,

Re: [PATCH iproute2] tc: fix typo in tc-tcindex man page

2017-09-20 Thread Stephen Hemminger
On Thu, 14 Sep 2017 17:00:46 +0200 Davide Caratti wrote: > fix mis-typed 'pass_on' keyword. > > Signed-off-by: Davide Caratti Applied, thanks Davide

Re: [iproute2 PATCH] tc/mirred: Clean up white-space noise

2017-09-20 Thread Stephen Hemminger
On Wed, 13 Sep 2017 03:05:13 -0700 Amritha Nambiar wrote: > Signed-off-by: Amritha Nambiar > --- > include/linux/tc_act/tc_mirred.h |6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/linux/tc_act/tc_mirred.h > b/include/linux/tc_act/tc_mirred.h > index 3

Re: [PATCH iproute2] Add information about COLORFGBG to ip.8 man page

2017-09-20 Thread Stephen Hemminger
On Fri, 15 Sep 2017 22:04:16 +0200 Roland Hopferwieser wrote: > diff --git a/man/man8/ip.8 b/man/man8/ip.8 > index ae018fdf..2a27a56e 100644 > --- a/man/man8/ip.8 > +++ b/man/man8/ip.8 > @@ -187,7 +187,8 @@ executes specified command over all objects, it > depends if command supports this > >

Re: [PATCH net-next 09/14] gtp: Allow configuring GTP interface as standalone

2017-09-20 Thread Tom Herbert
On Wed, Sep 20, 2017 at 5:13 PM, Harald Welte wrote: > Hi Tom, > > On Wed, Sep 20, 2017 at 09:24:07AM -0700, Tom Herbert wrote: >> On Wed, Sep 20, 2017 at 9:07 AM, Andreas Schultz wrote: >> > GTP isn't special, I just don't like to have testing only features in there >> > when the same goal can b

Re: [iproute PATCH v2] ipaddress: Fix segfault in 'addr showdump'

2017-09-20 Thread Stephen Hemminger
On Wed, 13 Sep 2017 11:20:34 +0200 Phil Sutter wrote: > Obviously, 'addr showdump' feature wasn't adjusted to json output > support. As a consequence, calls to print_string() in print_addrinfo() > tried to dereference a NULL FILE pointer. > > Cc: Julien Fortin > Fixes: d0e720111aad2 ("ip: ipadd

Re: [PATCH iproute2] tc: fq: support low_rate_threshold attribute

2017-09-20 Thread Stephen Hemminger
On Fri, 08 Sep 2017 14:12:59 -0700 Eric Dumazet wrote: > From: Eric Dumazet > > TCA_FQ_LOW_RATE_THRESHOLD sch_fq attribute was added in linux-4.9 > > Tested: > > lpaa5:/tmp# tc -qd add dev eth1 root fq > lpaa5:/tmp# tc -s qd sh dev eth1 > qdisc fq 8003: root refcnt 5 limit 1p flow_limit 1

Re: [RFC PATCH] can: m_can: Support higher speed CAN-FD bitrates

2017-09-20 Thread Franklin S Cooper Jr
On 09/20/2017 04:37 PM, Mario Hüttel wrote: > > > On 09/20/2017 10:19 PM, Franklin S Cooper Jr wrote: >> Hi Wenyou, >> >> On 09/17/2017 10:47 PM, Yang, Wenyou wrote: >>> >>> On 2017/9/14 13:06, Sekhar Nori wrote: On Thursday 14 September 2017 03:28 AM, Franklin S Cooper Jr wrote: > On

Re: [v2,3/3] can: m_can: Add PM Runtime

2017-09-20 Thread Franklin S Cooper Jr
On 08/24/2017 03:30 AM, Sekhar Nori wrote: > + OMAP mailing list > > On Tuesday 25 July 2017 04:21 AM, Franklin Cooper wrote: >> Add support for PM Runtime which is the new way to handle managing clocks. >> However, to avoid breaking SoCs not using PM_RUNTIME leave the old clk >> management appro

Re: [v2,1/3] can: m_can: Make hclk optional

2017-09-20 Thread Franklin S Cooper Jr
On 08/24/2017 03:00 AM, Sekhar Nori wrote: > + some OMAP folks and Linux OMAP list > > On Tuesday 25 July 2017 04:21 AM, Franklin Cooper wrote: >> Hclk is the MCAN's interface clock. However, for OMAP based devices such as >> DRA7 SoC family the interface clock is handled by hwmod. Therefore, th

Re: [PATCH net-next 08/14] gtp: Support encpasulating over IPv6

2017-09-20 Thread Tom Herbert
On Wed, Sep 20, 2017 at 5:04 PM, Harald Welte wrote: > Hi Tom, > > On Wed, Sep 20, 2017 at 01:40:54PM -0700, Tom Herbert wrote: >> On Wed, Sep 20, 2017 at 12:45 PM, David Miller wrote: >> > There is a socket associated with the tunnel to do the encapsulation >> > and it has an address family, rig

Re: [PATCH net-next 08/14] gtp: Support encpasulating over IPv6

2017-09-20 Thread Harald Welte
Hi Tom, On Wed, Sep 20, 2017 at 01:40:54PM -0700, Tom Herbert wrote: > On Wed, Sep 20, 2017 at 12:45 PM, David Miller wrote: > > There is a socket associated with the tunnel to do the encapsulation > > and it has an address family, right? > > If fd's are set from userspace for the sockets then w

Re: [PATCH net-next 09/14] gtp: Allow configuring GTP interface as standalone

2017-09-20 Thread Harald Welte
Hi Tom, On Wed, Sep 20, 2017 at 09:24:07AM -0700, Tom Herbert wrote: > On Wed, Sep 20, 2017 at 9:07 AM, Andreas Schultz wrote: > > GTP isn't special, I just don't like to have testing only features in there > > when the same goal can be reached without having to add extra stuff. Adding > > code t

[PATCH v2 20/31] net/core: Collapse redundant sk_timer callback data assignments

2017-09-20 Thread Kees Cook
The core sk_timer initializer can provide the common .data assignment instead of it being set separately in users. Cc: "David S. Miller" Cc: Ralf Baechle Cc: Andrew Hendry Cc: Eric Dumazet Cc: Paolo Abeni Cc: David Howells Cc: Colin Ian King Cc: Ingo Molnar Cc: linzhang Cc: netdev@vger.ke

[PATCH v2 30/31] appletalk: Remove unneeded synchronization

2017-09-20 Thread Kees Cook
The use of del_timer_sync() will make sure a timer is not rescheduled. As such, there is no need to add external signals to kill timers. In preparation for switching the timer callback argument to the timer pointer, this drops the .data argument since it doesn't serve a meaningful purpose here. Cc

[PATCH v2 19/31] timer: Remove open-coded casts for .data and .function

2017-09-20 Thread Kees Cook
This standardizes the callback and data prototypes in several places that perform casting, in an effort to remove more open-coded .data and .function uses in favor of setup_timer(). Cc: Samuel Ortiz Cc: Tyrel Datwyler Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: "Jam

[PATCH net-next] net: dsa: add port fdb dump

2017-09-20 Thread Vivien Didelot
Dumping a DSA port's FDB entries is not specific to a DSA slave, so add a dsa_port_fdb_dump function, similarly to dsa_port_fdb_add and dsa_port_fdb_del. Signed-off-by: Vivien Didelot --- net/dsa/dsa_priv.h | 1 + net/dsa/port.c | 11 +++ net/dsa/slave.c| 9 ++--- 3 files c

[PATCH net-next] net: dsa: better scoping of slave functions

2017-09-20 Thread Vivien Didelot
A few DSA slave functions take a dsa_slave_priv pointer as first argument, whereas the scope of the slave.c functions is the slave net_device structure. Fix this and rename dsa_netpoll_send_skb to dsa_slave_netpoll_send_skb. Signed-off-by: Vivien Didelot --- net/dsa/slave.c | 29 +++-

Re: [PATCH v2 1/3] can: m_can: Make hclk optional

2017-09-20 Thread Franklin S Cooper Jr
On 09/20/2017 05:00 PM, Mario Hüttel wrote: >> Hclk is the MCAN's interface clock. However, for OMAP based devices such as >> DRA7 SoC family the interface clock is handled by hwmod. Therefore, this >> interface clock is managed by hwmod driver via pm_runtime_get and >> pm_runtime_put calls. Ther

[PATCH v2 13/31] timer: Remove meaningless .data/.function assignments

2017-09-20 Thread Kees Cook
Several timer users needlessly reset their .function/.data fields during their timer callback, but nothing else changes them. Some users do not use their .data field at all. Each instance is removed here. Cc: Krzysztof Halasa Cc: Aditya Shankar Cc: Ganesh Krishna Cc: Greg Kroah-Hartman Cc: Jen

[PATCH v2 25/31] net/atm/mpc: Use separate static data field with with static timer

2017-09-20 Thread Kees Cook
In preparation for changing the timer callback argument to the timer pointer, move to a separate static data variable. Cc: "David S. Miller" Cc: Andrew Morton Cc: Alexey Dobriyan Cc: "Reshetova, Elena" Cc: netdev@vger.kernel.org Signed-off-by: Kees Cook --- net/atm/mpc.c | 10 +- 1 f

Re: [PATCH net-next 00/10] net/smc: updates 2017-09-20

2017-09-20 Thread David Miller
From: Ursula Braun Date: Wed, 20 Sep 2017 13:58:03 +0200 > here is a collection of small smc-patches built for net-next > improving the smc code in different areas. There are bug fixes in here, which should be targetted at 'net'.

[PATCH v2 31/31] timer: Switch to testing for .function instead of .data

2017-09-20 Thread Kees Cook
This is part of the work to support switching all struct timer_list callbacks to get the timer pointer as the argument (like other modern callback interfaces in the kernel) instead of from the .data field. This patch is one of several steps in removing open-coded users of the .data field: In sever

Re: [PATCH] net_sched: always reset qdisc backlog in qdisc_reset()

2017-09-20 Thread Cong Wang
On Wed, Sep 20, 2017 at 5:45 AM, Konstantin Khlebnikov wrote: > SKB stored in qdisc->gso_skb also counted into backlog. > > Some qdiscs don't reset backlog to zero in ->reset(), > for example sfq just dequeue and free all queued skb. > > Signed-off-by: Konstantin Khlebnikov > Fixes: 2f5fb43f

Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode slab cache

2017-09-20 Thread Christoph Hellwig
On Wed, Sep 20, 2017 at 02:21:45PM -0700, Kees Cook wrote: > This is why I included several other lists on the full CC (am I > unlucky enough to have you not subscribed to any of them?). Adding a > CC for everyone can result in a huge CC list, especially for the > forth-coming 300-patch timer_list

Re: [PATCH net-next v2] bridge: also trigger RTM_NEWLINK when interface is released from bridge

2017-09-20 Thread Stephen Hemminger
On Wed, 20 Sep 2017 15:57:16 -0600 David Ahern wrote: > On 9/20/17 3:09 PM, David Miller wrote: > > From: Vincent Bernat > > Date: Sat, 16 Sep 2017 16:18:33 +0200 > > > > David, I am CC:'ing you because you've done work in this area over the > > past year. I'm applying this patch, it's been si

Re: [PATCH net 0/9] TM related bugfixes for the HNS3 Ethernet Driver

2017-09-20 Thread David Miller
From: Yunsheng Lin Date: Wed, 20 Sep 2017 18:52:49 +0800 > This patch set contains a few bugfixes related to hclge_tm module. Series applied, thank you.

Re: [PATCH 0/2] Netfilter fixes for net

2017-09-20 Thread David Miller
From: Pablo Neira Ayuso Date: Wed, 20 Sep 2017 12:49:01 +0200 > The following patchset contains two Netfilter fixes for your net tree, > they are: > > 1) Fix NAt compilation with UP, from Geert Uytterhoeven. > > 2) Fix incorrect number of entries when dumping a set, from >Vishwanath Pai. P

Re: [PATCH net-next v5 1/4] bpf: add helper bpf_perf_event_read_value for perf event array map

2017-09-20 Thread David Miller
From: Peter Zijlstra Date: Wed, 20 Sep 2017 19:26:51 +0200 > Dave, could we have this in a topic tree of sorts, because I have a > pending series to rework all the timekeeping and it might be nice to not > have sfr run into all sorts of conflicts. If you want to merge it into your tree that's fi

Re: [PATCH net-next] cxgb4: add new T5 pci device id's

2017-09-20 Thread David Miller
From: Ganesh Goudar Date: Wed, 20 Sep 2017 11:32:07 +0530 > Add 0x50a5, 0x50a6, 0x50a7, 0x50a8 and 0x50a9 T5 device > id's. > > Signed-off-by: Ganesh Goudar Applied, thank you.

Re: [PATCH net] net/ncsi: Don't assume last available channel exists

2017-09-20 Thread David Miller
From: Samuel Mendoza-Jonas Date: Wed, 20 Sep 2017 14:12:51 +1000 > When handling new VLAN tags in NCSI we check the maximum allowed number > of filters on the last active ("hot") channel. However if the 'add' > callback is called before NCSI has configured a channel, this causes a > NULL derefere

Re: [PATCH] isdn/i4l: fetch the ppp_write buffer in one shot

2017-09-20 Thread David Miller
From: Meng Xu Date: Tue, 19 Sep 2017 21:49:55 -0400 > In isdn_ppp_write(), the header (i.e., protobuf) of the buffer is > fetched twice from userspace. The first fetch is used to peek at the > protocol of the message and reset the huptimer if necessary; while the > second fetch copies in the whol

Re: [PATCH 0/2] blackfin: Drop non-functional DSA code

2017-09-20 Thread David Miller
From: Florian Fainelli Date: Tue, 19 Sep 2017 18:03:44 -0700 > I sent those many months ago in the hope that the bfin-linux people > would pick those patches but nobody seems to be responding, can you > queue those via net-next since this affects DSA? Ok, if they aren't responding what can we do

Re: [PATCH net-next] bpf: Optimize lpm trie delete

2017-09-20 Thread Daniel Mack
On 09/20/2017 08:51 PM, Craig Gallek wrote: > On Wed, Sep 20, 2017 at 12:51 PM, Daniel Mack wrote: >> Hi Craig, >> >> Thanks, this looks much cleaner already :) >> >> On 09/20/2017 06:22 PM, Craig Gallek wrote: >>> diff --git a/kernel/bpf/lpm_trie.c b/kernel/bpf/lpm_trie.c >>> index 9d58a576b2ae..

[PATCH net 1/2] net: ethtool: Add back transceiver type

2017-09-20 Thread Florian Fainelli
Commit 3f1ac7a700d0 ("net: ethtool: add new ETHTOOL_xLINKSETTINGS API") deprecated the ethtool_cmd::transceiver field, which was fine in premise, except that the PHY library was actually using it to report the type of transceiver: internal or external. Use the first word of the reserved field to p

[PATCH net 2/2] net: phy: Keep reporting transceiver type

2017-09-20 Thread Florian Fainelli
With commit 2d55173e71b0 ("phy: add generic function to support ksetting support"), we lost the ability to report the transceiver type like we used to. Now that we have added back the transceiver type to ethtool_link_settings, we can report it back like we used to and have no loss of information.

[PATCH net 0/2] Bring back transceiver type for PHYLIB

2017-09-20 Thread Florian Fainelli
Hi With the introduction of the xLINKSETTINGS ethtool APIs, the transceiver type was deprecated, but in that process we lost some useful information that PHYLIB was consistently reporting about internal vs. external PHYs. This brings back transceiver as a read-only field that is only consumed in

Re: [PATCH net-next] net: dsa: Utilize dsa_slave_dev_check()

2017-09-20 Thread David Miller
From: Florian Fainelli Date: Tue, 19 Sep 2017 18:00:37 -0700 > Instead of open coding the check. > > Signed-off-by: Florian Fainelli Applied, thanks.

Re: [PATCH v2 0/2] enable hires timer to timeout datagram socket

2017-09-20 Thread Vallish Vaidyeshwara
On Sat, Sep 16, 2017 at 11:47:56AM +0200, Thomas Gleixner wrote: > On Fri, 8 Sep 2017, Eric Dumazet wrote: > > On Fri, 2017-09-08 at 11:55 -0700, Eduardo Valentin wrote: > > > Hello, > > > > > > On Fri, Sep 08, 2017 at 10:26:45AM -0700, David Miller wrote: > > > > From: David Woodhouse > > > > Da

Re: [PATCH v4 3/3] net: fec: return IRQ_HANDLED if fec_ptp_check_pps_event handled it

2017-09-20 Thread David Miller
From: Troy Kisky Date: Tue, 19 Sep 2017 17:33:09 -0700 > fec_ptp_check_pps_event will return 1 if FEC_T_TF_MASK caused > an interrupt. Don't return IRQ_NONE in this case. > > Signed-off-by: Troy Kisky > Acked-by: Fugang Duan Applied.

Re: [PATCH v4 1/3] net: fec: only check queue 0 if RXF_0/TXF_0 interrupt is set

2017-09-20 Thread David Miller
From: Troy Kisky Date: Tue, 19 Sep 2017 17:33:07 -0700 > Before queue 0 was always checked if any queue caused an interrupt. > It is better to just mark queue 0 if queue 0 has caused an interrupt. > > Signed-off-by: Troy Kisky > Acked-by: Fugang Duan Applied.

Re: [PATCH v4 2/3] net: fec: remove unused interrupt FEC_ENET_TS_TIMER

2017-09-20 Thread David Miller
From: Troy Kisky Date: Tue, 19 Sep 2017 17:33:08 -0700 > FEC_ENET_TS_TIMER is not checked in the interrupt routine > so there is no need to enable it. > > Signed-off-by: Troy Kisky > Acked-by: Fugang Duan Applied.

Re: [PATCH net] net: change skb->mac_header when Generic XDP calls adjust_head

2017-09-20 Thread David Miller
From: Edward Cree Date: Tue, 19 Sep 2017 18:45:56 +0100 > Since XDP's view of the packet includes the MAC header, moving the start- > of-packet with bpf_xdp_adjust_head needs to also update the offset of the > MAC header (which is relative to skb->head, not to the skb->data that was > changed)

Re: [PATCH net-next v2] bridge: also trigger RTM_NEWLINK when interface is released from bridge

2017-09-20 Thread David Miller
From: Vincent Bernat Date: Thu, 21 Sep 2017 00:12:53 +0200 > ❦ 20 septembre 2017 15:57 -0600, David Ahern  : > >> The DELLINK is for AF_BRIDGE family (ifi_family). Adding family to >> print_linkinfo and running the monitor I get: >> >> >> [LINK]family 0: 35: dummy1: mtu 1500 qdisc >> noqueue m

Re: [PATCH] net: compat: assert the size of cmsg copied in is as expected

2017-09-20 Thread David Miller
From: Meng Xu Date: Tue, 19 Sep 2017 13:19:13 -0400 > The actual length of cmsg fetched in during the second loop > (i.e., kcmsg - kcmsg_base) could be different from what we > get from the first loop (i.e., kcmlen). > > The main reason is that the two get_user() calls in the two > loops (i.e.,

[PATCH 1/1] ip: ip_print: if no json obj is initialized default fp is stdout

2017-09-20 Thread Julien Fortin
From: Julien Fortin The ip monitor didn't call `new_json_obj` (even for in non json context), so the static FILE* _fp variable wasn't initialized, thus raising a SIGSEGV in ipaddress.c. This patch should fix this issue for good, new paths won't have to call `new_json_obj`. $ ip -t mon label link

Re: [PATCH net-next v2] bridge: also trigger RTM_NEWLINK when interface is released from bridge

2017-09-20 Thread Vincent Bernat
❦ 20 septembre 2017 15:57 -0600, David Ahern  : > The DELLINK is for AF_BRIDGE family (ifi_family). Adding family to > print_linkinfo and running the monitor I get: > > > [LINK]family 0: 35: dummy1: mtu 1500 qdisc > noqueue master br0 state UNKNOWN group default > link/ether d6:c3:73:86:3c:7

Re: Latest net-next from GIT panic

2017-09-20 Thread Wei Wang
>>> bisected again and same result: >>> b838d5e1c5b6e57b10ec8af2268824041e3ea911 is the first bad commit >>> commit b838d5e1c5b6e57b10ec8af2268824041e3ea911 >>> Author: Wei Wang >>> Date: Sat Jun 17 10:42:32 2017 -0700 >>> >>> ipv4: mark DST_NOGC and remove the operation of dst_free() >>> >>

Re: [PATCH net-next 3/3] virtio-net: support XDP_REDIRECT

2017-09-20 Thread John Fastabend
On 09/19/2017 02:42 AM, Jason Wang wrote: > This patch tries to add XDP_REDIRECT for virtio-net. The changes are > not complex as we could use exist XDP_TX helpers for most of the > work. The rest is passing the XDP_TX to NAPI handler for implementing > batching. > > Cc: John Fastabend > Signed-o

Re: [PATCH v2 1/3] can: m_can: Make hclk optional

2017-09-20 Thread Mario Hüttel
> Hclk is the MCAN's interface clock. However, for OMAP based devices such as > DRA7 SoC family the interface clock is handled by hwmod. Therefore, this > interface clock is managed by hwmod driver via pm_runtime_get and > pm_runtime_put calls. Therefore, this interface clock isn't defined in DT >

Re: [PATCH net-next v2] bridge: also trigger RTM_NEWLINK when interface is released from bridge

2017-09-20 Thread David Ahern
On 9/20/17 3:09 PM, David Miller wrote: > From: Vincent Bernat > Date: Sat, 16 Sep 2017 16:18:33 +0200 > > David, I am CC:'ing you because you've done work in this area over the > past year. I'm applying this patch, it's been sitting since the 16th > and likes entirely correct to me. But if you

Re: [RFC PATCH] can: m_can: Support higher speed CAN-FD bitrates

2017-09-20 Thread Mario Hüttel
On 09/20/2017 10:19 PM, Franklin S Cooper Jr wrote: > Hi Wenyou, > > On 09/17/2017 10:47 PM, Yang, Wenyou wrote: >> >> On 2017/9/14 13:06, Sekhar Nori wrote: >>> On Thursday 14 September 2017 03:28 AM, Franklin S Cooper Jr wrote: On 08/18/2017 02:39 PM, Franklin S Cooper Jr wrote: > Duri

Re: TSN Scorecard, was Re: [RFC net-next 0/5] TSN: Add qdisc-based config interfaces for traffic shapers

2017-09-20 Thread Jesus Sanchez-Palencia
Hi, On 09/19/2017 10:49 PM, Richard Cochran wrote: (...) > > No, that is not what I meant. We need some minimal additional kernel > support in order to fully implement the TSN family of standards. Of > course, the bulk will have to be done in user space. It would be a > mistake to cram the s

Re: [PATCH] bgmac: Remove all offloading features, including GRO.

2017-09-20 Thread Florian Fainelli
On 09/20/2017 02:27 PM, ros...@gmail.com wrote: > Sorry for the noise. After more testing I've found out that the cause > was that I had BBR enabled on my laptop. Switching back to CUBIC fixed > the issue. > > In other words, this patch is detrimental. Quite unsurprisingly, thanks for coming back

Re: [PATCH net-next 1/3] virtio-net: remove unnecessary parameter of virtnet_xdp_xmit()

2017-09-20 Thread David Miller
From: Jason Wang Date: Tue, 19 Sep 2017 17:42:41 +0800 > CC: John Fastabend > Signed-off-by: Jason Wang Applied.

Re: [PATCH net-next] udp: do rmem bulk free even if the rx sk queue is empty

2017-09-20 Thread David Miller
From: Paolo Abeni Date: Tue, 19 Sep 2017 12:11:43 +0200 > The commit 6b229cf77d68 ("udp: add batching to udp_rmem_release()") > reduced greatly the cacheline contention between the BH and the US > reader batching the rmem updates in most scenarios. > > Such optimization is explicitly avoided if

Re: Latest net-next from GIT panic

2017-09-20 Thread Paweł Staszewski
W dniu 2017-09-20 o 23:25, Paweł Staszewski pisze: W dniu 2017-09-20 o 23:24, Paweł Staszewski pisze: W dniu 2017-09-20 o 23:10, Paweł Staszewski pisze: W dniu 2017-09-20 o 21:23, Paweł Staszewski pisze: W dniu 2017-09-20 o 21:13, Paweł Staszewski pisze: W dniu 2017-09-20 o 20:36,

Re: [PATCH net-next 2/3] virtio-net: add packet len average only when needed during XDP

2017-09-20 Thread David Miller
From: Jason Wang Date: Tue, 19 Sep 2017 17:42:42 +0800 > There's no need to add packet len average in the case of XDP_PASS > since it will be done soon after skb is created. > > Cc: John Fastabend > Signed-off-by: Jason Wang Applied.

Re: [PATCH net-next 3/3] virtio-net: support XDP_REDIRECT

2017-09-20 Thread David Miller
From: Jason Wang Date: Tue, 19 Sep 2017 17:42:43 +0800 > This patch tries to add XDP_REDIRECT for virtio-net. The changes are > not complex as we could use exist XDP_TX helpers for most of the > work. The rest is passing the XDP_TX to NAPI handler for implementing > batching. > > Cc: John Fastab

Re: Latest net-next from GIT panic

2017-09-20 Thread Paweł Staszewski
W dniu 2017-09-20 o 23:24, Paweł Staszewski pisze: W dniu 2017-09-20 o 23:10, Paweł Staszewski pisze: W dniu 2017-09-20 o 21:23, Paweł Staszewski pisze: W dniu 2017-09-20 o 21:13, Paweł Staszewski pisze: W dniu 2017-09-20 o 20:36, Cong Wang pisze: On Wed, Sep 20, 2017 at 11:30 AM, Er

Re: [PATCH] bgmac: Remove all offloading features, including GRO.

2017-09-20 Thread rosenp
Sorry for the noise. After more testing I've found out that the cause was that I had BBR enabled on my laptop. Switching back to CUBIC fixed the issue. In other words, this patch is detrimental. ~67mbps - gro off ~87mbps - gro on On Fri, 2017-09-15 at 23:04 -0700, Florian Fainelli wrote: > On Se

Re: [PATCH v4 2/4] samples/bpf: Enable cross compiler support

2017-09-20 Thread Daniel Borkmann
On 09/20/2017 06:11 PM, Joel Fernandes wrote: When cross compiling, bpf samples use HOSTCC for compiling the non-BPF part of the sample, however what we really want is to use the cross compiler to build for the cross target since that is what will load and run the BPF sample. Detect this and comp

Re: [PATCH v4 4/4] samples/bpf: Add documentation on cross compilation

2017-09-20 Thread Daniel Borkmann
On 09/20/2017 06:11 PM, Joel Fernandes wrote: Acked-by: Alexei Starovoitov Signed-off-by: Joel Fernandes (Minor typo pointed out by Randy, but rest looks fine.) Acked-by: Daniel Borkmann

  1   2   3   4   >