<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/drivers/net, branch v4.14.13</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.14.13</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.14.13'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2018-01-02T19:31:13Z</updated>
<entry>
<title>Revert "mlx5: move affinity hints assignments to generic code"</title>
<updated>2018-01-02T19:31:13Z</updated>
<author>
<name>Saeed Mahameed</name>
<email>saeedm@mellanox.com</email>
</author>
<published>2017-11-10T06:59:52Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=8f5bbb29b62c818e16de49c0bbc945b501c157cf'/>
<id>urn:sha1:8f5bbb29b62c818e16de49c0bbc945b501c157cf</id>
<content type='text'>
[ Upstream commit 231243c82793428467524227ae02ca451e6a98e7 ]

Before the offending commit, mlx5 core did the IRQ affinity itself,
and it seems that the new generic code have some drawbacks and one
of them is the lack for user ability to modify irq affinity after
the initial affinity values got assigned.

The issue is still being discussed and a solution in the new generic code
is required, until then we need to revert this patch.

This fixes the following issue:
echo &lt;new affinity&gt; &gt; /proc/irq/&lt;x&gt;/smp_affinity
fails with  -EIO

This reverts commit a435393acafbf0ecff4deb3e3cb554b34f0d0664.
Note: kept mlx5_get_vector_affinity in include/linux/mlx5/driver.h since
it is used in mlx5_ib driver.

Fixes: a435393acafb ("mlx5: move affinity hints assignments to generic code")
Cc: Sagi Grimberg &lt;sagi@grimberg.me&gt;
Cc: Thomas Gleixner &lt;tglx@linutronix.de&gt;
Cc: Jes Sorensen &lt;jsorensen@fb.com&gt;
Reported-by: Jes Sorensen &lt;jsorensen@fb.com&gt;
Signed-off-by: Saeed Mahameed &lt;saeedm@mellanox.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>phylink: ensure AN is enabled</title>
<updated>2018-01-02T19:31:12Z</updated>
<author>
<name>Russell King</name>
<email>rmk+kernel@armlinux.org.uk</email>
</author>
<published>2017-12-20T23:21:34Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=185a3475dee6c14dd67d1a94178b6b5615cf167b'/>
<id>urn:sha1:185a3475dee6c14dd67d1a94178b6b5615cf167b</id>
<content type='text'>
[ Upstream commit 74ee0e8c1bf9925c59cc8f1c65c29adf6e4cf603 ]

Ensure that we mark AN as enabled at boot time, rather than leaving
it disabled.  This is noticable if your SFP module is fiber, and
it supports faster speeds than 1G with 2.5G support in place.

Fixes: 9525ae83959b ("phylink: add phylink infrastructure")
Signed-off-by: Russell King &lt;rmk+kernel@armlinux.org.uk&gt;
Reviewed-by: Florian Fainelli &lt;f.fainelli@gmail.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>phylink: ensure the PHY interface mode is appropriately set</title>
<updated>2018-01-02T19:31:12Z</updated>
<author>
<name>Russell King</name>
<email>rmk+kernel@armlinux.org.uk</email>
</author>
<published>2017-12-20T23:21:28Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=39889c293371e826503b61d98050a49742ccaf9c'/>
<id>urn:sha1:39889c293371e826503b61d98050a49742ccaf9c</id>
<content type='text'>
[ Upstream commit 182088aa3c6c7f7c20a2c1dcc9ded4a3fc631f38 ]

When setting the ethtool settings, ensure that the validated PHY
interface mode is propagated to the current link settings, so that
2500BaseX can be selected.

Fixes: 9525ae83959b ("phylink: add phylink infrastructure")
Signed-off-by: Russell King &lt;rmk+kernel@armlinux.org.uk&gt;
Reviewed-by: Florian Fainelli &lt;f.fainelli@gmail.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>bnxt_en: Fix sources of spurious netpoll warnings</title>
<updated>2018-01-02T19:31:12Z</updated>
<author>
<name>Calvin Owens</name>
<email>calvinowens@fb.com</email>
</author>
<published>2017-12-08T17:05:26Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=7f6dcb82d04045f143758f56caf031376f9bf16c'/>
<id>urn:sha1:7f6dcb82d04045f143758f56caf031376f9bf16c</id>
<content type='text'>
[ Upstream commit 2edbdb3159d6f6bd3a9b6e7f789f2b879699a519 ]

After applying 2270bc5da3497945 ("bnxt_en: Fix netpoll handling") and
903649e718f80da2 ("bnxt_en: Improve -ENOMEM logic in NAPI poll loop."),
we still see the following WARN fire:

  ------------[ cut here ]------------
  WARNING: CPU: 0 PID: 1875170 at net/core/netpoll.c:165 netpoll_poll_dev+0x15a/0x160
  bnxt_poll+0x0/0xd0 exceeded budget in poll
  &lt;snip&gt;
  Call Trace:
   [&lt;ffffffff814be5cd&gt;] dump_stack+0x4d/0x70
   [&lt;ffffffff8107e013&gt;] __warn+0xd3/0xf0
   [&lt;ffffffff8107e07f&gt;] warn_slowpath_fmt+0x4f/0x60
   [&lt;ffffffff8179519a&gt;] netpoll_poll_dev+0x15a/0x160
   [&lt;ffffffff81795f38&gt;] netpoll_send_skb_on_dev+0x168/0x250
   [&lt;ffffffff817962fc&gt;] netpoll_send_udp+0x2dc/0x440
   [&lt;ffffffff815fa9be&gt;] write_ext_msg+0x20e/0x250
   [&lt;ffffffff810c8125&gt;] call_console_drivers.constprop.23+0xa5/0x110
   [&lt;ffffffff810c9549&gt;] console_unlock+0x339/0x5b0
   [&lt;ffffffff810c9a88&gt;] vprintk_emit+0x2c8/0x450
   [&lt;ffffffff810c9d5f&gt;] vprintk_default+0x1f/0x30
   [&lt;ffffffff81173df5&gt;] printk+0x48/0x50
   [&lt;ffffffffa0197713&gt;] edac_raw_mc_handle_error+0x563/0x5c0 [edac_core]
   [&lt;ffffffffa0197b9b&gt;] edac_mc_handle_error+0x42b/0x6e0 [edac_core]
   [&lt;ffffffffa01c3a60&gt;] sbridge_mce_output_error+0x410/0x10d0 [sb_edac]
   [&lt;ffffffffa01c47cc&gt;] sbridge_check_error+0xac/0x130 [sb_edac]
   [&lt;ffffffffa0197f3c&gt;] edac_mc_workq_function+0x3c/0x90 [edac_core]
   [&lt;ffffffff81095f8b&gt;] process_one_work+0x19b/0x480
   [&lt;ffffffff810967ca&gt;] worker_thread+0x6a/0x520
   [&lt;ffffffff8109c7c4&gt;] kthread+0xe4/0x100
   [&lt;ffffffff81884c52&gt;] ret_from_fork+0x22/0x40

This happens because we increment rx_pkts on -ENOMEM and -EIO, resulting
in rx_pkts &gt; 0. Fix this by only bumping rx_pkts if we were actually
given a non-zero budget.

Signed-off-by: Calvin Owens &lt;calvinowens@fb.com&gt;
Acked-by: Michael Chan &lt;michael.chan@broadcom.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>vxlan: restore dev-&gt;mtu setting based on lower device</title>
<updated>2018-01-02T19:31:12Z</updated>
<author>
<name>Alexey Kodanev</name>
<email>alexey.kodanev@oracle.com</email>
</author>
<published>2017-12-14T17:20:00Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=215b69e208089f42c29f9343ef11f4adc6ea6f6b'/>
<id>urn:sha1:215b69e208089f42c29f9343ef11f4adc6ea6f6b</id>
<content type='text'>
[ Upstream commit f870c1ff65a6d1f3a083f277280802ee09a5b44d ]

Stefano Brivio says:
    Commit a985343ba906 ("vxlan: refactor verification and
    application of configuration") introduced a change in the
    behaviour of initial MTU setting: earlier, the MTU for a link
    created on top of a given lower device, without an initial MTU
    specification, was set to the MTU of the lower device minus
    headroom as a result of this path in vxlan_dev_configure():

	if (!conf-&gt;mtu)
		dev-&gt;mtu = lowerdev-&gt;mtu -
			   (use_ipv6 ? VXLAN6_HEADROOM : VXLAN_HEADROOM);

    which is now gone. Now, the initial MTU, in absence of a
    configured value, is simply set by ether_setup() to ETH_DATA_LEN
    (1500 bytes).

    This breaks userspace expectations in case the MTU of
    the lower device is higher than 1500 bytes minus headroom.

This patch restores the previous behaviour on newlink operation. Since
max_mtu can be negative and we update dev-&gt;mtu directly, also check it
for valid minimum.

Reported-by: Junhan Yan &lt;juyan@redhat.com&gt;
Fixes: a985343ba906 ("vxlan: refactor verification and application of configuration")
Signed-off-by: Alexey Kodanev &lt;alexey.kodanev@oracle.com&gt;
Acked-by: Stefano Brivio &lt;sbrivio@redhat.com&gt;
Signed-off-by: Stefano Brivio &lt;sbrivio@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>net/mlx5: FPGA, return -EINVAL if size is zero</title>
<updated>2018-01-02T19:31:12Z</updated>
<author>
<name>Kamal Heib</name>
<email>kamalh@mellanox.com</email>
</author>
<published>2017-10-29T02:03:37Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=cea58617977baafe4a1be8f55c8e55d21c6cfb20'/>
<id>urn:sha1:cea58617977baafe4a1be8f55c8e55d21c6cfb20</id>
<content type='text'>
[ Upstream commit bae115a2bb479142605726e6aa130f43f50e801a ]

Currently, if a size of zero is passed to
mlx5_fpga_mem_{read|write}_i2c()
the "err" return value will not be initialized, which triggers gcc
warnings:

[..]/mlx5/core/fpga/sdk.c:87 mlx5_fpga_mem_read_i2c() error:
uninitialized symbol 'err'.
[..]/mlx5/core/fpga/sdk.c:115 mlx5_fpga_mem_write_i2c() error:
uninitialized symbol 'err'.

fix that.

Fixes: a9956d35d199 ('net/mlx5: FPGA, Add SBU infrastructure')
Signed-off-by: Kamal Heib &lt;kamalh@mellanox.com&gt;
Reviewed-by: Yevgeny Kliteynik &lt;kliteyn@mellanox.com&gt;
Signed-off-by: Saeed Mahameed &lt;saeedm@mellanox.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>net: phy: marvell: Limit 88m1101 autoneg errata to 88E1145 as well.</title>
<updated>2018-01-02T19:31:11Z</updated>
<author>
<name>Zhao Qiang</name>
<email>qiang.zhao@nxp.com</email>
</author>
<published>2017-12-18T02:26:43Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=5e255d684d0554893768b4452de404f293c71696'/>
<id>urn:sha1:5e255d684d0554893768b4452de404f293c71696</id>
<content type='text'>
[ Upstream commit c505873eaece2b4aefd07d339dc7e1400e0235ac ]

88E1145 also need this autoneg errata.

Fixes: f2899788353c ("net: phy: marvell: Limit errata to 88m1101")
Signed-off-by: Zhao Qiang &lt;qiang.zhao@nxp.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>mlxsw: spectrum: Disable MAC learning for ovs port</title>
<updated>2018-01-02T19:31:11Z</updated>
<author>
<name>Yuval Mintz</name>
<email>yuvalm@mellanox.com</email>
</author>
<published>2017-12-15T07:44:21Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=bcc029ff5dafc68d00a5fceadd93763d2b43e0e3'/>
<id>urn:sha1:bcc029ff5dafc68d00a5fceadd93763d2b43e0e3</id>
<content type='text'>
[ Upstream commit fccff0862838908d21eaf956d57e09c6c189f7c5 ]

Learning is currently enabled for ports which are OVS slaves -
even though OVS doesn't need this indication.
Since we're not associating a fid with the port, HW would continuously
notify driver of learned [&amp; aged] MACs which would be logged as errors.

Fixes: 2b94e58df58c ("mlxsw: spectrum: Allow ports to work under OVS master")
Signed-off-by: Yuval Mintz &lt;yuvalm@mellanox.com&gt;
Reviewed-by: Ido Schimmel &lt;idosch@mellanox.com&gt;
Signed-off-by: Jiri Pirko &lt;jiri@mellanox.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>net: dsa: bcm_sf2: Clear IDDQ_GLOBAL_PWR bit for PHY</title>
<updated>2018-01-02T19:31:10Z</updated>
<author>
<name>Florian Fainelli</name>
<email>f.fainelli@gmail.com</email>
</author>
<published>2017-11-22T01:37:46Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=ff1ff3815c2483ac19bd6f926a535dd4cea27e2e'/>
<id>urn:sha1:ff1ff3815c2483ac19bd6f926a535dd4cea27e2e</id>
<content type='text'>
[ Upstream commit 4b52d010113e11006a389f2a8315167ede9e0b10 ]

The PHY on BCM7278 has an additional bit that needs to be cleared:
IDDQ_GLOBAL_PWR, without doing this, the PHY remains stuck in reset out
of suspend/resume cycles.

Fixes: 0fe9933804eb ("net: dsa: bcm_sf2: Add support for BCM7278 integrated switch")
Signed-off-by: Florian Fainelli &lt;f.fainelli@gmail.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>sfc: pass valid pointers from efx_enqueue_unwind</title>
<updated>2018-01-02T19:31:10Z</updated>
<author>
<name>Bert Kenward</name>
<email>bkenward@solarflare.com</email>
</author>
<published>2017-12-07T17:18:58Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=701768dc9a1030d3f02b9b3e9b4690d15f637dad'/>
<id>urn:sha1:701768dc9a1030d3f02b9b3e9b4690d15f637dad</id>
<content type='text'>
[ Upstream commit d4a7a8893d4cdbc89d79ac4aa704bf8d4b67b368 ]

The bytes_compl and pkts_compl pointers passed to efx_dequeue_buffers
cannot be NULL. Add a paranoid warning to check this condition and fix
the one case where they were NULL.

efx_enqueue_unwind() is called very rarely, during error handling.
Without this fix it would fail with a NULL pointer dereference in
efx_dequeue_buffer, with efx_enqueue_skb in the call stack.

Fixes: e9117e5099ea ("sfc: Firmware-Assisted TSO version 2")
Reported-by: Jarod Wilson &lt;jarod@redhat.com&gt;
Signed-off-by: Bert Kenward &lt;bkenward@solarflare.com&gt;
Tested-by: Jarod Wilson &lt;jarod@redhat.com&gt;
Acked-by: Jarod Wilson &lt;jarod@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
</feed>
