On 2016-11-15 at 20:19:46 +0100, Florian Fainelli wrote:
> Implement ethtool::nway_reset using phy_ethtool_nway_reset. We are
> already using dev->phydev all over the place so this comes for free.
>
> Signed-off-by: Florian Fainelli
Reviewed-by: Tobias Klauser
From: Eric Dumazet
It seems many drivers do not respect napi_hash_del() contract.
When napi_hash_del() is used before netif_napi_del(), an RCU grace
period is needed before freeing NAPI object.
Fixes: 91815639d880 ("virtio-net: rx busy polling support")
Signed-off-by: Eric Dumazet
Cc: Jason Wa
Hi David,
On Tue, Nov 08, 2016 at 10:26:25AM +0100, Michal Tesar wrote:
> On Mon, Nov 07, 2016 at 08:13:45PM -0500, David Miller wrote:
>
> > From: Michal Tesar
> > Date: Thu, 3 Nov 2016 10:38:34 +0100
> >
> > > 2. If the received Query is a General Query, the interface timer is
> > > used
On Tue, Nov 15, 2016 at 08:42:03PM -0800, John Fastabend wrote:
> On 16-11-14 03:06 PM, Michael S. Tsirkin wrote:
> > On Thu, Nov 10, 2016 at 08:44:32PM -0800, John Fastabend wrote:
> >> Signed-off-by: John Fastabend
> >
> > This will naturally reduce the cache line bounce
> > costs, but so will
On 16-11-14 03:06 PM, Michael S. Tsirkin wrote:
> On Thu, Nov 10, 2016 at 08:44:32PM -0800, John Fastabend wrote:
>> Signed-off-by: John Fastabend
>
> This will naturally reduce the cache line bounce
> costs, but so will a _many API for ptr-ring,
> doing lock-add many-unlock.
>
> the number of a
On 16-11-15 05:32 AM, Jesper Dangaard Brouer wrote:
>
> (looks like my message didn't reach the netdev list, due to me sending
> from the wrong email, forwarded message again):
>
> On Thu, 10 Nov 2016 20:44:08 -0800 John Fastabend
> wrote:
>
>> ---
>> include/linux/ptr_ring_ll.h | 136
>> ++
On 16-11-14 03:01 PM, Michael S. Tsirkin wrote:
> On Thu, Nov 10, 2016 at 08:44:08PM -0800, John Fastabend wrote:
>>
>> ---
>> include/linux/ptr_ring_ll.h | 136
>> +++
>> include/linux/skb_array.h | 25
>> 2 files changed, 161 insertions(+)
>
No, this patch will not change dev->name,
It's care about ifa->ifa_label.
> - if (colon)
> - memcpy(ifa->ifa_label, ifr.ifr_name, IFNAMSIZ);
> - else
> - memcpy(ifa->ifa_label, dev->name, IFNAMSIZ);
When
From: Tobias Regnery
Date: Tue, 15 Nov 2016 12:43:07 +0100
> This patchset lays the groundwork for multi queue support in the alx driver
> and enables multi queue support for the tx path by default. The hardware
> supports up to 4 tx queues.
Series applied, thanks.
From: "Allan W. Nielsen"
Date: Tue, 15 Nov 2016 11:19:59 +0100
> Old cover letters included below.
Please do not form your cover letter by quoting older and older
cover letters.
Always write a clean, single, cover letter with a changelog of
changed between each and every version leading up to t
From: Sabrina Dubroca
Date: Tue, 15 Nov 2016 11:16:35 +0100
> rtnl_xdp_size() only considers the size of the actual payload attribute,
> and misses the space taken by the attribute used for nesting (IFLA_XDP).
>
> Fixes: d1fdd9138682 ("rtnl: add option for setting link xdp prog")
> Signed-off-by
From: Sabrina Dubroca
Date: Tue, 15 Nov 2016 10:39:03 +0100
> The size reported by rtnl_vfinfo_size doesn't match the space used by
> rtnl_fill_vfinfo.
>
> rtnl_vfinfo_size currently doesn't account for the nest attributes
> used by statistics (added in commit 3b766cd83232), nor for struct
> ifl
Hi Florian,
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Florian-Fainelli/net-ethernet-Allow-Marvell-Freescale-to-COMPILE_TEST/20161116-024633
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.
From: Madalin Bucur
Date: Tue, 15 Nov 2016 10:41:00 +0200
> This patch series adds the Ethernet driver for the Freescale
> QorIQ Data Path Acceleration Architecture (DPAA).
Series applied, thanks.
Some lines use tabs while others use spaces. Use spaces everywhere.
Signed-off-by: Mike Frysinger
---
misc/ifstat.c | 24
misc/nstat.c | 22 +++---
2 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/misc/ifstat.c b/misc/ifstat.c
index d551
From: YUAN Linyu
Date: Wed, 16 Nov 2016 03:13:31 +
> So assign label to request name will do same thing as original code.
Nope. dev->name does not have the colon, it was trimmed from the
string for the device lookup. So the found device's dev->name does
not have the colon character, even i
From: YUAN Linyu
Date: Wed, 16 Nov 2016 03:13:31 +
> orginal code means when reqest name have colon, then label name will have
> colon.
And that is intentional.
From: Andrei Vagin
Date: Mon, 14 Nov 2016 18:15:14 -0800
> The repair mode is used to get and restore sequence numbers and
> data from queues. It used to checkpoint/restore connections.
>
> Currently the repair mode can be enabled for sockets in the established
> and closed states, but for other
From: Eric Dumazet
Date: Mon, 14 Nov 2016 16:28:42 -0800
> From: Eric Dumazet
>
> Rolf Neugebauer reported very long delays at netns dismantle.
>
> Eric W. Biederman was kind enough to look at this problem
> and noticed synchronize_net() occurring from netif_napi_del() that was
> added in linu
The SPEED_UNFORCED indicates the MAC & PHY should perform
auto-negotiation to determine a speed which works. If this is called
for, don't set the force bit. If it is set, the MAC actually does
10Gbps, why the internal PHYs don't support.
Signed-off-by: Andrew Lunn
---
drivers/net/dsa/mv88e6xxx/p
From: Raghu Vatsavayi
Date: Mon, 14 Nov 2016 15:54:38 -0800
> Following is the V6 patch series for adding VF support on
> CN23XX devices. This version addressed:
> 1) Your concern for ordering of local variable declarations
>from longest to shortest line.
> 2) Removed module parameters max_vf
Hi Florian,
[auto build test WARNING on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Florian-Fainelli/net-gianfar_ptp-Rename-FS-bit-to-FIPERST/20161116-095805
config: sh-allmodconfig (attached as .config)
compiler: sh4-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
From: Alex
Date: Mon, 14 Nov 2016 13:54:57 -0800
>
>
> On 11/14/2016 01:25 PM, Florian Fainelli wrote:
>> On 11/14/2016 01:18 PM, David Miller wrote:
>>> From: Alexandru Gagniuc
>>> Date: Sat, 12 Nov 2016 15:32:13 -0800
>>>
+ if (phydev->interface == PHY_INTERFACE_MODE_RGMII_ID)
+
From: Pablo Neira Ayuso
Date: Mon, 14 Nov 2016 23:40:30 +0100
> Honor udptable parameter that is passed to __udp*_lib_mcast_deliver(),
> otherwise udplite broadcast/multicast use the wrong table and it breaks.
>
> Fixes: 2dc41cff7545 ("udp: Use hash2 for long hash1 chains in
> __udp*_lib_mcast_
From: Tom Lendacky
Date: Mon, 14 Nov 2016 16:28:27 -0600
> This patch series addresses some minor issues found in the recently
> accepted patch series for the AMD XGBE driver.
>
> The following fixes are included in this driver update series:
>
> - Fix how a mask is applied to a Clause 37 regis
hi david,
> -Original Message-
> From: netdev-ow...@vger.kernel.org [mailto:netdev-ow...@vger.kernel.org]
> On Behalf Of David Miller
> Sent: Wednesday, November 16, 2016 11:01 AM
> To: cug...@163.com
> Cc: netdev@vger.kernel.org; YUAN Linyu
> Subject: Re: [PATCH] net: ioctl SIOCSIFADDR mi
From: Michael Weiser
Date: Mon, 14 Nov 2016 18:58:04 +0100
> the following patches are what remains to be fixed in order to allow running a
> big-endian kernel on the Cubieboard2.
Series applied to net-next, thanks.
From: Florian Westphal
Date: Mon, 14 Nov 2016 16:42:01 +0100
> draft-ietf-tcpm-dctcp-02 says:
>
> ... when the sender receives an indication of congestion
> (ECE), the sender SHOULD update cwnd as follows:
>
> cwnd = cwnd * (1 - DCTCP.Alpha / 2)
>
> So, lets do this and reduce cwnd mo
From: yuan linyu
Date: Tue, 15 Nov 2016 20:44:59 +0800
> @@ -1063,10 +1063,7 @@ int devinet_ioctl(struct net *net, unsigned int cmd,
> void __user *arg)
> if (!ifa)
> break;
> INIT_HLIST_NODE(&ifa->hash);
> -
From: Florian Fainelli
Date: Tue, 15 Nov 2016 18:21:09 -0800
> Introduced a typo making the driver no longer build, *sigh*.
>
> Fixes: 42469bf5d9bb ("net: bcm63xx_enet: Utilize phy_ethtool_nway_reset")
> Signed-off-by: Florian Fainelli
Applied.
Introduced a typo making the driver no longer build, *sigh*.
Fixes: 42469bf5d9bb ("net: bcm63xx_enet: Utilize phy_ethtool_nway_reset")
Signed-off-by: Florian Fainelli
---
drivers/net/ethernet/broadcom/bcm63xx_enet.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/
Vicente Jiménez wrote:
> I agree that both patches try to solve the same problem in a very similar way.
> Florian Westphal's patch do two more things:
> 1- add warning with pr_warn_ratelimited. I like this idea. I also
> though about adding some message but I have no kernel experience and I
> pref
This patch adds the register dump format for FUJITSU Extended
Network device like the following:
# ethtool -d es0
0x: OWNER_EPID(Owner EPID) 0x0001
0x0004: MAX_EP(Maximum EP) 0x0008
0x0010: DCTL (Device Control)
On Sat, Nov 12, 2016 at 09:02:24PM -0500, David Miller wrote:
> From: Kast Bernd
> Date: Fri, 4 Nov 2016 00:33:06 +0100
>
> > This patch adds a module parameter in order to activate ASPM. By that
> > the CPU can enter deep sleep modes (PC6) and power consumption can be
> > reduced (for example fr
On error, remask the interrupts, release all maps, and remove the
domain. This cannot be done using the mv88e6xxx_g1_irq_free() because
some of these actions are not idempotent.
Signed-off-by: Andrew Lunn
---
drivers/net/dsa/mv88e6xxx/chip.c | 31 ---
1 file changed,
It is not possible to use devm_request_threaded_irq() because we have
two stacked interrupt controllers in one device. The lower interrupt
controller cannot be removed until the upper is fully removed. This
happens too late with the devm API, resulting in error messages about
removing a domain whil
Freeing interrupts requires switch register access to mask the
interrupts. Hence we must hold the register mutex.
Signed-off-by: Andrew Lunn
---
drivers/net/dsa/mv88e6xxx/chip.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net
Fix the g1 interrupt free code such that is masks any further
interrupts, and then releases the interrupt.
Signed-off-by: Andrew Lunn
---
drivers/net/dsa/mv88e6xxx/chip.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.
From: Giuseppe Cavallaro
Date: Mon, 14 Nov 2016 09:27:27 +0100
> This subset of patches aim to fix the PTP support
> for the stmmac and especially for 4.x chip series.
> While setting PTP on an ST box with 4.00a Ethernet
> core, the kernel panics due to a broken settings
> of the descriptors. The
Simple typos, s/g2/g1/
Signed-off-by: Andrew Lunn
---
drivers/net/dsa/mv88e6xxx/chip.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.c
index d6d9d66b81ce..6aa81d2d8f63 100644
--- a/drivers/net/dsa/mv88e6x
The interrupt code was never tested with a board who's probing
resulted in an -EPROBE_DEFFERED. So the clean up paths never got
tested. I now do have -EPROBE_DEFFERED, and things break badly during
cleanup. These are the fixes.
This is fixing code in net-next.
Andrew Lunn (6):
net: dsa: mv88e6x
Trying to remove an IRQ domain that was not created results in an
Opps. Add the necessary checks that the irqs were created before
freeing them.
Signed-off-by: Andrew Lunn
---
drivers/net/dsa/mv88e6xxx/chip.c | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/drive
On Wed, Nov 16, 2016 at 11:36:47AM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the net-next tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> kernel/bpf/bpf_lru_list.c: In function
> '__bpf_lru_list_rotate_inactive.isra.3':
> kernel/bpf/bpf_lru_list.c
From: Hangbin Liu
Date: Mon, 14 Nov 2016 16:16:28 +0800
> In commit 24cf3af3fed5 ("igmp: call ip_mc_clear_src..."), we forgot to remove
> igmpv3_clear_delrec() in ip_mc_down(), which also called ip_mc_clear_src().
> This make us clear all IGMPv3 source filter info after NETDEV_DOWN.
> Move igmpv3
On Thu, Nov 10, 2016 at 9:10 PM, Nicolas Pitre wrote:
> Some embedded systems have no use for them. This removes about
> 25KB from the kernel binary size when configured out.
>
> Corresponding syscalls are routed to a stub logging the attempt to
> use those syscalls which should be enough of a cl
Type of debugfs_real_fops() is longer than parameters and
the name, so there is no way to break the declaration nicely.
We have to go over 80 characters.
Signed-off-by: Jakub Kicinski
Acked-by: Greg Kroah-Hartman
---
Sorry, I forgot to include the target tree in the tag, resending.
This is fixup
On 11/16/16 1:00 AM, Hannes Frederic Sowa wrote:
On 15.11.2016 08:57, Erik Nordmark wrote:
Implemented RFC7527 Enhanced DAD.
IPv6 duplicate address detection can fail if there is some temporary
loopback of Ethernet frames. RFC7527 solves this by including a random
nonce in the NS messages used f
FS is a global symbol used by the x86 32-bit architecture, fixes builds
re-definitions:
>> drivers/net/ethernet/freescale/gianfar_ptp.c:75:0: warning: "FS"
>> redefined
#define FS(1<<28) /* FIPER start indication */
In file included from arch/x86/include/uapi/asm/ptrace
All Marvell Ethernet drivers actually build fine with COMPILE_TEST with
a few warnings.
Signed-off-by: Florian Fainelli
---
drivers/net/ethernet/marvell/Kconfig | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/marvell/Kconfig
b/drivers/net/etherne
There are only a handful of Freescale Ethernet drivers that don't
actually build with COMPILE_TEST:
* FEC, for which we would need to define a default register layout if no
supported architecture is defined
* UCC_GETH which depends on PowerPC cpm.h header (which could be moved
to a generic lo
FS is a global symbol used by the x86 32-bit architecture, fixes builds
re-definitions:
>> drivers/net/ethernet/freescale/gianfar_ptp.c:75:0: warning: "FS"
>> redefined
#define FS(1<<28) /* FIPER start indication */
In file included from arch/x86/include/uapi/asm/ptrace
There are only a handful of Freescale Ethernet drivers that don't
actually build with COMPILE_TEST:
* FEC, for which we would need to define a default register layout if no
supported architecture is defined
* UCC_GETH which depends on PowerPC cpm.h header (which could be moved
to a generic lo
Hi David,
This patch series allows most of the Freescale Ethernet drivers to be built
with COMPILE_TEST and all Marvell drivers to build with COMPILE_TEST.
This is helpful to increase build coverage without requiring hacking into
Kconfig anymore.
Changes in v3:
- reorder patches to avoid introd
On 11/15/2016 04:35 PM, Florian Fainelli wrote:
> Hi David,
>
> This patch series allows most of the Freescale Ethernet drivers to be built
> with COMPILE_TEST and all Marvell drivers to build with COMPILE_TEST.
>
> This is helpful to increase build coverage without requiring hacking into
> Kconf
Hi all,
After merging the net-next tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
kernel/bpf/bpf_lru_list.c: In function '__bpf_lru_list_rotate_inactive.isra.3':
kernel/bpf/bpf_lru_list.c:201:28: warning: 'next' may be used uninitialized in
this function [-Wmaybe-uni
All Marvell Ethernet drivers actually build fine with COMPILE_TEST with
a few warnings.
Signed-off-by: Florian Fainelli
---
drivers/net/ethernet/marvell/Kconfig | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/marvell/Kconfig
b/drivers/net/etherne
Hi David,
This patch series allows most of the Freescale Ethernet drivers to be built
with COMPILE_TEST and all Marvell drivers to build with COMPILE_TEST.
This is helpful to increase build coverage without requiring hacking into
Kconfig anymore.
Changes in v2:
- rename register define clash wh
FS is a global symbol used by the x86 32-bit architecture, fixes builds
re-definitions:
>> drivers/net/ethernet/freescale/gianfar_ptp.c:75:0: warning: "FS"
>> redefined
#define FS(1<<28) /* FIPER start indication */
In file included from arch/x86/include/uapi/asm/ptrace
There are only a handful of Freescale Ethernet drivers that don't
actually build with COMPILE_TEST:
* FEC, for which we would need to define a default register layout if no
supported architecture is defined
* UCC_GETH which depends on PowerPC cpm.h header (which could be moved
to a generic lo
On 11/15/2016 04:24 PM, kbuild test robot wrote:
> Hi Florian,
>
> [auto build test WARNING on net-next/master]
>
> url:
> https://github.com/0day-ci/linux/commits/Florian-Fainelli/net-ethernet-Allow-Marvell-Freescale-to-COMPILE_TEST/20161116-024633
> config: i386-allmodconfig (attached as .c
Hi Florian,
[auto build test WARNING on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Florian-Fainelli/net-ethernet-Allow-Marvell-Freescale-to-COMPILE_TEST/20161116-024633
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduc
Hi Andrew, Vivien,
On 11/15/2016 03:58 PM, Florian Fainelli wrote:
> We currently have a fundamental problem in how we treat the CPU port and
> its VLAN membership. As soon as a second VLAN is configured to be
> untagged, the CPU automatically becomes untagged for that VLAN as well,
> and yet, we
Helpers like bpf_prog_add(), bpf_prog_inc(), bpf_map_inc() can fail
with an error, so make sure the caller properly checks their return
value and not just ignores it (which could worst-case lead to use
after free).
Signed-off-by: Daniel Borkmann
Acked-by: Alexei Starovoitov
---
include/linux/bp
In mlx5e_create_rq(), when creating a new queue, we call bpf_prog_add() but
without checking the return value. bpf_prog_add() can fail since 92117d8443bc
("bpf: fix refcnt overflow"), so we really must check it. Take the reference
right when we assign it to the rq from priv->xdp_prog, and just drop
Various mlx5 bugs on eBPF refcount handling found during review.
Last patch in series adds a __must_check to BPF helpers to make
sure we won't run into it again w/o compiler complaining first.
v1 -> v2:
- After discussion with Alexei, we agreed upon rebasing the
patches against net-next.
- S
mlx5e_xdp_set() is currently the only place where we drop reference on the
prog sitting in priv->xdp_prog when it's exchanged by a new one. We also
need to make sure that we eventually release that reference, for example,
in case the netdev is dismantled.
Fixes: 86994156c736 ("net/mlx5e: XDP fast
There are multiple issues in mlx5e_xdp_set():
1) The batched bpf_prog_add() is currently not checked for errors! When
doing so, it should be done at an earlier point in time to makes sure
that we cannot fail anymore at the time we want to set the program for
each channel. This only means
We currently have a fundamental problem in how we treat the CPU port and
its VLAN membership. As soon as a second VLAN is configured to be
untagged, the CPU automatically becomes untagged for that VLAN as well,
and yet, we don't gracefully make sure that the CPU becomes tagged in
the other VLANs it
On 11/15/2016 01:57 PM, woojung@microchip.com wrote:
> From: Woojung Huh
>
> Relocate mdix code to phy driver to be called at config_init().
>
> Signed-off-by: Woojung Huh
> ---
> drivers/net/phy/microchip.c | 43 +-
> drivers/net/usb/lan78xx.c | 73
> ++-
Got it, thanks, I'll follow your discussion thread.
-Original Message-
From: Thadeu Lima de Souza Cascardo [mailto:casca...@cascardo.eti.br]
Sent: Wednesday, November 16, 2016 3:01 AM
To: Yang, Yi Y ; Jiri Benc ;
netdev@vger.kernel.org
Cc: d...@openvswitch.org; Simon Horman ;
egar...@re
On 16.11.2016 00:03, Andrew Lunn wrote:
>> > + val = MII_BMCR << 16 | SLIC_PCR_AUTONEG |
>> > + SLIC_PCR_AUTONEG_RST;
>> > + slic_write(sdev, SLIC_REG_WPHY, val);
>
>> Thats essentially what I meant by setting a fl
> > + val = MII_BMCR << 16 | SLIC_PCR_AUTONEG |
> > +SLIC_PCR_AUTONEG_RST;
> > + slic_write(sdev, SLIC_REG_WPHY, val);
> Thats essentially what I meant by setting a flag in the irq handler. The mdio
> function
On 15.11.2016 23:39, Florian Fainelli wrote:
> On 11/15/2016 02:34 PM, Lino Sanfilippo wrote:
>> On 15.11.2016 22:59, Andrew Lunn wrote:
The link state is retrieved by a command to the application processor that
is running
on the network card. Also the register to set the phy confi
On Tue, Nov 15, 2016 at 2:21 PM, Andrei Vagin wrote:
> On Tue, Nov 15, 2016 at 1:07 PM, Cong Wang wrote:
>> On Tue, Nov 15, 2016 at 12:48 PM, Andrei Vagin wrote:
>>> On Tue, Nov 15, 2016 at 10:50 AM, Cong Wang
>>> wrote:
On Tue, Nov 15, 2016 at 10:04 AM, Cong Wang
wrote:
> On M
On 11/15/2016 02:34 PM, Lino Sanfilippo wrote:
> On 15.11.2016 22:59, Andrew Lunn wrote:
>>> The link state is retrieved by a command to the application processor that
>>> is running
>>> on the network card. Also the register to set the phy configuration is
>>> write-only, so
>>> it is not even
On 15.11.2016 22:59, Andrew Lunn wrote:
>> The link state is retrieved by a command to the application processor that
>> is running
>> on the network card. Also the register to set the phy configuration is
>> write-only, so
>> it is not even possible to do the usual mdio bit-banging in the Phy r
On Tue, Nov 15, 2016 at 1:07 PM, Cong Wang wrote:
> On Tue, Nov 15, 2016 at 12:48 PM, Andrei Vagin wrote:
>> On Tue, Nov 15, 2016 at 10:50 AM, Cong Wang wrote:
>>> On Tue, Nov 15, 2016 at 10:04 AM, Cong Wang
>>> wrote:
On Mon, Nov 14, 2016 at 10:23 PM, Andrei Vagin wrote:
> Hi Nicola
This patch series addresses some minor issues found in the recently
accepted patch series for the AMD XGBE driver.
The following fixes are included in this driver update series:
- Fix a possibly uninitialized variable in the debugfs support
- Fix the GPIO pin number constraint check
This patch s
The GPIO support in the hardware allows for up to 16 GPIO pins, enumerated
from 0 to 15. The driver uses the wrong value (16) to validate the GPIO
pin range in the routines to set and clear the GPIO output pins. Update
the code to use the correct value (15).
Reported-by: Colin Ian King
Signed-o
The debugfs support in the driver uses a common routine to write the
debugfs values. In this routine, if the input file position is non-zero
then the write routine will not return an error and an output parameter
will not have been set. Because an error isn't returned an uninitialized
value will be
From: Philippe Reynes
Date: Sun, 13 Nov 2016 23:33:46 +0100
> The ethtool api {get|set}_settings is deprecated.
> We move this driver to new api {get|set}_link_ksettings.
>
> Signed-off-by: Philippe Reynes
Applied.
> The link state is retrieved by a command to the application processor that is
> running
> on the network card. Also the register to set the phy configuration is
> write-only, so
> it is not even possible to do the usual mdio bit-banging in the Phy read()
> and write()
> functions (however the
From: Woojung Huh
Relocate mdix code to phy driver to be called at config_init().
Signed-off-by: Woojung Huh
---
drivers/net/phy/microchip.c | 43 +-
drivers/net/usb/lan78xx.c | 73 ++---
2 files changed, 45 insertions(+), 71 de
On 11/13/2016 2:25 AM, Neftin, Sasha wrote:
On 11/13/2016 10:34 AM, Neftin, Sasha wrote:
On 11/11/2016 12:35 AM, Baicar, Tyler wrote:
Hello Sasha,
On 11/9/2016 11:19 PM, Neftin, Sasha wrote:
On 11/9/2016 11:41 PM, Tyler Baicar wrote:
Move IRQ free code so that it will happen regardless of th
On Tue, Nov 15, 2016 at 11:00:04AM -0800, Martin KaFai Lau wrote:
> gcc-6.2.1 gives the following warning:
> kernel/bpf/bpf_lru_list.c: In function
> ‘__bpf_lru_list_rotate_inactive.isra.3’:
> kernel/bpf/bpf_lru_list.c:201:28: warning: ‘next’ may be used uninitialized
> in this function [-Wmaybe-
From: Florian Fainelli
Date: Tue, 15 Nov 2016 10:06:29 -0800
> This patch series centralizes how ethtool::nway_reset is implemented
> by providing a PHYLIB function which calls into genphy_restart_aneg().
>
> All drivers below are converted to use this new helper function. Some
> other have spec
Hi Dave,
This series contains four humble mlx5 features.
from Gal,
- Add the support for PCIe statistics and expose them in ethtool
from Huy,
- Add the support for port module events reporting and statistics
- Add the support for driver version setting into FW (for display purposes
only)
>
From: Gal Pressman
This patch exposes two groups of PCIe counters:
- Performance counters.
- Timers and states counters.
Queried with ethtool -S .
Signed-off-by: Gal Pressman
Signed-off-by: Saeed Mahameed
---
.../net/ethernet/mellanox/mlx5/core/en_ethtool.c | 17
drivers/net/et
From: Mohamad Haj Yahia
Add more cache command size sets and more entries for each set based on
the current commands set different sizes and commands frequency.
Fixes: e126ba97dba9 ('mlx5: Add driver for Mellanox Connect-IB adapters')
Signed-off-by: Mohamad Haj Yahia
Signed-off-by: Saeed Mahame
From: Huy Nguyen
Add hardware structures and constants definitions needed for module
events support.
Signed-off-by: Huy Nguyen
Signed-off-by: Saeed Mahameed
Signed-off-by: Leon Romanovsky
---
include/linux/mlx5/device.h | 11 +++
include/linux/mlx5/mlx5_ifc.h | 3 ++-
include/linu
From: Gal Pressman
Add the needed infrastructure for future use of MPCNT register.
Signed-off-by: Gal Pressman
Signed-off-by: Saeed Mahameed
---
include/linux/mlx5/device.h | 5 +++
include/linux/mlx5/driver.h | 1 +
include/linux/mlx5/mlx5_ifc.h | 93 +++
From: Huy Nguyen
If driver_version capability bit is enabled, set driver version
to firmware after the init HCA command, for display purposes.
Example of driver version: "Linux,mlx5_core,3.0-1"
Signed-off-by: Huy Nguyen
Signed-off-by: Saeed Mahameed
---
drivers/net/ethernet/mellanox/mlx5/cor
From: Huy Nguyen
For each asynchronous port module event:
1. print with ratelimit to the dmesg log
2. increment the corresponding event counter
Signed-off-by: Huy Nguyen
Signed-off-by: Saeed Mahameed
---
drivers/net/ethernet/mellanox/mlx5/core/eq.c | 12 +
.../net/ethernet/mella
Add driver_version capability bit is enabled, and set driver
version command in mlx5_ifc firmware header. The only purpose
of this command is to store a driver version/OS string in FW
to be reported and displayed in various management systems,
such as IPMI/BMC.
Signed-off-by: Saeed Mahameed
Sign
From: Huy Nguyen
Add port module event counters to ethtool -S command
Signed-off-by: Huy Nguyen
Signed-off-by: Saeed Mahameed
---
.../net/ethernet/mellanox/mlx5/core/en_ethtool.c | 23 +-
drivers/net/ethernet/mellanox/mlx5/core/en_stats.h | 17
2 files c
Hi,
On 15.11.2016 21:54, Florian Fainelli wrote:
> On 11/15/2016 12:46 PM, Lino Sanfilippo wrote:
>>> Could this be pulled out into a standard PHY driver? All the SLIC
>>> SLIC_PCR_ defines seems to be the same as those in mii.h. This could
>>> be a standard PHY hidden behind a single register.
>>
> Subject: [PATCH 13/15] net: ethernet: lantiq_etop: Utilize
> phy_ethtool_nway_reset
>
> Signed-off-by: Florian Fainelli
> ---
> drivers/net/ethernet/lantiq_etop.c | 8 +---
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
Reviewed-by: Thomas Langer
On Tue, 2016-11-15 at 22:07 +0100, Jiri Pirko wrote:
> Tue, Nov 15, 2016 at 10:01:03PM CET, alexander.h.du...@intel.com wrote:
> >
> > On Tue, 2016-11-15 at 21:52 +0100, Jiri Pirko wrote:
> > >
> > > Tue, Nov 15, 2016 at 09:49:02PM CET, alexander.h.du...@intel.com wrote:
> > > >
> > > >
> > > >
Tue, Nov 15, 2016 at 10:01:03PM CET, alexander.h.du...@intel.com wrote:
>On Tue, 2016-11-15 at 21:52 +0100, Jiri Pirko wrote:
>> Tue, Nov 15, 2016 at 09:49:02PM CET, alexander.h.du...@intel.com wrote:
>> >
>> > On Tue, 2016-11-15 at 21:31 +0100, Jiri Pirko wrote:
>> > >
>> > > Tue, Nov 15, 2016 a
On Tue, Nov 15, 2016 at 12:48 PM, Andrei Vagin wrote:
> On Tue, Nov 15, 2016 at 10:50 AM, Cong Wang wrote:
>> On Tue, Nov 15, 2016 at 10:04 AM, Cong Wang wrote:
>>> On Mon, Nov 14, 2016 at 10:23 PM, Andrei Vagin wrote:
Hi Nicolas,
cleanup_net() calls idr_destroy(net->netns_ids) f
1 - 100 of 279 matches
Mail list logo