Signed-off-by: Wei Liu
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 0c55b0fedbe2..e212c6a42ddf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17295,7 +17295,7 @@ F: Documentation/ABI/stable/sysfs-hypervisor-xen
F: Do
In file included from net/netfilter/utils.c:5:
include/linux/netfilter_ipv6.h: In function 'nf_ipv6_br_defrag':
include/linux/netfilter_ipv6.h:110:9: error: implicit declaration of function
'nf_ct_frag6_gather'; did you mean 'nf_ct_attach'?
[-Werror=implicit-function-declaration]
return nf_ct_f
From: Nathan Chancellor
Clang warns:
In file included from drivers/net/ethernet/intel/iavf/iavf_main.c:4:
In file included from drivers/net/ethernet/intel/iavf/iavf.h:37:
In file included from drivers/net/ethernet/intel/iavf/iavf_type.h:8:
drivers/net/ethernet/intel/iavf/iavf_osdep.h:49:18: warn
From: Alice Michael
Update the old i40e function names to be iavf
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf_adminq.c| 6 +++---
drivers/net/ethernet/intel/iavf/iavf_prototype.h | 4 ++--
2 files changed, 5 i
From: "Gustavo A. R. Silva"
One of the more common cases of allocation size calculations is finding
the size of a structure that has a zero-sized array at the end, along
with memory for some number of elements for that array. For example:
struct foo {
int stuff;
struct boo entry[];
};
s
From: "Gustavo A. R. Silva"
One of the more common cases of allocation size calculations is finding
the size of a structure that has a zero-sized array at the end, along
with memory for some number of elements for that array. For example:
struct foo {
int stuff;
struct boo entry[];
};
s
From: Alice Michael
Some small things were missed with recent name changes
from i40e to iavf. Having a separate patch allows to
correct the small misses in one place.
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf_c
This series contains updates to the iavf driver.
Nathan Chancellor converts the use of gnu_printf to printf.
Aleksandr modifies the driver to limit the number of RSS queues to the
number of online CPUs in order to avoid creating misconfigured RSS
queues.
Gustavo A. R. Silva converts a couple of
From: Alice Michael
The defines in iavf_client.h were still vastly i40e, and they
should be iavf.
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf.h| 2 +-
drivers/net/ethernet/intel/iavf/iavf_client.c | 84
From: Alice Michael
rename the flags inside of iavf_status from I40E_*
to IAVF_*
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf_adminq.c | 42 +--
drivers/net/ethernet/intel/iavf/iavf_adminq.h | 2 +-
drivers/net/
From: Aleksandr Loktionov
Limiting RSS queues number to online CPUs number in order to
avoid issues with creating misconfigured RSS queues.
Signed-off-by: Aleksandr Loktionov
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf_virtchnl.c | 2 +-
1 fi
From: Alice Michael
Update the old variables and flags marked as i40e to match
the iavf name of the driver.
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf_adminq.c | 80 +--
drivers/net/ethernet/intel/iavf/iavf_admi
From: Sergey Nemov
Instead of typedefing the enum iavf_status_code with iavf_status,
just shorten the enum itself and get rid of typedef.
Signed-off-by: Sergey Nemov
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf.h| 2 +-
drivers/net/et
From: Alice Michael
There were a couple of erroneously missed i40e names to
update to iavf left after the larger chunks. Updated them
separately so now they should all be aligned as iavf.
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ether
From: Alice Michael
Renaming remaining defines from i40e to iavf
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/iavf.h| 4 ++--
drivers/net/ethernet/intel/iavf/iavf_client.c | 20 +--
2 files chan
From: Alice Michael
With the rename of the iavf driver, there were some
files that were missed in renaming. Update these to
be iavf as well.
Signed-off-by: Alice Michael
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/iavf/Makefile
> -Original Message-
> From: Wei Liu [mailto:wei.l...@citrix.com]
> Sent: 31 May 2019 08:31
> To: netdev@vger.kernel.org
> Cc: Xen-devel ; Paul Durrant
> ; David Miller
> ; Wei Liu
> Subject: [PATCH net-next] Update my email address
>
> Signed-off-by: Wei Liu
Acked-by: Paul Durrant
>
On Wed, May 29, 2019 at 12:49:55PM +0300, same...@amazon.com wrote:
> From: Arthur Kiyanovski
>
> This commit adds a mechanism for exposing different driver
> properties via ethtool's priv_flags.
>
> In this commit we:
>
> Add commands, structs and defines necessary for handling
> extra propert
From: Kevin Darbyshire-Bryant
ctinfo is an action restoring data stored in conntrack marks to various
fields. At present it has two independent modes of operation,
restoration of DSCP into IPv4/v6 diffserv and restoration of conntrack
marks into packet skb marks.
It understands a number of para
On Wed, May 29, 2019 at 7:48 AM Herbert Xu wrote:
>
> On Wed, May 29, 2019 at 07:43:51AM +0200, Dmitry Vyukov wrote:
> >
> > If fqdir->dead read/write are concurrent, then this still needs to be
> > READ_ONCE/WRITE_ONCE. Ordering is orthogonal to atomicity.
>
> No they do not. READ_ONCE/WRITE_ONC
There is a possible null pointer deference bugs in neigh_fill_info(),
which is similar to the bug which was fixed in commit 6adc5fd6a142
("net/neighbour: fix crash at dumping device-agnostic proxy entries").
Signed-off-by: Young Xiao <92siuy...@gmail.com>
---
net/core/neighbour.c | 2 +-
1 file c
> On 30 May 2019, at 20:59, Stephen Hemminger
> wrote:
>
> On Thu, 30 May 2019 16:43:20 +
> Kevin 'ldir' Darbyshire-Bryant wrote:
>
> Please don't use HTML encoded mail. I.e not exchange.
As far as I know I’m not sending HTML emails, or I’m trying hard to send
plain text from mac mail,
There is a possible null pointer deference bug in vxlan_fdb_info(),
which is similar to the bug which was fixed in commit 6adc5fd6a142
("net/neighbour: fix crash at dumping device-agnostic proxy entries").
Signed-off-by: Young Xiao <92siuy...@gmail.com>
---
drivers/net/vxlan.c | 2 +-
1 file chan
On 31.05.2019 11:29, Young Xiao wrote:
There is a possible null pointer deference bugs in neigh_fill_info(),
which is similar to the bug which was fixed in commit 6adc5fd6a142
("net/neighbour: fix crash at dumping device-agnostic proxy entries").
Have you seen this in real life?
I see nobody wh
On Fri, 2019-05-31 at 16:29 +0800, Young Xiao wrote:
> There is a possible null pointer deference bugs in neigh_fill_info(),
> which is similar to the bug which was fixed in commit 6adc5fd6a142
> ("net/neighbour: fix crash at dumping device-agnostic proxy entries").
>
> Signed-off-by: Young Xiao <
On Fri, 2019-05-31 at 16:34 +0800, Young Xiao wrote:
> There is a possible null pointer deference bug in vxlan_fdb_info(),
> which is similar to the bug which was fixed in commit 6adc5fd6a142
> ("net/neighbour: fix crash at dumping device-agnostic proxy entries").
>
> Signed-off-by: Young Xiao <92
On Thu, 2019-05-30 at 11:08 -0700, Stephen Hemminger wrote:
> On Thu, 30 May 2019 20:03:41 +0200
> Davide Caratti wrote:
>
> >
> > +static inline int tc_skb_pull_vlans(struct sk_buff *skb,
> > + unsigned int *hdr_count,
> > + __be16 *p
From: wenxu
When CONFIG_IPV6 is not build with modules and CONIFG_NF_CONNTRACK_BRIDGE=m
There will compile err:
net/ipv6/netfilter.c:242:2: error: unknown field 'br_defrag' specified in
initializer
.br_defrag = nf_ct_frag6_gather,
net/ipv6/netfilter.c:243:2: error: unknown field 'br_fragment'
+Cc we...@ucloud.cn
On Fri, May 31, 2019 at 10:11:43AM +0200, Pablo Neira Ayuso wrote:
> In file included from net/netfilter/utils.c:5:
> include/linux/netfilter_ipv6.h: In function 'nf_ipv6_br_defrag':
> include/linux/netfilter_ipv6.h:110:9: error: implicit declaration of function
> 'nf_ct_frag6
On 2019/5/31 16:02, Pablo Neira Ayuso wrote:
> On Fri, May 31, 2019 at 11:06:49AM +0800, Yuehaibing wrote:
>> +cc netdev
>>
>> On 2019/5/31 10:46, YueHaibing wrote:
>>> Fix gcc build error while CONFIG_IPV6 is not set
>>>
>>> In file included from net/netfilter/core.c:19:0:
>>> ./include/linux/n
This patch fixes a few problems with CONFIG_IPV6=y and
CONFIG_NF_CONNTRACK_BRIDGE=m:
In file included from net/netfilter/utils.c:5:
include/linux/netfilter_ipv6.h: In function 'nf_ipv6_br_defrag':
include/linux/netfilter_ipv6.h:110:9: error: implicit declaration of function
'nf_ct_frag6_gather';
This patch fixes a few problems with CONFIG_IPV6=y and
CONFIG_NF_CONNTRACK_BRIDGE=m:
In file included from net/netfilter/utils.c:5:
include/linux/netfilter_ipv6.h: In function 'nf_ipv6_br_defrag':
include/linux/netfilter_ipv6.h:110:9: error: implicit declaration of function
'nf_ct_frag6_gather';
On Fri, May 31, 2019 at 11:11:24AM +0200, Pablo Neira Ayuso wrote:
> This patch fixes a few problems with CONFIG_IPV6=y and
> CONFIG_NF_CONNTRACK_BRIDGE=m:
>
> In file included from net/netfilter/utils.c:5:
> include/linux/netfilter_ipv6.h: In function 'nf_ipv6_br_defrag':
> include/linux/netfilte
Hi,
On Fri, May 31, 2019 at 05:02:55PM +0800, we...@ucloud.cn wrote:
> From: wenxu
>
> When CONFIG_IPV6 is not build with modules and CONIFG_NF_CONNTRACK_BRIDGE=m
> There will compile err:
> net/ipv6/netfilter.c:242:2: error: unknown field 'br_defrag' specified in
> initializer
> .br_defrag
On 30 May 2019, at 13:19, Stefano Garzarella wrote:
>
> On Tue, May 28, 2019 at 04:01:00PM +, Jorgen Hansen wrote:
>>> On Thu, May 23, 2019 at 04:37:03PM +0100, Stefan Hajnoczi wrote:
On Tue, May 14, 2019 at 10:15:43AM +0200, Stefano Garzarella wrote:
>
>
>
> 2. listen() /
xdping allows us to get latency estimates from XDP. Output looks
like this:
./xdping -I eth4 192.168.55.8
Setting up XDP for eth4, please wait...
XDP setup disrupts network connectivity, hit Ctrl+C to quit
Normal ping RTT data
[Ignore final RTT; it is distorted by XDP using the reply]
PING 192.1
On Thu, May 30, 2019 at 08:06:16PM +0200, Andrew Lunn wrote:
> The kernel can now indicate if the PHY supports operating over a
> single pair at 100Mbps or 1000Mbps.
>
> Signed-off-by: Andrew Lunn
> ---
> ethtool.c | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/ethtool.c b/ethto
don't hardcode the 'pipe' control action, so that the following TDC test
b776 - Replace simple action with invalid goto chain control
can detect kernels that correctly validate 'goto chain' control action.
CC: Andrea Claudi
CC: Marcelo Ricardo Leitner
Signed-off-by: Davide Caratti
---
tc/m_
Signed-off-by: wenxu
On 5/31/2019 5:15 PM, Pablo Neira Ayuso wrote:
> This patch fixes a few problems with CONFIG_IPV6=y and
> CONFIG_NF_CONNTRACK_BRIDGE=m:
>
> In file included from net/netfilter/utils.c:5:
> include/linux/netfilter_ipv6.h: In function 'nf_ipv6_br_defrag':
> include/linux/netf
From: Björn Töpel
Remove all use of XDP_QUERY_PROG{,_HW}, since it was moved to the
generic code path.
Signed-off-by: Björn Töpel
---
drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.c | 4
.../net/ethernet/cavium/thunder/nicvf_main.c | 3 ---
.../net/ethernet/freescale/dpaa2/dpaa2-eth.c |
From: Björn Töpel
All XDP capable drivers need to implement the XDP_QUERY_PROG{,_HW}
command of ndo_bpf. The query code is fairly generic. This commit
refactors the query code up from the drivers to the netdev level.
The struct net_device has gained two new members: xdp_prog_hw and
xdp_flags. Th
Here's yet another attempt tomove the XDP_QUERY_PROG{,_HW} code out
from the drivers to generic netdev code.
The first patch in the series move the XDP query functionality, and
the second remove XDP_QUERY_PROG{,_HW} from all drivers.
Please refer to the individual commit messages for more details
On Fri, 31 May 2019 at 11:42, Björn Töpel wrote:
>
[...]
>
> I, hopefully, addressed all comments from Jakub and Saeed, except one;
> I did not move the XDP struct net_device into a struct of its own.
>
Uhm, the last sentence was weird.
What I meant was: I did not move the newly introduced XDP m
On Thu, May 30, 2019 at 03:50:09PM -0600, David Ahern wrote:
> On 5/30/19 2:52 PM, George Wilkie wrote:
> > This doesn't work for me (again, not using namespaces).
> > For traffic coming in on vrf-b to a destination on 10.200.2.0,
> > I see ARPs going out for the destination on xvrf2/in on xvrf1,
>
Hi Nirranjan,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Nirranjan-Kirubaharan/cxgb4-Set-initial-IRQ-affinity-hints/20190531-180720
config: xtensa-allyesconfig (attached as .config)
compiler
On 2019-05-30 20:57, Jonathan Lemon wrote:
Currently, the AF_XDP code uses a separate map in order to
determine if an xsk is bound to a queue. Instead of doing this,
have bpf_map_lookup_elem() return the queue_id, as a way of
indicating that there is a valid entry at the map index.
Rearrange so
On 2019-05-30 20:57, Jonathan Lemon wrote:
Use the recent change to XSKMAP bpf_map_lookup_elem() to test if
there is a xsk present in the map instead of duplicating the work
with qidconf.
Fix things so callers using XSK_LIBBPF_FLAGS__INHIBIT_PROG_LOAD
bypass any internal bpf maps, so xsk_socket_
> > @@ -634,10 +636,14 @@ static void dump_link_caps(const char *prefix, const
> > char *an_prefix,
> > "100baseT/Half" },
> > { 1, ETHTOOL_LINK_MODE_100baseT_Full_BIT,
> > "100baseT/Full" },
> > + { 1, ETHTOOL_LINK_MODE_100baseT1_Full_BIT,
> > +
while at it, fix missing square bracket near 'ptype' and a typo in the
action description (it's -> its).
Signed-off-by: Davide Caratti
---
man/man8/tc-skbedit.8 | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/man/man8/tc-skbedit.8 b/man/man8/tc-skbedit.8
index
Am Freitag, den 31.05.2019, 13:59 +0200 schrieb Andrew Lunn:
> > > @@ -634,10 +636,14 @@ static void dump_link_caps(const char *prefix,
> > > const char *an_prefix,
> > > > > > "100baseT/Half" },
> > > > > > { 1, ETHTOOL_LINK_MODE_100baseT_Full_BIT,
> > > > > >
On Fri, May 31, 2019 at 10:47:27AM +0300, Nikita Yushchenko wrote:
> This change allows phytool [1] and similar tools to read and write C45 phy
> registers from userspace.
>
> This is useful for debugging and for porting vendor phy diagnostics tools.
>
> [1] https://github.com/wkz/phytool
Hi Nik
I was wondering if there is a way to pop multiple MPLS labels at
once for local delivery using iproute2?
Adding multiple labels for encapsulation is supported via
label1/label2/... syntax, but I can't find a way to do the same
for popping multiple labels for local delivery.
For example if I run
v2:
- remove ifa_list iteration in afs instead of conversion
All other patches are unchanged.
Eric Dumazet reported following problem:
It looks that unless RTNL is held, accessing ifa_list needs proper RCU
protection. indev->ifa_list can be changed under us by another cpu
(which owns RTN
David Howell says:
I'm told that there's not really any point populating the list.
Current OpenAFS ignores it, as does AuriStor - and IBM AFS 3.6 will
do the right thing.
The list is actually useless as it's the client's view of the world,
not the servers, so if there's any NAT in the way
This also replaces spots that used for_primary_ifa().
for_primary_ifa() aborts the loop on the first secondary address seen.
Replace it with either the rcu or rtnl variant of in_dev_for_each_ifa(),
but two places will now also consider secondary addresses too:
inet_addr_onlink() and inet_ifa_bypr
Netfilter hooks are always running under rcu read lock, use
the new iterator macro so sparse won't complain once we add
proper __rcu annotations.
Signed-off-by: Florian Westphal
---
net/ipv4/netfilter/nf_tproxy_ipv4.c| 9 +++--
net/netfilter/nf_conntrack_broadcast.c | 9 +++--
net/ne
ifa_list is protected by rcu, yet code doesn't reflect this.
Add the __rcu annotations and fix up all places that are now reported by
sparse.
I've done this in the same commit to not add intermediate patches that
result in new warnings.
Reported-by: Eric Dumazet
Signed-off-by: Florian Westphal
Like previous patches, use the new iterator macros to avoid sparse
warnings once proper __rcu annotations are added.
Compile tested only.
Signed-off-by: Florian Westphal
---
drivers/infiniband/core/roce_gid_mgmt.c | 5 +++--
drivers/infiniband/hw/cxgb4/cm.c |
Use in_dev_for_each_ifa_rcu/rtnl instead.
This prevents sparse warnings once proper __rcu annotations are added.
Signed-off-by: Florian Westphal
t di# Last commands done (6 commands done):
---
net/ipv4/fib_frontend.c | 24 +---
net/ipv4/igmp.c | 5 +++--
net/ipv6/ad
The ifa_list is protected either by rcu or rtnl lock, but the
current iterators do not account for this.
This adds two iterators as replacement, a later patch in
the series will update them with the needed rcu/rtnl_dereference calls.
Its not done in this patch yet to avoid sparse warnings -- the
> That's not just theory. The Broadcom BCM54811 PHY supports both
> 100/1000baseT, as well as 100baseT1.
Hi Lucus
There does not appear to be a driver for it, which is why i've not
seen it, nor have we had this conversation before.
Do you have a driver to submit?
Andrew
The mlx5_core driver uses several indirect calls in fast-path, some of them
are invoked on each ingress packet, even for the XDP-only traffic.
This series leverage the indirect call wrappers infrastructure the avoid
the expansive RETPOLINE overhead for 2 indirect calls in fast-path.
Each call is
We can avoid another indirect call per packet wrapping the rx
handler call with the proper helper.
To ensure that even the last listed direct call experience
measurable gain, despite the additional conditionals we must
traverse before reaching it, I tested reversing the order of the
listed options
Experimental results[1] has shown that resorting to several branches
and a direct-call is faster than indirect call via retpoline, even
when the number of added branches go up 5.
This change adds two additional helpers, to cope with indirect calls
with up to 4 available direct call option. We will
We can avoid an indirect call per packet wrapping the skb creation
with the appropriate helper.
Signed-off-by: Paolo Abeni
---
.../net/ethernet/mellanox/mlx5/core/en_rx.c | 22 ++-
1 file changed, 17 insertions(+), 5 deletions(-)
diff --git a/drivers/net/ethernet/mellanox/mlx5
On Tue, May 28, 2019 at 10:27:21AM +0100, Russell King wrote:
> Ensure that we supply the same phy interface mode to mac_link_down() as
> we did for the corresponding mac_link_up() call. This ensures that MAC
> drivers that use the phy interface mode in these methods can depend on
> mac_link_down(
Am Freitag, den 31.05.2019, 14:30 +0200 schrieb Andrew Lunn:
> > That's not just theory. The Broadcom BCM54811 PHY supports both
> > 100/1000baseT, as well as 100baseT1.
>
> Hi Lucus
>
> There does not appear to be a driver for it, which is why i've not
> seen it, nor have we had this conversatio
Zdravstvuyte! Vas interesuyut kliyentskiye bazy dannykh?
On Fri, May 31, 2019 at 03:02:26PM +0200, Andrew Lunn wrote:
> On Tue, May 28, 2019 at 10:27:21AM +0100, Russell King wrote:
> > Ensure that we supply the same phy interface mode to mac_link_down() as
> > we did for the corresponding mac_link_up() call. This ensures that MAC
> > drivers that use t
Import the latest ethtool.h and add two new links modes.
v2:
Move the new speeds to the end of the all_advertised_modes_bits[].
Remove the same_line bit for the new moved
Add the new modes to the man page.
Andrew Lunn (2):
ethtool: sync ethtool-copy.h with linux-next from 30/05/2019
ethtool:
The kernel can now indicate if the PHY supports operating over a
single pair at 100Mbps or 1000Mbps.
Signed-off-by: Andrew Lunn
---
ethtool.8.in | 2 ++
ethtool.c| 6 ++
2 files changed, 8 insertions(+)
diff --git a/ethtool.8.in b/ethtool.8.in
index 430d11b915af..6af63455c636 100644
---
Sync ethtool-copy.h with linux-next from 22/05/2019. This provides
access to the new link modes for 100BaseT1 and 1000BaseT1.
Signed-off-by: Andrew Lunn
---
ethtool-copy.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/ethtool-copy.h b/ethtool-copy.h
index 92ab10d65fc9
On Thu, 30 May 2019 14:50:21 -0700
Tom Herbert wrote:
> Mutable fields related to segment routing are: destination address,
> segments left, and modifiable TLVs (those whose high order bit is set).
>
> Add support to rearrange a segment routing (type 4) routing header to
> handle these mutabilit
On Fri, May 31, 2019 at 10:24:08AM +0200, Dmitry Vyukov wrote:
>
> OK, let's call it barrier. But we need more than a barrier here then.
READ_ONCE/WRITE_ONCE is not some magical dust that you sprinkle
around in your code to make it work without locks. You need to
understand exactly why you need t
On Fri, May 31, 2019 at 7:05 AM Ahmed Abdelsalam wrote:
>
> On Thu, 30 May 2019 14:50:21 -0700
> Tom Herbert wrote:
>
> > Mutable fields related to segment routing are: destination address,
> > segments left, and modifiable TLVs (those whose high order bit is set).
> >
> > Add support to rearrang
On 5/31/19 4:38 AM, George Wilkie wrote:
> What are your thoughts on creating a "vrfdefault" for "local" table?
>ip link add vrfdefault type vrf table local
>ip link set dev vrfdefault up
>ip ro add vrf vrfA 10.10.3.0/24 dev vrfdefault
>ip ro add 10.10.2.0/24 dev vrfA
>ip -6 ro
On 5/31/19 1:29 AM, Young Xiao wrote:
> There is a possible null pointer deference bugs in neigh_fill_info(),
> which is similar to the bug which was fixed in commit 6adc5fd6a142
> ("net/neighbour: fix crash at dumping device-agnostic proxy entries").
>
> Signed-off-by: Young Xiao <92siuy...@gm
On 5/31/19 1:34 AM, Young Xiao wrote:
> There is a possible null pointer deference bug in vxlan_fdb_info(),
> which is similar to the bug which was fixed in commit 6adc5fd6a142
> ("net/neighbour: fix crash at dumping device-agnostic proxy entries").
>
> Signed-off-by: Young Xiao <92siuy...@gmai
Hi Richard,
I love your patch! Yet something to improve:
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Richard-Cochran/Peer-to-Peer-One-Step-time-stamping/20190531-213601
config: s390-debug_defconfig (attached as .config)
compiler: s390-linux-gcc
Florian Westphal wrote:
> David Howell says:
"Howells"
Apart from that:
Tested-by: David Howells
On 5/31/19 7:45 AM, Herbert Xu wrote:
> On Fri, May 31, 2019 at 10:24:08AM +0200, Dmitry Vyukov wrote:
>>
>> OK, let's call it barrier. But we need more than a barrier here then.
>
> READ_ONCE/WRITE_ONCE is not some magical dust that you sprinkle
> around in your code to make it work without lo
On Thu, May 30, 2019 at 9:34 PM Gen Zhang wrote:
>
> In selinux_sb_eat_lsm_opts(), 'arg' is allocated by kmemdup_nul(). It
> returns NULL when fails. So 'arg' should be checked.
>
> Signed-off-by: Gen Zhang
> Reviewed-by: Ondrej Mosnacek
> Fixes: 99dbbb593fe6 ("selinux: rewrite selinux_sb_eat_ls
On Fri, May 31, 2019 at 03:57:48PM +0200, Andrew Lunn wrote:
> The kernel can now indicate if the PHY supports operating over a
> single pair at 100Mbps or 1000Mbps.
>
> Signed-off-by: Andrew Lunn
> ---
Reviewed-by: Michal Kubecek
> ethtool.8.in | 2 ++
> ethtool.c| 6 ++
> 2 files ch
On Fri, May 31, 2019 at 03:57:47PM +0200, Andrew Lunn wrote:
> Sync ethtool-copy.h with linux-next from 22/05/2019. This provides
> access to the new link modes for 100BaseT1 and 1000BaseT1.
>
> Signed-off-by: Andrew Lunn
> ---
Reviewed-by: Michal Kubecek
BtW, this differs from the file "make
> On May 31, 2019, at 2:24 AM, Alan Maguire wrote:
>
> xdping allows us to get latency estimates from XDP. Output looks
> like this:
>
> ./xdping -I eth4 192.168.55.8
> Setting up XDP for eth4, please wait...
> XDP setup disrupts network connectivity, hit Ctrl+C to quit
>
> Normal ping RTT
On Fri, May 31, 2019 at 2:12 PM Davide Caratti wrote:
>
> while at it, fix missing square bracket near 'ptype' and a typo in the
> action description (it's -> its).
>
> Signed-off-by: Davide Caratti
> ---
> man/man8/tc-skbedit.8 | 13 ++---
> 1 file changed, 10 insertions(+), 3 deletions
> Reviewed-by: Michal Kubecek
>
> BtW, this differs from the file "make headers_install" produces in
> net-next but only in white space so that it doesn't really matter and it
> gets sorted in a future sync.
Yes, there is something odd going on. It looks like two tabs have been
converted to spac
Hi Richard,
I love your patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Richard-Cochran/Peer-to-Peer-One-Step-time-stamping/20190531-213601
config: m68k-allmodconfig (attached as .config)
compiler: m68k-linux
On Fri, 2019-05-31 at 01:15 -0700, Jeff Kirsher wrote:
> From: Nathan Chancellor
> We can convert from gnu_printf to printf without any side effects
[]
> diff --git a/drivers/net/ethernet/intel/iavf/iavf_osdep.h
> b/drivers/net/ethernet/intel/iavf/iavf_osdep.h
[]
> @@ -46,7 +46,7 @@ struct iavf_v
On Fri, 31 May 2019 08:51:43 +0200 Tom Barbette wrote:
> CCing mlx5 maintainers and commiters of bce2b2b. TLDK: there is a huge
> CPU increase on CX5 when introducing a XDP program.
>
> See https://www.youtube.com/watch?v=o5hlJZbN4Tk&feature=youtu.be
> around 0:40. We're talking something like
On 31 May 2019, at 4:49, Björn Töpel wrote:
> On 2019-05-30 20:57, Jonathan Lemon wrote:
>> Currently, the AF_XDP code uses a separate map in order to
>> determine if an xsk is bound to a queue. Instead of doing this,
>> have bpf_map_lookup_elem() return the queue_id, as a way of
>> indicating th
v3: fix typo in patch1 commit message
All other patches are unchanged.
v2: remove ifa_list iteration in afs instead of conversion
Eric Dumazet reported following problem:
It looks that unless RTNL is held, accessing ifa_list needs proper RCU
protection. indev->ifa_list can be changed und
David Howells says:
I'm told that there's not really any point populating the list.
Current OpenAFS ignores it, as does AuriStor - and IBM AFS 3.6 will
do the right thing.
The list is actually useless as it's the client's view of the world,
not the servers, so if there's any NAT in the wa
This also replaces spots that used for_primary_ifa().
for_primary_ifa() aborts the loop on the first secondary address seen.
Replace it with either the rcu or rtnl variant of in_dev_for_each_ifa(),
but two places will now also consider secondary addresses too:
inet_addr_onlink() and inet_ifa_bypr
Netfilter hooks are always running under rcu read lock, use
the new iterator macro so sparse won't complain once we add
proper __rcu annotations.
Signed-off-by: Florian Westphal
---
net/ipv4/netfilter/nf_tproxy_ipv4.c| 9 +++--
net/netfilter/nf_conntrack_broadcast.c | 9 +++--
net/ne
Like previous patches, use the new iterator macros to avoid sparse
warnings once proper __rcu annotations are added.
Compile tested only.
Signed-off-by: Florian Westphal
---
drivers/infiniband/core/roce_gid_mgmt.c | 5 +++--
drivers/infiniband/hw/cxgb4/cm.c |
Use in_dev_for_each_ifa_rcu/rtnl instead.
This prevents sparse warnings once proper __rcu annotations are added.
Signed-off-by: Florian Westphal
t di# Last commands done (6 commands done):
---
net/ipv4/fib_frontend.c | 24 +---
net/ipv4/igmp.c | 5 +++--
net/ipv6/ad
The ifa_list is protected either by rcu or rtnl lock, but the
current iterators do not account for this.
This adds two iterators as replacement, a later patch in
the series will update them with the needed rcu/rtnl_dereference calls.
Its not done in this patch yet to avoid sparse warnings -- the
ifa_list is protected by rcu, yet code doesn't reflect this.
Add the __rcu annotations and fix up all places that are now reported by
sparse.
I've done this in the same commit to not add intermediate patches that
result in new warnings.
Reported-by: Eric Dumazet
Signed-off-by: Florian Westphal
David Howells wrote:
> Florian Westphal wrote:
>
> > David Howell says:
>
> "Howells"
My bad.
> Apart from that:
>
> Tested-by: David Howells
Thanks, a lot, I've re-submitted this as v3 retaining your tested-by.
1 - 100 of 299 matches
Mail list logo