[PATCH next] loopback: fix lockdep splat

2019-07-02 Thread Mahesh Bandewar
dev_init_scheduler() and dev_activate() expect the caller to hold RTNL. Since we don't want blackhole device to be initialized per ns, we are initializing at init. [3.855027] Call Trace: [3.855034] dump_stack+0x67/0x95 [3.855037] lockdep_rcu_suspicious+0xd5/0x110 [3.855044] dev_

[PATCH net-next v2 1/1] qed: Add support for Timestamping the unicast PTP packets.

2019-07-02 Thread Sudarsana Reddy Kalluru
This patch adds driver changes to detect/timestamp the unicast PTP packets. Changes from previous version: --- v2: Defined a macro for unicast ptp param mask. Please consider applying this to "net-next". Signed-off-by: Sudarsana Reddy Kalluru Signed-off-by: Ariel Eli

Re: [PATCH net-next] net: socionext: remove set but not used variable 'pkts'

2019-07-02 Thread Ilias Apalodimas
On Wed, Jul 03, 2019 at 02:42:13AM +, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/net/ethernet/socionext/netsec.c: In function 'netsec_clean_tx_dring': > drivers/net/ethernet/socionext/netsec.c:637:15: warning: > variable 'pkts' set but not used [-Wunused-bu

RE: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and port attribute

2019-07-02 Thread Parav Pandit
> -Original Message- > From: Jakub Kicinski > Sent: Wednesday, July 3, 2019 7:46 AM > To: Parav Pandit > Cc: Jiri Pirko ; netdev@vger.kernel.org; Saeed > Mahameed > Subject: Re: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and > port attribute > > On Wed, 3 Jul 2019 02

Re: [PATCH net-next] skbuff: increase verbosity when dumping skb data

2019-07-02 Thread Yunsheng Lin
On 2019/7/3 3:39, Willem de Bruijn wrote: > From: Willem de Bruijn > > skb_warn_bad_offload and netdev_rx_csum_fault trigger on hard to debug > issues. Dump more state and the header. > > Optionally dump the entire packet and linear segment. This is required > to debug checksum bugs that may inc

Re: [PATCH net] tcp: refine memory limit test in tcp_fragment()

2019-07-02 Thread Tony Lu
Hello Eric, We have applied that commit e358f4af19db ("tcp: tcp_fragment() should apply sane memory limits") as a hotpatch in production environment. We found that it will make tcp long connection reset during sending out packet when applying that commit.

Re: [PATCH bpf-next v2 0/8] bpf: TCP RTT sock_ops bpf callback

2019-07-02 Thread Lawrence Brakmo
On 7/2/19, 10:55 AM, "netdev-ow...@vger.kernel.org on behalf of Y Song" wrote: On Tue, Jul 2, 2019 at 9:14 AM Stanislav Fomichev wrote: > > Congestion control team would like to have a periodic callback to > track some TCP statistics. Let's add a sock_ops callback that can be

RE: [PATCH net-next 1/1] devlink: Add APIs to publish/unpublish the port parameters.

2019-07-02 Thread Sudarsana Reddy Kalluru
> -Original Message- > From: Parav Pandit > Sent: Tuesday, July 2, 2019 10:49 PM > To: Andrew Lunn > Cc: Sudarsana Reddy Kalluru ; > da...@davemloft.net; netdev@vger.kernel.org; Michal Kalderon > ; Ariel Elior ; > j...@resnulli.us > Subject: [EXT] RE: [PATCH net-next 1/1] devlink: Add A

[PATCH net-next 0/3] tc-testing: Add JSON verification and simple traffic generation

2019-07-02 Thread Lucas Bates
This patchset introduces JSON as a verification method in tdc and adds a new plugin, scapyPlugin, as a way to send traffic to test tc filters and actions. The first patch adds the JSON verification to the core tdc script. The second patch makes a change to the TdcPlugin module that will allow tdc

[PATCH net-next 3/3] tc-testing: introduce scapyPlugin for basic traffic

2019-07-02 Thread Lucas Bates
The scapyPlugin allows for simple traffic generation in tdc to test various tc features. It was tested with scapy v2.4.2, but should work with any successive version. In order to use the plugin's functionality, scapy must be installed. This can be done with: pip3 install scapy or to install 2.

[PATCH net-next 2/3] tc-testing: Allow tdc plugins to see test case data

2019-07-02 Thread Lucas Bates
Instead of only passing the test case name and ID, pass the entire current test case down to the plugins. This change allows plugins to start accepting commands and directives from the test cases themselves, for greater flexibility in testing. --- tools/testing/selftests/tc-testing/TdcPlugin.py |

[PATCH net-next 1/3] tc-testing: Add JSON verification to tdc

2019-07-02 Thread Lucas Bates
This patch allows tdc to process JSON output to perform secondary verification of the command under test. If the verifyCmd generates JSON, one can provide the 'matchJSON' key to process it instead of a regex. matchJSON has two elements: 'path' and 'value'. The 'path' key is a list of integers and

[PATCH net-next] net: socionext: remove set but not used variable 'pkts'

2019-07-02 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/ethernet/socionext/netsec.c: In function 'netsec_clean_tx_dring': drivers/net/ethernet/socionext/netsec.c:637:15: warning: variable 'pkts' set but not used [-Wunused-but-set-variable] It is not used since commit ba2b232108d3 ("net: netse

RE: [PATCH net-next 1/1] qed: Add support for Timestamping the unicast PTP packets.

2019-07-02 Thread Sudarsana Reddy Kalluru
> -Original Message- > From: David Miller > Sent: Wednesday, July 3, 2019 12:53 AM > To: Sudarsana Reddy Kalluru > Cc: netdev@vger.kernel.org; Michal Kalderon ; > Ariel Elior > Subject: Re: [PATCH net-next 1/1] qed: Add support for Timestamping the > unicast PTP packets. > > From: Sudar

[PATCH AUTOSEL 5.1 18/39] net: stmmac: fixed new system time seconds value calculation

2019-07-02 Thread Sasha Levin
From: Roland Hii [ Upstream commit a1e5388b4d5fc78688e5e9ee6641f779721d6291 ] When ADDSUB bit is set, the system time seconds field is calculated as the complement of the seconds part of the update value. For example, if 3.1 seconds need to be subtracted from the system time, this field

[PATCH AUTOSEL 5.1 20/39] sis900: fix TX completion

2019-07-02 Thread Sasha Levin
From: Sergej Benilov [ Upstream commit 8ac8a01092b2added0749ef937037bf1912e13e3 ] Since commit 605ad7f184b60cfaacbc038aa6c55ee68dee3c89 "tcp: refine TSO autosizing", outbound throughput is dramatically reduced for some connections, as sis900 is doing TX completion within idle states only. Make

[PATCH AUTOSEL 5.1 16/39] be2net: fix link failure after ethtool offline test

2019-07-02 Thread Sasha Levin
From: Petr Oros [ Upstream commit 2e5db6eb3c23e5dc8171eb8f6af7a97ef9fcf3a9 ] Certain cards in conjunction with certain switches need a little more time for link setup that results in ethtool link test failure after offline test. Patch adds a loop that waits for a link setup finish. Changes in v

Re: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and port attribute

2019-07-02 Thread Jakub Kicinski
On Wed, 3 Jul 2019 02:08:39 +, Parav Pandit wrote: > > If you want to expose some device specific > > eswitch port ID please add a new attribute for that. > > The fact that that ID may match port_number for your device today is > > coincidental. port_number, and split attributes should not be

[PATCH AUTOSEL 4.9 3/8] be2net: fix link failure after ethtool offline test

2019-07-02 Thread Sasha Levin
From: Petr Oros [ Upstream commit 2e5db6eb3c23e5dc8171eb8f6af7a97ef9fcf3a9 ] Certain cards in conjunction with certain switches need a little more time for link setup that results in ethtool link test failure after offline test. Patch adds a loop that waits for a link setup finish. Changes in v

Re: [PATCH net-next v5 0/5] Add MPLS actions to TC

2019-07-02 Thread Willem de Bruijn
On Tue, Jul 2, 2019 at 8:32 PM John Hurley wrote: > > This patchset introduces a new TC action module that allows the > manipulation of the MPLS headers of packets. The code impliments > functionality including push, pop, and modify. > > Also included are tests for the new funtionality. Note that

[PATCH AUTOSEL 4.4 4/6] ppp: mppe: Add softdep to arc4

2019-07-02 Thread Sasha Levin
From: Takashi Iwai [ Upstream commit aad1dcc4f011ea409850e040363dff1e59aa4175 ] The arc4 crypto is mandatory at ppp_mppe probe time, so let's put a softdep line, so that the corresponding module gets prepared gracefully. Without this, a simple inclusion to initrd via dracut failed due to the mi

RE: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and port attribute

2019-07-02 Thread Parav Pandit
> -Original Message- > From: Jakub Kicinski > Sent: Wednesday, July 3, 2019 5:13 AM > To: Parav Pandit > Cc: Jiri Pirko ; netdev@vger.kernel.org; Saeed > Mahameed > Subject: Re: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and > port attribute > > On Tue, 2 Jul 2019 18

Re: [PATCH net-next 1/3] macsec: remove superfluous function calls

2019-07-02 Thread Willem de Bruijn
On Sun, Jun 30, 2019 at 9:57 PM Willem de Bruijn wrote: > > On Sun, Jun 30, 2019 at 4:48 PM Andreas Steinmetz wrote: > > > > Remove superfluous skb_share_check() and skb_unshare(). > > macsec_decrypt is only called by macsec_handle_frame which > > already does a skb_unshare(). > > There is a subt

Re: [PATCH] net: core: page_pool: add user refcnt and reintroduce page_pool_destroy

2019-07-02 Thread Ivan Khoronzhuk
On Tue, Jul 02, 2019 at 09:58:40PM +0300, Ivan Khoronzhuk wrote: On Tue, Jul 02, 2019 at 08:29:07PM +0200, Jesper Dangaard Brouer wrote: On Tue, 2 Jul 2019 18:21:13 +0300 Ivan Khoronzhuk wrote: On Tue, Jul 02, 2019 at 05:10:29PM +0200, Jesper Dangaard Brouer wrote: On Tue, 2 Jul 2019 17:56:1

Re: [PATCH net 2/2] macsec: fix checksumming after decryption

2019-07-02 Thread Willem de Bruijn
On Tue, Jul 2, 2019 at 3:48 PM Andreas Steinmetz wrote: > > On Tue, 2019-07-02 at 10:35 -0400, Willem de Bruijn wrote: > > On Tue, Jul 2, 2019 at 12:25 AM Andreas Steinmetz wrote: > > > On Sun, 2019-06-30 at 21:47 -0400, Willem de Bruijn wrote: > > > > On Sun, Jun 30, 2019 at 4:48 PM Andreas Stei

Re: [PATCH v2 bpf-next 1/4] bpf: unprivileged BPF access via /dev/bpf

2019-07-02 Thread Kees Cook
On Mon, Jul 01, 2019 at 06:59:13PM -0700, Andy Lutomirski wrote: > I think I'm understanding your motivation. You're not trying to make > bpf() generically usable without privilege -- you're trying to create > a way to allow certain users to access dangerous bpf functionality > within some limits.

Re: [PATCH net-next 3/7] net/rds: Wait for the FRMR_IS_FREE (or FRMR_IS_STALE) transition after posting IB_WR_LOCAL_INV

2019-07-02 Thread Gerd Rausch
On 02/07/2019 09.49, santosh.shilim...@oracle.com wrote: > On 7/1/19 10:11 PM, Gerd Rausch wrote: >> For the registration work-requests there is a benefit to wait a short >> amount of time only (the trade-off described in patch #1 of this series). >> > Actually we should just switch this code to wh

Re: [PATCH net 0/6] gtp: fix several bugs

2019-07-02 Thread Taehee Yoo
Hi Harald, On Wed, 3 Jul 2019 at 09:10, Harald Welte wrote: > > Hi Taehee, > > On Wed, Jul 03, 2019 at 12:20:34AM +0900, Taehee Yoo wrote: > > This patch series fixes several bugs in the gtp module. > > thanks a lot for your patches, they are much appreciated. > > They look valid to me after a br

Re: [PATCH net-next 3/7] net/rds: Wait for the FRMR_IS_FREE (or FRMR_IS_STALE) transition after posting IB_WR_LOCAL_INV

2019-07-02 Thread Gerd Rausch
On 02/07/2019 14.18, santosh.shilim...@oracle.com wrote: > On 7/2/19 2:05 PM, Gerd Rausch wrote: >> What do you call "RDS_GET_MR" semantics? >> > Its a blocking socket call. Meaning after this call return to the > user, the key must be valid. With async registration that can't be > guaranteed. >

Re: net: micrel: confusion about phyids used in driver

2019-07-02 Thread Uwe Kleine-König
Hello Yuiko Oshino, On Fri, May 10, 2019 at 09:22:43AM +0200, Uwe Kleine-König wrote: > On Thu, May 09, 2019 at 11:07:45PM +0200, Andrew Lunn wrote: > > On Thu, May 09, 2019 at 10:55:29PM +0200, Heiner Kallweit wrote: > > > On 09.05.2019 22:29, Uwe Kleine-König wrote: > > > > I have a board here t

Re: [PATCH v2 bpf-next 1/4] bpf: unprivileged BPF access via /dev/bpf

2019-07-02 Thread Song Liu
> On Jul 3, 2019, at 5:32 AM, Andy Lutomirski wrote: > > On Tue, Jul 2, 2019 at 2:04 PM Kees Cook wrote: >> >> On Mon, Jul 01, 2019 at 06:59:13PM -0700, Andy Lutomirski wrote: >>> I think I'm understanding your motivation. You're not trying to make >>> bpf() generically usable without privi

RE: net: micrel: confusion about phyids used in driver

2019-07-02 Thread Yuiko.Oshino
>-Original Message- >From: Uwe Kleine-König >Sent: Tuesday, July 2, 2019 4:32 PM >To: Yuiko Oshino - C18177 >Cc: netdev@vger.kernel.org; Andrew Lunn ; Florian Fainelli >; ker...@pengutronix.de; Heiner Kallweit > >Subject: Re: net: micrel: confusion about phyids used in driver > >Extern

[PATCH v2] net: don't warn in inet diag when IPV6 is disabled

2019-07-02 Thread Stephen Hemminger
If IPV6 was disabled, then ss command would cause a kernel warning because the command was attempting to dump IPV6 socket information. The fix is to just remove the warning. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=202249 Fixes: 432490f9d455 ("net: ip, diag -- Add diag interface for r

Re: [PATCH v2 bpf-next 1/4] bpf: unprivileged BPF access via /dev/bpf

2019-07-02 Thread Andy Lutomirski
On Tue, Jul 2, 2019 at 2:04 PM Kees Cook wrote: > > On Mon, Jul 01, 2019 at 06:59:13PM -0700, Andy Lutomirski wrote: > > I think I'm understanding your motivation. You're not trying to make > > bpf() generically usable without privilege -- you're trying to create > > a way to allow certain users

Re: [PATCH v2 net-next] r8169: add random MAC address fallback

2019-07-02 Thread David Miller
From: Heiner Kallweit Date: Tue, 2 Jul 2019 20:46:09 +0200 > It was reported that the GPD MicroPC is broken in a way that no valid > MAC address can be read from the network chip. The vendor driver deals > with this by assigning a random MAC address as fallback. So let's do > the same. > > Signe

Re: [PATCH v2 bpf-next 6/9] libbpf: use negative fd to specify missing BTF

2019-07-02 Thread Stanislav Fomichev
On 07/02, Stanislav Fomichev wrote: > On 05/29, Andrii Nakryiko wrote: > > 0 is a valid FD, so it's better to initialize it to -1, as is done in > > other places. Also, technically, BTF type ID 0 is valid (it's a VOID > > type), so it's more reliable to check btf_fd, instead of > > btf_key_type_id,

Re: [PATCH net-next] Revert "r8169: improve handling VLAN tag"

2019-07-02 Thread David Miller
From: Heiner Kallweit Date: Tue, 2 Jul 2019 07:59:17 +0200 > This reverts commit 759d095741721888b6ee51afa74e0a66ce65e974. > > The patch was based on a misunderstanding. As Al Viro pointed out [0] > it's simply wrong on big endian. So let's revert it. > > [0] https://marc.info/?t=1562009756

Re: [PATCH net-next] r8169: add random MAC address fallback

2019-07-02 Thread David Miller
From: Heiner Kallweit Date: Tue, 2 Jul 2019 18:49:14 +0200 > On 02.07.2019 08:18, Heiner Kallweit wrote: >>>From 1c8bacf724f1450e5256c68fbff407305faf9cbd Mon Sep 17 00:00:00 2001 >> >> >> > > Sorry, something went wrong when preparing the commit message. I'll resubmit. It helps to speak clos

Re: [PATCH 1/2 nf-next v3] netfilter: nft_meta: Add NFT_META_BRI_IIFVPROTO support

2019-07-02 Thread Pablo Neira Ayuso
On Fri, Jun 28, 2019 at 08:49:34AM +0800, we...@ucloud.cn wrote: > From: wenxu > > This patch provide a meta to get the bridge vlan proto > > nft add rule bridge firewall zones counter meta br_vlan_proto 0x8100 Applied, thanks.

Re: [PATCH v4 bpf-next] bpf: Add support for fq's EDT to HBM

2019-07-02 Thread Yonghong Song
On 7/2/19 3:09 PM, brakmo wrote: > Adds support for fq's Earliest Departure Time to HBM (Host Bandwidth > Manager). Includes a new BPF program supporting EDT, and also updates > corresponding programs. > > It will drop packets with an EDT of more than 500us in the future > unless the packet belo

Re: [PATCH net] net: don't warn in inet diag when IPV6 is disabled

2019-07-02 Thread David Miller
From: Stephen Hemminger Date: Mon, 1 Jul 2019 08:23:03 -0700 > @@ -19,9 +19,11 @@ raw_get_hashinfo(const struct inet_diag_req_v2 *r) > { > if (r->sdiag_family == AF_INET) { > return &raw_v4_hashinfo; > -#if IS_ENABLED(CONFIG_IPV6) > } else if (r->sdiag_family == AF_INE

Re: [PATCH net-next 0/2] Mellanox, mlx5 devlink versions query

2019-07-02 Thread Jakub Kicinski
On Tue, 2 Jul 2019 23:55:07 +, Saeed Mahameed wrote: > Hi Dave, > > This humble 2 patch series from Shay adds the support for devlink fw > versions query to mlx5 driver. > > In the first patch we implement the needed fw commands to support this > feature. > In the 2nd patch we implement the d

Re: [PATCH net-next] mlxsw: spectrum_ptp: Fix validation in mlxsw_sp1_ptp_packet_finish()

2019-07-02 Thread David Miller
From: Petr Machata Date: Tue, 2 Jul 2019 19:06:47 + > Before mlxsw_sp1_ptp_packet_finish() sends the packet back, it validates > whether the corresponding port is still valid. However the condition is > incorrect: when mlxsw_sp_port == NULL, the code dereferences the port to > compare it to s

Re: [PATCH net 0/2] macsec: fix some bugs in the receive path

2019-07-02 Thread David Miller
From: Andreas Steinmetz Date: Sun, 30 Jun 2019 22:46:38 +0200 > This series fixes some bugs in the receive path of macsec. The first > is a use after free when processing macsec frames with a SecTAG that > has the TCI E bit set but the C bit clear. In the 2nd bug, the driver > leaves an invalid c

Re: [PATCH net] net: don't warn in inet diag when IPV6 is disabled

2019-07-02 Thread Stephen Hemminger
On Tue, 02 Jul 2019 14:23:47 -0700 (PDT) David Miller wrote: > From: Stephen Hemminger > Date: Mon, 1 Jul 2019 08:23:03 -0700 > > > @@ -19,9 +19,11 @@ raw_get_hashinfo(const struct inet_diag_req_v2 *r) > > { > > if (r->sdiag_family == AF_INET) { > > return &raw_v4_hashinfo; >

Re: [PATCH net 0/6] gtp: fix several bugs

2019-07-02 Thread Harald Welte
Hi Taehee, On Wed, Jul 03, 2019 at 12:20:34AM +0900, Taehee Yoo wrote: > This patch series fixes several bugs in the gtp module. thanks a lot for your patches, they are much appreciated. They look valid to me after a brief initial review. However, I'm currently on holidays and don't have the ab

Re: [PATCH net-next] tipc: remove ub->ubsock checks

2019-07-02 Thread David Miller
From: Xin Long Date: Tue, 2 Jul 2019 00:57:19 +0800 > Both tipc_udp_enable and tipc_udp_disable are called under rtnl_lock, > ub->ubsock could never be NULL in tipc_udp_disable and cleanup_bearer, > so remove the check. > > Also remove the one in tipc_udp_enable by adding "free" label. > > Sig

Re: [PATCH net-next] bonding/main: fix NULL dereference in bond_select_active_slave()

2019-07-02 Thread David Miller
From: Eric Dumazet Date: Mon, 1 Jul 2019 10:48:51 -0700 > A bonding master can be up while best_slave is NULL. ... > Fixes: e2a7420df2e0 ("bonding/main: convert to using slave printk macros") > Signed-off-by: Eric Dumazet > Reported-by: John Sperbeck Applied, thanks.

Re: Validation of forward_delay seems wrong...

2019-07-02 Thread Nikolay Aleksandrov
On 03/07/2019 00:19, Nikolay Aleksandrov wrote: > On 02/07/2019 23:47, Andrew Lunn wrote: >> Hi Nikolay >> >> The man page says that the bridge forward_delay is in units of >> seconds, and should be between 2 and 30. >> >> I've tested on a couple of different kernel versions, and this appears >> to

Re: [PATCH v2 bpf-next 6/9] libbpf: use negative fd to specify missing BTF

2019-07-02 Thread Stanislav Fomichev
On 05/29, Andrii Nakryiko wrote: > 0 is a valid FD, so it's better to initialize it to -1, as is done in > other places. Also, technically, BTF type ID 0 is valid (it's a VOID > type), so it's more reliable to check btf_fd, instead of > btf_key_type_id, to determine if there is any BTF associated w

Re: [RFC iproute2] netns: add mounting state file for each netns

2019-07-02 Thread Stephen Hemminger
On Sun, 30 Jun 2019 21:29:33 +0200 Matteo Croce wrote: > @@ -737,6 +746,14 @@ static int netns_add(int argc, char **argv, bool create) > } > close(fd); > > + fd = open(tmp_path, O_RDONLY|O_CREAT|O_EXCL, 0); > + if (fd < 0) { > + fprintf(stderr, "Cannot create nam

Re: [PATCH] net: ethernet: mediatek: Allow non TRGMII mode with MT7621 DDR2 devices

2019-07-02 Thread David Miller
From: René van Dorst Date: Sat, 29 Jun 2019 14:24:51 +0200 > No reason to error out on a MT7621 device with DDR2 memory when non > TRGMII mode is selected. > Only MT7621 DDR2 clock setup is not supported for TRGMII mode. > But non TRGMII mode doesn't need any special clock setup. > > Signed-off-

Re: [PATCH] ipv4: Fix off-by-one in route dump counter without netlink strict checking

2019-07-02 Thread David Miller
From: Stefano Brivio Date: Sat, 29 Jun 2019 19:55:08 +0200 > In commit ee28906fd7a1 ("ipv4: Dump route exceptions if requested") I > added a counter of per-node dumped routes (including actual routes and > exceptions), analogous to the existing counter for dumped nodes. Dumping > exceptions means

Re: [PATCH net-next] tipc: use rcu dereference functions properly

2019-07-02 Thread David Miller
From: Xin Long Date: Tue, 2 Jul 2019 00:54:55 +0800 > For these places are protected by rcu_read_lock, we change from > rcu_dereference_rtnl to rcu_dereference, as there is no need to > check if rtnl lock is held. > > For these places are protected by rtnl_lock, we change from > rcu_dereference

Re: [PATCH net-next] af_packet: convert pending frame counter to atomic_t

2019-07-02 Thread David Miller
From: Neil Horman Date: Fri, 28 Jun 2019 10:52:06 -0400 > Given that the socket transmit path is an exclusive path (locked via the > pg_vec_lock mutex), we do not have the ability to increment this counter > on multiple cpus in parallel. This implementation also seems to have > the potential to

Re: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and port attribute

2019-07-02 Thread Jakub Kicinski
On Tue, 2 Jul 2019 18:50:31 +, Parav Pandit wrote: > > > I didn't see any immediate need to report, at the same time didn't > > > find any reason to treat such port flavours differently than existing > > > one. It just gives a clear view of the device's eswitch. Might find it > > > useful durin

Re: [PATCH net-next 3/7] net/rds: Wait for the FRMR_IS_FREE (or FRMR_IS_STALE) transition after posting IB_WR_LOCAL_INV

2019-07-02 Thread santosh . shilimkar
On 7/2/19 3:12 PM, Gerd Rausch wrote: On 02/07/2019 14.18, santosh.shilim...@oracle.com wrote: On 7/2/19 2:05 PM, Gerd Rausch wrote: What do you call "RDS_GET_MR" semantics? Its a blocking socket call. Meaning after this call return to the user, the key must be valid. With async registration

[PATCH net-next v5 2/5] net: core: move pop MPLS functionality from OvS to core helper

2019-07-02 Thread John Hurley
Open vSwitch provides code to pop an MPLS header to a packet. In preparation for supporting this in TC, move the pop code to an skb helper that can be reused. Remove the, now unused, update_ethertype static function from OvS. Signed-off-by: John Hurley Reviewed-by: Jakub Kicinski Reviewed-by: S

[PATCH net-next v5 1/5] net: core: move push MPLS functionality from OvS to core helper

2019-07-02 Thread John Hurley
Open vSwitch provides code to push an MPLS header to a packet. In preparation for supporting this in TC, move the push code to an skb helper that can be reused. Signed-off-by: John Hurley Reviewed-by: Jakub Kicinski Reviewed-by: Simon Horman --- include/linux/skbuff.h| 1 + net/core/skbuf

[PATCH net-next v5 3/5] net: core: add MPLS update core helper and use in OvS

2019-07-02 Thread John Hurley
Open vSwitch allows the updating of an existing MPLS header on a packet. In preparation for supporting similar functionality in TC, move this to a common skb helper function. Signed-off-by: John Hurley Reviewed-by: Jakub Kicinski Reviewed-by: Simon Horman --- include/linux/skbuff.h| 1 +

[PATCH net-next v5 4/5] net: sched: add mpls manipulation actions to TC

2019-07-02 Thread John Hurley
Currently, TC offers the ability to match on the MPLS fields of a packet through the use of the flow_dissector_key_mpls struct. However, as yet, TC actions do not allow the modification or manipulation of such fields. Add a new module that registers TC action ops to allow manipulation of MPLS. Thi

[PATCH net-next v5 0/5] Add MPLS actions to TC

2019-07-02 Thread John Hurley
This patchset introduces a new TC action module that allows the manipulation of the MPLS headers of packets. The code impliments functionality including push, pop, and modify. Also included are tests for the new funtionality. Note that these will require iproute2 changes to be submitted soon. NOT

[PATCH net-next v5 5/5] selftests: tc-tests: actions: add MPLS tests

2019-07-02 Thread John Hurley
Add a new series of selftests to verify the functionality of act_mpls in TC. Signed-off-by: John Hurley Reviewed-by: Simon Horman Acked-by: Jakub Kicinski --- .../tc-testing/tc-tests/actions/mpls.json | 812 + 1 file changed, 812 insertions(+) create mode 100644

[PATCH v3 bpf-next] bpf: Add support for fq's EDT to HBM

2019-07-02 Thread brakmo
Adds support for fq's Earliest Departure Time to HBM (Host Bandwidth Manager). Includes a new BPF program supporting EDT, and also updates corresponding programs. It will drop packets with an EDT of more than 500us in the future unless the packet belongs to a flow with less than 2 packets in fligh

[PATCH] gve: Fix u64_stats_sync to initialize start

2019-07-02 Thread Catherine Sullivan
u64_stats_fetch_begin needs to initialize start. Signed-off-by: Catherine Sullivan Reported-by: kbuild test robot --- drivers/net/ethernet/google/gve/gve_ethtool.c | 6 -- drivers/net/ethernet/google/gve/gve_main.c| 6 -- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git

[PATCH net-next 3/3 v2] macsec: add brackets and indentation after calling macsec_decrypt

2019-07-02 Thread Andreas Steinmetz
At this point, skb could only be a valid pointer, so this patch does not introduce any functional change. Change: The patch now only moves the IS_ERR(skb) case under the block where macsec_decrypt() is called, but not the call to macsec_post_decrypt(). Signed-off-by: Andreas Steinmetz --- a/dr

[PATCH net-next 1/3 v2] macsec: remove superfluous function calls

2019-07-02 Thread Andreas Steinmetz
Remove superfluous skb_share_check() and skb_unshare(). macsec_decrypt is only called by macsec_handle_frame which already does a skb_unshare(). Signed-off-by: Andreas Steinmetz --- a/drivers/net/macsec.c 2019-06-30 22:02:54.906908179 +0200 +++ b/drivers/net/macsec.c 2019-06-30 22:03:0

[PATCH net-next 2/3 v2] macsec: remove superfluous zeroing of skb reference

2019-07-02 Thread Andreas Steinmetz
Remove superfluous zeroing of skb pointer for the RX_HANDLER_CONSUMED case, since in that case, __netif_receive_skb_core will simply ignore the value. Signed-off-by: Andreas Steinmetz --- a/drivers/net/macsec.c 2019-06-30 22:04:10.003613880 +0200 +++ b/drivers/net/macsec.c 2019-06-30 2

[PATCH net-next 0/3 v2] macsec: a few cleanups in the receive path

2019-07-02 Thread Andreas Steinmetz
This patchset removes some unnecessary code in the receive path of the macsec driver, and re-indents the error handling after calling macsec_decrypt to make the post-processing clearer. This is a combined effort of Sabrina Dubroca and me. Change in 3/3: The patch now only moves the IS_ERR(skb)

[PATCH net-next 0/2] Mellanox, mlx5 devlink versions query

2019-07-02 Thread Saeed Mahameed
Hi Dave, This humble 2 patch series from Shay adds the support for devlink fw versions query to mlx5 driver. In the first patch we implement the needed fw commands to support this feature. In the 2nd patch we implement the devlink callbacks themselves. I am not sending this as a pull request sin

[PATCH net-next 2/2] net/mlx5: Added devlink info callback

2019-07-02 Thread Saeed Mahameed
From: Shay Agroskin The callback is invoked using 'devlink dev info ' command and returns the running and pending firmware version of the HCA and the name of the kernel driver. If there is a pending firmware version (a new version is burned but the HCA still runs with the previous) it is returne

[PATCH net-next 1/2] net/mlx5: Added fw version query command

2019-07-02 Thread Saeed Mahameed
From: Shay Agroskin Using the MCQI and MCQS registers, we query the running and pending fw version of the HCA. The MCQS is queried with sequentially increasing component index, until a component of type BOOT_IMG is found. Querying this component's version using the MCQI register yields the runnin

[PATCH v4 bpf-next] bpf: Add support for fq's EDT to HBM

2019-07-02 Thread brakmo
Adds support for fq's Earliest Departure Time to HBM (Host Bandwidth Manager). Includes a new BPF program supporting EDT, and also updates corresponding programs. It will drop packets with an EDT of more than 500us in the future unless the packet belongs to a flow with less than 2 packets in fligh

Re: [PATCH net-next 3/7] net/rds: Wait for the FRMR_IS_FREE (or FRMR_IS_STALE) transition after posting IB_WR_LOCAL_INV

2019-07-02 Thread santosh . shilimkar
On 7/2/19 2:05 PM, Gerd Rausch wrote: On 02/07/2019 09.49, santosh.shilim...@oracle.com wrote: On 7/1/19 10:11 PM, Gerd Rausch wrote: For the registration work-requests there is a benefit to wait a short amount of time only (the trade-off described in patch #1 of this series). Actually we sho

Re: [PATCH net 2/2] macsec: fix checksumming after decryption

2019-07-02 Thread Andreas Steinmetz
On Tue, 2019-07-02 at 10:35 -0400, Willem de Bruijn wrote: > On Tue, Jul 2, 2019 at 12:25 AM Andreas Steinmetz wrote: > > On Sun, 2019-06-30 at 21:47 -0400, Willem de Bruijn wrote: > > > On Sun, Jun 30, 2019 at 4:48 PM Andreas Steinmetz > > > wrote: > > > > Fix checksumming after decryption. > >

[PATCH net-next] skbuff: increase verbosity when dumping skb data

2019-07-02 Thread Willem de Bruijn
From: Willem de Bruijn skb_warn_bad_offload and netdev_rx_csum_fault trigger on hard to debug issues. Dump more state and the header. Optionally dump the entire packet and linear segment. This is required to debug checksum bugs that may include bytes past skb_tail_pointer(). Both call sites cal

Re: veth pair ping fail if one of them enslaved into a VRF

2019-07-02 Thread Zoltán Elek
Thank you for the detailed answer, I tried your solution and it works as expected. Ido Schimmel ezt írta (időpont: 2019. júl. 2., K, 21:21): > > On Tue, Jul 02, 2019 at 08:42:15PM +0200, Zoltán Elek wrote: > > Hi! > > > > I have a simple scenario, with a veth pair, IP addresses assigned from > >

Re: [PATCH v2 bpf-next 1/4] libbpf: capture value in BTF type info for BTF-defined map defs

2019-07-02 Thread Andrii Nakryiko
On Tue, Jul 2, 2019 at 12:19 PM Y Song wrote: > > On Fri, Jun 28, 2019 at 8:26 AM Andrii Nakryiko wrote: > > > > Change BTF-defined map definitions to capture compile-time integer > > values as part of BTF type definition, to avoid split of key/value type > > information and actual type/size/flag

Re: [PATCH v2 bpf-next] bpf: Add support for fq's EDT to HBM

2019-07-02 Thread Lawrence Brakmo
Yonhhong, thank you for the feedbac. Repies inlined. I will submit a new revision later today. On 7/1/19, 2:04 PM, "Yonghong Song" wrote: On 6/28/19 12:41 PM, brakmo wrote: > Adds support for fq's Earliest Departure Time to HBM (Host Bandwidth > Manager). Includes a new BPF program

Re: [PATCH v2 bpf-next 1/4] bpf: unprivileged BPF access via /dev/bpf

2019-07-02 Thread Andrii Nakryiko
On Thu, Jun 27, 2019 at 1:20 PM Song Liu wrote: > > This patch introduce unprivileged BPF access. The access control is > achieved via device /dev/bpf. Users with write access to /dev/bpf are able > to call sys_bpf(). > > Two ioctl command are added to /dev/bpf: > > The two commands enable/disable

Re: [PATCH net-next 1/1] qed: Add support for Timestamping the unicast PTP packets.

2019-07-02 Thread David Miller
From: Sudarsana Reddy Kalluru Date: Tue, 2 Jul 2019 08:04:12 -0700 > The register definition in the header file captures more details on > the individual bits. Where is this register definition in the header file and why aren't CPP defines from there being used instead of a mask constant mask?

Re: veth pair ping fail if one of them enslaved into a VRF

2019-07-02 Thread Ido Schimmel
On Tue, Jul 02, 2019 at 08:42:15PM +0200, Zoltán Elek wrote: > Hi! > > I have a simple scenario, with a veth pair, IP addresses assigned from > the same subnet. They can ping eachother. But when I put one of them > into a VRF (in the example below, I put veth in-vrf into the test-vrf > VRF) the pi

Re: [PATCH v2 bpf-next 1/4] libbpf: capture value in BTF type info for BTF-defined map defs

2019-07-02 Thread Y Song
On Fri, Jun 28, 2019 at 8:26 AM Andrii Nakryiko wrote: > > Change BTF-defined map definitions to capture compile-time integer > values as part of BTF type definition, to avoid split of key/value type > information and actual type/size/flags initialization for maps. > > Signed-off-by: Andrii Nakryi

Re: [PATCH net-next 07/12] net: use tcf_block_setup() infrastructure

2019-07-02 Thread Marcelo Ricardo Leitner
On Thu, Jun 20, 2019 at 09:49:12PM +0200, Pablo Neira Ayuso wrote: ... > @@ -1173,8 +1191,10 @@ static int tcf_block_offload_cmd(struct tcf_block > *block, > struct tc_block_offload bo = {}; > int err; > > + bo.net = dev_net(dev); > bo.command = command; > bo.binder_t

Re: [PATCH net-next 02/12] net: sched: add tcf_block_cb_alloc()

2019-07-02 Thread Marcelo Ricardo Leitner
On Thu, Jun 20, 2019 at 09:49:07PM +0200, Pablo Neira Ayuso wrote: > Add a new helper function to allocate tcf_block_cb objects. > > Signed-off-by: Pablo Neira Ayuso > --- > include/net/pkt_cls.h | 8 > net/sched/cls_api.c | 23 +++ > 2 files changed, 27 insertion

[PATCH net-next] mlxsw: spectrum_ptp: Fix validation in mlxsw_sp1_ptp_packet_finish()

2019-07-02 Thread Petr Machata
Before mlxsw_sp1_ptp_packet_finish() sends the packet back, it validates whether the corresponding port is still valid. However the condition is incorrect: when mlxsw_sp_port == NULL, the code dereferences the port to compare it to skb->dev. The condition needs to check whether the port is present

RE: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and port attribute

2019-07-02 Thread Parav Pandit
> -Original Message- > From: Jakub Kicinski > Sent: Tuesday, July 2, 2019 11:17 PM > To: Parav Pandit > Cc: Jiri Pirko ; netdev@vger.kernel.org; Saeed > Mahameed > Subject: Re: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and > port attribute > > On Tue, 2 Jul 2019 04:2

[PATCH v2 net-next] r8169: add random MAC address fallback

2019-07-02 Thread Heiner Kallweit
It was reported that the GPD MicroPC is broken in a way that no valid MAC address can be read from the network chip. The vendor driver deals with this by assigning a random MAC address as fallback. So let's do the same. Signed-off-by: Heiner Kallweit --- v2: - fix broken commit message --- drive

veth pair ping fail if one of them enslaved into a VRF

2019-07-02 Thread Zoltán Elek
Hi! I have a simple scenario, with a veth pair, IP addresses assigned from the same subnet. They can ping eachother. But when I put one of them into a VRF (in the example below, I put veth in-vrf into the test-vrf VRF) the ping fails. My first question: that is the expected behavior? And my second

[PATCH bpf-next] selftests: bpf: standardize to static __always_inline

2019-07-02 Thread Jiri Benc
The progs for bpf selftests use several different notations to force function inlining. Standardize to what most of them use, static __always_inline. Suggested-by: Song Liu Signed-off-by: Jiri Benc --- tools/testing/selftests/bpf/progs/pyperf.h| 9 +++-- .../testing/selftests/bpf/progs/str

Re: [PATCH bpf v2] selftests: bpf: fix inlines in test_lwt_seg6local

2019-07-02 Thread Y Song
On Tue, Jul 2, 2019 at 10:41 AM Jiri Benc wrote: > > Selftests are reporting this failure in test_lwt_seg6local.sh: > > + ip netns exec ns2 ip -6 route add fb00::6 encap bpf in obj > test_lwt_seg6local.o sec encap_srh dev veth2 > Error fetching program/map! > Failed to parse eBPF program: Operati

Re: [PATCH bpf-next v2 0/8] bpf: TCP RTT sock_ops bpf callback

2019-07-02 Thread Y Song
On Tue, Jul 2, 2019 at 9:14 AM Stanislav Fomichev wrote: > > Congestion control team would like to have a periodic callback to > track some TCP statistics. Let's add a sock_ops callback that can be > selectively enabled on a socket by socket basis and is executed for > every RTT. BPF program frequ

Re: [PATCH bpf-next] selftests/bpf: fix compiling loop{1,2,3}.c on s390

2019-07-02 Thread Y Song
On Tue, Jul 2, 2019 at 8:40 AM Ilya Leoshkevich wrote: > > Use PT_REGS_RC(ctx) instead of ctx->rax, which is not present on s390. > > Pass -D__TARGET_ARCH_$(ARCH) to selftests in order to choose a proper > PT_REGS_RC variant. > > Fix s930 -> s390 typo. > > On s390, provide the forward declaration

Re: [PATCH bpf-next] selftests/bpf: fix compiling loop{1,2,3}.c on s390

2019-07-02 Thread Y Song
On Tue, Jul 2, 2019 at 9:58 AM Ilya Leoshkevich wrote: > > > Am 02.07.2019 um 18:42 schrieb Y Song : > > > > On Tue, Jul 2, 2019 at 8:40 AM Ilya Leoshkevich wrote: > >> > >> -#elif defined(__s390x__) > >> - #define bpf_target_s930x > > > > I see in some other places (e.g., bcc) where > > ma

Re: [PATCH net-next 1/3] devlink: Introduce PCI PF port flavour and port attribute

2019-07-02 Thread Jakub Kicinski
On Tue, 2 Jul 2019 04:26:47 +, Parav Pandit wrote: > > On Mon, 1 Jul 2019 07:27:32 -0500, Parav Pandit wrote: > > > In an eswitch, PCI PF may have port which is normally represented > > > using a representor netdevice. > > > To have better visibility of eswitch port, its association with PF,

[PATCH net-next v2] bonding: add an option to specify a delay between peer notifications

2019-07-02 Thread Vincent Bernat
Currently, gratuitous ARP/ND packets are sent every `miimon' milliseconds. This commit allows a user to specify a custom delay through a new option, `peer_notif_delay'. Like for `updelay' and `downdelay', this delay should be a multiple of `miimon' to avoid managing an additional work queue. The c

[PATCH bpf v2] selftests: bpf: fix inlines in test_lwt_seg6local

2019-07-02 Thread Jiri Benc
Selftests are reporting this failure in test_lwt_seg6local.sh: + ip netns exec ns2 ip -6 route add fb00::6 encap bpf in obj test_lwt_seg6local.o sec encap_srh dev veth2 Error fetching program/map! Failed to parse eBPF program: Operation not permitted The problem is __attribute__((always_inline))

Re: [PATCH bpf-next v2] bpftool: Add BPF_F_QUERY_EFFECTIVE support in bpftool cgroup [show|tree]

2019-07-02 Thread Jakub Kicinski
On Tue, 2 Jul 2019 09:48:43 -0700, Takshak Chahande wrote: > + cgroup_fd = open(cgroup_path, O_RDONLY); > if (cgroup_fd < 0) { > - p_err("can't open cgroup %s", argv[1]); > + p_err("can't open cgroup %s", cgroup_path); AFAICS the bpf tree has not been merged into

Re: [PATCH bpf] selftests: bpf: fix inlines in test_lwt_seg6local

2019-07-02 Thread Jiri Benc
On Mon, 1 Jul 2019 11:39:27 -0700, Y Song wrote: > By default, we have > # define __always_inlineinline __attribute__((always_inline)) > > So just use __always_inline should be less verbose in your patch. I'll resubmit, converting everything to __always_inline, targeting bpf-next. > BTW,

RE: [PATCH net-next 1/1] devlink: Add APIs to publish/unpublish the port parameters.

2019-07-02 Thread Parav Pandit
> -Original Message- > From: Andrew Lunn > Sent: Tuesday, July 2, 2019 10:19 PM > To: Parav Pandit > Cc: Sudarsana Reddy Kalluru ; da...@davemloft.net; > netdev@vger.kernel.org; mkalde...@marvell.com; ael...@marvell.com; > j...@resnulli.us > Subject: Re: [PATCH net-next 1/1] devlink: A

  1   2   >