Hi Markus,
On 10/27/2017 10:30 PM, SF Markus Elfring wrote:
From: Markus Elfring
Date: Fri, 27 Oct 2017 22:22:24 +0200
* Add a jump target so that a bit of exception handling can be better
reused at the end of this function.
* Adjust two condition checks.
This issue was detected by using
On Fri, Oct 27, 2017 at 11:15 PM, Jakub Kicinski wrote:
> On Sat, 28 Oct 2017 14:05:04 +0800, Yunsheng Lin wrote:
>> Hi, Michael
>>
>> On 2017/10/28 13:56, Michael Chan wrote:
>> > Fix undefined symbols when CONFIG_VLAN_8021Q or CONFIG_INET is not set.
>> >
>> > Fixes: 8c95f773b4a3 ("bnxt_en: add
On Sat, 28 Oct 2017 14:05:04 +0800, Yunsheng Lin wrote:
> Hi, Michael
>
> On 2017/10/28 13:56, Michael Chan wrote:
> > Fix undefined symbols when CONFIG_VLAN_8021Q or CONFIG_INET is not set.
> >
> > Fixes: 8c95f773b4a3 ("bnxt_en: add support for Flower based vxlan
> > encap/decap offload")
> > R
Hi, Michael
On 2017/10/28 13:56, Michael Chan wrote:
> Fix undefined symbols when CONFIG_VLAN_8021Q or CONFIG_INET is not set.
>
> Fixes: 8c95f773b4a3 ("bnxt_en: add support for Flower based vxlan encap/decap
> offload")
> Reported-by: Jakub Kicinski
> Signed-off-by: Michael Chan
> ---
> driv
Fix undefined symbols when CONFIG_VLAN_8021Q or CONFIG_INET is not set.
Fixes: 8c95f773b4a3 ("bnxt_en: add support for Flower based vxlan encap/decap
offload")
Reported-by: Jakub Kicinski
Signed-off-by: Michael Chan
---
drivers/net/ethernet/broadcom/bnxt/bnxt_tc.c | 6 ++
1 file changed, 6
Hi, Dan
On 2017/10/27 19:52, Dan Carpenter wrote:
> On Fri, Oct 27, 2017 at 05:32:42PM +0800, Yunsheng Lin wrote:
>> Hi, Dan
>>
>> On 2017/10/27 14:40, Dan Carpenter wrote:
>>> There are several places where we accidentally return success when
>>> kcalloc() fails.
>>>
>>> Fixes: fcb39f6c10b2 ("qed
From: Roopa Prabhu
This config maps to IFLA_BRPORT_VLAN_TUNNEL bridge port netlink
flag attribute. This flag enables vlan to tunnel mapping on a bridge
port. It is off by default.
set vlan_tunnel attribute on bridge port vxlan0:
$ip link set dev vxlan0 type bridge_slave vlan_tunnel on
$ip link
From: Roopa Prabhu
This patch uses kernel bridge vlan attribute
IFLA_BRIDGE_VLAN_TUNNEL_INFO to set/delete/show per vlan tunnel info.
$bridge vlan add dev vxlan0 vid 2000 tunnel_info id 2000
$bridge vlan add dev vxlan0 vid 1000-1001 tunnel_info id 2000-2001
$bridge vlan tunnelshow
portvlan
From: Roopa Prabhu
This series adds the required options to iplink and bridge commands
to enable per vlan tunnel info on a bridge port provided by
netlink attributes IFLA_BRPORT_VLAN_TUNNEL (flag) and
IFLA_BRIDGE_VLAN_TUNNEL_INFO
enable vlan-tunnel mapping on a bridge port:
$ip link set dev vxla
Davide found the following script triggers a NULL pointer
dereference:
ip l a name eth0 type dummy
tc q a dev eth0 parent :1 handle 1: htb
This is because for a freshly created netdevice noop_qdisc
is attached and when passing 'parent :1', kernel actually
tries to match the major handle which is
Remove platform_device_put() call after platform_device_unregister()
from function bcmgenet_mii_exit(), otherwise, we will call
platform_device_put() twice.
Fixes: 9a4e79697009 ("net: bcmgenet: utilize generic Broadcom UniMAC
MDIO controller driver")
Signed-off-by: Wei Yongjun
---
drivers/net/et
Fixes the following sparse warnings:
drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c:224:5: warning:
symbol 'aq_ethtool_get_coalesce' was not declared. Should it be static?
drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c:245:5: warning:
symbol 'aq_ethtool_set_coalesce' was not declared.
On Fri, 27 Oct 2017 20:28:16 -0700, Michael Chan wrote:
> On Fri, Oct 27, 2017 at 8:18 PM, Jakub Kicinski wrote:
> > On Thu, 26 Oct 2017 11:51:29 -0400, Michael Chan wrote:
> >> From: Sathya Perla
> >>
> >> This patch adds IPv4 vxlan encap/decap action support to TC-flower
> >> offload.
> >>
>
On Fri, Oct 27, 2017 at 8:18 PM, Jakub Kicinski wrote:
> On Thu, 26 Oct 2017 11:51:29 -0400, Michael Chan wrote:
>> From: Sathya Perla
>>
>> This patch adds IPv4 vxlan encap/decap action support to TC-flower
>> offload.
>>
>> For vxlan encap, the driver maintains a tunnel encap hash-table.
>> Whe
On Thu, 26 Oct 2017 11:51:29 -0400, Michael Chan wrote:
> From: Sathya Perla
>
> This patch adds IPv4 vxlan encap/decap action support to TC-flower
> offload.
>
> For vxlan encap, the driver maintains a tunnel encap hash-table.
> When a new flow with a tunnel encap action arrives, this table
> i
Your comments on the tunnel VRF and underlay VRF being different make
sense and is more flexible. I think assigning the dev/link to the
same VRF as the tunnel master accomplishes the same thing that I
originally had in mind.
ping v4 loopback...
jeff@VM2:~$ ip route list vrf myvrf
127.0.0.0/8 dev
On 10/27/17 3:23 PM, Sebastian Andrzej Siewior wrote:
Hi,
while looking at other things here I stumbled at this in
kernel/bpf/percpu_freelist.c:
|void pcpu_freelist_populate(struct pcpu_freelist *s, void *buf, u32 elem_size,
|u32 nr_elems)
|{
…
|/* disable ir
On Fri, Oct 27, 2017 at 05:28:22PM -0700, Tushar Dave wrote:
> Default rlimit RLIMIT_MEMLOCK is 64KB, causes bpf map failure.
> e.g.
> [root@labbpf]# ./xdp_redirect_map $( > $( failed to create a map: 1 Operation not permitted
>
> The failure is 100% when multiple xdp programs are running. Fix it.
On Fri, Oct 27, 2017 at 04:12:30PM -0700, Tushar Dave wrote:
> Default rlimit RLIMIT_MEMLOCK is 64KB, causes bpf map failure.
> e.g.
> [root@lab bpf]#./xdp1 -N $( failed to create a map: 1 Operation not permitted
>
> Fix it.
>
> Signed-off-by: Tushar Dave
Acked-by: Alexei Starovoitov
On Fri, Oct 27, 2017 at 3:22 PM, Subash Abhinov Kasiviswanathan
wrote:
>> This doesn't make sense to me, maybe I am missing something. What
>> "real device" is setting the skb->destructor() and doing it to somehow
>> handle recycling? The only driver I can find that is setting
>> skb->desctructor(
On Fri, 2017-10-27 at 17:09 -0700, Tom Herbert wrote:
> Call gro_cells_receive instead of netif_rx.
>
...
>
> static void gtp_link_setup(struct net_device *dev)
> {
> + struct gtp_dev *gtp = netdev_priv(dev);
> +
> dev->netdev_ops = >p_netdev_ops;
> dev->needs_free_ne
On Fri, 2017-10-27 at 17:09 -0700, Tom Herbert wrote:
> Move gro_cells_init to ndo_init and properly check return value.
>
> Signed-off-by: Tom Herbert
> ---
> drivers/net/vxlan.c | 11 +--
> 1 file changed, 9 insertions(+), 2 deletions(-)
Reported-by: Eric Dumazet
On October 27, 2017 5:30:00 PM PDT, Florian Fainelli
wrote:
>Due to a documentation mistake, the IPG length was set to 0x12 while it
>should have been 12 (decimal). This would affect short packet (64B
>typically) performance since the IPG was bigger than necessary.
>
David, I will be sending a v
On Fri, 27 Oct 2017 09:27:30 +0200, Jiri Pirko wrote:
> >>> 2. Deleting the ingress qdisc fails to remove filters added in
> >>> HW. Filters in SW gets deleted.
> >>>
> >>> We haven’t exactly root-caused this, the changes being extensive, but
> >>> our guess is again something wrong with the
Adding a FIB rule on a spectrum platform silently aborts FIB offload:
$ ip ru add pref 99 from all to 192.168.1.1 table 10
$ dmesg -c
[ 623.144736] mlxsw_spectrum :03:00.0: FIB abort triggered. Note that
FIB entries are no longer being offloaded to this device.
This patch reworks
Add extack to fib_notifier_info and plumb through stack to
call_fib_rule_notifiers, call_fib_entry_notifiers and
call_fib6_entry_notifiers. This allows notifer handlers to
return messages to user.
Signed-off-by: David Ahern
---
include/net/fib_notifier.h | 1 +
net/core/fib_rules.c | 9 +
The NLMSGERR API already carries data (eg, a cookie) on the success path.
Allow a message string to be returned as well.
Signed-off-by: David Ahern
---
net/netlink/af_netlink.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/net/netlink/af_netlink.c b/net/n
There are many cases where networking subsystems throw non-fatal warning
messages that end up in dmesg / kernel log to which a user making the
change is completely oblivious. This set makes the extack facility
usable for returning such messages.
The case in point here is spectrum and adding FIB ru
From: Wei Wang
Cached routes should only be created by the system when receiving pmtu
discovery or ip redirect msg. Users should not be allowed to create
cached routes.
Furthermore, after the patch series to move cached routes into exception
table, user added cached routes will trigger the follo
Due to a documentation mistake, the IPG length was set to 0x12 while it
should have been 12 (decimal). This would affect short packet (64B
typically) performance since the IPG was bigger than necessary.
Fixes: 44a4524c54af ("net: systemport: Add support for SYSTEMPORT Lite")
Signed-off-by: Florian
Default rlimit RLIMIT_MEMLOCK is 64KB, causes bpf map failure.
e.g.
[root@labbpf]# ./xdp_redirect_map $( $(
---
samples/bpf/xdp_redirect_map_user.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/samples/bpf/xdp_redirect_map_user.c
b/samples/bpf/xdp_redirect_map_user.c
index d4d86a2..9
On Fri, Oct 27, 2017 at 3:34 PM, Paweł Staszewski wrote:
> Hi
>
>
>
>
> I have many problems with 40e driver
>
> memleaks , kernel panics , stack traces , tx hungx , tx timeouts and many
> many others :)
>
>
> But the main problem that can't be resolved in linux is resolved in freebsd
>
> problem
The gtp pktinfo structure is unnecessary and needs a lot of code to
manage it. Remove it. Also, add per pdp port configuration for transmit.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c| 177 +--
include/uapi/linux/gtp.h | 1 +
2 files ch
Add new configuration of GTP interfaces that allow specifying a port to
listen on (as opposed to having to get sockets from a userspace control
plane). This allows GTP interfaces to be configured and the data path
tested without requiring a GTP-C daemon.
Signed-off-by: Tom Herbert
---
drivers/ne
ip_tunnel_get_route and ip6_tnl_get_route are created to return
routes for a tunnel. These functions are derived from the VXLAN
functions.
Signed-off-by: Tom Herbert
---
include/net/ip6_tunnel.h | 31 +++
include/net/ip_tunnels.h | 32
Call gro_cells_receive instead of netif_rx.
Signed-off-by: Tom Herbert
---
drivers/net/Kconfig | 1 +
drivers/net/gtp.c | 18 +-
2 files changed, 18 insertions(+), 1 deletion(-)
diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
index 0936da592e12..720c2af0bb88 100644
---
Allows using GTP datapath over IPv6. Remote peers are indicated by IPv6.
Note this is experimental, more work is needed to make this
compliant with 3GPP standard.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c| 248 ++-
include/uapi/linux/gt
Call ip_tunnel_get_route and ip6_tnl_get_route to handle getting a route
and dealing with the dst_cache.
Signed-off-by: Tom Herbert
---
drivers/net/vxlan.c | 84 -
1 file changed, 5 insertions(+), 79 deletions(-)
diff --git a/drivers/net/vxlan
Allow IPv6 mobile subscriber packets. This entails adding an IPv6 mobile
subscriber address to pdp context and IPv6 specific variants to find pdp
contexts by address.
Note that this is experimental support of IPv6, more work is
necessary to make this compliant with 3GPP standard.
Signed-off-by: T
Call ip_tunnel_get_route and dst_cache to pdp context which should
improve performance by obviating the need to perform a route lookup
on every packet.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 62 +++
1 file changed, 35 insertions(+),
Replace mtu handling with call to __iptunnel_update_pmtu.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 36 ++--
1 file changed, 6 insertions(+), 30 deletions(-)
diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index 8b7014aa45c2..7b9a482b3d22 100644
--- a
Add __iptunnel_update_pmtu exported function which does not take
an iptunnel argument but instead includes the fields from the
iptunnel structure as arguments which are needed in the function.
iptunnel_update_pmtu was modified to call __iptunnel_update_pmtu.
Signed-off-by: Tom Herbert
---
inclu
Create separate UDP receive functions for GTP version 0 and version 1.
Set encap_rcv appropriately when configuring a socket.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 100 ++
1 file changed, 49 insertions(+), 51 deletions(-)
diff --g
Move gro_cells_init to ndo_init and properly check return value.
Signed-off-by: Tom Herbert
---
drivers/net/vxlan.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
index d7c49cf1d5e9..a54dd3d2a824 100644
--- a/drivers/net/
Consolidate return logic to make it easier to extend.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 26 +-
1 file changed, 17 insertions(+), 9 deletions(-)
diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index 265fa9fdbf3d..2daf434ae543 100644
--- a/drivers/net/g
This patch set builds upon the initial GTP implementation to make
support closer to that enjoyed by other encapsulation protocols.
The major items are:
- Experimental IPv6 support
- Configurable networking interfaces so that GTP kernel can be
used and tested without needing GSN network em
On Fri, 27 Oct 2017 12:09:17 -0700, Felix Manlunas wrote:
> From: Vijaya Mohan Guvva
>
> Enable switchdev for SRIOV capable LiquidIO NIC. It registers
> a representor netdev (with switchdev_ops) for each SRIOV VF created.
> It also has changes to send representor interface configurations like
> a
On 17/10/27 (金) 19:19, Nikolay Aleksandrov wrote:
Before this patch there was no way to tell if the vlan add operation
actually changed anything, thus we would always generate a notification
on adds. Let's make the notifications more precise and generate them
only if anything changed, so use the
On Thu, Oct 26, 2017 at 11:26 AM, Davide Caratti wrote:
> the following script generates a NULL pointer dereference error:
>
> ip l a name eth0 type dummy
> tc q a dev eth0 parent :1 handle 1: htb
>
> upon creation of classful qdiscs, qdisc_alloc() dereferences dev_queue->dev
> assuming it is not
e1000 sourceforge is a bad place to make anything with Your problems
Just checked this now if something changed :)
But when I post reply to some bug that i was have same problem somebody
closed the ticket and delete my message :)
So rly :)
W dniu 2017-10-25 o 23:49, Pavlos Parissis pisz
Default rlimit RLIMIT_MEMLOCK is 64KB, causes bpf map failure.
e.g.
[root@lab bpf]#./xdp1 -N $(
---
samples/bpf/xdp1_user.c | 8
1 file changed, 8 insertions(+)
diff --git a/samples/bpf/xdp1_user.c b/samples/bpf/xdp1_user.c
index 2431c03..fdaefe9 100644
--- a/samples/bpf/xdp1_user.c
+++
W dniu 2017-10-28 o 00:34, Paweł Staszewski pisze:
Hi
I have many problems with 40e driver
memleaks , kernel panics , stack traces , tx hungx , tx timeouts and
many many others :)
But the main problem that can't be resolved in linux is resolved in
freebsd
problem in freebsd with thi
On 28 October 2017 at 00:21, Paweł Staszewski wrote:
> from today it is in net.git
>
> https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git/log/?qt=grep&q=i40e
>
> It will be later in net-next
>
>
I have to admit I am not that familiar with repo layout, I though
net-next holds what is go
bcm_sf2 and b53 replicate the same operations: clear all VLANs and set
their ports to the default VLAN tag (1 for these devices) so export the
b53 function doing just that.
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_common.c | 6 --
drivers/net/dsa/b53/b53_priv.h | 1 +
On 10/27/17 2:59 PM, Jeff Barnhill wrote:
> w/regards to this comment:
>You have a remote address with no qualification about which VRF to
> use for the lookup.
>
> I was using this to enslave the tunnel:
> sudo ip link set jtun vrf myvrf
>
> and assumed this would be enough to cause all tunn
Hi
I have many problems with 40e driver
memleaks , kernel panics , stack traces , tx hungx , tx timeouts and
many many others :)
But the main problem that can't be resolved in linux is resolved in freebsd
problem in freebsd with this:
[2501243.181829] i40e :01:00.1 eno2: VSI_seid 39
Hi,
while looking at other things here I stumbled at this in
kernel/bpf/percpu_freelist.c:
|void pcpu_freelist_populate(struct pcpu_freelist *s, void *buf, u32 elem_size,
|u32 nr_elems)
|{
…
|/* disable irq to workaround lockdep false positive
| * in bp
This doesn't make sense to me, maybe I am missing something. What
"real device" is setting the skb->destructor() and doing it to somehow
handle recycling? The only driver I can find that is setting
skb->desctructor() is the Chelsio drivers, and they appear to be using
it to just clean-up DMA mappi
from today it is in net.git
https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git/log/?qt=grep&q=i40e
It will be later in net-next
Also can You please tell me what firmware You are using with Your nics ?
Those are X710 ?
Thanks
Paweł
W dniu 2017-10-27 o 23:20, Pavlos Parissis pi
Quoting Steffen Klassert :
On Thu, Oct 26, 2017 at 06:31:35AM -0500, Gustavo A. R. Silva wrote:
Use BUG_ON instead of if condition followed by BUG.
This issue was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
Applied to ipsec-next, thanks Gustavo!
Glad to help
On Thu, Oct 26, 2017 at 2:40 PM, Roman Mashak wrote:
> Userland client should be able to read an event, and reflect it back to
> the kernel, therefore it needs to extract complete set of netlink flags.
>
> For example, this will allow "tc monitor" to distinguish Add and Replace
> qdisc operations.
On Sat, Oct 28, 2017 at 02:13:29AM +0800, Xin Long wrote:
> Now when migrating sock to another one in sctp_sock_migrate(), it only
> resets owner sk for the data in receive queues, not the chunks on out
> queues.
>
> It would cause that data chunks length on the sock is not consistent
> with sk sk
On Fri, Oct 27, 2017 at 1:49 PM, Manish Kurup wrote:
> static inline u32 tcf_vlan_action(const struct tc_action *a)
> {
> - return to_vlan(a)->tcfv_action;
> + u32 tcfv_action;
> +
> + rcu_read_lock();
> + tcfv_action = rcu_dereference(to_vlan(a)->vlan_p)->tcfv_action;
>
Creating a macvtap interface with the liquidio VF driver as lower device
causes this alarming message to show up in dmesg:
liquidio_link_ctrl_cmd_completion Unknown cmd 27
That's actually a false alarm because cmd 27 is the value of the macro
OCTNET_CMD_SET_UC_LIST which is known. It's a con
On Fri, Oct 27, 2017 at 5:06 PM, Jiri Pirko wrote:
> Fri, Oct 27, 2017 at 10:54:06PM CEST, steven.l...@broadcom.com wrote:
>>Adding DEVLINK_PERM_CONFIG_SRIOV_ENABLED permanent config
>>parameter. Value is permanent, so becomes the new default
>
> Avoid the double space.
Ok.
>
>
>>value for this
On Fri, Oct 27, 2017 at 5:04 PM, Jiri Pirko wrote:
> Fri, Oct 27, 2017 at 10:54:05PM CEST, steven.l...@broadcom.com wrote:
>>Add support for permanent config parameter get/set commands. Used
>>for persistent device configuration parameters.
>>
>>Signed-off-by: Steve Lin
>>Acked-by: Andy Gospodare
On 23 October 2017 at 01:15, Paweł Staszewski wrote:
> Yes can confirm that after adding patch:
>
> [jkirsher/net-queue PATCH] i40e: Add programming descriptors to
> cleaned_count
>
>
> There is no memleak.
>
>
Somehow this patch isn't present in the current net-next repo.
Shouldn't it be there?
On Thu, Oct 26, 2017 at 11:00 PM, Dmitry Vyukov wrote:
> On Thu, Oct 26, 2017 at 7:58 PM, Tejun Heo wrote:
>> Hello,
>>
>> On Thu, Oct 26, 2017 at 09:35:44AM -0700, syzbot wrote:
>>> BUG: KASAN: use-after-free in __read_once_size
>>> include/linux/compiler.h:276 [inline]
>>> BUG: KASAN: use-after
Fri, Oct 27, 2017 at 10:54:06PM CEST, steven.l...@broadcom.com wrote:
>Adding DEVLINK_PERM_CONFIG_SRIOV_ENABLED permanent config
>parameter. Value is permanent, so becomes the new default
Avoid the double space.
>value for this device.
>
> DEVLINK_PERM_CONFIG_DISABLE = Disable SR-IOV
> DEVLIN
Fri, Oct 27, 2017 at 10:54:05PM CEST, steven.l...@broadcom.com wrote:
>Add support for permanent config parameter get/set commands. Used
>for persistent device configuration parameters.
>
>Signed-off-by: Steve Lin
>Acked-by: Andy Gospodarek
>---
> include/net/devlink.h| 7 ++
> include/u
On Fri, Oct 27, 2017 at 1:30 PM, Subash Abhinov Kasiviswanathan
wrote:
> For deaggregation, the real device receives a large linear skb and
> passes it on to rmnet. rmnet creates new skbs from this large frame.
>
> If the real device supports recycling, it does not need to allocate
> the large skb
w/regards to this comment:
You have a remote address with no qualification about which VRF to
use for the lookup.
I was using this to enslave the tunnel:
sudo ip link set jtun vrf myvrf
and assumed this would be enough to cause all tunnel traffic to be
part of this VRF.
You are right about th
On 10/27/17 1:38 PM, Alban Crequy wrote:
Hi,
On 25 October 2017 at 01:57, Song Liu wrote:
This tracepoint can be used to trace synack retransmits. It maintains
pointer to struct request_sock.
We cannot simply reuse trace_tcp_retransmit_skb() here, because the
sk here is the LISTEN socket. The
Implements get and set of configuration parameters using new devlink
config get/set API. Parameters themselves defined in later patches.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 258 +-
drivers/net/ethernet/b
Add support for permanent config parameter get/set commands. Used
for persistent device configuration parameters.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
include/net/devlink.h| 7 ++
include/uapi/linux/devlink.h | 25
net/core/devlink.c | 287 +++
Adding DEVLINK_PERM_CONFIG_NUM_MSIX_VECT_PER_VF permanent config
parameter. Value is permanent, so becomes the new default value
for this device.
Value defines number of MSI-X vectors allocated per VF.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
include/uapi/linux/devlink.h | 4 +++
Adding permanent config parameter for SR-IOV enablement, using
devlink API for get/set operation.
DEVLINK_PERM_CONFIG_DISABLE = SR-IOV disabled
DEVLINK_PERM_CONFIG_ENABLE = SR-IOV enabled
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c
Adding permanent config parameter for number MSI-X vectors
per VF, using devlink API for get/set operation.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/ethernet/broad
Adding permanent config parameter for number of VFs per PF,
using devlink API for get/set operation.
Value sets the number of VFs per PF in SR-IOV mode.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 4
1 file changed, 4 insertio
Adding permanent config parameter for maximum number of PF
MSI-X vectors.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c
b/drivers
Adding DEVLINK_PERM_CONFIG_MAX_NUM_PF_MSIX_VECT permanent config
parameter. Value is permanent, so becomes the new default value
for this device.
Value sets the maximum number of PF MSI-X vectors.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
include/uapi/linux/devlink.h | 4
ne
Adding DEVLINK_PERM_CONFIG_NUM_VF_PER_PF permanent config
parameter. Value is permanent, so becomes the new default
value for this device.
The value sets the number of VFs per PF in SR-IOV mode.
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
include/uapi/linux/devlink.h | 4
net/
Changes since v3:
* Using union instead of void * to pass parameter values to/from drivers.
* Updated parameter comments / naming and added enum for enable/disable
to add clarity.
* Various code cleanup in bnxt:
- change hwrm call to use mutex protected version
- use roundup() macro
- remo
Adding DEVLINK_PERM_CONFIG_SRIOV_ENABLED permanent config
parameter. Value is permanent, so becomes the new default
value for this device.
DEVLINK_PERM_CONFIG_DISABLE = Disable SR-IOV
DEVLINK_PERM_CONFIG_ENABLE = Enable SR-IOV
Signed-off-by: Steve Lin
Acked-by: Andy Gospodarek
---
includ
Using a spinlock in the VLAN action causes performance issues when the VLAN
action is used on multiple cores. Rewrote the VLAN action to use RCU read
locking for reads and updates instead.
Acked-by: Jamal Hadi Salim
Acked-by: Jiri Pirko
Signed-off-by: Manish Kurup
---
include/net/tc_act/tc_vla
The VLAN action maintains one set of stats across all cores, and uses a
spinlock to synchronize updates to it from the same. Changed this to use a
per-CPU stats context instead.
This change will result in better performance.
Acked-by: Jamal Hadi Salim
Acked-by: Jiri Pirko
Signed-off-by: Manish K
Hi everyone,
Apologies for the resend (newbie mistakes), but I made the following
corrections to the cover letter and patches:
* Added version and comment info to subjects of patches and all the changes
(below). I used v2, since my earlier mails had no versions.
* Added ACKs from Jamal and Jiri
On Wed, Oct 25, 2017 at 10:37 PM, Yuchung Cheng wrote:
> On Wed, Oct 25, 2017 at 7:07 PM, Alexei Starovoitov
> wrote:
>>
>> On Thu, Sep 28, 2017 at 04:36:58PM -0700, Yuchung Cheng wrote:
>> > On Thu, Sep 28, 2017 at 1:14 AM, Oleksandr Natalenko
>> > wrote:
>> > > Hi.
>> > >
>> > > Won't tell abo
Hi,
On 25 October 2017 at 01:57, Song Liu wrote:
> This tracepoint can be used to trace synack retransmits. It maintains
> pointer to struct request_sock.
>
> We cannot simply reuse trace_tcp_retransmit_skb() here, because the
> sk here is the LISTEN socket. The IP addresses and ports should be
>
For deaggregation, the real device receives a large linear skb and
passes it on to rmnet. rmnet creates new skbs from this large frame.
If the real device supports recycling, it does not need to allocate
the large skbs during packet reception and can instead reuse them.
Signed-off-by: Subash Abhi
From: Markus Elfring
Date: Fri, 27 Oct 2017 22:22:24 +0200
* Add a jump target so that a bit of exception handling can be better
reused at the end of this function.
* Adjust two condition checks.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
dr
Hi David,
Quoting David Ranch :
Hello Gustavo,
I appreciate you working on keeping up the kernel and maintaining
some of the older feature areas like AX.25, Netrom, etc. Other than
auditing your code changes, can you tell me what you're changing?
I've been attempting to find who / where
Fetching the master device can be done directly when a port is parsed
from device tree or pdata, instead of waiting until dsa_dst_parse.
Now that -EPROBE_DEFER is returned before we add the switch to the tree,
there is no need to check for this error after dsa_dst_parse.
Signed-off-by: Vivien Did
On 10/26/17 10:06 PM, Song Liu wrote:
On Oct 26, 2017, at 7:01 PM, Cong Wang wrote:
On Thu, Oct 26, 2017 at 4:50 PM, Song Liu wrote:
In this case, we are putting CONFIG_IPV6 in TRACE_EVENT macro, which generates
warnings like:
./include/trace/events/tcp.h:274:1: error: directive in argumen
There is no point into hiding the -EINVAL error code in ERR_PTR from a
dsa_get_ports function, simply get the "ports" node directly from within
the dsa_parse_ports_dn function.
This also has the effect to make the pdata and device tree handling code
symmetrical inside _dsa_register_switch.
At the
Now that slave dsa_port always have their name set, there is no need to
pass it to dsa_slave_create() anymore. Remove this argument.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c | 3 +--
net/dsa/dsa_priv.h | 2 +-
net/dsa/legacy.c | 2 +-
net/dsa/slave.c| 3 ++-
4 files changed, 5
This patchset adds port parsing functions called early in the new
bindings parsing stage, which regroup all the fetching of static data
available at the port level, including the port's type, name and CPU
master interface.
This simplifies the rest of the code which does not need to dig into
device
Add symmetrical DSA port parsing functions for pdata and device tree,
used to parse and validate a given port node or platform data.
They don't do much for the moment but will be extended later on to
assign a port type and get device references.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c
In the case of pdata, the dsa_cpu_parse function calls dev_put() before
making sure it isn't NULL. Fix this.
Fixes: 71e0bbde0d88 ("net: dsa: Add support for platform data")
Signed-off-by: Vivien Didelot
Reviewed-by: Florian Fainelli
---
net/dsa/dsa2.c | 7 ---
1 file changed, 4 insertions(+
Get the optional "label" property and assign a default one directly at
parse time instead of doing it when creating the slave.
For legacy, simply assign the port name stored in cd->port_names.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c | 10 +-
net/dsa/legacy.c | 1 +
2 files c
1 - 100 of 261 matches
Mail list logo
