On Mon, 21 Aug 2017 15:35:42 -0700
Alexei Starovoitov wrote:
> On Mon, Aug 21, 2017 at 09:25:06PM +0200, Jesper Dangaard Brouer wrote:
> >
> > Third gotcha(3): You got this far, loaded xdp on both interfaces, and
> > notice now that (with default setup) you can RX with 14Mpps but only
> > TX wit
netif_napi_del() should be paired with netif_napi_add(), however no
such call takes place in ftgmac100_probe(). This triggers a NULL
pointer dereference if e.g. no PHY is found by the MDIO probe:
[ 2.77] libphy: Fixed MDIO Bus: probed
[ 2.77] ftgmac100 1e66.ethernet:
The JIT compiler emits ARM 32 bit instructions. Currently, It supports
eBPF only. Classic BPF is supported because of the conversion by BPF core.
This patch is essentially changing the current implementation of JIT compiler
of Berkeley Packet Filter from classic to internal with almost all
instruc
On Tue, Aug 22, 2017 at 11:57 AM, Richard Cochran
wrote:
> On Mon, Aug 21, 2017 at 11:01:12PM +0530, Bhumika Goyal wrote:
>> File ptp_ixp46x.c is not tested as I could not find any architecture
>> to cross compile it.
>
> No problem. Thanks for test compiling the other drivers.
>
By testing I me
The JIT compiler emits ARM 32 bit instructions. Currently, It supports
eBPF only. Classic BPF is supported because of the conversion by BPF core.
This patch is essentially changing the current implementation of JIT compiler
of Berkeley Packet Filter from classic to internal with almost all
instruc
Signed-off-by: Tonghao Zhang
---
include/net/tcp.h | 3 +--
net/ipv4/tcp_fastopen.c | 6 ++
net/ipv4/tcp_input.c| 2 +-
3 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/include/net/tcp.h b/include/net/tcp.h
index afdab3781425..a995004ae946 100644
--- a/include/net/tcp.
tcp_peer_is_proven needs a proper route to make the
determination, but dst always is NULL. This bug may
be there at the beginning of git tree. This does not
look serious enough to deserve backports to stable
versions.
Signed-off-by: Tonghao Zhang
---
net/ipv4/tcp_input.c | 9 -
1 file ch
Just simplify the tcp_conn_request function.
Tonghao Zhang (2):
tcp: Get a proper dst before checking it.
tcp: Remove the unused parameter for tcp_try_fastopen.
include/net/tcp.h | 3 +--
net/ipv4/tcp_fastopen.c | 6 ++
net/ipv4/tcp_input.c| 11 +--
3 files changed, 8
The JIT compiler emits ARM 32 bit instructions. Currently, It supports
eBPF only. Classic BPF is supported because of the conversion by BPF core.
This patch is essentially changing the current implementation of JIT compiler
of Berkeley Packet Filter from classic to internal with almost all
instruc
The JIT compiler emits ARM 32 bit instructions. Currently, It supports
eBPF only. Classic BPF is supported because of the conversion by BPF core.
This patch is essentially changing the current implementation of JIT compiler
of Berkeley Packet Filter from classic to internal with almost all
instruc
On Mon, Aug 21, 2017 at 11:01:12PM +0530, Bhumika Goyal wrote:
> File ptp_ixp46x.c is not tested as I could not find any architecture
> to cross compile it.
No problem. Thanks for test compiling the other drivers.
Acked-by: Richard Cochran
On Mon, Aug 21, 2017 at 10:36:50PM +0530, Bhumika Goyal wrote:
> Make these const as they are only used in a copy operation.
> Done using Coccinelle.
Acked-by: Richard Cochran
On Mon, Aug 21, 2017 at 06:22:10PM +, Vallish Vaidyeshwara wrote:
> AWS Lambda is affected by this change in behavior in
> system call. Following links has more information:
> https://en.wikipedia.org/wiki/AWS_Lambda
Quote:
Unlike Amazon EC2, which is priced by the hour, AWS Lambda is
On Tue, Aug 22, 2017 at 11:12 AM, Akshat Kakkar wrote:
> There are multiple hosts/clients. All are mainly windows based.
>
> Timestamp is not used as my clients mainly are windows based and in
> that it tcp timestamp is by defauly disabled.
>
> sysctl is as follows:
>
> kernel.shmmax = 68719476736
There are multiple hosts/clients. All are mainly windows based.
Timestamp is not used as my clients mainly are windows based and in
that it tcp timestamp is by defauly disabled.
sysctl is as follows:
kernel.shmmax = 68719476736
kernel.shmall = 4294967296
kernel.pid_max=4194303
vm.max_map_count=1
+ priv = netdev_priv(rmnet_dev);
+ memset(priv, 0, sizeof(struct rmnet_priv));
Netdev private area is always zero on creation. See alloc_netdev().
+ if (!rmnet_dev)
+ return 0;
Do not confuse 0 with NULL. Did you run sparse?
+
+ priv = netdev_priv(rmne
On Thu, Aug 17, 2017 at 07:19:03PM +, Nick Huber wrote:
> I've been experience the following traceback since upgrading from the 4.9
> kernel to the 4.11 branch. I've only seen this in a few VMWare guests and I
> haven't been able to narrow down what exactly is causing it. I'm not familiar
>
+#define ETH_P_MAP 0xDA1A /* Multiplexing and
Aggregation Protocol
+* NOT AN OFFICIALLY REGISTERED ID ]
+*/
Hi Subash
This list is sorted. So this entry should go earlier.
+ netdev_info(real
Hi Rob,
On Mon, Aug 21, 2017 at 02:10:33PM -0500, Rob Herring wrote:
> On Sun, Aug 20, 2017 at 5:28 AM, Baruch Siach wrote:
> > Add device-tree binding documentation SFP transceivers. Support for SFP
> > transceivers has been recently introduced (drivers/net/phy/sfp.c).
> >
> > Signed-off-by: Bar
On Mon, Aug 21, 2017 at 10:15 AM, David Lamparter wrote:
> Hi all,
>
>
> this is an update on the earlier "[RFC net-next] VPLS support". Note
> I've changed the subject lines on some of the patches to better reflect
> what they really do (tbh the earlier subject lines were crap.)
>
> As previousl
Some boards uses a PHY with internal delay with an Allwinner SoC.
Support these PHY modes in the driver.
As the driver has no configuration registers for these modes, just treat
them as ordinary RGMII.
Signed-off-by: Icenowy Zheng
---
drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 3 +++
Some Pine64+ boards come with bad RTL8211E PHYs, which cannot work reliably
unless do some hack. According to Pine64 people, Realtek describes the hack
as totally disabling RX delay, and it's not documented at all.
This patchset introduces the workaround on Pine64+.
The first patch adds RGMII var
From: Icenowy Zheng
The page select register also exists on RTL8211E PHY (although it
behaves slightly differently).
Change the register macro name to remove the F.
Signed-off-by: Icenowy Zheng
---
drivers/net/phy/realtek.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
d
On Fri, 18 Aug 2017 05:54:08 -0700
William Tu wrote:
> The patch adds ERSPAN type II tunnel support. The implementation
> is based on the draft at https://tools.ietf.org/html/draft-foschiano-erspan-01
> One of the purposes is for Linux box to be able to receive ERSPAN
> monitoring traffic sent f
> +void rmnet_vnd_setup(struct net_device *rmnet_dev)
> +{
> + struct rmnet_priv *priv;
> +
> + /* Clear out private data */
> + priv = netdev_priv(rmnet_dev);
> + memset(priv, 0, sizeof(struct rmnet_priv));
Netdev private area is always zero on creation. See alloc_netdev().
> +s
From: Icenowy Zheng
Some RTL8211E chips have broken GbE function, which needs a hack to
fix. It's said that this fix will affect the performance on not-buggy
PHYs, so it should only be enabled on boards with the broken PHY.
Currently only some Pine64+ boards are known to have this issue.
This ha
Some Pine64+ boards have a broken RTL8211E PHY, which cannot work
reliably in 1000Base-T mode with default configuration.
A solution is passed to Pine64, which is said to be disabling the
internal RX delay of the PHY.
Enable the hack by set the PHY mode to RGMII-TXID.
Signed-off-by: Icenowy Zhen
>>> Interesting, deadlock could be treated as a a radical case of the
>>> discussion
>>> here https://patchwork.kernel.org/patch/3787671/.
>>>
>>> git grep tells more similar skb_orphan() cases. Do we need to change them
>>> all (or part)?
>>
>> Most skb_orphan calls are not relevant to the issue o
On 2017年08月21日 23:41, Willem de Bruijn wrote:
On Mon, Aug 21, 2017 at 8:33 AM, Jason Wang wrote:
On 2017年08月19日 14:38, Koichiro Den wrote:
Facing the possible unbounded delay relying on freeing on xmit path,
we also better to invoke and clear the upper layer zerocopy callback
beforehand to
From: Haiyang Zhang
The parameter "nvdev" is not in use.
Signed-off-by: Haiyang Zhang
---
drivers/net/hyperv/netvsc_drv.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c
index 4677d21..d8612b1 100644
From: Haiyang Zhang
The patch set adds the functions to switch UDP hash level between
L3 and L4 by ethtool command. UDP over IPv4 and v6 can be set
differently. The default hash level is L4. We currently only
allow switching TX hash level from within the guests.
The ethtool callback function is
From: Haiyang Zhang
Update Documentation/networking/netvsc.txt for UDP hash level setting
and related info.
Signed-off-by: Haiyang Zhang
---
Documentation/networking/netvsc.txt | 22 +-
1 files changed, 17 insertions(+), 5 deletions(-)
diff --git a/Documentation/networki
From: Haiyang Zhang
The patch add the functions to switch UDP hash level between
L3 and L4 by ethtool command. UDP over IPv4 and v6 can be set
differently. The default hash level is L4. We currently only
allow switching TX hash level from within the guests.
On Azure, fragmented UDP packets have
From: Haiyang Zhang
The parameter "sk" is not in use.
Signed-off-by: Haiyang Zhang
---
drivers/net/hyperv/netvsc_drv.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c
index b33f050..4677d21 100644
---
On Mon, Aug 21, 2017 at 06:19:44PM -0700, Felix Manlunas wrote:
> From: Weilin Chang
>
> Signed-off-by: Weilin Chang
> Signed-off-by: Felix Manlunas
> ---
> .../net/ethernet/cavium/liquidio/octeon_console.c | 85
> ++
> .../net/ethernet/cavium/liquidio/octeon_device.h |
Hi all,
Today's linux-next merge of the net-next tree got a conflict in:
arch/arm64/boot/dts/rockchip/rk3328-evb.dts
between commits:
ab78718bda79 ("arm64: dts: rockchip: Enable tsadc module on RK3328 eavluation
board")
1e28037ec88e ("arm64: dts: rockchip: add rk805 node for rk3328-evb")
From: Weilin Chang
Signed-off-by: Weilin Chang
Signed-off-by: Felix Manlunas
---
.../net/ethernet/cavium/liquidio/octeon_console.c | 85 ++
.../net/ethernet/cavium/liquidio/octeon_device.h | 5 ++
2 files changed, 90 insertions(+)
diff --git a/drivers/net/ethernet/cavi
> +static int rmnet_unregister_real_device(struct net_device *real_dev)
> +{
> + struct rmnet_real_dev_info *rdinfo;
> + struct list_head *iter;
> +
> + ASSERT_RTNL();
> +
> + if (!rmnet_is_real_dev_registered(real_dev) ||
> + netdev_lower_get_next(real_dev, &iter))
> +
On Mon, Aug 21, 2017 at 8:12 PM, Willem de Bruijn
wrote:
> On Mon, Aug 21, 2017 at 6:40 PM, Eric Dumazet wrote:
>> On Mon, 2017-08-21 at 17:39 -0400, Willem de Bruijn wrote:
>>> From: Willem de Bruijn
>>>
>>> When peeking, if a bad csum is discovered, the skb is unlinked from
>>> the queue with
On Mon, Aug 21, 2017 at 04:36:57PM -0600, Subash Abhinov Kasiviswanathan wrote:
> Define the multiplexing and aggregation (MAP) ether type 0xDA1A. This
> is needed for receiving data in the MAP protocol like RMNET. This is
> not an officially registered ID.
>
> Signed-off-by: Subash Abhinov Kasivi
On Mon, Aug 21, 2017 at 10:56 PM, Eric Dumazet wrote:
> Please do not top post.
Got it, thanks.
> On Mon, 2017-08-21 at 21:24 +0800, Tonghao Zhang wrote:
>> Thanks, yes this is a bug. I found this bug exists from 3.17~ 4.13.
>> The commit is d94e0417
>>
>
> This bug was there at the beginning o
On Mon, Aug 21, 2017 at 5:46 PM, Gao Feng wrote:
> Hi Cong,
>
> Thanks your reminder firstly.
> But I had used the get_maintainer.pl actually before sent the patch.
>
> The following is the output.
> [fgao@ikuai8 net-next]#./scripts/get_maintainer.pl
> patch_ScheCheck/0001-net-sched-Add-the-inval
At 2017-08-22 03:58:03, "Cong Wang" wrote:
>On Mon, Aug 21, 2017 at 10:47 AM, David Miller wrote:
>> From: gfree.w...@vip.163.com
>> Date: Fri, 18 Aug 2017 15:23:24 +0800
>>
>>> From: Gao Feng
>>>
>>> Add the invalid handle "0" check to avoid unnecessary search, because
>>> the qdisc uses the sk
On 08/21/2017 07:24 AM, Andrew Lunn wrote:
> On Mon, Aug 21, 2017 at 01:45:30PM +0200, Romain Perier wrote:
>> Currently, if this logging function is used prior the phy driver is
>> bound to the phy device (that is usually done from .ndo_open),
>> 'phydev->drv' might be NULL, resulting in a kernel
On 08/21/2017 04:45 AM, Romain Perier wrote:
> This code is no longer used, the logging function was changed by commit
> fbca164776e4 ("net: stmmac: Use the right logging functi").
>
> Fixes: fbca164776e4 ("net: stmmac: Use the right logging functi")
> Signed-off-by: Romain Perier
> ---
> driver
On Mon, 21 Aug 2017 16:46:51 +0200
Phil Sutter wrote:
> If fopen() succeeded but len != PATH_MAX, the function leaks the open
> FILE pointer. Fix this by checking len value before calling fopen().
>
> Signed-off-by: Phil Sutter
> Acked-by: Daniel Borkmann
Thanks, Applied
On Mon, 21 Aug 2017 19:08:08 +0200
Phil Sutter wrote:
> This shouldn't happen but relying upon external data without checking
> may lead to unexpected results.
>
> Signed-off-by: Phil Sutter
> ---
> misc/ss.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/misc/ss.c b
On Mon, Aug 21, 2017 at 05:01:51PM -0700, Stephen Hemminger wrote:
> On Mon, 21 Aug 2017 19:15:17 +0200 David Lamparter wrote:
> > > P.S.: For a little context on the bridge FDB changes - I'm hoping to
> > > find some time to extend this to the MDB to allow aggregating dst
> > > metadata and handi
On Mon, 21 Aug 2017 18:36:49 +0200
Phil Sutter wrote:
> This series collects patches from v1 dealing with spots where error
> checking is necessary or recommended.
>
> Minor changes to patches 1 and 2, patch 3 remains unchanged.
>
> Phil Sutter (3):
> iproute: Check mark value input
> iplin
On Mon, 21 Aug 2017 15:23:36 +0200
Phil Sutter wrote:
> Signed-off-by: Phil Sutter
> ---
> ip/xfrm_state.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/ip/xfrm_state.c b/ip/xfrm_state.c
> index e11c93bf1c3b5..7c0389038986e 100644
> --- a/ip/xfrm_state.c
> +++ b/ip
On Mon, 21 Aug 2017 19:08:07 +0200
Phil Sutter wrote:
> Introduce a wrapper which does the sanity checking and returns NULL
> in case fd is invalid.
>
> Signed-off-by: Phil Sutter
> ---
> misc/nstat.c | 15 +++
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/mi
On Mon, 21 Aug 2017 12:03:05 +0200
Phil Sutter wrote:
> If the string at 'p' contains neither space not newline, 'p' will become
> NULL. Make sure this isn't the case before dereferencing it.
>
> Signed-off-by: Phil Sutter
> ---
> Changes since v2:
> - Call abort() if 'p' becomes NULL.
> ---
>
On Mon, 21 Aug 2017 11:26:58 +0200
Phil Sutter wrote:
> This series collects patches from v1 which resolve situations where
> garbage might be read, either due to missing initialization of
> variables or accessing data which went out of scope.
>
> Changes since v2:
> - Rebased onto current maste
On Thu, 17 Aug 2017 19:09:24 +0200
Phil Sutter wrote:
> This series collects patches from v1 which deal with dead code, either
> by removing it or changing context so it is accessed again if that makes
> sense.
>
> No changes to the actual patches, just splitting into smaller series.
>
> Phil S
On Sun, 20 Aug 2017 12:58:20 +0300
Leon Romanovsky wrote:
> From: Leon Romanovsky
>
> This is fifth revision of series implementing the RDAMtool - the tool
> to configure RDMA devices.
>
> It looks like everyone who was interested to read cover letter already did it,
> so I'll start from the
On Mon, Aug 21, 2017 at 6:40 PM, Eric Dumazet wrote:
> On Mon, 2017-08-21 at 17:39 -0400, Willem de Bruijn wrote:
>> From: Willem de Bruijn
>>
>> When peeking, if a bad csum is discovered, the skb is unlinked from
>> the queue with __sk_queue_drop_skb and the peek operation restarted.
>>
>> __sk_
On 08/21/2017 04:23 PM, Florian Fainelli wrote:
> On 08/04/2017 10:11 AM, Eric Dumazet wrote:
>> On Fri, 2017-08-04 at 08:51 -0700, Florian Fainelli wrote:
>>> On 08/03/2017 10:36 PM, Eric Dumazet wrote:
On Thu, 2017-08-03 at 21:33 -0700, Florian Fainelli wrote:
> During testing with a bac
On Mon, 21 Aug 2017 19:15:17 +0200
David Lamparter wrote:
> Hi all,
>
>
> this is an update on the earlier "[RFC net-next] VPLS support". Note
> I've changed the subject lines on some of the patches to better reflect
> what they really do (tbh the earlier subject lines were crap.)
>
> As prev
From: Eric Biggers
Switch the DO_ONCE() macro from the deprecated jump label API to the new
one. The new one is more readable, and for DO_ONCE() it also makes the
generated code more icache-friendly: now the one-time initialization
code is placed out-of-line at the jump target, rather than at th
On 08/04/2017 10:11 AM, Eric Dumazet wrote:
> On Fri, 2017-08-04 at 08:51 -0700, Florian Fainelli wrote:
>> On 08/03/2017 10:36 PM, Eric Dumazet wrote:
>>> On Thu, 2017-08-03 at 21:33 -0700, Florian Fainelli wrote:
During testing with a background iperf pushing 1Gbit/sec worth of
traffic
All proper patch series must have a header "[PATCH xxx 0/N]" posting
which explains at a high level what the patch series does, how it does
it, and why it is doing it that way.
Therefore, please resubmit this patch series with a proper header
posting.
Thank you.
On Mon, 2017-08-21 at 17:39 -0400, Willem de Bruijn wrote:
> From: Willem de Bruijn
>
> When peeking, if a bad csum is discovered, the skb is unlinked from
> the queue with __sk_queue_drop_skb and the peek operation restarted.
>
> __sk_queue_drop_skb only drops packets that match the queue head.
On Mon, Aug 21, 2017 at 5:39 PM, Willem de Bruijn
wrote:
> From: Willem de Bruijn
>
> When peeking, if a bad csum is discovered, the skb is unlinked from
> the queue with __sk_queue_drop_skb and the peek operation restarted.
>
> __sk_queue_drop_skb only drops packets that match the queue head. Wi
Define the multiplexing and aggregation (MAP) ether type 0xDA1A. This
is needed for receiving data in the MAP protocol like RMNET. This is
not an officially registered ID.
Signed-off-by: Subash Abhinov Kasiviswanathan
---
include/uapi/linux/if_ether.h | 4 +++-
1 file changed, 3 insertions(+), 1
Define the raw IP type. This is needed for raw IP net devices
like rmnet.
Signed-off-by: Subash Abhinov Kasiviswanathan
---
include/uapi/linux/if_arp.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/uapi/linux/if_arp.h b/include/uapi/linux/if_arp.h
index cf73510..a2a6356 100644
---
RmNet driver provides a transport agnostic MAP (multiplexing and
aggregation protocol) support in embedded module. Module provides
virtual network devices which can be attached to any IP-mode
physical device. This will be used to provide all MAP functionality
on future hardware in a single consiste
This patch adds support for the rmnet driver which is required to
support recent chipsets using Qualcomm Technologies, Inc. modems. The data
from hardware follows the multiplexing and aggregation protocol (MAP).
This driver can be used to register onto any physical network device in
IP mode. Physi
On Mon, Aug 21, 2017 at 09:25:06PM +0200, Jesper Dangaard Brouer wrote:
>
> Third gotcha(3): You got this far, loaded xdp on both interfaces, and
> notice now that (with default setup) you can RX with 14Mpps but only
> TX with 6.9Mpps (and might have 5% idle cycles). I debugged this via
> perf tr
On Tue, 22 Aug 2017 00:06:17 +0200, Christophe JAILLET wrote:
> Check memory allocation failure and return -ENOMEM in such a case, as
> already done a few lines below.
>
> As 'dev->tx_q' can be NULL, we also need to check for that in
> 'mt7601u_free_tx()', and return early.
>
> Signed-off-by: Chr
On 08/21/2017 03:15 PM, woojung@microchip.com wrote:
> Florian,
>
>> -Original Message-
>> From: Florian Fainelli [mailto:f.faine...@gmail.com]
>> Sent: Monday, August 21, 2017 3:42 PM
>> To: netdev@vger.kernel.org
>> Cc: da...@davemloft.net; and...@lunn.ch;
>> vivien.dide...@savoirfai
Florian,
> -Original Message-
> From: Florian Fainelli [mailto:f.faine...@gmail.com]
> Sent: Monday, August 21, 2017 3:42 PM
> To: netdev@vger.kernel.org
> Cc: da...@davemloft.net; and...@lunn.ch;
> vivien.dide...@savoirfairelinux.com; Woojung Huh - C21699; Florian Fainelli
> Subject: [PAT
Le 21/08/2017 à 23:41, Jakub Kicinski a écrit :
On Mon, 21 Aug 2017 14:34:30 -0700, Jakub Kicinski wrote:
On Mon, 21 Aug 2017 22:59:56 +0200, Christophe JAILLET wrote:
Check memory allocation failure and return -ENOMEM in such a case, as
already done a few lines below
Signed-off-by: Christophe
On 08/14/2017 03:22 PM, Vivien Didelot wrote:
> This commit adds the boiler plate to create a DSA related debug
> filesystem entry as well as a "tree" file, containing the tree index.
>
> # cat switch1/tree
> 0
>
> Signed-off-by: Vivien Didelot
Reviewed-by: Florian Fainelli
--
Florian
Check memory allocation failure and return -ENOMEM in such a case, as
already done a few lines below.
As 'dev->tx_q' can be NULL, we also need to check for that in
'mt7601u_free_tx()', and return early.
Signed-off-by: Christophe JAILLET
---
v2: avoid another NULL pointer dereference in 'mt7601u_
From: Haiyang Zhang
The parameter "sk" is not in use.
Signed-off-by: Haiyang Zhang
---
drivers/net/hyperv/netvsc_drv.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c
index b33f050..4677d21 100644
---
From: Haiyang Zhang
The parameter "nvdev" is not in use.
Signed-off-by: Haiyang Zhang
---
drivers/net/hyperv/netvsc_drv.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c
index 4677d21..d8612b1 100644
From: Haiyang Zhang
The patch add the functions to switch UDP hash level between
L3 and L4 by ethtool command. UDP over IPv4 and v6 can be set
differently. The default hash level is L4. We currently only
allow switching TX hash level from within the guests.
On Azure, fragmented UDP packets have
From: Haiyang Zhang
Update Documentation/networking/netvsc.txt for UDP hash level setting
and related info.
Signed-off-by: Haiyang Zhang
---
Documentation/networking/netvsc.txt | 22 +-
1 files changed, 17 insertions(+), 5 deletions(-)
diff --git a/Documentation/networki
On Mon, 21 Aug 2017 14:34:30 -0700, Jakub Kicinski wrote:
> On Mon, 21 Aug 2017 22:59:56 +0200, Christophe JAILLET wrote:
> > Check memory allocation failure and return -ENOMEM in such a case, as
> > already done a few lines below
> >
> > Signed-off-by: Christophe JAILLET
>
> Acked-by: Jakub K
From: Willem de Bruijn
When peeking, if a bad csum is discovered, the skb is unlinked from
the queue with __sk_queue_drop_skb and the peek operation restarted.
__sk_queue_drop_skb only drops packets that match the queue head. With
sk_peek_off, the skb need not be at head, causing the call to fai
On Mon, 21 Aug 2017 22:59:56 +0200, Christophe JAILLET wrote:
> Check memory allocation failure and return -ENOMEM in such a case, as
> already done a few lines below
>
> Signed-off-by: Christophe JAILLET
Acked-by: Jakub Kicinski
Thanks!
On 8/21/17 2:00 PM, Daniel Borkmann wrote:
On 08/21/2017 10:44 PM, Edward Cree wrote:
On 21/08/17 21:27, Daniel Borkmann wrote:
On 08/21/2017 08:36 PM, Edward Cree wrote:
On 19/08/17 00:37, Alexei Starovoitov wrote:
[...]
I'm tempted to just rip out env->varlen_map_value_access and always
ch
On 8/21/17 1:24 PM, Edward Cree wrote:
On 18/08/17 15:16, Edward Cree wrote:
On 18/08/17 04:21, Alexei Starovoitov wrote:
It seems you're trying to sort-of do per-fake-basic block liveness
analysis, but our state_list_marks are not correct if we go with
canonical basic block definition, since w
From: Eric Dumazet
Date: Fri, 18 Aug 2017 20:40:01 -0700
> Let look at tun->pcpu_stats, for example.
>
> It is allocated at line 1831, before the register_netdevice()
>
> drivers/net/tun.c does not provide ndo_init()
I see the problem now.
And it's done this way because several steps need to
From: Subash Abhinov Kasiviswanathan
Date: Fri, 18 Aug 2017 23:35:31 -0600
> diff --git a/drivers/net/ethernet/qualcomm/Makefile
> b/drivers/net/ethernet/qualcomm/Makefile
> index 92fa7c4..c4f38bd 100644
> --- a/drivers/net/ethernet/qualcomm/Makefile
> +++ b/drivers/net/ethernet/qualcomm/Makefil
On 08/21/2017 10:44 PM, Edward Cree wrote:
On 21/08/17 21:27, Daniel Borkmann wrote:
On 08/21/2017 08:36 PM, Edward Cree wrote:
On 19/08/17 00:37, Alexei Starovoitov wrote:
[...]
I'm tempted to just rip out env->varlen_map_value_access and always check
the whole thing, because honestly I d
Check memory allocation failure and return -ENOMEM in such a case, as
already done a few lines below
Signed-off-by: Christophe JAILLET
---
drivers/net/wireless/mediatek/mt7601u/dma.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/wireless/mediatek/mt7601u/dma.c
b/drivers/net/
On 21/08/17 21:27, Daniel Borkmann wrote:
> On 08/21/2017 08:36 PM, Edward Cree wrote:
>> On 19/08/17 00:37, Alexei Starovoitov wrote:
> [...]
>> I'm tempted to just rip out env->varlen_map_value_access and always check
>> the whole thing, because honestly I don't know what it was meant to do
>>
From: gfree.w...@vip.163.com
Date: Fri, 18 Aug 2017 15:23:24 +0800
> From: Gao Feng
>
> Add the invalid handle "0" check to avoid unnecessary search, because
> the qdisc uses the skb->priority as the handle value to look up, and
> it is "0" usually.
>
> Signed-off-by: Gao Feng
Applied, thanks
From: Jon Maloy
Date: Mon, 21 Aug 2017 17:59:30 +0200
> When the broadcast send link after 100 attempts has failed to
> transfer a packet to all peers, we consider it stale, and reset
> it. Thereafter it needs to re-synchronize with the peers, something
> currently done by just resetting and re-e
On 08/21/2017 08:36 PM, Edward Cree wrote:
On 19/08/17 00:37, Alexei Starovoitov wrote:
[...]
I'm tempted to just rip out env->varlen_map_value_access and always check
the whole thing, because honestly I don't know what it was meant to do
originally or how it can ever do any useful pruning.
On 18/08/17 15:16, Edward Cree wrote:
> On 18/08/17 04:21, Alexei Starovoitov wrote:
>> It seems you're trying to sort-of do per-fake-basic block liveness
>> analysis, but our state_list_marks are not correct if we go with
>> canonical basic block definition, since we mark the jump insn and
>> not
On 17-08-21 03:58 PM, Cong Wang wrote:
On Mon, Aug 21, 2017 at 10:47 AM, David Miller wrote:
From: gfree.w...@vip.163.com
Date: Fri, 18 Aug 2017 15:23:24 +0800
From: Gao Feng
Add the invalid handle "0" check to avoid unnecessary search, because
the qdisc uses the skb->priority as the handle
On Fri, Aug 18, 2017 at 11:44 AM, Vallish Vaidyeshwara
wrote:
> - *timeo_p = schedule_timeout(*timeo_p);
> + /* Wait using highres timer */
> + expires = ktime_add_ns(ktime_get(), jiffies_to_nsecs(*timeo_p));
> + pre_sched_time = jiffies;
> + if (schedule_hrtimeout(&e
skb_put_padto() will free the sk_buff passed as reference in case of
errors, but we still need to check its return value and decide what to
do.
Detected by CoverityScan, CID#1416688 ("CHECKED_RETURN")
Fixes: ee1c27977284 ("net/hsr: Added support for HSR v1")
Signed-off-by: Florian Fainelli
---
On Mon, Aug 21, 2017 at 10:47 AM, David Miller wrote:
> From: gfree.w...@vip.163.com
> Date: Fri, 18 Aug 2017 15:23:24 +0800
>
>> From: Gao Feng
>>
>> Add the invalid handle "0" check to avoid unnecessary search, because
>> the qdisc uses the skb->priority as the handle value to look up, and
>> i
On 08/21/2017 07:53 AM, icen...@aosc.io wrote:
> 在 2017-05-05 02:29，Florian Fainelli 写道：
>> On 05/04/2017 11:26 AM, Icenowy Zheng wrote:
>>>
>>>
>>> 于 2017年5月5日 GMT+08:00 上午2:21:29, Florian Fainelli
>>> 写到:
On 05/04/2017 11:10 AM, icen...@aosc.io wrote:
> 在 2017-04-22 08:22，Florian Fainel
On Mon, Aug 21, 2017 at 12:41:43PM -0700, Florian Fainelli wrote:
> skb_put_padto() already frees the passed sk_buff reference upon error,
> so calling kfree_skb() on it again is not necessary.
>
> Detected by CoverityScan, CID#1416687 ("USE_AFTER_FREE")
>
> Fixes: e71cb9e00922 ("net: dsa: ksz: f
On Sat, Aug 19, 2017 at 03:41:14PM +0900, Koichiro Den wrote:
> Signed-off-by: Koichiro Den
> ---
> drivers/vhost/vhost.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
> index e4613a3c362d..93e909afc1c3 100644
> --- a/drive
During access_ok checks, addr increases as we iterate over the data
structure, thus addr + len - 1 will point beyond the end of region we
are translating. Harmless since we then verify that the region covers
addr, but let's not waste cpu cycles.
Reported-by: Koichiro Den
Signed-off-by: Michael S
1 - 100 of 300 matches
Mail list logo
