From: Mahesh Bandewar
When we create IPvlan slave; we use ether_setup() and that
sets up default MTU to 1500 while the master device may have
lower / different MTU. Any subsequent changes to the masters'
MTU are reflected into the slaves' MTU setting. However if those
don't happen (most likely sc
I have also noticed that for gso, all gso segs will have exactly same
outer udp checksum, this is also because inner checksum cancellation.
Can we also optimize that outer udp checksum should be only calculated
once for all gso segs ?
On 01/09/2016 03:45 AM, Edward Cree wrote:
The arithmetic
From: tndave
Date: Wed, 27 Jan 2016 17:50:14 -0800
> Hi,
>
> i40e driver has 'struct i40e_dma_mem' defined with 'packed' directive
> causing kernel unaligned errors on sparc (when
> 40e_allocate_dma_mem_d()
> is being called)
>
> log_unaligned: 1031 callbacks suppressed
> Kernel unaligned acces
Linus Torvalds writes:
> On Wed, Jan 27, 2016 at 2:15 PM, Michael S. Tsirkin wrote:
>>>
>>> Anyway, pulled. Just curious about how that thing happened.
>>
>> That's because apparently diffstat obeys orderfile rules:
>
> Ugh. I guess that makes sense, but it's still very annoying for
> something
On Thu, Jan 28, 2016 at 01:02:15AM -0500, Jarod Wilson wrote:
> On Wed, Jan 27, 2016 at 01:09:47PM -0800, Eric Dumazet wrote:
> > On Wed, 2016-01-27 at 15:21 -0500, Jarod Wilson wrote:
> >
> > > diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> > > index 289c231..7973ab5 100644
On Thu, Jan 28, 2016 at 01:02:15AM -0500, Jarod Wilson wrote:
> On Wed, Jan 27, 2016 at 01:09:47PM -0800, Eric Dumazet wrote:
> > On Wed, 2016-01-27 at 15:21 -0500, Jarod Wilson wrote:
> >
> > > diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> > > index 289c231..7973ab5 100644
On Wed, Jan 27, 2016 at 01:09:47PM -0800, Eric Dumazet wrote:
> On Wed, 2016-01-27 at 15:21 -0500, Jarod Wilson wrote:
>
> > diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> > index 289c231..7973ab5 100644
> > --- a/include/linux/netdevice.h
> > +++ b/include/linux/netdevice.h
Hmm. So my daughter has a little Gigabyte Brix that has rtl8821ae
wireless in it. Yeah, nasty, I know, but it has actually worked
reasonably well.
.. except now I upgraded the nearest access point, and now wireless on
that machine no longer works.
Or rather, it actually *does* work in the sense t
Dear Sir/Madam,
Clean tested working pulls CPUs and QTYs in stock.
115 X X5650
65 X E5410
75 X X5660
145 X E5530
100 X E5645
40 X X5680
75 X X5690
Brand new sealed IP phones and QTYs in stock.
55 x CP-7937G
77 x CP-7942G
54 x CP-7945G
75 x CP-7962G
..
45 x Avaya 9630
6
There are cases where qdisc_dequeue_peeked can return NULL, and the result
is dereferenced later on in the function.
Similarly to the other qdisc dequeue functions, check whether the skb
pointer is NULL and if it is, goto out.
Signed-off-by: Bernie Harris
---
net/sched/sch_drr.c | 2 ++
1 file
Thank you. I will add unlikely() and re-submit the patch.
From: Cong Wang
Sent: 27 January 2016 07:03
To: Bernie Harris
Cc: Linux Kernel Network Developers; David Miller; Jamal Hadi Salim;
sta...@vger.kernel.org
Subject: Re: [PATCH] net_sched: drr: check
On Wed, Jan 27, 2016 at 12:47 PM, Jesper Dangaard Brouer
wrote:
> On Mon, 25 Jan 2016 23:10:16 +0100
> Jesper Dangaard Brouer wrote:
>
>> On Mon, 25 Jan 2016 09:50:16 -0800 John Fastabend
>> wrote:
>>
>> > On 16-01-25 09:09 AM, Tom Herbert wrote:
>> > > On Mon, Jan 25, 2016 at 5:15 AM, Jesper D
Convert the driver to use ns_to_timespec64() and timespec64_to_ns()
instead of open coding the same logic.
Signed-off-by: Kefeng Wang
---
drivers/ptp/ptp_ixp46x.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/ptp/ptp_ixp46x.c b/drivers/ptp/ptp_ixp46x.c
index
Hi,
i40e driver has 'struct i40e_dma_mem' defined with 'packed' directive
causing kernel unaligned errors on sparc (when 40e_allocate_dma_mem_d()
is being called)
log_unaligned: 1031 callbacks suppressed
Kernel unaligned access at TPC[448ae8] dma_4v_alloc_coherent+0x188/0x2e0
Kernel unaligned ac
PHY devices may only list compatibility with clause 22, 45, and if
they need to be more specific, their PHY identifier values. No other
compatible strings are allowed. Make this clear in the documentation,
and remove examples where make/model compatible strings are listed.
Signed-off-by: Andrew L
White list PHY compatible values which indicate PHYs. Issue a warning
when one is encountered.
Update the documentation to make it clear what is expected in the
compatible string.
v2:
Fix Grammar, reword changelog, add Tested-by and Acked-by.
Andrew Lunn (2):
of: of_mdio: Add a whitelist of P
Some phy nodes list a compatible value indicating the PHY make/model.
This is never used to match the device to the driver. However it does
confuse the code to separate a PHY from a generic MDIO device like a
switch. Generic MDIO devices must have a compatible value, PHYs can
list clause 22 or 45,
On Wed, Jan 27, 2016 at 4:58 PM, Joe Perches wrote:
> On Wed, 2016-01-27 at 16:45 -0800, Kees Cook wrote:
>> Some callers of strtobool were passing a pointer to unterminated strings.
>> This fixes the issue and consolidates some logic in cifs.
>
> This may be incomplete as it duplicates the behavi
On Wed, 2016-01-27 at 16:45 -0800, Kees Cook wrote:
> Some callers of strtobool were passing a pointer to unterminated strings.
> This fixes the issue and consolidates some logic in cifs.
This may be incomplete as it duplicates the behavior for
the old number of characters, but this is not a solut
Some callers of strtobool were passing a pointer to unterminated strings.
This fixes the issue and consolidates some logic in cifs.
Signed-off-by: Kees Cook
Cc: Amitkumar Karwar
Cc: Nishant Sarmukadam
Cc: Kalle Valo
Cc: Steve French
Cc: linux-c...@vger.kernel.org
---
This is preparation for a
On Wed, Jan 27, 2016 at 10:56:08PM +0200, Julian Anastasov wrote:
>
> Hello,
>
> On Wed, 27 Jan 2016, Arnd Bergmann wrote:
>
> > ip_vs_fill_iph_skb_off() may not find an IP header, and gcc has
> > determined that ip_vs_sip_fill_param() then incorrectly accesses
> > the protocol fields:
> >
On Wed, Jan 27, 2016 at 10:01:42PM +0200, Julian Anastasov wrote:
>
> Hello,
>
> On Wed, 27 Jan 2016, Arnd Bergmann wrote:
>
> > The proc_create() and remove_proc_entry() functions do not reference
> > their arguments when CONFIG_PROC_FS is disabled, so we get a couple
> > of warnings abou
Throttle TX path only at slower than SuperSpeed USB.
SuperSpeed USB has enough bandwidth to maintain GigE.
Signed-off-by: Woojung Huh
---
drivers/net/usb/lan78xx.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c
index 6
Update lan78xx to use patch of commit 4f2aaf7dd95b
("Merge branch 'fix-phy-ignore-interrupts'").
Signed-off-by: Woojung Huh
---
drivers/net/usb/lan78xx.c | 30 ++
1 file changed, 14 insertions(+), 16 deletions(-)
diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/u
Depends on chip, some EEPROM pins are muxed with LED function.
Disable & restore LED function to access EEPROM.
Signed-off-by: Woojung Huh
---
drivers/net/usb/lan78xx.c | 98 ++-
1 file changed, 71 insertions(+), 27 deletions(-)
diff --git a/drivers/n
Woojung Huh (3):
lan78xx: change to use updated phy-ignore-interrupts
lan78xx: Add to handle mux control per chip id
lan78xx: throttle TX path at slower than SuperSpeed USB
drivers/net/usb/lan78xx.c | 132 ++
1 file changed, 88 insertions(+), 44 d
On Wed, Jan 27, 2016 at 2:15 PM, Michael S. Tsirkin wrote:
>>
>> Anyway, pulled. Just curious about how that thing happened.
>
> That's because apparently diffstat obeys orderfile rules:
Ugh. I guess that makes sense, but it's still very annoying for
something like a pull request, where now diffe
On Wed, Jan 27, 2016 at 11:57:52AM -0800, Linus Torvalds wrote:
> On Wed, Jan 27, 2016 at 11:12 AM, Michael S. Tsirkin wrote:
> >
> > arch/sh/include/asm/barrier.h| 1 -
> > tools/virtio/asm/barrier.h | 22 +-
> > tools/virtio/linux/compiler.h| 9 +
> >
From: Parthasarathy Bhuvaragan
Date: Wed, 27 Jan 2016 11:35:59 +0100
> In 'commit 7fe8097cef5f ("tipc: fix nullpointer bug when subscribing
> to events")', we terminate the connection if the subscription
> creation fails.
> In the same commit, the subscription creation result was based on
> the v
From: Kefeng Wang
Date: Wed, 27 Jan 2016 17:34:37 +0800
> Convert the driver to use ns_to_timespec64() to keep consistency
> with timespec64_to_ns() instead of open coding the same logic.
>
> Signed-off-by: Kefeng Wang
Applied, thank you.
On Wed, Jan 27, 2016 at 09:47:50PM +0100, Jesper Dangaard Brouer wrote:
> Sum: 18.75 % => calc: 30.0 ns (sum: 30.0 ns) => Total: 159.9 ns
>
> To get around the cache-miss in eth_type_trans(), I created a
> "icache-loop" in mlx5e_poll_rx_cq() and pull all RX-ring packets "out",
> before calling et
Signed-off-by: Jörg Thalheim
---
include/net/tcp.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/net/tcp.h b/include/net/tcp.h
index 051dc5c2..74d8eb4 100644
--- a/include/net/tcp.h
+++ b/include/net/tcp.h
@@ -216,7 +216,7 @@ void tcp_time_wait(struct sock *sk, int s
From: Eric Dumazet
Date: Tue, 26 Jan 2016 16:59:42 -0800
> From: Eric Dumazet
>
> We should not assume a valid protocol header is present,
> as this is not the case for IPv4 fragments.
>
> Lets avoid extra cache line misses and potential bugs
> if we actually find a socket and incorrectly uses
On Wed, 2016-01-27 at 15:21 -0500, Jarod Wilson wrote:
> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> index 289c231..7973ab5 100644
> --- a/include/linux/netdevice.h
> +++ b/include/linux/netdevice.h
> @@ -180,6 +180,7 @@ struct net_device_stats {
> unsigned long tx
On Wed, 2016-01-27 at 11:50 -0800, Cong Wang wrote:
> Hmm? I think nfc_llcp_send_ui_frame() needs to do some fragmention
> with this temporary memory, or you are saying msg_iter has some
> API available to seek the pointer? Even if so, it doesn't look like
> suitable for -stable.
>
memcpy_from_m
Hello,
On Wed, 27 Jan 2016, Arnd Bergmann wrote:
> ip_vs_fill_iph_skb_off() may not find an IP header, and gcc has
> determined that ip_vs_sip_fill_param() then incorrectly accesses
> the protocol fields:
>
> net/netfilter/ipvs/ip_vs_pe_sip.c: In function 'ip_vs_sip_fill_param':
> net/n
On Mon, 25 Jan 2016 23:10:16 +0100
Jesper Dangaard Brouer wrote:
> On Mon, 25 Jan 2016 09:50:16 -0800 John Fastabend
> wrote:
>
> > On 16-01-25 09:09 AM, Tom Herbert wrote:
> > > On Mon, Jan 25, 2016 at 5:15 AM, Jesper Dangaard Brouer
> > > wrote:
> > >>
> [...]
> > >>
> > >> There ar
4.2.8-ckt3 -stable review patch. If anyone has any objections, please let me
know.
---8<
From: Vijay Pandurangan
[ Upstream commit ce8c839b74e3017996fad4e1b7ba2e2625ede82f ]
Packets that arrive from real hardware devices have ip_sum
The network core tries to keep track of dropped packets, but some packets
you wouldn't really call dropped, so much as intentionally ignored, under
certain circumstances. One such case is that of bonding and team device
slaves that are currently inactive. Their respective rx_handler functions
retu
This adds an rx_unhandled stat counter, along with a sysfs statistics
node, and copies the counter out via netlink as well.
CC: "David S. Miller"
CC: Eric Dumazet
CC: Jiri Pirko
CC: Daniel Borkmann
CC: Tom Herbert
CC: Jay Vosburgh
CC: Veaceslav Falico
CC: Andy Gospodarek
CC: netdev@vger.ke
CC: Jiri Pirko
CC: netdev@vger.kernel.org
Signed-off-by: Jarod Wilson
---
drivers/net/team/team.c | 10 +++---
include/linux/if_team.h | 1 +
2 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/net/team/team.c b/drivers/net/team/team.c
index 718ceea..4460a8c 100644
--- a/
The rx_unhandled counters get output here in a column after drop with
a heading name of unh. I'm not sure if perhaps /proc/net/dev should be
considered a stable interface that shouldn't be mucked with. If so,
this can certainly be dropped, without impacting the core functionality.
CC: netdev@vger.
Sample output with this set applied for an active-backup bond:
$ cat /proc/net/dev
Inter-| Receive |
Transmit
face |bytespackets errs drop unh fifo frame compressed multicast|bytes
packets errs drop fifo colls carrier compressed
This is a note to let you know that I have just added a patch titled
=?UTF-8?q?veth:=20don=E2=80=99t=20modify=20ip=5Fsummed;=20doing?=
=?UTF-8?q?=20so=20treats=20packets=20with=20bad=20checksums=20as=20good.?=
to the linux-4.2.y-queue branch of the 4.2.y-ckt extended stable tree
which can b
From: Hans Westgaard Ry
Date: Wed, 27 Jan 2016 14:20:29 +0100
> Devices may have limits on the number of fragments in an skb they support.
Then this belongs as a global networking setting not a protocol
specfic one.
Hello,
On Wed, 27 Jan 2016, Arnd Bergmann wrote:
> The proc_create() and remove_proc_entry() functions do not reference
> their arguments when CONFIG_PROC_FS is disabled, so we get a couple
> of warnings about unused variables in IPVS:
>
> ipvs/ip_vs_app.c:608:14: warning: unused variab
On 12/01/16 11:10, Gregory CLEMENT wrote:
> This basic implementation allows to share code between driver using
> hardware buffer management. As the code is hardware agnostic, there is
> few helpers, most of the optimization brought by the an HW BM has to be
> done at driver level.
>
> Signed-off-
>-Original Message-
>From: Jay Vosburgh [mailto:jay.vosbu...@canonical.com]
>Sent: Monday, January 25, 2016 10:01 PM
>To: zhuyj
>Cc: mkube...@suse.cz; vfal...@gmail.com; go...@cumulusnetworks.com;
>netdev@vger.kernel.org; Shteinbock, Boris (Wind River); Tantilov, Emil S
>Subject: Re: [PATCH
On Wed, Jan 27, 2016 at 11:12 AM, Michael S. Tsirkin wrote:
>
> arch/sh/include/asm/barrier.h| 1 -
> tools/virtio/asm/barrier.h | 22 +-
> tools/virtio/linux/compiler.h| 9 +
> tools/virtio/linux/kernel.h | 1 +
> tools/virtio/ringtest/ma
On Tue, Jan 26, 2016 at 6:23 PM, Eric Dumazet wrote:
> On Tue, 2016-01-26 at 15:12 -0800, Cong Wang wrote:
>> On Tue, Jan 26, 2016 at 2:55 PM, Julian Calaby
>> wrote:
>> > Hi Cong,
>> >
>> > On Wed, Jan 27, 2016 at 4:53 AM, Cong Wang
>> > wrote:
>> >
>> > A commit message would be nice. A brie
+1
'temp' seems indeed not to be intuitive enough as 'static' in this context.
Regards,
Rami Rosen
Intel Corporation
-Original Message-
From: netdev-ow...@vger.kernel.org [mailto:netdev-ow...@vger.kernel.org] On
Behalf Of Roopa Prabhu
Sent: Wednesday, January 27, 2016 7:10 PM
To: step..
On Wed, Jan 27, 2016 at 02:45:26PM +0100, Arnd Bergmann wrote:
> To ensure we get consistent error handling here, this changes the code
> to only set rlen if we actually read data correctly, which also takes
> care of the warning.
It may be a good idea to do the job better. Looking at the code:
On Wed, 2016-01-27 at 14:28 +0100, Corinna Vinschen wrote:
> Problem: When switching off VLAN offloading on an i350, the VLAN
> interface gets unusable. For testing, set up a VLAN on an i350
> and some remote machine, e.g.:
>
> $ ip link add link eth0 name eth0.42 type vlan id 42
> $ ip addr
The following changes since commit 92e963f50fc74041b5e9e744c330dca48e04f08d:
Linux 4.5-rc1 (2016-01-24 13:06:47 -0800)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git tags/for_linus
for you to fetch changes up to 481eaec37e91e2b33f17275901
From: Eric Dumazet
With some combinations of user provided flags in netlink command,
it is possible to call tcp_get_info() with a buffer that is not 8-bytes
aligned.
It does matter on some arches, so we need to use put_unaligned() to
store the u64 fields.
Current iproute2 package does not trigg
Em 27-01-2016 16:35, Eric Dumazet escreveu:
On Wed, 2016-01-27 at 15:06 -0200, Marcelo Ricardo Leitner wrote:
sctp GSO requires it and sctp can be compiled as a module, so export
this function.
Signed-off-by: Marcelo Ricardo Leitner
---
net/core/skbuff.c | 1 +
1 file changed, 1 insertion(+
On 27.01.2016 14:45, Paolo Abeni wrote:
diff --git a/net/ipv6/route.c b/net/ipv6/route.c
index 3c8834b..973cb73 100644
--- a/net/ipv6/route.c
+++ b/net/ipv6/route.c
@@ -1183,11 +1183,10 @@ static struct rt6_info *ip6_pol_route_output(struct net
*net, struct fib6_table
return ip6_pol_rout
On Wed, 2016-01-27 at 15:06 -0200, Marcelo Ricardo Leitner wrote:
> sctp GSO requires it and sctp can be compiled as a module, so export
> this function.
>
> Signed-off-by: Marcelo Ricardo Leitner
> ---
> net/core/skbuff.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/net/core/skbuff
This commit adds a callback which allows to adjust the maximum transmit
bitrate the card can output. This makes it possible to get a smooth
traffic instead of the default burst-y behaviour when trying to output
e.g. a video stream.
Much of the logic needed to get a correct bcnrc_val was taken from
On 27/01/16 05:54, Antonio Ospite wrote:
> Fix the 0x0x prefix in an integer constant.
>
> In this case, while at it, also fix a typo (s/unitcast/unicast/).
>
> Signed-off-by: Antonio Ospite
> Cc: Florian Fainelli
Acked-by: Florian Fainelli
--
Florian
On 27.01.2016 16:15, Eric Dumazet wrote:
On Wed, 2016-01-27 at 14:20 +0100, Hans Westgaard Ry wrote:
Devices may have limits on the number of fragments in an skb they support.
Current codebase uses a constant as maximum for number of fragments one
skb can hold and use.
When enabling scatter/gath
> Only case I can see the need for a make/model string is if there's a
> need to add model-specific properties since you'd need a compatible
> then (or make those properties shared between all phy bindings).
Documentation/devicetree/bindings/net/micrel-ksz90x1.txt
It adds optional properties to t
On Wed, Jan 27, 2016 at 9:11 AM, Andrew Lunn wrote:
> On Wed, Jan 27, 2016 at 08:41:56AM -0800, Olof Johansson wrote:
>> On Tue, Jan 26, 2016 at 4:11 PM, Andrew Lunn wrote:
>> > PHY devices may only list clause 22, 45, and their PHY identifier
>> > values as compatible values. No other compatible
On Wed, Jan 27, 2016 at 08:41:56AM -0800, Olof Johansson wrote:
> On Tue, Jan 26, 2016 at 4:11 PM, Andrew Lunn wrote:
> > PHY devices may only list clause 22, 45, and their PHY identifier
> > values as compatible values. No other compatible strings are allowed.
> > Make this clear in the documenta
From: Roopa Prabhu
There is no intuitive option to add static fdb entries today.
'temp' seems to have a side effect of adding
'static' fdb entries. But the name and intent
of 'temp' does not say anything about it being static.
example:
bridge fdb add operates as follows:
$bridge fdb add 00:01:0
This patch just adds initial offloading bits, just to ease reviewing of
the next one. It will be merged with GSO patch itself in the actual
submission.
Signed-off-by: Marcelo Ricardo Leitner
---
include/linux/skbuff.h | 2 ++
include/net/sctp/sctp.h | 4
net/sctp/Makefile | 3 ++-
This patchset is merely a RFC for the moment. There are some
controversial points that I'd like to discuss before actually proposing
the patches.
The points are more detailed in the 3rd patch.
Another approach that I can think of, is using something related to
what Dave Miller mentioned on the th
This patch enables SCTP to do GSO.
SCTP has this pecualiarty that its packets cannot be just segmented to
(P)MTU. Its chunks must be contained in IP segments, padding respected.
So we can't just generate a big skb, set gso_size to the fragmentation
point and deliver it to IP layer.
Instead, this
sctp GSO requires it and sctp can be compiled as a module, so export
this function.
Signed-off-by: Marcelo Ricardo Leitner
---
net/core/skbuff.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/net/core/skbuff.c b/net/core/skbuff.c
index
b2df375ec9c2173a8132b8efa1c3062f0510284b..704b69682085
From: Nikolay Aleksandrov
Currently when a macvlan is being initialized and the lower device is
netif_carrier_ok(), the macvlan device doesn't run through
rfc2863_policy() and is left with UNKNOWN operstate. Fix it by adding an
unconditional linkwatch event for the new macvlan device. Similar fix
On Tue, Jan 26, 2016 at 4:11 PM, Andrew Lunn wrote:
> PHY devices may only list clause 22, 45, and their PHY identifier
> values as compatible values. No other compatible strings are allowed.
> Make this clear in the documentation, and remove examples where
> make/model compatible strings are list
On 1/10/16, 2:28 AM, Thomas Graf wrote:
> On 01/09/16 at 10:39am, roopa wrote:
>> On 1/6/16, 5:33 AM, David Wragg wrote:
>>> Allow the MTU of vxlan devices without an underlying device to be set to
>>> larger values (up to a maximum based on IP packet limits and vxlan
>>> overhead).
>>>
>>> Previou
On January 26, 2016 5:57:55 PM PST, Andrew Lunn wrote:
>> Well, we do now, since that is one of the results used by
>> of_mdiobus_child_is_phy(),
>
>It uses it, but do not rely on it, since for backwards compatibility,
>we cannot assume it is there.
>
>You can never change an optional parameter to
[[PATCH 3/9] net: bgmac: clarify CONFIG_BCMA dependency] On 27/01/2016 (Wed
15:04) Arnd Bergmann wrote:
> The bgmac driver depends on BCMA_HOST_SOC, which is only used
> when CONFIG_BCMA is enabled. However, it is a bool option and can
> be set when CONFIG_BCMA=m, and then bgmac can be built-in,
Le 27/01/2016 16:51, Nicolas Ferre a écrit :
> Le 27/01/2016 15:04, Arnd Bergmann a écrit :
>> The macb_clk_init function returns three clock pointers, unless
>> the it fails to get the first ones. We correctly handle the
>> failure case by propagating the error from macb_probe, but
>> gcc does not
On Wed, 2016-01-27 at 22:40 +0800, Zhouyi Zhou wrote:
> From: Zhouyi Zhou
>
> I think hackers chould build a malicious h323 packet to overflow
> the pointer p which will panic during the memcpy(addr, p, len)
>
> For example, he may fabricate a very large taddr->ipAddress.ip;
>
> Signed-off-by:
Wed, Jan 27, 2016 at 04:51:14PM CET, and...@lunn.ch wrote:
>On Wed, Jan 27, 2016 at 03:46:57PM +0100, Jiri Pirko wrote:
>> Hi All.
>>
>> Just want to send a small note here about what we use for mlxsw driver
>> automated testing. It can be handy for other people using switchdev
>> infra, namely DS
Le 27/01/2016 15:04, Arnd Bergmann a écrit :
> The macb_clk_init function returns three clock pointers, unless
> the it fails to get the first ones. We correctly handle the
> failure case by propagating the error from macb_probe, but
> gcc does not realize this and incorrectly warns about a later
>
On Wed, Jan 27, 2016 at 03:46:57PM +0100, Jiri Pirko wrote:
> Hi All.
>
> Just want to send a small note here about what we use for mlxsw driver
> automated testing. It can be handy for other people using switchdev
> infra, namely DSA guys.
Hi Jiri
I have a few hacked together python scripts com
On Wednesday 27 January 2016 14:13:16 Måns Rullgård wrote:
> Arnd Bergmann writes:
>
> > The nb8800_poll() function initializes the 'next' variable in the
> > loop looking for new input data. We know this will be called at
> > least once because 'budget' is a guaranteed to be a positive number
>
On Wed, 27 Jan 2016, Arnd Bergmann wrote:
> This adds code to ensure that the BAR values are initialized
> even in the impossible case when a device gets probed that
> does not belong to any bus. This shuts up the warning.
>
> Signed-off-by: Arnd Bergmann
> ---
> drivers/net/fddi/defxx.c | 5 ++
On Wed, 2016-01-27 at 14:20 +0100, Hans Westgaard Ry wrote:
> Devices may have limits on the number of fragments in an skb they support.
> Current codebase uses a constant as maximum for number of fragments one
> skb can hold and use.
> When enabling scatter/gather and running traffic with many sma
From: Saeed Mahameed
Currently our netdevice ops is a static global variable which
is referenced by all mlx5e netdevice instances. This can be
problematic when different driver instances do not share same
HW capabilities (e.g SRIOV PF and VFs probed to the host).
Remove the global variable and a
From: Amir Vadai
We need to handle flow table entry destinations only if the action
associated with the rule is
forwarding (MLX5_FLOW_CONTEXT_ACTION_FWD_DEST).
Fixes: 26a8145390b3 ('net/mlx5_core: Introduce flow steering firmware commands')
Signed-off-by: Amir Vadai
Signed-off-by: Maor Gottlieb
Hi Dave,
Amir fixed a case where we could attempt to set destination for
a non-forwarding (e.g drop) steering rule. Saeed fixed the case
where VFs probed to the host shared the PF ndo_set_vf_yyy NDOs
which is wrong.
Or.
Amir Vadai (1):
net/mlx5_core: Set flow steering dest only for forward r
Hi Zhouyi,
[auto build test ERROR on nf-next/master]
[also build test ERROR on v4.5-rc1 next-20160127]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Zhouyi-Zhou/netfilter-h323-avoid
From: Zhouyi Zhou
I think hackers chould build a malicious h323 packet to overflow
the pointer p which will panic during the memcpy(addr, p, len)
For example, he may fabricate a very large taddr->ipAddress.ip;
Signed-off-by: Zhouyi Zhou
---
net/netfilter/nf_conntrack_h323_main.c |8 +
Hi All.
Just want to send a small note here about what we use for mlxsw driver
automated testing. It can be handy for other people using switchdev
infra, namely DSA guys.
The framework is called LNST:
http://lnst-project.org/
Switchdev recipes are located in LNST git repo here:
https://github.co
Until now all stations (notably in AP mode) would
share 4 AC queues exposed via netdev. This meant
that whenever a single station's AC became full
all other stations' AC were considered full and
stopped as well.
This was suboptimal and could lead to per station
traffic starvation and/or suboptimal
From: Ido Schimmel
When STP state is set to DISABLED the port is assumed to be inactive, but
currently we forward packets ingressing through it.
Instead, set the port's STP state in hardware to DISCARDING, which means
it doesn't forward packets or perform any learning, but it does trap
control p
From: Ido Schimmel
When removing a net device from a bridge we should flush the FDB entries
associated with this net device. Up until now, we relied upon bridge
code to do that for us, but it is possible for user to prevent hardware
from syncing with the software bridge (learning_sync=0), so we n
From: Ido Schimmel
When dumping the hardware FDB we should report entries pointing to VLAN
devices with VLAN 0, as packets coming into the bridge are untagged.
Likewise, pass FDB_{ADD,DEL} notifications with VLAN 0 for these
devices.
Fixes: 54a732018d8e ("mlxsw: spectrum: Adjust switchdev ops fo
From: Ido Schimmel
When dumping the FDB we can't compare the actual pointers of the ports
structs, as it's possible the struct represents a vPort instead of the
underlying physical port.
Solve this by comparing the local port number instead, as it's shared
between the physical ports and all the
From: Ido Schimmel
As explained in previous commit, we should always take care of flushing
the FDB in the driver and not rely on bridge code.
We need to distinguish between two cases with regards to LAG:
1) Port is leaving LAG while LAG is bridged (or VLAN devices on top of
it). In this case do
From: Ido Schimmel
LAG FDB entries pointing to VLAN devices should be reported to the
bridge with the matching VLAN device and not the underlying LAG device.
Fixes: aac78a440887 ("mlxsw: spectrum: Adjust FDB notifications for VLAN
devices")
Signed-off-by: Ido Schimmel
Signed-off-by: Jiri Pirko
From: Ido Schimmel
When port is put into LISTENING state it shouldn't populate the FDB, so
set the port's STP state in hardware to DISCARDING instead of LEARNING.
It will therefore keep listening to BPDU packets, but discard other
non-control packets and won't perform any learning.
Fixes: 56ade8
From: Ido Schimmel
It is possible for a user to remove a port from a LAG device, while the
LAG device or VLAN devices on top of it are bridged. In these cases,
bridge's teardown sequence is never issued, so we need to take care of
it ourselves.
When LAG's unlinking event is received by port netd
From: Ido Schimmel
LAG FDB records can only point to LAG devices or VLAN devices configured
on top of them. Therefore, when dumping the FDB we shouldn't associate
these records with the underlying physical ports.
Fixes: 8a1ab5d76639 ("mlxsw: spectrum: Implement FDB add/remove/dump for LAG")
Sign
From: Ido Schimmel
The rx_lane, tx_lane and module fields in the PMLP register don't have
an additional offset besides the base one (0x04), so set it to 0x00.
Fixes: 4ec14b7634b2 ("mlxsw: Add interface to access registers and process
events")
Signed-off-by: Ido Schimmel
Signed-off-by: Jiri Pir
From: Ido Schimmel
When we disable learning on bridge port we should still update the
software bridge's FDB when entry pointing to this bridge port is
aged-out. We can otherwise have an inconsistency between software and
hardware tables.
Fixes: 8a1ab5d76639 ("mlxsw: spectrum: Implement FDB add/r
1 - 100 of 132 matches
Mail list logo
