Hi!
On 06/04/17 10:15, Herbert Xu wrote:
> On Thu, Mar 16, 2017 at 03:16:29PM +0100, Alexander Sverdlin wrote:
>> This is a regression caused by 856e3f4092
>> ("crypto: seqiv - Add support for new AEAD interface")
>>
>> As I've said above, I can offer one of the two solutions, which patch should
From: Gao Feng
Because TCP_MIB_OUTRSTS is an important count, so always increase it
whatever send it successfully or not.
Now move the increment of TCP_MIB_OUTRSTS to the top of
tcp_send_active_reset to make sure it is increased always even though
fail to alloc skb.
Signed-off-by: Gao Feng
---
On 04/06/2017 07:04 AM, Andrew Lunn wrote:
> On Thu, Apr 06, 2017 at 06:47:35AM -0700, Florian Fainelli wrote:
>> When CONFIG_OF=n, we will see the following warning:
>>
>> drivers/net/dsa/mv88e6xxx/chip.c:2834:34: warning:
>> 'mv88e6xxx_mdio_external_match' defined but not used
>> [-Wunused-co
On Thu, Apr 06, 2017 at 06:53:10AM -0700, Florian Fainelli wrote:
>
>
> On 04/05/2017 12:32 PM, Andrew Lunn wrote:
> >> +static const struct of_device_id lan9303_mdio_of_match[] = {
> >> + { .compatible = "smsc,lan9303" },
> >> + { /* sentinel */ },
> >> +};
> >> +MODULE_DEVICE_TABLE(of, lan930
Hi Neal,
> -Original Message-
> From: Neal Cardwell [mailto:ncardw...@google.com]
> Sent: Thursday, April 6, 2017 10:01 PM
> To: Gao Feng
> Cc: David Miller ; Alexey Kuznetsov
> ; James Morris ; Patrick McHardy
> ; Netdev ; Gao Feng
>
> Subject: Re: [PATCH net 1/1] net: tcp: Don't increa
From: Johannes Berg
It's rather confusing that the netlink message flags are
numbered 1, 2, 4, 8, 16, 32, , 0x100. Make that
more understandable by numbering the lower ones with hex
constants as well.
Signed-off-by: Johannes Berg
---
include/uapi/linux/netlink.h | 12 ++--
1 file chang
From: Andreas Pape
Some of the bla debug messages are extended and additional messages are
added for easier bla debugging. Some debug messages introduced with the
dat changes in prior patches of this patch series have been changed to
be more compliant to other existing debug messages.
Acked-by:
From: Andreas Pape
Consider the following situation which has been found in a test setup:
Gateway B has claimed client C and gateway A has the same backbone
network as B. C sends a broad- or multicast to B and directly after
this packet decides to send another packet to A due to a better TQ
value
On Thu, Apr 6, 2017 at 10:05 AM, Gao Feng wrote:
> If so, we should increase the TCP_MIB_OUTRSTS too when fail to alloc skb.
> When machine is overloaded and mem is exhausted, it may fail to alloc skb.
Moving the increment of TCP_MIB_OUTRSTS to the top of
tcp_send_active_reset() sounds fine to me
From: Linus Lüssing
This patch refactors the num_packets counter of a forw_packet in the
following three ways:
1) Removed dual-use of forw_packet::num_packets:
-> now for aggregation purposes only
2) Using forw_packet::skb::cb::num_bcasts instead:
-> for easier access in aggregation code l
From: Tobias Klauser
Instead of using a private copy of struct net_device_stats in struct
batadv_priv, use stats from struct net_device.
Signed-off-by: Tobias Klauser
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-adv/distributed-arp-table.c | 9 +
net/b
From: Sven Eckelmann
The .get_settings function pointer and the related API was deprecated.
Fortunately, batman-adv is a virtual interface and never provided any
useful information via .get_settings. The stub can therefore be
removed.
This also avoids that incorrect information is shown in ethto
From: Sven Eckelmann
The ethtool code was spread in soft-interface.c. This makes reading the
code and working on it unnecessary complicated. Having everything in a
common place next to the other code which references it, makes it slightly
easier.
Signed-off-by: Sven Eckelmann
Acked-by: Marek Li
From: Linus Lüssing
An skb is assigned to a forw_packet only once, shortly after the
forw_packet allocation.
With this patch the assignment is moved into the this allocation
function.
Signed-off-by: Linus Lüssing
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-a
From: Andreas Pape
If none of the backbone gateways in a bla setup has already knowledge of
the mac address searched for in an incoming ARP request from the backbone
an address resolution via the DHT of DAT is started. The gateway can send
several ARP requests to different DHT nodes and therefore
From: Sven Eckelmann
The BATADV_PRINT_VID is not free of of possible side-effects. This can be
avoided when the the macro is converted to a simple inline function.
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-adv/bridge_loop_avoidance.c | 38 +--
Signed-off-by: Simon Wunderlich
---
net/batman-adv/main.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/batman-adv/main.h b/net/batman-adv/main.h
index 57a8103dbce7..f87b17c613a6 100644
--- a/net/batman-adv/main.h
+++ b/net/batman-adv/main.h
@@ -24,7 +24,7 @@
#define BA
From: Sven Eckelmann
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-adv/translation-table.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/net/batman-adv/translation-table.c
b/net/batman-adv/translation-table.c
index 6077a87d46f0..a75f0
From: Tobias Klauser
The memory for netdev_priv is allocated using kzalloc in alloc_netdev
(or alloc_netdev_mq respectively) so there is no need to set it to 0
again.
Signed-off-by: Tobias Klauser
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-adv/soft-interface
From: Sven Eckelmann
An argument of a macro should not be evaluated multiple times. Otherwise
embedded operations in these arguments will be executed multiple times.
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-adv/log.h | 5 +++--
1 file changed, 3 insertions(
From: Sven Eckelmann
batadv devices don't support msglevel. The ethtool stubs therefore returned
that it isn't supported. But instead, the complete function can be dropped
to avoid that bogus values are shown in ethtool.
Signed-off-by: Sven Eckelmann
Acked-by: Marek Lindner
Signed-off-by: Simo
From: Andreas Pape
Additional dropping of unicast packets received from another backbone gw if
the same backbone network before being forwarded to the same backbone again
is necessary. It was observed in a test setup that in rare cases these
frames lead to looping unicast traffic backbone->mesh->
From: Sven Eckelmann
The ethtool_ops of batman-adv never contained more than a stub for the
get_link function pointer. It was always returning that a link exists even
when the devices was not yet up and therefore nothing resampling a link
could have been available.
Instead use the ethtool helper
From: Sven Eckelmann
The name of the function might change in which these messages are printed.
It is therefore better to let the compiler handle the insertion of the
correct function name.
Signed-off-by: Sven Eckelmann
Signed-off-by: Simon Wunderlich
---
net/batman-adv/tp_meter.c | 7 ---
From: Sven Eckelmann
It is not necessary to disable these code sections in case other kernel
features are disabled. Instead the IS_ENABLED tests can be added directly
in the code and the compiler can remove the unnecessary code parts during
its optimization run.
Signed-off-by: Sven Eckelmann
Si
On Thu, Apr 06, 2017 at 06:47:36AM -0700, Florian Fainelli wrote:
> Fixes the following warnings:
>
> drivers/net/dsa/mv88e6xxx/global2.c:571:5: warning: no previous prototype for
> 'mv88e6xxx_g2_smi_phy_read_c45' [-Wmissing-prototypes]
> int mv88e6xxx_g2_smi_phy_read_c45(struct mv88e6xxx_chip *
From: Andreas Pape
If dat is enabled it must be made sure that only the backbone gw which has
claimed the remote destination for the ARP request answers the ARP request
directly if the MAC address is known due to the local dat table. This
prevents multiple ARP replies in a common backbone if more
the git repository at:
git://git.open-mesh.org/linux-merge.git tags/batadv-next-for-davem-20170406
for you to fetch changes up to ab044f8e3eaf84c4cc95d7606fadfdfa006dc8ec:
batman-adv: Use net_device_stats from struct net_device (2017-04-05 15:41:24
+0200
On Thu, Apr 06, 2017 at 06:47:35AM -0700, Florian Fainelli wrote:
> When CONFIG_OF=n, we will see the following warning:
>
> drivers/net/dsa/mv88e6xxx/chip.c:2834:34: warning:
> 'mv88e6xxx_mdio_external_match' defined but not used
> [-Wunused-const-variable=]
> static const struct of_device_id
On Thu, Apr 6, 2017 at 9:35 AM, wrote:
> From: Gao Feng
>
> When fail to transmit RST, don't increase TCP_MIB_OUTRSTS in func
> tcp_send_active_reset like the case that it only increases
> LINUX_MIB_TCPABORTFAILED when fail to alloc skb.
>
> Signed-off-by: Gao Feng
> ---
I would be concerned t
On 04/05/2017 12:32 PM, Andrew Lunn wrote:
>> +static const struct of_device_id lan9303_mdio_of_match[] = {
>> +{ .compatible = "smsc,lan9303" },
>> +{ /* sentinel */ },
>> +};
>> +MODULE_DEVICE_TABLE(of, lan9303_mdio_of_match);
>
> If i'm reading this right, i think you have the same co
On 04/06/2017 06:46 AM, Juergen Borleis wrote:
>>> + phy-reset-gpios = <&gpio7 6 GPIO_ACTIVE_LOW>;
>>> + phy-reset-duration = <200>;
>>> +
>>> + dsa,member = <0 0>;
>>> +
>>> + ports {
>>> + #address-cells = <1>;
>>> + #si
Fixes the following warnings:
drivers/net/dsa/mv88e6xxx/global2.c:571:5: warning: no previous prototype for
'mv88e6xxx_g2_smi_phy_read_c45' [-Wmissing-prototypes]
int mv88e6xxx_g2_smi_phy_read_c45(struct mv88e6xxx_chip *chip, int addr,
^
drivers/net/dsa/mv88e6xxx
When CONFIG_OF=n, we will see the following warning:
drivers/net/dsa/mv88e6xxx/chip.c:2834:34: warning:
'mv88e6xxx_mdio_external_match' defined but not used [-Wunused-const-variable=]
static const struct of_device_id mv88e6xxx_mdio_external_match[] = {
^
Hi all,
This fixes a bunch of warnings exposed by a make W=1, no functional changes
Florian Fainelli (2):
net: dsa: mv88e6xxx: Fix warning with CONFIG_OF=n
net: dsa: mv88e6xxx: Provide prototypes for SMI read/write functions
drivers/net/dsa/mv88e6xxx/chip.c| 2 ++
drivers/net/dsa/mv88e6
Hi Andrew,
On Wednesday 05 April 2017 20:21:55 Andrew Lunn wrote:
> [...]
> > +SMSC/MicroChip LAN9303 three port ethernet switch
> > +-
> > +
> > +Required properties:
> > +
> > +- compatible: should be "smsc,lan9303"
> > +- #size-cells: must be 0
>
From: Gao Feng
When fail to transmit RST, don't increase TCP_MIB_OUTRSTS in func
tcp_send_active_reset like the case that it only increases
LINUX_MIB_TCPABORTFAILED when fail to alloc skb.
Signed-off-by: Gao Feng
---
net/ipv4/tcp_output.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(
Hi Andrew,
On Thursday 06 April 2017 13:59:00 Andrew Lunn wrote:
> [...]
> > > Does the MDIO bus go to the outside world? Could there be external
> > > PHYs?
> >
> > This device includes two phys (at port 1 and 2) and these
> > functions are called to detect their state.
>
> Some switches hav
Hi Andrew,
On Wednesday 05 April 2017 19:10:01 Andrew Lunn wrote:
> [...]
> > +static int lan9303_rcv(struct sk_buff *skb, struct net_device *dev,
> > + struct packet_type *pt, struct net_device *orig_dev)
> > +{
> > + u16 *lan9303_tag;
> > + struct dsa_switch_tree *dst = dev-
Thu, Apr 06, 2017 at 07:41:28AM CEST, lucien@gmail.com wrote:
>Commit f6988cb63a4e ("team: don't call netdev_change_features under
>team->lock") fixed the issue calling netdev_change_features under
>team->lock for team_compute_features.
>
>But there are still two places where it calls netdev_ch
On Wed, Apr 05, 2017 at 07:00:54PM -0700, Chenbo Feng wrote:
> From: Chenbo Feng
>
>
> In the current kernel socket cookie implementation, there is no simple
> and direct way to retrieve the socket cookie based on file descriptor. A
> process mat need to get it from sock fd if it want to correla
On Thu, Apr 06, 2017 at 01:10:52PM +0800, Xin Long wrote:
> Now sctp doesn't check sock's state before listening on it. It could
> even cause changing a sock with any state to become a listening sock
> when doing sctp_listen.
>
> This patch is to fix it by checking sock's state in sctp_listen, so
From: Michal Kalderon
No need to maintain the various open archipelagos as a list -
The maximal number of them is known, and we can use the CID
as key for random-access into the array.
Signed-off-by: Michal Kalderon
Signed-off-by: Yuval Mintz
---
drivers/net/ethernet/qlogic/qed/qed_ooo.c | 10
Now that management firmware is capable of telling us the number of CQs
available for a given PF, qed needs to communicate the number to qedi
so it would know have many to use.
Signed-off-by: Yuval Mintz
---
drivers/net/ethernet/qlogic/qed/qed.h | 3 ++-
drivers/net/ethernet/qlogic/qed/qe
Management firmware can query for some basic iSCSI-related statistics.
Provide those just as we do for other protocols.
Signed-off-by: Yuval Mintz
---
drivers/net/ethernet/qlogic/qed/qed_iscsi.c | 20
drivers/net/ethernet/qlogic/qed/qed_iscsi.h | 14 +-
drivers/n
Thu, Apr 06, 2017 at 02:23:03PM CEST, h...@lst.de wrote:
>Trivial conversion as only one vector is supported, but at least we
>lose the useless msix_entry member in the per-device structure.
>
>Signed-off-by: Christoph Hellwig
Acked-by: Jiri Pirko
Before initializing the chip's engine, driver currently closes a set
of registers on the HW's ingress flow to prevent packets from slipping
in while they're not supposed to.
This configuration is insufficient, as there are some scenarios where
packets would still arrive even when said registers ar
From: Rahul Verma
In order to access HW registers driver needs to acquire a PTT entry
[mapping between bar memory and internal chip address].
Since acquiring PTT entries could fail [at least in theory] as their
number is finite and other flows can hold them, we reserve special PTT
entries for 'im
From: Tomer Tayar
Default HW configuration is optimal for an architecture where cache
line size is 64B.
During chip initialization, properly initialize the cache line size
in HW to avoid possible redundant PCI transactions.
Signed-off-by: Tomer Tayar
Signed-off-by: Yuval Mintz
---
drivers/ne
Firmware provides a statistic for the number of out-of-order isles
it used - fill it in the iscsi-related statistics.
Signed-off-by: Yuval Mintz
---
drivers/net/ethernet/qlogic/qed/qed_iscsi.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/ethernet/qlogic/qed/qed_iscsi.c
b/dr
Patches #1 and #2 revolve around register access performed by driver;
The first merely adds some debug, while the second does some fixing
of incorrect PTT usage as well as preventing issues similar to those
fixed by 6f437d431930 ("qed: Don't use attention PTT for configuring BW").
Patch #3 better
PTT entries are per-hwfn; If some errneous flow is trying
to use a PTT belonging to a differnet hwfn warn user, as this
can break every register accessing flow later and is very hard
to root-cause.
Signed-off-by: Yuval Mintz
---
drivers/net/ethernet/qlogic/qed/qed_hw.c | 7 +++
1 file change
On Tue, Apr 04, 2017 at 09:45:36PM -0600, Tycho Andersen wrote:
> Hi Kees,
>
> On Tue, Apr 04, 2017 at 03:17:57PM -0700, Kees Cook wrote:
> > On Tue, Apr 4, 2017 at 3:08 PM, Tycho Andersen wrote:
> > > The goal of this patch is to protect the JIT against an attacker with a
> > > write-in-memory p
From: Colin Ian King
On the case where nn->eth_port is null the warning message
is printing the port by dereferencing this null pointer.
Remove the deference to avoid a crash when printing the
warning message.
Detected by CoverityScan, CID#1426198 ("Dereference after null check")
Fixes: ce22f5a
On 04/06/2017 01:05 PM, Shubham Bansal wrote:
Gentle Reminder.
Sorry for late reply.
Anybody can tell me how to test the JIT compiler ?
There's lib/test_bpf.c, see Documentation/networking/filter.txt +1349
for some more information. It basically contains various test cases that
have the pur
On Thu, Apr 06, 2017 at 02:23:03PM +0200, Christoph Hellwig wrote:
> Trivial conversion as only one vector is supported, but at least we
> lose the useless msix_entry member in the per-device structure.
>
> Signed-off-by: Christoph Hellwig
For net-next:
Reviewed-by: Ido Schimmel
Tested-by: Ido
On Wed, Apr 05, 2017 at 10:59:49PM -0400, Aaron Conole wrote:
> Hi Daniel,
>
> Daniel Borkmann writes:
>
> > On 04/04/2017 08:33 PM, Aaron Conole wrote:
> >> The eBPF framework is used for more than just socket level filtering. It
> >> can also provide tracing, and even change the way packets c
On 6 April 2017 at 13:29, Alexei Starovoitov
wrote:
> On Wed, Apr 05, 2017 at 09:17:25PM +0100, David Howells wrote:
>> From: Chun-Yi Lee
>>
>> There are some bpf functions can be used to read kernel memory:
>> bpf_probe_read, bpf_probe_write_user and bpf_trace_printk. These allow
>> private key
On Thu, Apr 06, 2017 at 09:49:09AM +0100, Joao Pinto wrote:
[...]
> err_init_rx_buffers:
> - while (--i >= 0)
> - stmmac_free_rx_buffer(priv, i);
> + while (queue >= 0) {
I /think/ this could simply be:
while (queue--) {
...
}
That evaluates b
Hi!
On 06/04/17 10:16, Herbert Xu wrote:
> This patch removes the hard-coded 64-byte limit on the length
> of the algorithm name through bind(2). The address length can
> now exceed that. The user-space structure remains unchanged.
> In order to use a longer name simply extend the salg_name arra
On Wed, Apr 05, 2017 at 09:17:25PM +0100, David Howells wrote:
> From: Chun-Yi Lee
>
> There are some bpf functions can be used to read kernel memory:
> bpf_probe_read, bpf_probe_write_user and bpf_trace_printk. These allow
> private keys in kernel memory (e.g. the hibernation image signing key)
Hi all,
this series removes the remaining callers of the pci_enable_msix()
function and then the function itself. All remaining users in in network
drivers, so if Bjorn is fine with it I'd like to handle it entirely
through the netdev tree.
Changes since V1:
- replace the two previous thunderx
From: Thanneeru Srinivasulu
Remove deprecated pci_enable_msix API in favour of it's
successor pci_alloc_irq_vectors.
Signed-off-by: Thanneeru Srinivasulu
Signed-off-by: Sunil Goutham
Reviewed-by: Christoph Hellwig
---
drivers/net/ethernet/cavium/thunder/nic.h| 2 -
drivers/net/ether
Remove the deprecated pci_enable_msix API in favour of it's successor,
and make sure to handle errors during IRQ setup properly.
Signed-off-by: Christoph Hellwig
---
drivers/net/ethernet/atheros/alx/alx.h | 6 --
drivers/net/ethernet/atheros/alx/main.c | 128 ++--
Remove the deprecated pci_enable_msix API in favour of it's successor.
Signed-off-by: Christoph Hellwig
---
drivers/net/ethernet/amazon/ena/ena_netdev.c | 55 ++--
drivers/net/ethernet/amazon/ena/ena_netdev.h | 2 -
2 files changed, 12 insertions(+), 45 deletions(-)
dif
Unused now that all callers switched to pci_alloc_irq_vectors.
Signed-off-by: Christoph Hellwig
---
drivers/pci/msi.c | 21 -
include/linux/pci.h | 4
2 files changed, 25 deletions(-)
diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c
index d571bc330686..0042c365b29b
Trivial conversion as only one vector is supported, but at least we
lose the useless msix_entry member in the per-device structure.
Signed-off-by: Christoph Hellwig
---
Changes since V1:
- rebase due to a trivial conflict in net-next
drivers/net/ethernet/mellanox/mlxsw/pci.c | 13 ++--
> > Do you really need more than one GPIO? A single gpio would make all
> > this code a lot simpler.
> >
>
> Yes we need. Some of our boards have separate GPIO RESET lines for
> different PHYs on the same MDIO bus.
If you have a one-to-one mapping of GPIO and PHY, you should really be
modelling
> > > +static int lan9303_phy_write(struct dsa_switch *ds, int phy, int regnum,
> > > + u16 val)
> > > +{
> > > + struct lan9303 *chip = ds_to_lan9303(ds);
> > > + int phy_base = chip->phy_addr_sel_strap;
> > > +
> > > + if (phy == phy_base)
> > > + return lan9303
On 2017/4/5 21:05, John Garry wrote:
> On 02/04/2017 07:49, Ding Tianhong wrote:
>>
>>
>> On 2017/4/2 2:26, David Miller wrote:
>>> From: Ding Tianhong
>>> Date: Sat, 1 Apr 2017 15:25:51 +0800
>>>
Till now only the Intel ixgbe could support enable
Relaxed ordering in the drivers for spe
On Thu, Apr 06, 2017 at 11:40:23AM +0200, Christoph Hellwig wrote:
> Trivial conversion as only one vector is supported, but at least we
> lose the useless msix_entry member in the per-device structure.
>
> Signed-off-by: Christoph Hellwig
It doesn't apply cleanly to net-next. Can you please re-
On Thu, Apr 06, 2017 at 08:38:10AM +0200, Wouter Verhelst wrote:
> On Wed, Apr 05, 2017 at 01:30:31PM +0200, Michal Hocko wrote:
> > On Wed 05-04-17 09:46:59, Vlastimil Babka wrote:
> > > We now have memalloc_noreclaim_{save,restore} helpers for robust setting
> > > and
> > > clearing of PF_MEMALL
On Thu, Apr 06, 2017 at 12:28:44PM +0200, Peter Zijlstra wrote:
> On Wed, Apr 05, 2017 at 05:31:05PM -0700, Stephen Hemminger wrote:
> > On Sun, 02 Apr 2017 06:28:41 +0200
> > Mike Galbraith wrote:
> >
> > > Livelock can be triggered by setting kworkers to SCHED_FIFO, then
> > > suspend/resume..
Gentle Reminder.
Anybody can tell me how to test the JIT compiler ?
Best,
Shubham Bansal
On Thu, Mar 30, 2017 at 7:34 PM, Shubham Bansal
wrote:
> Thanks Daniel.
>
> Can you tell me how to test the eBPF JIT compiler? It would be great
> if you could tell me starting from compiling to proper test
On Wed, 2017-04-05 at 15:33 -0700, Cong Wang wrote:
> Good find! I missed the refcnt in rt_set_nexthop() before that commit.
>
> We need to revert that commit to restore the refcnt for fib_info.
Well, there are other spots , in decnet and IPv6.
This is why my original mail stated the problem wa
From: Thanneeru Srinivasulu
Adding support for TSO and checksum hardware offloads for ipv6.
Signed-off-by: Thanneeru Srinivasulu
Signed-off-by: Sunil Goutham
---
drivers/net/ethernet/cavium/thunder/nicvf_main.c |8 +---
drivers/net/ethernet/cavium/thunder/nicvf_queues.c | 10 +
Hi Andrew,
On Wednesday 05 April 2017 20:12:32 Andrew Lunn wrote:
> [...]
> > drivers/net/phy/lan9303-core.c | 924
> > +
> > drivers/net/phy/lan9303.h | 21 +
>
> drivers/net/dsa please.
Done in v2.
> One general comment. I'm assuming parts of this
On Wed, Apr 05, 2017 at 05:31:05PM -0700, Stephen Hemminger wrote:
> On Sun, 02 Apr 2017 06:28:41 +0200
> Mike Galbraith wrote:
>
> > Livelock can be triggered by setting kworkers to SCHED_FIFO, then
> > suspend/resume.. you come back from sleepy-land with a spinning
> > kworker. For whatever re
On Thu, 2017-04-06 at 03:26 -0700, Eric Dumazet wrote:
> Hi Thushar, thank you for working on this issue.
I am sorry for the typo Tushar !
Thanks !
On Tue, Apr 04, 2017 at 10:25:19PM -0700, Cong Wang wrote:
> On Tue, Apr 4, 2017 at 8:20 PM, Mike Galbraith wrote:
> > - while (some_qdisc_is_busy(dev))
> > - yield();
> > + swait_event_timeout(swait, !some_qdisc_is_busy(dev), 1);
> > }
>
> I don
On Wed, 2017-04-05 at 19:06 -0700, Tushar Dave wrote:
> Reducing real_num_tx_queues needs to be in sync with skb queue_mapping
> otherwise skbs with queue_mapping greater than real_num_tx_queues
> can be sent to the underlying driver and can result in kernel panic.
>
> One such event is running ne
Add a tracepoint (rxrpc_connect_call) to log the combination of rxrpc_call
pointer, afs_call pointer/user data and wire call parameters to make it
easier to match the tracebuffer contents to captured network packets.
Signed-off-by: David Howells
---
include/trace/events/rxrpc.h | 26 +
Add a tracepoint (rxrpc_rx_abort) to record received aborts.
Signed-off-by: David Howells
---
include/trace/events/rxrpc.h | 24
net/rxrpc/input.c|4 +++-
2 files changed, 27 insertions(+), 1 deletion(-)
diff --git a/include/trace/events/rxrpc.h b/inc
Add a tracepoint (rxrpc_rx_rwind_change) to log changes in a call's receive
window size as imposed by the peer through an ACK packet.
Signed-off-by: David Howells
---
include/trace/events/rxrpc.h | 27 +++
net/rxrpc/input.c|2 ++
2 files changed, 29 ins
Add a tracepoint (rxrpc_rx_proto) to record protocol errors in received
packets. The following changes are made:
(1) Add a function, __rxrpc_abort_eproto(), to note a protocol error on a
call and mark the call aborted. This is wrapped by
rxrpc_abort_eproto() that makes the why string
Make rxrpc_kernel_abort_call() return an indication as to whether it
actually aborted the operation or not so that kafs can trace the failure of
the operation. Note that 'success' in this context means changing the
state of the call, not necessarily successfully transmitting an ABORT
packet.
Sign
In the rxkad security module, when we encounter a temporary error (such as
ENOMEM) from which we could conceivably recover, don't abort the
connection, but rather permit retransmission of the relevant packets to
induce a retry.
Note that I'm leaving some places that could be merged together to ins
Use negative error codes in struct rxrpc_call::error because that's what
the kernel normally deals with and to make the code consistent. We only
turn them positive when transcribing into a cmsg for userspace recvmsg.
Signed-off-by: David Howells
---
fs/afs/rxrpc.c | 12 ++--
l/git/dhowells/linux-fs.git/log/?h=rxrpc-rewrite
Tagged thusly:
git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git
rxrpc-rewrite-20170406
David
---
David Howells (7):
rxrpc: Use negative error codes in rxrpc_call struct
rxrpc: Note a successfully abor
Apologies.
Need coffee ...
Corrected in previous email.
cheers,
jamal
On 17-04-06 06:12 AM, Jamal Hadi Salim wrote:
The tech committee is pleased to announce a keynote by Jesse Brandeburg
titled "Linux and the Network"
In this talk Jesse will cover topics such as, a brief history of the
networ
Hi Andrew,
v2 of the patches will follow.
On Wednesday 05 April 2017 20:12:32 Andrew Lunn wrote:
> [...]
> > + do {
> > + ret = regmap_read(regmap, offset, reg);
> > + if (ret == -EAGAIN)
> > + msleep(500);
> > + } while (ret == -EAGAIN);
>
> Please limit
The tech committee is pleased to announce a keynote by Jesse Brandeburg
titled "Linux and the Network"
In this talk Jesse will cover topics such as, a brief history of the
network in Linux, Intel’s expectations of the number of devices on the
internet causing a dramatic expansion in the number o
The tech committee is pleased to announce a keynote by Jesse Brandeburg
titled "Linux and the Network"
In this talk Jesse will cover topics such as, a brief history of the
network in Linux, Intel’s expectations of the number of devices on the
internet causing a dramatic expansion in the number o
Simplify the loop in phy_supported_speeds().
Signed-off-by: Russell King
---
drivers/net/phy/phy.c | 14 --
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
index b50b6e61aace..f3015e6a52ab 100644
--- a/drivers/net/phy/phy.c
phylib has some undesirable behaviour when forcing a link mode through
ethtool. phylib uses this code:
idx = phy_find_valid(phy_find_setting(phydev->speed, phydev->duplex),
features);
to find an index in the settings table. phy_find_setting() starts at
index 0, a
This set of two patches fixes and cleans up the phylib code associated
with selecting the fixed mode.
phylib currently assumes that all PHYs will support the 10baseT/Half mode
of operation irrespective of the supported bitmask, because of the way
phy_find_valid() and phy_find_setting() operate.
T
Trivial conversion as only one vector is supported, but at least we
lose the useless msix_entry member in the per-device structure.
Signed-off-by: Christoph Hellwig
---
drivers/net/ethernet/mellanox/mlxsw/pci.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/dri
shouldn't you include and like in
commit 8ec2ef2b66ea2f that fixes blk-mq-pci.c ?
Not really. We can lose these from blk-mq-pci.c as well.
+#include
+#include
+#include
+#include
+#include "blk-mq.h"
Is this include needed ?
You're right, I can just keep:
+#include
+#include
+#in
Hi,
On 05/04/17 18:03, Andrew Lunn wrote:
> On Wed, Apr 05, 2017 at 11:33:57AM +0300, Roger Quadros wrote:
>> Some boards [1] leave the PHYs at an invalid state
>> during system power-up or reset thus causing unreliability
>> issues with the PHY like not being detected by the mdio bus
>> or link n
Survived 10/10 reboot + ping test
Tested-by: Niklas Cassel
On 04/06/2017 10:49 AM, Joao Pinto wrote:
> This patch adds the structure stmmac_tx_queue which contains
> tx queues specific data (previously in stmmac_priv).
>
> Signed-off-by: Joao Pinto
> ---
> changes v1->v3:
> - just to keep up w
101 - 200 of 228 matches
Mail list logo