Hi Yonghong,
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Yonghong-Song/bpf-add-two-helpers-to-read-perf-event-enabled-running-time/20170919-134113
config: m68k-allyesconfig (attached as .config)
compiler: m68k-linux-gcc (GCC) 4.9.0
reproduce:
On Thu, Sep 14, 2017 at 09:19:49PM +0200, Andrew Lunn wrote:
> > > Is the MDIO controller "allwinner,sun8i-h3-emac" or "snps,dwmac-mdio"?
> > > If the latter, then I think the node is fine, but then the mux should be
> > > a child node of it. IOW, the child of an MDIO controller should either
>
On Mon, Sep 18, 2017 at 04:06:28PM -0700, Vinicius Costa Gomes wrote:
> That's the point, the application does not need to know that, and asking
> that would be stupid.
On the contrary, this information is essential to the application.
Probably you have never seen an actual Ethernet field bus in
o
From: Eric Dumazet
skb->rbnode shares space with skb->next, skb->prev and skb->tstamp
Current uses (TCP receive ofo queue and netem) need to save/restore
tstamp.
Since we might use an RB tree for TCP retransmit queue at some point
to speedup SACK processing with large rtx queues, this patch
Hi all,
Commit
129c6cda2de2 ("8139too: revisit napi_complete_done() usage")
is missing a Signed-off-by from its author.
--
Cheers,
Stephen Rothwell
From: Tom Herbert
Date: Mon, 18 Sep 2017 17:39:02 -0700
> Add configuration to control use of zero checksums on transmit for both
> IPv4 and IPv6, and control over accepting zero IPv6 checksums on
> receive.
>
> Signed-off-by: Tom Herbert
I thought we were trying to move away from this special
From: Tom Herbert
Date: Mon, 18 Sep 2017 17:39:01 -0700
> Allow applications or encapsulation protocols to register a GSO segment
> function to their specific protocol. To faciliate this I reserved the
> upper four bits in the gso_type to indicate the application specific GSO
> type. Zero in thes
From: Tom Herbert
Date: Mon, 18 Sep 2017 17:38:58 -0700
> Allow peers to be specified by IPv6 addresses.
>
> Signed-off-by: Tom Herbert
Hmmm, can you just check the socket family or something like that?
From: Tom Herbert
Date: Mon, 18 Sep 2017 17:38:57 -0700
> @@ -98,6 +104,7 @@ static void pdp_context_delete(struct pdp_ctx *pctx);
> static inline u32 gtp0_hashfn(u64 tid)
> {
> u32 *tid32 = (u32 *) &tid;
> +
> return jhash_2words(tid32[0], tid32[1], gtp_h_initval);
> }
>
> @@ -1
From: Tom Herbert
Date: Mon, 18 Sep 2017 17:38:53 -0700
> 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
Not caused by your changes, but something to thin
On 2017-09-18 21:45, Eric W. Biederman wrote:
> Richard Guy Briggs writes:
>
> > On 2017-09-14 12:33, Eric W. Biederman wrote:
> >> Richard Guy Briggs writes:
> >>
> >> > The trigger is a pseudo filesystem (proc, since PID tree already exists)
> >> > write of a u64 representing the container ID
From: Florian Fainelli
Date: Mon, 18 Sep 2017 16:31:30 -0700
> There are several problems with commit 10377ba7673d ("net: systemport:
> Support 64bit statistics", first one got fixed in 7095c973453e ("net:
> systemport: Fix 64-bit stats deadlock").
>
> The second problem is that this specific co
On Tue, Sep 19, 2017 at 02:06:21AM +0100, Salil Mehta wrote:
> This patch set presents some bug fixes for the HNS3 Ethernet driver,
> identified
> during internal testing & stabilization efforts.
>
> This patch series is meant for Linux 4.14 kernel.
>
> Lipeng (6):
> net: hns3: get phy addr from
From: Eric Dumazet
Date: Mon, 18 Sep 2017 13:03:43 -0700
> From: Eric Dumazet
>
> It seems we have to be more careful in napi_complete_done()
> use. This patch is not a revert, as it seems we can
> avoid bug that Ville reported by moving the napi_complete_done()
> test in the spinlock section.
Hi Michal,
On Mon, Sep 18, 2017 at 09:55:05AM +0200, Michal Kubecek wrote:
> > +static int rtnl_recvmsg(int fd, struct msghdr *msg, char **answer)
> > +{
> > + struct iovec *iov;
> > + int len = -1, buf_len = 32768;
> > + char *bufp, *buf = NULL;
> > +
> > + int flag = MSG_PEEK | MSG_TRUNC
Richard Guy Briggs writes:
> On 2017-09-14 12:33, Eric W. Biederman wrote:
>> Richard Guy Briggs writes:
>>
>> > The trigger is a pseudo filesystem (proc, since PID tree already exists)
>> > write of a u64 representing the container ID to a file representing a
>> > process that will become the
Export b53_{enable,disable}_port and use these two functions in
bcm_sf2_port_setup and bcm_sf2_port_disable. The generic functions
cannot be used without wrapping because we need to manage additional
switch integration details (PHY, Broadcom tag etc.).
Signed-off-by: Florian Fainelli
---
drivers
In preparation for migrating the EEE code from bcm_sf2 to b53, define the full
EEE register page and offsets within that page.
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_regs.h | 41 +
1 file changed, 41 insertions(+)
diff --git a/drivers
bcm_sf2 and b53 do exactly the same thing, so share that piece.
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_common.c | 3 ++-
drivers/net/dsa/b53/b53_priv.h | 1 +
drivers/net/dsa/bcm_sf2.c| 23 +--
3 files changed, 4 insertions(+), 23 deletions(-)
Instead of repeating the same pattern: acquire mutex, read/write,
release mutex, define a macro: b53_build_op() which takes the type
(read|write), I/O size, and value (scalar or pointer). This helps with
fixing bugs that could exist (e.g: missing barrier, lock etc.).
Reviewed-by: Vivien Didelot
S
The code to enable Broadcom tags/headers is largely switch independent,
and in preparation for enabling it for multiple devices with b53, move
the code we have in bcm_sf2.c to b53_common.c
Reviewed-by: Vivien Didelot
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_common.c | 47
The magic number 8 in 3 locations in bcm_sf2_cfp.c actually designates the
number of switch port egress queues, so use that define instead of open-coding
it.
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/bcm_sf2_cfp.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
Add support for enabling and disabling EEE, as well as re-negotiating it in
.adjust_link() and in .port_enable().
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_common.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/ds
Move the bcm_sf2 EEE-related functions to the b53 driver because this is shared
code amongst Gigabit capable switch, only 5325 and 5365 are too old to support
that.
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_common.c | 63 ++
drivers/net/dsa/b
From: Eric Dumazet
Date: Mon, 18 Sep 2017 12:36:22 -0700
> From: Eric Dumazet
>
> HTB qdisc overlimits counter is properly increased, but we have no per
> class counter, meaning it is difficult to diagnose HTB problems.
>
> This patch adds this counter, visible in "tc -s class show dev eth0",
There is no need to configure the enabled ports once in bcm_sf2_sw_setup() and
then a second time around when dsa_switch_ops::port_enable is called, just do
it when port_enable is called which is better in terms of power consumption and
correctness.
Reviewed-by: Vivien Didelot
Signed-off-by: Flor
There is no need to configure the enabled ports once in b53_setup() and then a
second time around when dsa_switch_ops::port_enable is called, just do it when
port_enable is called which is better in terms of power consumption and
correctness.
Reviewed-by: Vivien Didelot
Signed-off-by: Florian Fai
In preparation for future changes allowing the configuring of multiple
CPU ports, make b53_enable_cpu_port() take a port argument.
Reviewed-by: Vivien Didelot
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_common.c | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
This is not used anywhere, so remove it.
Reviewed-by: Vivien Didelot
Signed-off-by: Florian Fainelli
---
drivers/net/dsa/b53/b53_priv.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/net/dsa/b53/b53_priv.h b/drivers/net/dsa/b53/b53_priv.h
index 01bd8cbe9a3f..7528b22aeb03 100644
Hi all,
This patch series is a first pass set of clean-ups to reduce the number of LOCs
between b53 and bcm_sf2 and sharing as many functions as possible.
There is a number of additional cleanups queued up locally that require more
thorough testing.
Thanks!
Changes in v2:
- added Reviewed-by t
On Tue, Sep 19, 2017 at 03:16:44AM +0200, Daniel Borkmann wrote:
> Commit 109980b894e9 ("bpf: don't select potentially stale
> ri->map from buggy xdp progs") passed the pointer to the prog
> itself to be loaded into r4 prior on bpf_redirect_map() helper
> call, so that we can store the owner into r
On Mon, Sep 18, 2017 at 8:47 PM, Jason A. Donenfeld wrote:
> The best I've come up with is, in a sleep loop, writing to the tun
> device's fd something with a NULL or invalid payload. If the interface
> is down, the kernel returns -EIO. If the interface is up, the kernel
> returns -EINVAL. This se
Commit 109980b894e9 ("bpf: don't select potentially stale
ri->map from buggy xdp progs") passed the pointer to the prog
itself to be loaded into r4 prior on bpf_redirect_map() helper
call, so that we can store the owner into ri->map_owner out of
the helper.
Issue with that is that the actual addre
This patch set presents some bug fixes for the HNS3 Ethernet driver, identified
during internal testing & stabilization efforts.
This patch series is meant for Linux 4.14 kernel.
Lipeng (6):
net: hns3: get phy addr from NCL_config
net: hns3: fix the command used to unmap ring from vector
n
From: Lipeng
This patch add INT_GL and VF id to vector configure when bind ring
with vector. INT_GL means Interrupt Gap Limiting. Vector id starts
from 0 in each VF, so the bind command must specify VF id.
Signed-off-by: Lipeng
Signed-off-by: Mingguang Qu
Signed-off-by: Salil Mehta
---
drive
This patch replaces the ethernet address copy instance with more
appropriate ether_addr_copy() function.
Signed-off-by: Salil Mehta
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns
From: Lipeng
HNS3 driver get mac address from NCL_config file and set the mac address
to HW. If the mac address in NCL_config is invalid, driver will set a
random mac address, and use this address.
The current code will set random mac address to HW, but will not set the
valid mac address from NC
From: Lipeng
When register/unregister ae_dev, ae_dev should match all client
in the client_list. Enet and roce can co-exists together so we
should continue checking for enet and roce presence together.
So break should not be there.
Above caused problems in loading and unloading of modules.
Sign
> -Original Message-
> From: netdev-ow...@vger.kernel.org [mailto:netdev-
> ow...@vger.kernel.org] On Behalf Of Joe Perches
> Sent: Sunday, September 17, 2017 23:15
> To: Jon Maloy ; Thomas Meyer
>
> Cc: Ying Xue ; netdev@vger.kernel.org; tipc-
> discuss...@lists.sourceforge.net; linux-k
From: Lipeng
When there is no vlan id in the packets, hardware will treat the vlan id
as 0 and look for the mac_vlan table. This patch set the default vlan id
of PF as 0. Without this config, it will fail when look for mac_vlan
table, and hardware will drop packets.
Signed-off-by: Mingguang Qu
From: Lipeng
NCL_config file defines phy address for every port. Driver should get
phy address from NCL_config file.If do not get the right phy address,
every port will use the default phy address 0, different port use the
same phy address will cause error.
Signed-off-by: Lipeng
Signed-off-by:
From: Lipeng
When unmap ring from vector, it use wrong command, this will cause
error if the unmap action need multi command description. This patch
fix the error.
Signed-off-by: Lipeng
Signed-off-by: Salil Mehta
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 2 +-
1 file chang
> I am not really sure why this is such a concern for you so soon when
> your driver is not even included yet. You should really aim for baby
> steps here: get the basic driver(s) included, with a limited set of
> features, and gradually add more features to the driver. When
> fwd_offload_mark and
Hi all,
Where is the ID field of the IPv4 header created when the DF flag is
set? I am looking at ip_build_and_send_pkt. The code seems to have
changed in 4.4-rc1:
if (ip_dont_fragment(sk, &rt->dst)) {
iph->frag_off = htons(IP_DF);
iph->id = 0;
} else {
iph->frag_off = 0;
__ip_sel
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| 167 ---
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
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 | 59 ++-
1 file changed, 32 insertions(+),
Populate GRO receive and GRO complete functions for GTP-Uv0 and v1.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 204 ++
1 file changed, 204 insertions(+)
diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index b53946f8b10b..2f9d810cf19
Allow peers to be specified by IPv6 addresses.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c| 198 +--
include/uapi/linux/gtp.h | 1 +
include/uapi/linux/if_link.h | 3 +
3 files changed, 158 insertions(+), 44 deletions(-)
diff --gi
Need to define a gtp_gso_segment since the GTP header includes a length
field that must be set per packet. Also, GPv0 header includes a sequence
number that is incremented per packet.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c| 176 +++
i
Add a net field to gtp that is derived from src_net. Use net_eq to make
cross net argument for transmit functions.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index 1870469a4
Add configuration to control use of zero checksums on transmit for both
IPv4 and IPv6, and control over accepting zero IPv6 checksums on
receive.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c| 35 +--
include/uapi/linux/if_link.h | 4
2 files
Allow applications or encapsulation protocols to register a GSO segment
function to their specific protocol. To faciliate this I reserved the
upper four bits in the gso_type to indicate the application specific GSO
type. Zero in these bits indicates no application GSO, so there are
fifteen instance
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
Removes MTU handling in gtp_build_skb_ip4. This is non standard relative
to how other tunneling protocols handle MTU. The model espoused is that
the inner interface should set it's MTU to be less than the expected
path MTU on the overlay network. Path MTU discovery is not typically
used for modifyi
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.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c| 259 +--
include/uapi/linux
This patch set builds upon the initial GTP implementation to make
support closer to that enjoyed by other encapsulation protocols.
The major items are:
- IPv6 support
- Configurable networking interfaces so that GTP kernel can be
used and tested without needing GSN network emulation (i.e.
Create separate UDP receive functions for GTP version 0 and version 1.
Set encap_rcv appropriately when configuring a socket. Also, convert to
using gro_cells.
Signed-off-by: Tom Herbert
---
drivers/net/gtp.c | 130 +-
1 file changed, 71 insert
ip_tunnel_get_route and ip6_tnl_get_route are create to return
routes for a tunnel. These functions are derived from the VXLAN
functions.
Signed-off-by: Tom Herbert
---
include/net/ip6_tunnel.h | 33 +
include/net/ip_tunnels.h | 33
From: Yuchung Cheng
Date: Mon, 18 Sep 2017 11:05:16 -0700
> remove tcp_may_send_now and tcp_snd_test that are no longer used
>
> Fixes: 840a3cbe8969 ("tcp: remove forward retransmit feature")
> Signed-off-by: Yuchung Cheng
> Signed-off-by: Neal Cardwell
> Signed-off-by: Eric Dumazet
Applied,
From: Antoine Tenart
Date: Mon, 18 Sep 2017 15:04:06 +0200
> The dev->dma_mask usually points to dev->coherent_dma_mask. This is an
> issue as setting both of them will override the other. This is
> problematic here as the PPv2 driver uses a 32-bit-mask for coherent
> accesses (txq, rxq, bm) and
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@osuosl.org] On Behalf
> Of Benjamin Poirier
> Sent: Friday, July 21, 2017 11:36 AM
> To: Kirsher, Jeffrey T
> Cc: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org; linux-
> ker...@vger.kernel.org; Lennart Sorensen
> Subject: [Intel-w
From: Tobias Klauser
Date: Mon, 18 Sep 2017 15:03:46 +0200
> If bpf_map_precharge_memlock in dev_map_alloc, -ENOMEM is returned
> regardless of the actual error produced by bpf_map_precharge_memlock.
> Fix it by passing on the error returned by bpf_map_precharge_memlock.
>
> Also return -EINVAL
From: Sathya Perla
Date: Mon, 18 Sep 2017 17:05:37 +0530
> Check for ingress-only qdisc for flower offload, as other qdiscs
> are not supported for flower offload.
>
> Suggested-by: Jiri Pirko
> Signed-off-by: Sathya Perla
Applied, thanks.
From: Colin King
Date: Mon, 18 Sep 2017 12:40:38 +0100
> From: Colin Ian King
>
> Pointer tcm is being initialized and is never read, it is only being used
> to determine the size of struct tcmsg. Clean this up by removing
> variable tcm and explicitly using the sizeof struct tcmsg rather than
From: Roman Yeryomin
Date: Sun, 17 Sep 2017 20:23:53 +0300
> Changes from v1:
> - use GRO instead of increasing ring size
> - use NAPI_POLL_WEIGHT instead of defining own NAPI_WEIGHT
> - optimize rx descriptor flags processing
Series applied, thank you.
On 09/18/2017 04:44 PM, tristram...@microchip.com wrote:
>>> In the old DSA implementation all the ports are partitioned into its own
>> device
>>> and the bridge joining them will do all the forwarding. This is useful for
>> quick
>>> testing with some protocols like RSTP but it is probably usele
> > In the old DSA implementation all the ports are partitioned into its own
> device
> > and the bridge joining them will do all the forwarding. This is useful for
> quick
> > testing with some protocols like RSTP but it is probably useless for real
> > operation.
>
> It is a good minimal driver
From: Sudip Mukherjee
Date: Sun, 17 Sep 2017 12:46:20 +0100
> Modify baycom driver to use the new parallel port device model.
>
> Signed-off-by: Sudip Mukherjee
Applied to net-next, thanks.
From: Randy Dunlap
Date: Sat, 16 Sep 2017 13:10:06 -0700
> From: Randy Dunlap
>
> Fix ASCII art in Documentation/networking/switchdev.txt:
>
> Change non-ASCII "spaces" to ASCII spaces.
>
> Change 2 erroneous '+' characters in ASCII art to '-' (at the '*'
> characters below):
>
> line 32:
>
This patch fixes a bug exhibited by the following scenario:
1. fd1 = perf_event_open with attr.config = ID1
2. attach bpf program prog1 to fd1
3. fd2 = perf_event_open with attr.config = ID1
4. user program closes fd2 and prog1 is detached from the tracepoint.
5. user program with f
From: Davide Caratti
Date: Sat, 16 Sep 2017 14:02:21 +0200
> this script, edited from Linux Advanced Routing and Traffic Control guide
>
> tc q a dev en0 root handle 1: htb default a
> tc c a dev en0 parent 1: classid 1:1 htb rate 6mbit burst 15k
> tc c a dev en0 parent 1:1 classid 1:a htb rate
From: Xin Long
Date: Fri, 15 Sep 2017 15:58:33 +0800
> If ipv6 has been disabled from cmdline since kernel started, it makes
> no sense to allow users to create any ip6 tunnel. Otherwise, it could
> some potential problem.
>
> Jianlin found a kernel crash caused by this in ip6_gre when he set
>
From: Fahad Kunnathadi
Date: Fri, 15 Sep 2017 12:01:58 +0530
> To clear Speed Selection in MDIO control register(0x10),
> ie, clear bits 6 and 13 to zero while keeping other bits same.
> Before AND operation,The Mask value has to be perform with bitwise NOT
> operation (ie, ~ operator)
>
> This
There are several problems with commit 10377ba7673d ("net: systemport:
Support 64bit statistics", first one got fixed in 7095c973453e ("net:
systemport: Fix 64-bit stats deadlock").
The second problem is that this specific code updates the
stats64.tx_{packets,bytes} from ndo_get_stats64() and that
From: Florian Fainelli
Date: Mon, 18 Sep 2017 14:46:48 -0700
> On 09/18/2017 02:41 PM, Florian Fainelli wrote:
>> Hi all,
>>
>> This patch series is a first pass set of clean-ups to reduce the number of
>> LOCs
>> between b53 and bcm_sf2 and sharing as many functions as possible.
>>
>> There i
On Mon, Sep 18, 2017 at 06:30:49PM +0300, Mika Westerberg wrote:
> From: Amir Levy
>
> ThunderboltIP is a protocol created by Apple to tunnel IP/ethernet
> traffic over a Thunderbolt cable. The protocol consists of configuration
> phase where each side sends ThunderboltIP login packets (the proto
Hi Richard,
Richard Cochran writes:
> On Thu, Aug 31, 2017 at 06:26:20PM -0700, Vinicius Costa Gomes wrote:
>> * Time-aware shaper (802.1Qbv):
>
> I just posted a working alternative showing how to handle 802.1Qbv and
> many other Ethernet field buses.
>
>>The idea we are currently explorin
The bpf sample program tracex6 is enhanced to use the new
helper to read enabled/running time as well.
Signed-off-by: Yonghong Song
---
samples/bpf/tracex6_kern.c| 26 ++
samples/bpf/tracex6_user.c| 13 -
tools/include/uapi/linu
Hardware pmu counters are limited resources. When there are more
pmu based perf events opened than available counters, kernel will
multiplex these events so each event gets certain percentage
(but not 100%) of the pmu time. In case that multiplexing happens,
the number of samples or counter value w
This patch adds helper bpf_perf_prog_read_cvalue for perf event based bpf
programs, to read event counter and enabled/running time.
The enabled/running time is accumulated since the perf event open.
The typical use case for perf event based bpf program is to attach itself
to a single event. In suc
The bpf sample program trace_event is enhanced to use the new
helper to print out enabled/running time.
Signed-off-by: Yonghong Song
---
samples/bpf/trace_event_kern.c| 10 ++
samples/bpf/trace_event_user.c| 13 -
tools/include/uapi/linux/bpf.h
Hardware pmu counters are limited resources. When there are more
pmu based perf events opened than available counters, kernel will
multiplex these events so each event gets certain percentage
(but not 100%) of the pmu time. In case that multiplexing happens,
the number of samples or counter value w
On 9/18/17 12:30 PM, Craig Gallek wrote:
From: Craig Gallek
Extend the 'random' operation tests to include a delete operation
(delete half of the nodes from both lpm implementions and ensure
that lookups are still equivalent).
Also, add a simple IPv4 test which verifies lookup behavior as node
On 9/18/17 12:30 PM, Craig Gallek wrote:
From: Craig Gallek
The 'trivial' lpm implementation in this test allows equivalent nodes
to be added (that is, nodes consisting of the same prefix and prefix
length). For lookup operations, this is fine because insertion happens
at the head of the (sing
On 9/18/17 12:30 PM, Craig Gallek wrote:
From: Craig Gallek
This is a simple non-recursive delete operation. It prunes paths
of empty nodes in the tree, but it does not try to further compress
the tree as nodes are removed.
Signed-off-by: Craig Gallek
---
kernel/bpf/lpm_trie.c | 80
> In the old DSA implementation all the ports are partitioned into its own
> device
> and the bridge joining them will do all the forwarding. This is useful for
> quick
> testing with some protocols like RSTP but it is probably useless for real
> operation.
It is a good minimal driver, to get s
On Mon, Sep 18, 2017 at 08:55:17PM +, tristram...@microchip.com wrote:
> > > > This is ugly. We have a clean separation between a switch driver and a
> > > > tag driver. Here you are mixing them together. Don't. Look at the
> > > > mailing lists for what Florian and I suggested to Pavel. It wil
On 09/18/2017 03:29 PM, Vivien Didelot wrote:
> Hi Florian,
>
> Florian Fainelli writes:
>
>> @@ -1000,6 +1005,9 @@ static void b53_adjust_link(struct dsa_switch *ds, int
>> port,
>> b53_write8(dev, B53_CTRL_PAGE, po_reg, gmii_po);
>> }
>> }
>> +
>> +/
On 09/18/2017 03:27 PM, Vivien Didelot wrote:
> Hi Florian,
>
> Florian Fainelli writes:
>
>> @@ -649,7 +595,7 @@ static void bcm_sf2_sw_adjust_link(struct dsa_switch
>> *ds, int port,
>> core_writel(priv, reg, offset);
>>
>> if (!phydev->is_pseudo_fixed_link)
>> -p->eee
Hi Florian,
Florian Fainelli writes:
> @@ -1000,6 +1005,9 @@ static void b53_adjust_link(struct dsa_switch *ds, int
> port,
> b53_write8(dev, B53_CTRL_PAGE, po_reg, gmii_po);
> }
> }
> +
> + /* Re-negotiate EEE if it was enabled already */
> + p
Hi Florian,
Florian Fainelli writes:
> @@ -649,7 +595,7 @@ static void bcm_sf2_sw_adjust_link(struct dsa_switch *ds,
> int port,
> core_writel(priv, reg, offset);
>
> if (!phydev->is_pseudo_fixed_link)
> - p->eee_enabled = bcm_sf2_eee_init(ds, port, phydev);
> +
Florian Fainelli writes:
> The code to enable Broadcom tags/headers is largely switch independent,
> and in preparation for enabling it for multiple devices with b53, move
> the code we have in bcm_sf2.c to b53_common.c
>
> Signed-off-by: Florian Fainelli
Reviewed-by: Vivien Didelot
Hi Florian,
Florian Fainelli writes:
> Instead of repeating the same pattern: acquire mutex, read/write, release
> mutex, define a macro: b53_build_op() which takes the type (read|write), I/O
> size, and value (scalar or pointer). This helps with fixing bugs that could
> exit (e.g: missing barri
On 09/18/2017 11:32 AM, jo...@toxicpanda.com wrote:
> From: Josef Bacik
>
> These self tests are just self contained binaries, they are not run by
> any of the scripts in the directory. This means they need to be marked
> with TEST_GEN_PROGS to actually be run, not TEST_GEN_FILES.
>
> Signed-of
On Mon, Sep 18, 2017 at 2:55 PM, hiren panchasara
wrote:
> On 09/18/17 at 02:46P, Yuchung Cheng wrote:
>> On Mon, Sep 18, 2017 at 2:29 PM, hiren panchasara
>> wrote:
>> > On 09/18/17 at 02:18P, Eric Dumazet wrote:
>> >> On Mon, 2017-09-18 at 13:14 -0700, hiren panchasara wrote:
>> >> > Hi all, I
Florian Fainelli writes:
> There is no need to configure the enabled ports once in bcm_sf2_sw_setup() and
> then a second time around when dsa_switch_ops::port_enable is called, just do
> it when port_enable is called which is better in terms of power consumption
> and
> correctness.
>
> Signed-
From: Valentin Longchamp
Date: Fri, 15 Sep 2017 07:58:47 +0200
> uf_info.regs is resource_size_t i.e. phys_addr_t that can be either u32
> or u64 according to CONFIG_PHYS_ADDR_T_64BIT.
>
> The printk format is thus adaptet to u64 and the regs value cast to u64
> to take both u32 and u64 into acc
Florian Fainelli writes:
> There is no need to configure the enabled ports once in b53_setup() and then a
> second time around when dsa_switch_ops::port_enable is called, just do it when
> port_enable is called which is better in terms of power consumption and
> correctness.
>
> Signed-off-by: Fl
On 09/18/17 at 02:46P, Yuchung Cheng wrote:
> On Mon, Sep 18, 2017 at 2:29 PM, hiren panchasara
> wrote:
> > On 09/18/17 at 02:18P, Eric Dumazet wrote:
> >> On Mon, 2017-09-18 at 13:14 -0700, hiren panchasara wrote:
> >> > Hi all, I am trying to disable rack to see 3dupacks in action during
> >> >
1 - 100 of 282 matches
Mail list logo
