It seem to be non intentionaly changed to tx in
ms commit: adc810900a703ee78fe88fd65e086d359fec04b2
ixgbe: Refactor busy poll socket code to address multiple issues
Lock is taken from ixgbe_low_latency_recv, and there under this
lock we use ixgbe_clean_rx_irq so it looks wrong for me to increment
On Tue, 2015-12-08 at 10:12 -0800, Anjali Singhai Jain wrote:
> Update the Kconfig file with dependency for supporting GENEVE tunnel
> offloads.
>
> Signed-off-by: Anjali Singhai Jain
> Signed-off-by: Kiran Patil
Acked-by: Jeff Kirsher
> ---
> drivers/net/ethernet/intel/Kconfig | 10
On Tue, 2015-12-08 at 10:12 -0800, Anjali Singhai Jain wrote:
> This patch adds driver hooks to implement ndo_ops to add/del
> udp port in the HW to identify GENEVE tunnels.
>
> Signed-off-by: Anjali Singhai Jain
> Signed-off-by: Kiran Patil
Acked-by: Jeff Kirsher
> ---
> drivers/net/etherne
On Thu, Dec 10, 2015 at 09:23:25PM +, Alex Bennée wrote:
> Stefan Hajnoczi writes:
>
> > From: Asias He
> >
> > VM sockets virtio transport implementation. This module runs in guest
> > kernel.
>
> checkpatch warns on a bunch of whitespace/tab issues.
Will fix in the next version.
> > +st
On Thu, Dec 10, 2015 at 10:17:07AM +, Alex Bennée wrote:
> Stefan Hajnoczi writes:
>
> > From: Asias He
> >
> > This module contains the common code and header files for the following
> > virtio-vsock and virtio-vhost kernel modules.
>
> General comment checkpatch has a bunch of warnings ab
G'day Giuseppe,
On 11/12/2015 1:16 AM, Giuseppe CAVALLARO wrote:
Hi
also pls fix this typo
stmmac: create of compatible mdio bus for stmacc driver
stmmac
Will do.
On 12/9/2015 9:39 AM, Phil Reid wr
Add the rhashtable_replace_fast function. This replaces one object in
the table with another atomically. The hashes of the new and old objects
must be equal.
Signed-off-by: Tom Herbert
---
include/linux/rhashtable.h | 82 ++
1 file changed, 82 insertio
This patch implements an ILA tanslation table. This table can be
configured with identifier to locator mappings, and can be be queried
to resolve a mapping. Queries can be parameterized based on interface,
direction (incoming or outoing), and matching locator. The table is
implemented using rhasht
Create ila directory in preparation for supporting other hooks in the
kernel than LWT for doing ILA. This includes:
- Moving ila.c to ila/ila_lwt.c
- Splitting out some common functions into ila_common.c
Signed-off-by: Tom Herbert
---
net/ipv6/Makefile | 2 +-
net/ipv6/ila.c
The start callback allows the caller to set up a context for the
dump callbacks. Presumably, the context can then be destroyed in
the done callback.
Signed-off-by: Tom Herbert
---
include/linux/netlink.h | 2 ++
include/net/genetlink.h | 2 ++
net/netlink/af_netlink.c | 4
net/netlink/
In the current implementation of ILA, LWT is used to perform
translation on both the input and output paths. This is functional,
however there is a big performance hit in the receive path. Early
demux occurs before the routing lookup (a hit actually obviates the
route lookup). Therefore the stack c
Corinna Vinschen :
[...]
> It's still a bit weird. On the machines I tested this on, if I disable
> LanWake and shutdown the machine, I can send, e.g., MagicPackets as much
> as I like, the machined don't come up. Isn't it a bit misleading then
> if ethtool reports that some WoL method is enable
On 10 December 2015 at 13:43, Or Gerlitz wrote:
> On Thu, Dec 10, 2015 at 11:23 PM, Joe Stringer wrote:
>> On 10 December 2015 at 13:06, Or Gerlitz wrote:
>>> On Wed, Dec 9, 2015 at 2:22 AM, Joe Stringer wrote:
>
>> As far as the mask, I briefly discussed this with Jarno and it seems
>>
Hi,
I'm currently seeing a suspicous RCU usage warning:
===
[ INFO: suspicious RCU usage. ]
4.4.0-rc4-next-20151210 #23 Not tainted
---
drivers/net/wireless/intel/iwlwifi/mvm/sta.c:1226 suspicious
rcu_dereference_protected()
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:11 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 11/11] ixgbe: Cle
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 10/11] ixgbe: Cle
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 04/11] ixgbe: Red
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 09/11] ixgbe: Fix
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 02/11] ixgbe: Fix
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 05/11] ixgbe: Sim
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 08/11] ixgbe: Add
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 07/11] ixgbe: Reo
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 06/11] ixgbe: Add
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:10 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 03/11] ixgbe: Sim
> -Original Message-
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On
> Behalf Of Alexander Duyck
> Sent: Monday, November 02, 2015 5:09 PM
> To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH 01/11] ixgbe: Ret
Hello.
On 10/12/15 19:16, Marcel Holtmann wrote:
Hi Stefan,
From: Markus Elfring
Date: Mon, 16 Nov 2015 13:50:23 +0100
The kfree_skb() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the
On Dec 10 21:40, Francois Romieu wrote:
> Corinna Vinschen :
> [...]
> > I could do this (after I could lay my hands on such a board, that is),
> > but I'm not convinced that this makes a lot of sense for two reasons.
>
> Ok, let's get this change applied. Whatever happens should not be
> hard to
> On Dec 10, 2015, at 11:14 AM, Pablo Neira Ayuso wrote:
>
> On Tue, Dec 08, 2015 at 05:01:04PM -0800, Jarno Rajahalme wrote:
>> Define a new inline function to map conntrack status to enum
>> ip_conntrack_info. This removes the need to otherwise duplicate this
>> code in a later patch ("openvs
On Thu, Dec 10, 2015 at 11:23 PM, Joe Stringer wrote:
> On 10 December 2015 at 13:06, Or Gerlitz wrote:
>> On Wed, Dec 9, 2015 at 2:22 AM, Joe Stringer wrote:
> As far as the mask, I briefly discussed this with Jarno and it seems
> like it could be something as simple as zeroing the ip_
Add support for renaming and hard links to the fs. Most of this can be
implemented by using simple library operations under the same constraints
that we don't use a reserved name like elsewhere. Linking can be useful
to share/manage things like maps across subsystem users. It works within
the file
On 10 December 2015 at 13:06, Or Gerlitz wrote:
> On Wed, Dec 9, 2015 at 2:22 AM, Joe Stringer wrote:
>> On 8 December 2015 at 13:23, Or Gerlitz wrote:
>>> On Tue, Dec 8, 2015 at 9:20 PM, Joe Stringer wrote:
>
> Apologies for the delayed response, we haven't found anything
> interesting
Stefan Hajnoczi writes:
> From: Asias He
>
> VM sockets virtio transport implementation. This module runs in guest
> kernel.
checkpatch warns on a bunch of whitespace/tab issues.
>
> Signed-off-by: Asias He
> Signed-off-by: Stefan Hajnoczi
> ---
> v2:
> * Fix total_tx_buf accounting
> * A
On Thu, Dec 10, 2015 at 12:37 PM, Tom Herbert wrote:
> Add support for remote checksum offload in both the normal and GRO
> paths. netlinks command are used to enable sending of the Remote
> Checksum Data, and allow processing of it on receive.
>
> Signed-off-by: Tom Herbert
Tom, can you please
> 10 dec. 2015 kl. 16:37 skrev Neal Cardwell :
>
>> On Thu, Dec 10, 2015 at 1:51 AM, Per Hurtig wrote:
>>
>>> On 08 Dec 2015, at 14:47, Eric Dumazet wrote:
>>>
>>> On Tue, 2015-12-08 at 10:19 +0100, Per Hurtig wrote:
>>>
+static u32 tcp_unsent_pkts(const struct sock *sk, u32 ulimit)
>>
On Wed, Dec 9, 2015 at 2:22 AM, Joe Stringer wrote:
> On 8 December 2015 at 13:23, Or Gerlitz wrote:
>> On Tue, Dec 8, 2015 at 9:20 PM, Joe Stringer wrote:
Apologies for the delayed response, we haven't found anything
interesting yet although we've mostly looked at plain set-field
>>>
Edward Cree wrote:
> I have just realised something startling. Assuming the inner protocol uses
> the ones complement checksum in the way IP, UDP and TCP do, the outer
> checksum can be computed *without looking at the payload*. Why? Because the
> ones complement sum of (say) a correctly ch
Corinna Vinschen :
[...]
> I could do this (after I could lay my hands on such a board, that is),
> but I'm not convinced that this makes a lot of sense for two reasons.
Ok, let's get this change applied. Whatever happens should not be
hard to manage (I'm thinking about other boards or BIOSes rel
Fixe a couple of cast errors found by sparse.
Signed-off-by: Tom Herbert
---
include/net/checksum.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/include/net/checksum.h b/include/net/checksum.h
index 9fcaedf..10a16b5 100644
--- a/include/net/checksum.h
+++ b/include/net/c
Add support for remote checksum offload in both the normal and GRO
paths. netlinks command are used to enable sending of the Remote
Checksum Data, and allow processing of it on receive.
Signed-off-by: Tom Herbert
---
drivers/net/geneve.c | 162 ---
This patch set adds UDP checksum configuration via netlink and
Remote Checksum Offload for Geneve,
v2:
- Fix type in commi log
v3:
- Fix issue of taking sizeof a pointer instead of the actual object (for real)
Testing (10Gbps mlx4):
Single connection TCP_STREAM in netperf
- No UDP checks
Add support to enable and disable UDP checksums via netlink. This is
similar to how VXLAN and GUE allow this. This includes support for
enabling the UDP zero checksum (for both TX and RX).
Signed-off-by: Tom Herbert
---
drivers/net/geneve.c | 93 +-
Fixe a couple of cast errors found by sparse.
Signed-off-by: Tom Herbert
---
include/net/checksum.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/include/net/checksum.h b/include/net/checksum.h
index 9fcaedf..10a16b5 100644
--- a/include/net/checksum.h
+++ b/include/net/c
This patch set adds UDP checksum configuration via netlink and
Remote Checksum Offload for Geneve,
v2:
- Fix issue of taking sizeof a pointer instead of the actual object
- Fix type in commi log
Testing (10Gbps mlx4):
Single connection TCP_STREAM in netperf
- No UDP checksums, no RCO
Add support for remote checksum offload in both the normal and GRO
paths. netlinks command are used to enable sending of the Remote
Checksum Data, and allow processing of it on receive.
Signed-off-by: Tom Herbert
---
drivers/net/geneve.c | 162 ---
Add support to enable and disable UDP checksums via netlink. This is
similar to how VXLAN and GUE allow this. This includes support for
enabling the UDP zero checksum (for both TX and RX).
Signed-off-by: Tom Herbert
---
drivers/net/geneve.c | 93 +-
On 10/12/15 16:26, Tom Herbert wrote:
> It sounds like potentially interesting work. You'll probably want my patches
> that provider helper functions that allow a driver to verify that it can
> offload a checksum. We'll have to update those also to allow two checksums.
I have just realised somet
The problem seen is that when adding a route with a nexthop with no
via address specified, iproute2 generates bogus output:
# ip -f mpls route add 100 dev lo
# ip -f mpls route list
100 via inet 0.0.8.0 dev lo
The reason for this is that the kernel generates an RTA_VIA attribute
with the fa
If an L2 via address for an mpls nexthop is specified, the length of
the L2 address must match that expected by the output device,
otherwise it could access memory beyond the end of the via address
buffer in the route.
This check was present prior to commit f8efb73c97e2 ("mpls: multipath
route sup
These four fixes all apply to the case of having an mpls route with an
output device, but without a nexthop.
Patches 2 and 3 could really have been combined in one patch, but I
wanted to separate the fix for some recent breakage from the fix for a
day-1 issue.
Robert Shearman (4):
mpls: validat
The via address is optional for a single path route, yet is mandatory
when the multipath attribute is used:
# ip -f mpls route add 100 dev lo
# ip -f mpls route add 101 nexthop dev lo
RTNETLINK answers: Invalid argument
Make them consistent by making the via address optional when the
RTA_MU
When a via address isn't specified, the via table is left initialised
to 0 (NEIGH_ARP_TABLE), and the via address length also left
initialised to 0. This results in a via address array of length 0
being allocated (contiguous with route and nexthop array), meaning
that when a packet is sent using ne
On Tue, Dec 08, 2015 at 05:01:04PM -0800, Jarno Rajahalme wrote:
> Define a new inline function to map conntrack status to enum
> ip_conntrack_info. This removes the need to otherwise duplicate this
> code in a later patch ("openvswitch: Find existing conntrack entry
> after upcall.").
>
> Signed
On Tue, Dec 08, 2015 at 05:01:10PM -0800, Jarno Rajahalme wrote:
> - /* Call the helper right after nf_conntrack_in() for confirmed
> - * connections, but only when commiting for unconfirmed connections.
> - */
> ct = nf_ct_get(skb, &ctinfo);
> - if (ct && (nf_ct_is_confirme
In commit 2a04ae8acb14 ("hv_netvsc: remove locking in netvsc_send()"), the
locking for MSD (Multi-Send Data) field was removed. This could cause a
race condition between RNDIS control messages and data packets processing,
because these two types of traffic are not synchronized.
This patch fixes thi
Hello,
On Thu, Dec 10, 2015 at 5:24 AM, Гаврилов Игорь wrote:
> Improved CTTUPLE macro with code from sched/act_connmark.c, so it be able to
> get unNATed addresses from nf_conntrack.
Please follow the guideline to submit a formal kernel patch:
https://www.kernel.org/doc/Documentation/Submittin
On 12/10/2015 07:25 PM, David Ahern wrote:
> The VRF driver cycles netdevs when an interface is enslaved or released:
> the down event is used to flush neighbor and route tables and the up
> event (if the interface was already up) effectively moves local and
> connected routes to the proper table.
The VRF driver cycles netdevs when an interface is enslaved or released:
the down event is used to flush neighbor and route tables and the up
event (if the interface was already up) effectively moves local and
connected routes to the proper table.
As of 4f823defdd5b the local route is left hanging
Hi Stefan,
>>> From: Markus Elfring
>>> Date: Mon, 16 Nov 2015 13:50:23 +0100
>>>
>>> The kfree_skb() function tests whether its argument is NULL and then
>>> returns immediately. Thus the test around the call is not needed.
>>>
>>> This issue was detected by using the Coccinelle software.
>>>
From: Bert Kenward
Date: Thu, 10 Dec 2015 13:30:08 +
> Without this filter insertion on a VF would fail if only one channel
> was in use. This would include the unicast station filter and therefore
> no traffic would be received.
>
> Signed-off-by: Bert Kenward
> ---
> drivers/net/ethernet
On 12/10/2015 06:18 AM, Otto Sabart wrote:
*) Is irqbalance disabled and the IRQs set the same each time, or might
there be variability possible there? Each of the five netperf runs will be
a different four-tuple which means each may (or may not) get RSS hashed/etc
differently.
The irqbalance
From: Eric Dumazet
> Sent: 10 December 2015 15:58
>
> BTW, are you even using IPv6 SCTP sessions ?
Our M3UA/SCTP protocol stack supports them and defaults to using
IPv6 listening sockets for IPv4 connections.
I very much doubt than any customers have used them yet.
So most of the IPv6 connections
Hi
also pls fix this typo
stmmac: create of compatible mdio bus for stmacc driver
stmmac
On 12/9/2015 9:39 AM, Phil Reid wrote:
The DSA driver needs to be passed a reference to an mdio bus. Typically
th
The file ila.h used for lightweight tunnels is being used by iproute2
but is not exported yet.
Signed-off-by: Stephen Hemminger
---
include/uapi/linux/Kbuild | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/uapi/linux/Kbuild b/include/uapi/linux/Kbuild
index 628e6e6..c2e5d6c 100644
--
On Tue, 01 Dec 2015 01:17:06 +0300
Konstantin Khlebnikov wrote:
> Though dumping such entries crashes present kernels.
>
> Signed-off-by: Konstantin Khlebnikov
Applied thanks, but I nobody will use it until after your
kernel patch makes it to stable.
--
To unsubscribe from this list: send the
On Wed, 2 Dec 2015 00:25:36 +0100
Daniel Borkmann wrote:
> Improve example files further and add a more generic set of possible
> helpers for them that can be used.
>
> Signed-off-by: Daniel Borkmann
Sure, applied to net-next
--
To unsubscribe from this list: send the line "unsubscribe netdev
On Mon, 30 Nov 2015 14:57:28 -0800
Tom Herbert wrote:
> This patch:
> - Adds a utility function for parsing a 64 bit address
> - Adds a utility function for converting a 64 bit address to ASCII
> - Adds and ILA encap type in lwt tunnels
>
> Signed-off-by: Tom Herbert
Looks good applied.
Yo
On Tue, 8 Dec 2015 12:24:44 -0800
David Ahern wrote:
> Currently, the table id for VRF devices requires an integer. Convert
> it to use rtnl_rttable_a2n which handles table names from the iproute2
> directory.
>
> This also fixes a bug in the original commit where table name are not
> properly
On Wed, 2 Dec 2015 12:50:22 +
Phil Sutter wrote:
> Older kernels use -1 internally as indicator to use the sysctl default,
> but they still export the setting. Newer kernels use 0 to indicate that
> (which is why the conversion from -1 to 0 was done here), but they also
> stopped exporting th
On Thu, 3 Dec 2015 17:13:48 +0100
Nicolas Dichtel wrote:
> There is two variables named 'len' in rtnl_talk. In fact, commit
> c079e121a73a didn't work. For example, it was possible to trigger
> a seg fault with this command:
> $ ip link set gre2 type ip6gre hoplimit 32
>
> Let's rename the argu
On Thu, 10 Dec 2015, Jens Axboe wrote:
> It's a balance, as we also should not make APIs out of everything. As I said,
> purely my opinion, but I think the is_last/is_first have jumped the shark.
I don't have a strong opinion either way.
What I think we should do though, is to either have both (
On Thu, 10 Dec 2015 13:24:51 +0100
Phil Sutter wrote:
> Just a typo there, it's spelled correctly in SEE ALSO section..
>
> Signed-off-by: Phil Sutter
Applied
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordo
On 12/9/15 6:35 PM, David Ahern wrote:
The VRF driver cycles netdevs when an interface is enslaved or released:
the down event is used to flush neighbor and route tables and the up
event (if the interface was already up) effectively moves local and
connected routes to the proper table.
As of 4f8
On Thu, Dec 10, 2015 at 7:49 AM, Edward Cree wrote:
> On 09/12/15 18:00, Tom Herbert wrote:
>> That is not at all true. If the stack has set up VXLAN RCO and the > device
>> decides to set the inner checksum itself then the checksum > will be bad.
>> The checksum interface is very specific pleas
On Thu, 2015-12-10 at 12:26 +, David Laight wrote:
> Yes, I'm worried about whether our M3UA code is likely to crash customer
> systems, not whether hostile applications can crash it.
> These boxes ought to be on private networks since the sigtran protocols
> themselves have nothing that even
On 09/12/15 18:00, Tom Herbert wrote:
> That is not at all true. If the stack has set up VXLAN RCO and the > device
> decides to set the inner checksum itself then the checksum > will be bad. The
> checksum interface is very specific please read it > carefully (sk_buff.h),
> if the driver/device
On Thu, Dec 10, 2015 at 09:23:57AM -0600, Josh Poimboeuf wrote:
> Personally I would disagree. Something like:
>
> if (list_is_first(&rq->queuelist, &nd->queue))
>
> is much more readable to me than:
>
> if (rq->queuelist.prev == &nd->queue)
>
> The first one takes no effort for me -- it's
On 12/10/2015 08:23 AM, Josh Poimboeuf wrote:
On Thu, Dec 10, 2015 at 08:10:34AM -0700, Jens Axboe wrote:
On 12/10/2015 07:17 AM, Geliang Tang wrote:
We already have list_is_last(), it makes sense to also add
list_is_first() for consistency. This list utility function
to check for first element
On Thu, Dec 10, 2015 at 1:51 AM, Per Hurtig wrote:
>
>> On 08 Dec 2015, at 14:47, Eric Dumazet wrote:
>>
>> On Tue, 2015-12-08 at 10:19 +0100, Per Hurtig wrote:
>>
>>> +static u32 tcp_unsent_pkts(const struct sock *sk, u32 ulimit)
>>> +{
>>> +struct sk_buff *skb = tcp_send_head(sk);
>>> +
On Thu, Dec 10, 2015 at 08:10:34AM -0700, Jens Axboe wrote:
> On 12/10/2015 07:17 AM, Geliang Tang wrote:
> >We already have list_is_last(), it makes sense to also add
> >list_is_first() for consistency. This list utility function
> >to check for first element in a list.
>
> Honestly, I think we a
Mark address pointer with __iomem in the IO accessors.
Otherwise we will get a sparse complain like following
.../hns/hns_dsaf_reg.h:991:36: warning: incorrect type in argument 1 (different
address spaces)
.../hns/hns_dsaf_reg.h:991:36:expected unsigned char [noderef] [usertype]
*base
.../h
printf() has a dedicated specifier to print MAC addresses. Use it instead of
pushing each byte via stack.
Signed-off-by: Andy Shevchenko
---
drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 49 +++---
1 file changed, 14 insertions(+), 35 deletions(-)
diff --git a/drivers/net
From: Maor Gottlieb
Rename en_flow_table.c to en_fs.c in order to be aligned
with the new flow steering files.
Signed-off-by: Maor Gottlieb
Signed-off-by: Moni Shoua
Signed-off-by: Matan Barak
Signed-off-by: Saeed Mahameed
---
drivers/net/ethernet/mellanox/mlx5/core/Makefile |2 +-
..
From: Maor Gottlieb
Introduce new Flow Steering (FS) firmware commands,
in-order to support the new flow steering infrastructure.
Signed-off-by: Maor Gottlieb
Signed-off-by: Moni Shoua
Signed-off-by: Matan Barak
Signed-off-by: Saeed Mahameed
---
drivers/net/ethernet/mellanox/mlx5/core/Makef
Under SRIOV there might be a case where VFs are loaded
without pre-assigned MAC address. In this case, the VF
will randomize its own MAC. This will address the case
of administrator not assigning MAC to the VF through
the PF OS APIs and keep udev happy.
Signed-off-by: Saeed Mahameed
---
drivers
From: Maor Gottlieb
Introduce the flow steering mlx5_flow_namespace (Namespace)
and fs_prio (Flow Steering Priority) tree nodes.
Namespaces are used in order to isolate different usages or types
of steering (for example, downstream patches will add a different
namespaces for the NIC driver and f
From: Maor Gottlieb
Expose the new flow steering API and remove the old
one.
Few changes are required:
1. The Ethernet flow steering follows the existing implementation, but uses
the new steering API. The old flow steering implementation is removed.
2. Move the E-switch FDB management to use t
From: Maor Gottlieb
Introducing the following objects:
mlx5_flow_root_namespace: represent the root of specific flow table
type tree(e.g NIC receive, FDB, etc..)
mlx5_flow_group: define the mask of the flow specification.
fs_fte(flow steering flow table entry): defines the value of the
flow sp
Hi Dave,
First two patches fixes some minor issues in recently
introduced SRIOV code.
The other seven patches modifies the driver's code that
manages flow steering rules with Connectx-4 devices.
Basic introduction:
The flow steering device specification model is composed of the following
enti
From: Maor Gottlieb
Flow steering initialization is based on static tree which
illustrates the flow steering tree when the driver is loaded. The
initialization considers the max supported flow table level of the device,
a minimum of 2 kernel flow tables(vlan and mac) are required to have
kernel f
From: Maor Gottlieb
Introducing the base data structure and its operations that are
going to represent ConnectX-4 Flow Steering, this data structure
is basically a tree and all Flow steering objects such as
(Flow Table/Flow Group/FTE/etc ..) are represented as fs_node(s).
fs_node is the base obj
E-Switch capabilities should be queried only if E-Switch flow table
is supported and not only when vport group manager.
Fixes: d753c6e8 ("net/mlx5: E-Switch, Introduce HCA cap and E-Switch vport
context")
Signed-off-by: Saeed Mahameed
---
drivers/net/ethernet/mellanox/mlx5/core/fw.c |2
On 12/10/2015 07:17 AM, Geliang Tang wrote:
We already have list_is_last(), it makes sense to also add
list_is_first() for consistency. This list utility function
to check for first element in a list.
Honestly, I think we already have way too many of these kind of helpers.
IMHO they don't real
Hi Sascha,
On jeu., déc. 10 2015, Sascha Hauer wrote:
> Hi Gregory,
>
> On Wed, Dec 09, 2015 at 06:49:43PM +0100, Gregory CLEMENT wrote:
>> With device tree it is no more possible to reset the PHY at board
>> level. Furthermore, doing in the driver allow to power down the PHY when
>> the netwo
For better readability, use list_is_{first,last}() instead of open-coded.
Signed-off-by: Geliang Tang
---
block/noop-iosched.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/noop-iosched.c b/block/noop-iosched.c
index a163c48..d44326e 100644
--- a/block/noop-iosche
We already have list_is_last(), it makes sense to also add
list_is_first() for consistency. This list utility function
to check for first element in a list.
Signed-off-by: Geliang Tang
---
include/linux/list.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/include/linux/list.h
For better readability, use list_is_first() instead of open-coded.
Signed-off-by: Geliang Tang
---
kernel/livepatch/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c
index bc2c85c..be64106 100644
--- a/kernel/livepatch/co
For better readability, use list_is_first() instead of open-coded.
Signed-off-by: Geliang Tang
---
net/netfilter/ipset/ip_set_list_set.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/netfilter/ipset/ip_set_list_set.c
b/net/netfilter/ipset/ip_set_list_set.c
index bbede9
Hi Rick,
> *) It is good to be binding netperf and netserver - helps with
> reproducibility, but why the two -T options? A brief look at src/netsh.c
> suggests it will indeed set the two binding options separately but that is
> merely a side-effect of how I wrote the code. It wasn't an intentio
Hi Tilman,
On 12/09/2015 03:10 AM, Tilman Schmidt wrote:
> Am 09.12.2015 um 00:12 schrieb Paul Bolle:
>
>>> --- a/drivers/isdn/gigaset/ser-gigaset.c
>>> +++ b/drivers/isdn/gigaset/ser-gigaset.c
>>> @@ -370,19 +370,23 @@ static void gigaset_freecshw(struct cardstate
>>> *cs)
>>> tasklet_kill(&
On Friday 06 November 2015 09:32:46 Greg KH wrote:
> On Thu, Nov 05, 2015 at 11:18:37AM +0100, Patrick Schaaf wrote:
> > bnx2x: Don't notify about scratchpad parities
> >
> > This is a (trivial) "backport" of ad6afbe9578d1fa26680faf78c846bd8c00d1d6e
> > to stable kernel 3.14.56.
>
> This patch is
1 - 100 of 115 matches
Mail list logo
