[PATCH 1/1] net: usb: set error code when usb_alloc_urb fails

In function lan78xx_probe(), variable ret takes the errno code on failures. However, when the call to usb_alloc_urb() fails, its value will keeps 0. 0 indicates success in the context, which is inconsistent with the execution result. This patch fixes the bug, assigning "-ENOMEM" to ret when

Re: [[PATCH iproute2/net-next v2] 2/4] tc: flower: document SCTP ip_proto

On Fri, Dec 02, 2016 at 03:06:08PM -0800, Stephen Hemminger wrote: > On Fri, 2 Dec 2016 09:45:19 +0100 > Simon Horman wrote: > > > Add SCTP ip_proto to help text and man page. > > > > Signed-off-by: Simon Horman > > This doesn't apply

[PATCH iproute2/net-next v3 2/3] tc: flower: correct name of ip_proto parameter to flower_parse_port()

This corrects a typo. Fixes: a1fb0d484237 ("tc: flower: Support matching on SCTP ports") Signed-off-by: Simon Horman --- tc/f_flower.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tc/f_flower.c b/tc/f_flower.c index

[PATCH iproute2/net-next v3 3/3] tc: flower: make use of flower_port_attr_type() safe and silent

Make use of flower_port_attr_type() safe: * flower_port_attr_type() may return a valid index into tb[] or -1. Only access tb[] in the case of the former. * Do not access null entries in tb[] Also make usage silent - it is valid for ip_proto to be invalid, for example if it is not specified as

[PATCH iproute2/net-next v3 1/3] tc: flower: document SCTP ip_proto

Add SCTP ip_proto to help text and man page. Signed-off-by: Simon Horman --- man/man8/tc-flower.8 | 14 +++--- tc/f_flower.c| 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/man/man8/tc-flower.8 b/man/man8/tc-flower.8 index

[PATCH iproute2/net-next v3 0/3] tc: flower: SCTP and other port fixes

Hi Stephen, this short series: * Makes some improvements to the documentation of flower; A follow-up to recent work by Paul Blakey and myself. * Corrects some errors introduced when SCTP port matching support was recently added. Changes v2->v3: * Rebase * Dropped merged patch Changes

[PATCH 1/1] net: wireless: intersil: fix improper return value

Function orinoco_ioctl_commit() returns 0 (indicates success) when the call to orinoco_lock() fails. Thus, the return value is inconsistent with the execution status. It may be better to return "-EBUSY" when the call to orinoco_lock() fails. Bugzilla:

[PATCH 1/1] net: caif: fix ineffective error check

In function caif_sktinit_module(), the check of the return value of sock_register() seems ineffective. This patch fixes it. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751 Signed-off-by: Pan Bian --- net/caif/caif_socket.c | 2 +- 1 file changed, 1

[PATCH 1/1] net: bridge: set error code on failure

Function br_sysfs_addbr() does not set error code when the call kobject_create_and_add() returns a NULL pointer. It may be better to return "-ENOMEM" when kobject_create_and_add() fails. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188781 Signed-off-by: Pan Bian

[net-next 06/18] i40e: Add Clause22 implementation

From: Michal Kosiarz Some external PHYs require Clause22 method for accessing registers. This patch also adds some defines to support blink led on devices using 10CBaseT PHY. Change-ID: I868a4326911900f6c89e7e522fda4968b0825f14 Signed-off-by: Michal Kosiarz

[net-next 10/18] i40evf: Move some i40evf_reset_task code to separate function

From: Joe Perches The i40evf_reset_task function is a couple hundred lines and it has a separable block that disables VF. Move that block to a new i40evf_disable_vf function to shorten i40evf_reset_task a bit. Signed-off-by: Joe Perches Tested-by: Andrew

[net-next 01/18] i40e: Be much more verbose about what we can and cannot offload

From: Alexander Duyck This change makes it so that we are much more robust about defining what we can and cannot offload. Previously we were just checking for the L4 tunnel header length, however there are other fields we should be verifying as there are multiple

[net-next 00/18][pull request] 40GbE Intel Wired LAN Driver Updates 2016-12-02

This series contains updates to i40e and i40evf only. Alex provides changes so that we are much more robust about defining what we can and cannot offload in i40e and i40evf by doing additional checks other than L4 tunnel header length. Jake provides several fixes/changes, first cleaning up a

Re: bpf bounded loops. Was: [flamebait] xdp

On Fri, Dec 2, 2016 at 4:20 PM, Alexei Starovoitov wrote: > On Fri, Dec 02, 2016 at 11:42:15AM -0800, John Fastabend wrote: >> >> As far as pattern search for DNS packets... >> >> it was requested by Cloudflare guys back in March: >> >>

[net-next 09/18] i40e: fix panic on SPARC while changing num of desc

From: Tushar Dave On SPARC, writel() should not be used to write directly to memory address but only to memory mapped I/O address otherwise it causes data access exception. Commit 147e81ec75689 ("i40e: Test memory before ethtool alloc succeeds") introduced a code that

[net-next 07/18] i40e: fix trivial typo in naming of i40e_sync_filters_subtask

From: Jacob Keller A comment incorrectly referred to i40e_vsi_sync_filters_subtask which does not actually exist. Reference the correct function instead. Change-ID: I6bd805c605741ffb6fe34377259bb0d597edfafd Signed-off-by: Jacob Keller

[net-next 08/18] i40e: Add protocols over MCTP to i40e_aq_discover_capabilities

From: Piotr Raczynski Add logical_id to I40E_AQ_CAP_ID_MNG_MODE capability starting from major version 2. Change-ID: Idb29214b172ea5c70cbd45a99e6745c0215af7e4 Signed-off-by: Piotr Raczynski Tested-by: Andrew Bowers

[net-next 14/18] i40evf: protect against NULL msix_entries and q_vectors pointers

From: Jacob Keller Update the functions which free msix_entries and q_vectors so that they are safe against NULL values. This allows calling code to not care whether these have already been freed when disabling and freeing them. Change-ID:

[PATCH net-next] net: remove abuse of VLAN DEI/CFI bit

This All-in-one patch removes abuse of VLAN CFI bit, so it can be passed intact through linux networking stack. Signed-off-by: Michał Mirosław --- Dear NetDevs I guess this needs to be split to the prep..convert[]..finish sequence, but if you like it as is,

[PATCH 1/1] netdev: broadcom: propagate error code

Function bnxt_hwrm_stat_ctx_alloc() always returns 0, even if the call to _hwrm_send_message() fails. It may be better to propagate the errors to the caller of bnxt_hwrm_stat_ctx_alloc(). Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188661 Signed-off-by: Pan Bian

[PATCH 1/1] net: wireless: intersil: fix improper return value

Function orinoco_ioctl_commit() returns 0 (indicates success) when the call to orinoco_lock() fails. Thus, the return value is inconsistent with the execution status. It may be better to return "-EBUSY" when the call to orinoco_lock() fails. Bugzilla:

Re: [PATCH v2 net-next 1/2] flow dissector: ICMP support

Fri, Dec 02, 2016 at 09:31:41PM CET, simon.hor...@netronome.com wrote: >Allow dissection of ICMP(V6) type and code. This re-uses transport layer >port dissection code as although ICMP is not a transport protocol and their >type and code are not ports this allows sharing of both code and storage. >

[PATCH 1/1] net: wireless: marvell: fix improper return value

Function lbs_cmd_802_11_sleep_params() always return 0, even if the call to lbs_cmd_with_response() fails. In this case, the parameter @sp will keep uninitialized. Because the return value is 0, its caller (say lbs_sleepparams_read()) will not detect the error, and will copy the uninitialized

[net-next 18/18] i40e: change message to only appear when extra debug info is wanted

From: Carolyn Wyborny This patch changes an X722 informational message so that it only appears when extra messages are desired. Without this patch, on X722 devices, this message appears at load, potentially causing unnecessary alarm. Change-ID:

[net-next 02/18] i40evf: Be much more verbose about what we can and cannot offload

From: Alexander Duyck This change makes it so that we are much more robust about defining what we can and cannot offload. Previously we were performing no checks. This should bring us up to parity with the i40e PF driver. In addition the device only supports GSO

[net-next 16/18] i40e: set broadcast promiscuous mode for each active VLAN

From: Jacob Keller A previous workaround added to ensure receipt of all broadcast frames incorrectly set the broadcast promiscuous mode unconditionally regardless of active VLAN status. Replace this partial workaround with a complete solution that sets the broadcast

Re: [PATCH v3 00/13] net: ethernet: ti: cpts: update and fixes

On Fri, Dec 02, 2016 at 02:30:10PM -0600, Grygorii Strashko wrote: > It is preparation series intended to clean up and optimize TI CPTS driver to > facilitate further integration with other TI's SoCs like Keystone 2. > > Changes in v3: > - patches reordered: fixes and small updates moved first >

[net-next 13/18] i40e: Pass unknown PHY type for unknown PHYs

From: Henry Tieman The PHY type value for unrecognized PHYs and cables was changed based on firmware version number. Newer hardware use lower firmware version numbers and this was causing some PHYs to be identified as type 0x16 instead of 0xe (unknown). Without this

[net-next 05/18] i40e: avoid duplicate private flags definitions

From: Jacob Keller Separate the global private flags and the regular private flags per interface into two arrays. Future additions of private flags will not need to be duplicated which may lead to buggy code. Also rename "i40e_priv_flags_strings_gl" to

[net-next 15/18] i40e: Fix for ethtool Supported link modes

From: Harshitha Ramamurthy This patch fixes the problem where the ethtool Supported link modes list backplane interfaces on X722 devices for 10GbE with SFP+ and Cortina retimer. This patch fixes the problem by setting and using a flag for this particular device

[net-next 12/18] i40e: Remove unreachable code

From: Henry Tieman The code at the end of i40e_read_phy_register_clause22() contained unreachable code and redundant control statements. This change removes the unreachable code. And deletes the redundant goto statement and if statement. Change-ID:

[net-next 04/18] i40e: remove second check of VLAN_N_VID in i40e_vlan_rx_add_vid

From: Jacob Keller Replace a check of magic number 4095 with VLAN_N_VID. This makes it obvious that a later check against VLAN_N_VID is always true and can be removed. Change-ID: I28998f127a61a529480ce63d8a07e266f6c63b7b Signed-off-by: Jacob Keller

[net-next 03/18] i40e: remove error_param_int label from i40e_vc_config_promiscuous_mode_msg

From: Jacob Keller This label is unnecessary, as are jumping to a block that checks aq_ret and then immediately skipping it and returning. So just jump straight to the error_param and remove this unnecessary label. Also use goto error_param even in the last check for

[net-next 17/18] i40e/i40evf: replace for memcpy with single memcpy call in ethtool

From: Jacob Keller memcpy replaced with single memcpy call in ethtool. Change-ID: I3f5bef6bcc593412c56592c6459784db41575a0a Signed-off-by: Jacob Keller Tested-by: Andrew Bowers Signed-off-by: Jeff Kirsher

[net-next 11/18] i40evf: check for msix_entries null dereference

From: Alan Brady It is possible for msix_entries to be freed by a previous suspend/remove before a VF is closed. This patch fixes the issue by checking for NULL before dereferencing msix_entries and returning early in the case where it is NULL within the i40evf_close code

[PATCH] net: af_mpls.c add space before open parenthesis

Adding space after switch keyword before open parenthesis for readability purpose. This patch fixes the checkpatch.pl warning: space required before the open parenthesis '(' Signed-off-by: Suraj Deshmukh --- net/mpls/af_mpls.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH iproute2/net-next] ss: initialise variables outside of for loop

On Fri, Dec 02, 2016 at 02:18:17PM -0800, Stephen Hemminger wrote: > On Fri, 2 Dec 2016 12:56:05 +0100 > Simon Horman wrote: > > > Initialise for loops outside of for loops. GCC flags this as being > > out of spec unless C99 or C11 mode is used. > > > > With this

Re: net: use-after-free in worker_thread

On Sat, Dec 3, 2016 at 9:41 AM, Cong Wang wrote: > On Sat, Dec 3, 2016 at 4:56 AM, Andrey Konovalov > wrote: >> Hi! >> >> I'm seeing lots of the following error reports while running the >> syzkaller fuzzer. >> >> Reports appeared when I updated

[PATCH v2 net-next 8/8] tcp: tsq: move tsq_flags close to sk_wmem_alloc

tsq_flags being in the same cache line than sk_wmem_alloc makes a lot of sense. Both fields are changed from tcp_wfree() and more generally by various TSQ related functions. Prior patch made room in struct sock and added sk_tsq_flags, this patch deletes tsq_flags from struct tcp_sock.

[PATCH v2 net-next 7/8] net: reorganize struct sock for better data locality

Group fields used in TX path, and keep some cache lines mostly read to permit sharing among cpus. Gained two 4 bytes holes on 64bit arches. Added a place holder for tcp tsq_flags, next to sk_wmem_alloc to speed up tcp_wfree() in the following patch. I have not added

[PATCH v2 net-next 5/8] tcp: tsq: add a shortcut in tcp_small_queue_check()

Always allow the two first skbs in write queue to be sent, regardless of sk_wmem_alloc/sk_pacing_rate values. This helps a lot in situations where TX completions are delayed either because of driver latencies or softirq latencies. Test is done with no cache line misses. Signed-off-by: Eric

[PATCH v2 net-next 3/8] tcp: tsq: add shortcut in tcp_tasklet_func()

Under high stress, I've seen tcp_tasklet_func() consuming ~700 usec, handling ~150 tcp sockets. By setting TCP_TSQ_DEFERRED in tcp_wfree(), we give a chance for other cpus/threads entering tcp_write_xmit() to grab it, allowing tcp_tasklet_func() to skip sockets that already did an xmit cycle. In

[PATCH v2 net-next 6/8] tcp: tcp_mtu_probe() is likely to exit early

Adding a likely() in tcp_mtu_probe() moves its code which used to be inlined in front of tcp_write_xmit() We still have a cache line miss to access icsk->icsk_mtup.enabled, we will probably have to reorganize fields to help data locality. Signed-off-by: Eric Dumazet ---

Hello Beautiful

Hello beautiful, How you doing today? I hope you are doing well. My name is Bentley, from the US. I'm in Syria right now fighting ISIS. I want to get to know you better, if I may be so bold. I consider myself an easy-going man, and I am currently looking for a relationship in which I feel

Re: [PATCH net-next 1/4] bpf: xdp: Allow head adjustment in XDP prog

On 12/03/2016 08:32 PM, Martin KaFai Lau wrote: On Sat, Dec 03, 2016 at 04:24:13PM +0100, Jesper Dangaard Brouer wrote: On Fri, 2 Dec 2016 15:23:30 -0800 Martin KaFai Lau wrote: -bool bpf_helper_changes_skb_data(void *func) +BPF_CALL_2(bpf_xdp_adjust_head, struct xdp_buff *,

[PATCH net-next] bpf: Preserve const register type on const OR alu ops

From: Gianluca Borello Occasionally, clang (e.g. version 3.8.1) translates a sum between two constant operands using a BPF_OR instead of a BPF_ADD. The verifier is currently not handling this scenario, and the destination register type becomes UNKNOWN_VALUE even if it's

Re: [PATCH] netlink: 2-clause nla_ok()

From: Alexey Dobriyan Date: Fri, 2 Dec 2016 03:59:06 +0300 > nla_ok() consists of 3 clauses: > > 1) int rem >= (int)sizeof(struct nlattr) > > 2) u16 nla_len >= sizeof(struct nlattr) > > 3) u16 nla_len <= int rem > > The statement is that clause (1) is

[PATCH v2 net-next 0/8] tcp: tsq: performance series

Under very high TX stress, CPU handling NIC TX completions can spend considerable amount of cycles handling TSQ (TCP Small Queues) logic. This patch series avoids some atomic operations, but most notable patch is the 3rd one, allowing other cpus processing ACK packets and calling tcp_write_xmit()

Re: [RFC PATCH 2/2] Documentation: devictree: Add macb mdio bindings

On Mon, Nov 28, 2016 at 03:19:27PM +0530, Harini Katakam wrote: > Add documentations for macb mdio driver. Bindings document h/w, not drivers. > > Signed-off-by: Harini Katakam > --- > .../devicetree/bindings/net/macb-mdio.txt | 31 > ++ > 1

Re: Possible regression due to "net/sched: cls_flower: Add offload support using egress Hardware device"

On Sat, Dec 03, 2016 at 06:06:08PM +0200, Or Gerlitz wrote: > On Sat, Dec 3, 2016 at 3:17 PM, Simon Horman wrote: > > > in net-next I am observing what appears to be an regression in net-next due > > to: > > 7091d8c7055d ("net/sched: cls_flower: Add offload support using

[Patch net-next] act_mirred: fix a typo in get_dev

Cc: Hadar Hen Zion Cc: Jiri Pirko Signed-off-by: Cong Wang --- net/sched/act_mirred.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/sched/act_mirred.c b/net/sched/act_mirred.c index bb09ba3..2d9fa6e

Re: [PATCH V2 net-next] net: hns: Fix to conditionally convey RX checksum flag to stack

From: Salil Mehta Date: Thu, 1 Dec 2016 12:09:22 + > But maybe now since we don't have any method to de-multiplex the kind of > checksum error (cannot depend upon register) we can have below code > re-arrangement: > > hns_nic_rx_checksum() { > /* check

Re: [PATCH] stmmac: cleanup documenation, make it match reality

From: Pavel Machek Date: Thu, 1 Dec 2016 11:32:18 +0100 > Fix english in documentation, make documentation match reality, remove > options that were removed from code. > > Signed-off-by: Pavel Machek Applied.

Re: [PATCH 2/2] net: stmmac: unify mdio functions

From: Corentin Labbe Date: Thu, 1 Dec 2016 16:19:41 +0100 > stmmac_mdio_{read|write} and stmmac_mdio_{read|write}_gmac4 are not > enought different for being split. > The only differences between thoses two functions are shift/mask for > addr/reg/clk_csr. > > This

Re: [PATCH 1/7] net: ethernet: ti: cpdma: am437x: allow descs to be plased in ddr

From: Grygorii Strashko Date: Thu, 1 Dec 2016 17:34:26 -0600 > @@ -167,10 +167,10 @@ static struct cpdma_control_info controls[] = { > > /* various accessors */ > #define dma_reg_read(ctlr, ofs) __raw_readl((ctlr)->dmaregs + > (ofs)) > -#define

Re: [PATCH 1/3] netns: publish net_generic correctly

All 3 patches applied to net-next, thanks.

Re: [PATCH net-next 0/2] samples, bpf: Refactor; Add automated tests for cgroups

From: Sargun Dhillon Date: Fri, 2 Dec 2016 02:42:03 -0800 > These two patches are around refactoring out some old, reusable code from the > existing test_current_task_under_cgroup_user test, and adding a new, > automated > test. > > There is some generic cgroupsv2 setup &

Re: [PATCH net-next] samples/bpf: silence compiler warnings

From: Alexei Starovoitov Date: Thu, 1 Dec 2016 18:31:12 -0800 > silence some of the clang compiler warnings like: > include/linux/fs.h:2693:9: warning: comparison of unsigned enum expression < > 0 is always false > arch/x86/include/asm/processor.h:491:30: warning: taking address of

Re: [PATCH 1/1] net: caif: fix ineffective error check

On 12/03/2016 06:38 PM, Pan Bian wrote: In function caif_sktinit_module(), the check of the return value of sock_register() seems ineffective. This patch fixes it. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751 Signed-off-by: Pan Bian ---

Re: [PATCH 1/6] net: stmmac: return error if no DMA configuration is found

When you post a series of related changes as a patch set, you must provide a proper "[PATCH 0/N] ..." posting which explains what the series is doing at a high level, how it is doing it, and why it is doing it that way. Please repost this entire series with a proper header posting included.

Re: [PATCH] pull request for net: batman-adv 2016-12-02

From: Simon Wunderlich Date: Fri, 2 Dec 2016 17:13:22 +0100 > here is another bugfix which we would like to see integrated into net, > if this is possible now. > > Please pull or let me know of any problem! Pulled, thanks.

Re: [PATCH net-next] udp: be less conservative with sock rmem accounting

From: Paolo Abeni Date: Fri, 2 Dec 2016 17:35:49 +0100 > Before commit 850cbaddb52d ("udp: use it's own memory accounting > schema"), the udp protocol allowed sk_rmem_alloc to grow beyond > the rcvbuf by the whole current packet's truesize. After said commit > we allow

Re: [PATCH net-next] sfc: remove EFX_BUG_ON_PARANOID, use EFX_WARN_ON_[ONCE_]PARANOID instead

From: Edward Cree Date: Fri, 2 Dec 2016 15:51:33 + > Logically, EFX_BUG_ON_PARANOID can never be correct. For, BUG_ON should > only be used if it is not possible to continue without potential harm; > and since the non-DEBUG driver will continue regardless (as the

Re: [PATCH net-next] net_sched: gen_estimator: account for timer drifts

From: Eric Dumazet Date: Fri, 02 Dec 2016 08:11:00 -0800 > From: Eric Dumazet > > Under heavy stress, timer used in estimators tend to slowly be delayed > by a few jiffies, leading to inaccuracies. > > Lets remember what was the last scheduled

Re: net: use-after-free in worker_thread

On Sat, Dec 3, 2016 at 4:56 AM, Andrey Konovalov wrote: > Hi! > > I'm seeing lots of the following error reports while running the > syzkaller fuzzer. > > Reports appeared when I updated to 3c49de52 (Dec 2) from 2caceb32 (Dec 1). > >

pull request: bluetooth-next 2016-12-03

Hi Dave, Here's a set of Bluetooth & 802.15.4 patches for net-next (i.e. 4.10 kernel): - Fix for a potential NULL deref in the ieee802154 netlink code - Fix for the ED values of the at86rf2xx driver - Documentation updates to ieee802154 - Cleanups to u8 vs __u8 usage - Timer API usage

Re: [PATCH net-next 1/4] bpf: xdp: Allow head adjustment in XDP prog

On Sat, Dec 03, 2016 at 04:24:13PM +0100, Jesper Dangaard Brouer wrote: > On Fri, 2 Dec 2016 15:23:30 -0800 > Martin KaFai Lau wrote: > > > -bool bpf_helper_changes_skb_data(void *func) > > +BPF_CALL_2(bpf_xdp_adjust_head, struct xdp_buff *, xdp, int, offset) > > +{ > > + /* Both

Re: pull-request: can-next 2016-12-01,pull-request: can-next 2016-12-01

From: Marc Kleine-Budde Date: Thu, 1 Dec 2016 21:21:44 +0100 > this is a pull request of 4 patches for net-next/master. > > There are two patches by Chris Paterson for the rcar_can and rcar_canfd > device tree binding documentation. And a patch by Geert Uytterhoeven > that

Re: [PATCH -next] net: ethernet: ti: davinci_cpdma: add missing EXPORTs

From: Paul Gortmaker Date: Thu, 1 Dec 2016 15:25:28 -0500 > As of commit 8f32b90981dcdb355516fb95953133f8d4e6b11d > ("net: ethernet: ti: davinci_cpdma: add set rate for a channel") the > ARM allmodconfig builds would fail modpost with: > > ERROR:

Re: [PATCH 1/1] netdev: broadcom: propagate error code

On Sat, Dec 3, 2016 at 1:56 AM, Pan Bian wrote: > Function bnxt_hwrm_stat_ctx_alloc() always returns 0, even if the call > to _hwrm_send_message() fails. It may be better to propagate the errors > to the caller of bnxt_hwrm_stat_ctx_alloc(). > > Bugzilla:

Re: [Patch net-next] act_mirred: fix a typo in get_dev

On Sat, 2016-12-03 at 10:36 -0800, Cong Wang wrote: > Cc: Hadar Hen Zion > Cc: Jiri Pirko > Signed-off-by: Cong Wang > --- > net/sched/act_mirred.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [flamebait] xdp, well meaning but pointless

On 16-12-03 08:19 AM, Willem de Bruijn wrote: > On Fri, Dec 2, 2016 at 12:22 PM, Jesper Dangaard Brouer > wrote: >> >> On Thu, 1 Dec 2016 10:11:08 +0100 Florian Westphal wrote: >> >>> In light of DPDKs existence it make a lot more sense to me to provide >>> a).

Re: [PATCH] irda: w83977af_ir: fix damaged whitespace

From: Arnd Bergmann Date: Mon, 28 Nov 2016 15:19:43 +0100 > As David Miller pointed out for for the previous patch, the whitespace > in some functions looks rather odd. This was caused by commit 6329da5f258a > ("obsolete config in kernel source: USE_INTERNAL_TIMER"), which removed

Re: [PATCH V2 net-next] net: hns: Fix to conditionally convey RX checksum flag to stack

From: Salil Mehta Date: Thu, 1 Dec 2016 16:59:14 + > It looks to me the cumbersome check in the PATCH V2 should > be retained. I really want something simpler with small checks that are done in logical pieces in a straigtforward progression. The code in V2 is

Re: [PATCH 1/2] net: stmmac: avoid Camelcase naming

From: Corentin Labbe Date: Thu, 1 Dec 2016 16:19:40 +0100 > This patch simply rename regValue to value, like it was named in other > mdio functions. > > Signed-off-by: Corentin Labbe Applied.

Re: [net-next 0/5] use reset to set headers

From: Zhang Shengju Date: Fri, 2 Dec 2016 09:51:02 +0800 > This patch serial replace 'set' function to 'reset', since the > offset is zero. It's not necessary to use set, reset function is > straightforward, and will remove the unnecessary add operation in >

Re: [PATCH v2 net-next 1/2] flow dissector: ICMP support

On Sat, Dec 3, 2016 at 2:49 AM, Jiri Pirko wrote: > Fri, Dec 02, 2016 at 09:31:41PM CET, simon.hor...@netronome.com wrote: >>Allow dissection of ICMP(V6) type and code. This re-uses transport layer >>port dissection code as although ICMP is not a transport protocol and their

[PATCH v2 net-next 2/8] tcp: tsq: remove one locked operation in tcp_wfree()

Instead of atomically clear TSQ_THROTTLED and atomically set TSQ_QUEUED bits, use one cmpxchg() to perform a single locked operation. Since the following patch will also set TCP_TSQ_DEFERRED here, this cmpxchg() will make this addition free. Signed-off-by: Eric Dumazet ---

[PATCH v2 net-next 1/8] tcp: tsq: add tsq_flags / tsq_enum

This is a cleanup, to ease code review of following patches. Old 'enum tsq_flags' is renamed, and a new enumeration is added with the flags used in cmpxchg() operations as opposed to single bit operations. Signed-off-by: Eric Dumazet --- include/linux/tcp.h | 11

[PATCH v2 net-next 4/8] tcp: tsq: avoid one atomic in tcp_wfree()

Under high load, tcp_wfree() has an atomic operation trying to schedule a tasklet over and over. We can schedule it only if our per cpu list was empty. Signed-off-by: Eric Dumazet --- net/ipv4/tcp_output.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff

[PATCH net-next] r8169: Add support for restarting auto-negotiation

Implement ethtooll::nway_restart by utilizing mii_nway_restart. Signed-off-by: Florian Fainelli --- drivers/net/ethernet/realtek/r8169.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c

Re: [PATCH net-next 0/8] drivers: net: xgene: Add Jumbo and Pause frame support

From: Iyappan Subramanian Date: Thu, 1 Dec 2016 16:41:36 -0800 > This patch set adds, > > 1. Jumbo frame support > 2. Pause frame based flow control > > and fixes RSS for non-TCP/UDP packets. > > Signed-off-by: Iyappan Subramanian Series applied,

Re: [PATCH 1/1] net: ethernet: 3com: set error code on failures

Hi, On 03.12.2016 14:24, Pan Bian wrote: > From: Pan Bian > > In function typhoon_init_one(), returns the value of variable err on > errors. However, on some error paths, variable err is not set to a > negative errno. This patch assigns "-EIO" to err on those paths. > >

Re: [PATCH net] team: team_port_add should check link_up before enable port

On Sat, Dec 03, 2016 at 09:42:11PM +0800, Xin Long wrote: > Now when users add a nic to team dev, the option 'enable' of the port > is true by default, as team_port_enable enables it after dev_open in > team_port_add. > > But even if the port_dev has no carrier, like it's cable was unpluged, >

Re: [flamebait] xdp, well meaning but pointless

On Fri, Dec 2, 2016 at 12:22 PM, Jesper Dangaard Brouer wrote: > > On Thu, 1 Dec 2016 10:11:08 +0100 Florian Westphal wrote: > >> In light of DPDKs existence it make a lot more sense to me to provide >> a). a faster mmap based interface (possibly AF_PACKET

Re: [PATCH v2 net] ixgbevf: fix invalid uses of napi_hash_del()

On Sat, 2016-12-03 at 07:00 -0800, Eric Dumazet wrote: > On Wed, 2016-11-16 at 07:26 -0800, Eric Dumazet wrote: > > From: Eric Dumazet > > > > Calling napi_hash_del() before netif_napi_del() is dangerous > > if a synchronize_rcu() is not enforced before NAPI struct freeing.

Re: arp_filter and IPv6 ND

On 2 December 2016 at 20:39, Hannes Frederic Sowa wrote: Hey, > E.g. you can use IP addresses bound to other interfaces to send replys > on another interface. This can be useful if you have a limited amount of > IP addresses on the system but much more interfaces.

[patch net-next v4 01/10] ipv4: fib: Export free_fib_info()

From: Ido Schimmel The FIB notification chain is going to be converted to an atomic chain, which means switchdev drivers will have to offload FIB entries in deferred work, as hardware operations entail sleeping. However, while the work is queued fib info might be freed, so

Re: Possible regression due to "net/sched: cls_flower: Add offload support using egress Hardware device"

On Sat, Dec 3, 2016 at 3:17 PM, Simon Horman wrote: > in net-next I am observing what appears to be an regression in net-next due > to: > 7091d8c7055d ("net/sched: cls_flower: Add offload support using egress > Hardware device") > > The problem occurs when adding a flower

Re: [PATCH 1/1] net: caif: fix ineffective error check

From: PanBian Hello Sergei, On Sat, Dec 03, 2016 at 04:17:51PM +0300, Sergei Shtylyov wrote: > Hello. > > On 12/3/2016 2:18 PM, Pan Bian wrote: > > >In function caif_sktinit_module(), the check of the return value of > >sock_register() seems ineffective. This patch fixes

net: use-after-free in worker_thread

Hi! I'm seeing lots of the following error reports while running the syzkaller fuzzer. Reports appeared when I updated to 3c49de52 (Dec 2) from 2caceb32 (Dec 1). == BUG: KASAN: use-after-free in worker_thread+0x17d8/0x18a0 Read of

[patch net-next v4 08/10] ipv4: fib: Convert FIB notification chain to be atomic

From: Ido Schimmel In order not to hold RTNL for long periods of time we're going to dump the FIB tables using RCU. Convert the FIB notification chain to be atomic, as we can't block in RCU critical sections. Signed-off-by: Ido Schimmel Signed-off-by:

[patch net-next v4 03/10] mlxsw: core: Create an ordered workqueue for FIB offload

From: Ido Schimmel We're going to start processing FIB entries addition / deletion events in deferred work. These work items must be processed in the order they were submitted or otherwise we can have differences between the kernel's FIB table and the device's. Solve this

[PATCH 1/1] atm: lanai: set error code when ioremap fails

In function lanai_dev_open(), when the call to ioremap() fails, the value of return variable result is 0. 0 means no error in this context. This patch fixes the bug, assigning "-ENOMEM" to result when ioremap() returns a NULL pointer. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188791

Possible regression due to "net/sched: cls_flower: Add offload support using egress Hardware device"

Hi Hadar, in net-next I am observing what appears to be an regression in net-next due to: 7091d8c7055d ("net/sched: cls_flower: Add offload support using egress Hardware device") The problem occurs when adding a flower filter (without offload to a virtio device). # ethtool -d eth0 ethtool -i

Re: [PATCH 1/1] net: caif: fix ineffective error check

Hello. On 12/3/2016 2:18 PM, Pan Bian wrote: In function caif_sktinit_module(), the check of the return value of sock_register() seems ineffective. This patch fixes it. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751 Signed-off-by: Pan Bian ---

Re: [PATCH v2 net] ixgbevf: fix invalid uses of napi_hash_del()

On Wed, 2016-11-16 at 07:26 -0800, Eric Dumazet wrote: > From: Eric Dumazet > > Calling napi_hash_del() before netif_napi_del() is dangerous > if a synchronize_rcu() is not enforced before NAPI struct freeing. > > Lets leave this detail to core networking stack to get it

Re: net: use-after-free in worker_thread

On Sat, Dec 3, 2016 at 2:49 PM, Eric Dumazet wrote: > On Sat, 2016-12-03 at 14:05 +0100, Andrey Konovalov wrote: >> On Sat, Dec 3, 2016 at 1:58 PM, Andrey Konovalov >> wrote: >> > +syzkal...@googlegroups.com >> > >> > On Sat, Dec 3, 2016 at 1:56

[PATCH 1/1] net: ethernet: 3com: set error code on failures

From: Pan Bian In function typhoon_init_one(), returns the value of variable err on errors. However, on some error paths, variable err is not set to a negative errno. This patch assigns "-EIO" to err on those paths. Signed-off-by: Pan Bian ---

[PATCH 1/1] net: dcb: set error code on failures

From: Pan Bian In function dcbnl_cee_fill(), returns the value of variable err on errors. However, on some error paths (e.g. nla put fails), its value may be 0. It may be better to explicitly set a negative errno to variable err before returning. Bugzilla:

  1   2   >