Usually an skb does not have up to MAX_SKB_FRAGS frags. So no need to
initialize the unuse part of sg. This patch initialize the sg based on
the real number it will used:
- during xmit, it could be inferred from nr_frags and can_push.
- for small receive buffer, it will also be 2.
Cc: Michael S.
Add netlink interface to configure Geneve UDP port number.
So that user can configure it for a Gevene device.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
Acked-by: John W. Linville
---
drivers/net/geneve.c | 25 +
include/uap
Introduce function udp_tun_rx_dst() to initialize tunnel dst on
receive path.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
---
drivers/net/vxlan.c| 29 ++--
include/net/dst_metadata.h | 61
With help of tunnel metadata mode OVS can directly use
Geneve devices to implement Geneve tunnels.
This patch removes all of the OVS specific Geneve code
and make OVS use a Geneve net_device. Basic geneve vport
is still there to handle compatibility with current
userspace application.
Signed-off-b
geneve_core module handles send and receive functionality.
This way OVS could use the Geneve API. Now with use of
tunnel meatadata mode OVS can directly use Geneve netdevice.
So there is no need for separate module for Geneve. Following
patch consolidates Geneve protocol processing in single module
Following patch create new tunnel flag which enable
tunnel metadata collection on given device. These devices
can be used by tunnel metadata based routing or by OVS.
Geneve Consolidation patch get rid of collect_md_tun to
simplify tunnel lookup further.
Signed-off-by: Pravin B Shelar
Reviewed-by:
On packet transmit path geneve need to lookup route. Following
patch improves route lookup using more parameters.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
Acked-by: John W. Linville
---
drivers/net/geneve.c |3 +++
1 files changed, 3 insertions(+), 0 d
This change simplifies Geneve Tunnel hash table management.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Reviewed-by: John W. Linville
---
drivers/net/geneve.c | 43 +--
1 files changed, 17 insertions(+), 26 deletions(-)
diff --git a/driver
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
Acked-by: John W. Linville
---
drivers/net/geneve.c |4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c
index 897e1a3..95e9da0 100644
--- a/drivers/
Following patches adds support for Geneve tunnel metadata
mode. OVS can make use of Geneve net-device with tunnel
metadata API from kernel.
This also allows us to consolidate Geneve implementation
from two kernel modules geneve_core and geneve to single
geneve module. geneve_core module was target
Thu, Aug 27, 2015 at 08:36:03AM CEST, da...@davemloft.net wrote:
>From: Jiri Pirko
>Date: Thu, 27 Aug 2015 08:27:04 +0200
>
>> I'm not saying it is not possible, it certainly is. But I think that
>> for example rocker internals have no value to default user, he
>> should not care and he cannot fin
From: Jiri Pirko
Date: Thu, 27 Aug 2015 08:27:04 +0200
> I'm not saying it is not possible, it certainly is. But I think that
> for example rocker internals have no value to default user, he
> should not care and he cannot find out what is going on there
> without knowledge or rocker.c code. The
This enables bridge vlan_protocol to be configured through netlink.
When CONFIG_BRIDGE_VLAN_FILTERING is disabled, kernel behaves the
same way as this feature is not implemented.
Signed-off-by: Toshiaki Makita
---
v2: Fix u16 to __be16
include/uapi/linux/if_link.h | 1 +
net/bridge/br_netlink
Thu, Aug 27, 2015 at 08:23:13AM CEST, sfel...@gmail.com wrote:
>On Wed, Aug 26, 2015 at 10:43 PM, Jiri Pirko wrote:
>> Wed, Aug 26, 2015 at 07:43:18PM CEST, sfel...@gmail.com wrote:
>>>On Wed, Aug 26, 2015 at 9:36 AM, Jiri Pirko wrote:
From: Jiri Pirko
Add this helper so code can
Thu, Aug 27, 2015 at 08:23:13AM CEST, sfel...@gmail.com wrote:
>On Wed, Aug 26, 2015 at 10:43 PM, Jiri Pirko wrote:
>> Wed, Aug 26, 2015 at 07:43:18PM CEST, sfel...@gmail.com wrote:
>>>On Wed, Aug 26, 2015 at 9:36 AM, Jiri Pirko wrote:
From: Jiri Pirko
Add this helper so code can
%s specifier makes bpf program and kernel debugging easier.
To make sure that trace_printk won't crash the unsafe string
is copied into stack and unsafe pointer is substituted.
String is also sanitized for printable characters.
The cost of swapping FS in probe_kernel_read is
amortized over 4 chars
Thu, Aug 27, 2015 at 08:01:15AM CEST, da...@davemloft.net wrote:
>From: Jiri Pirko
>Date: Thu, 27 Aug 2015 07:40:04 +0200
>
>> Switch object itselt would not help you to expose rocker internals. I
>> don't think that you can find generic way, same for all drivers, to
>> expose internal tables and
On Wed, Aug 26, 2015 at 10:43 PM, Jiri Pirko wrote:
> Wed, Aug 26, 2015 at 07:43:18PM CEST, sfel...@gmail.com wrote:
>>On Wed, Aug 26, 2015 at 9:36 AM, Jiri Pirko wrote:
>>> From: Jiri Pirko
>>>
>>> Add this helper so code can easily figure out if netdev is openswitch.
>>>
>>> Signed-off-by: Jir
From: Jiri Pirko
Date: Thu, 27 Aug 2015 07:40:04 +0200
> Switch object itselt would not help you to expose rocker internals. I
> don't think that you can find generic way, same for all drivers, to
> expose internal tables and stuff. That is hw specific.
Tables are datastructures with names and t
This enables bridge vlan_protocol to be configured through netlink.
When CONFIG_BRIDGE_VLAN_FILTERING is disabled, kernel behaves the
same way as this feature is not implemented.
Signed-off-by: Toshiaki Makita
---
include/uapi/linux/if_link.h | 1 +
net/bridge/br_netlink.c | 34 ++
The updated NV document with the new experiments and a table with all the
experimental results are now available at
(http://www.brakmo.org/networking/tcp-nv/TCPNV.html).
- Lawrence
On 8/25/15, 4:33 PM, "Lawrence Brakmo" wrote:
>Changes from v5: cleaning of NV code, changing some default param
Wed, Aug 26, 2015 at 07:43:18PM CEST, sfel...@gmail.com wrote:
>On Wed, Aug 26, 2015 at 9:36 AM, Jiri Pirko wrote:
>> From: Jiri Pirko
>>
>> Add this helper so code can easily figure out if netdev is openswitch.
>>
>> Signed-off-by: Jiri Pirko
>> ---
>> include/linux/netdevice.h| 8
Wed, Aug 26, 2015 at 07:24:55PM CEST, f.faine...@gmail.com wrote:
>On 26/08/15 09:36, Jiri Pirko wrote:
>> From: Jiri Pirko
>>
>> Add this helper so code can easily figure out if netdev is openswitch.
>>
>> Signed-off-by: Jiri Pirko
>> ---
>> include/linux/netdevice.h| 8
>
Wed, Aug 26, 2015 at 08:21:59PM CEST, sfel...@gmail.com wrote:
>On Wed, Aug 26, 2015 at 10:49 AM, David Miller wrote:
>> From: Jiri Pirko
>> Date: Wed, 26 Aug 2015 09:37:57 +0200
>>
>>> I don't think that are much more cases like this. Therefore I think that
>>> for this cases, debugfs might be a
This adds support for 3 new PCI device combinations -
1077:16a1, 1077:16a4 and 1077:16ad.
Signed-off-by: Yuval Mintz
---
Hi Dave,
Please consider applying this to 'net-next'.
Thanks,
Yuval
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 3 +++
1 file changed, 3 insertions(+)
diff --git
On 8/26/15, 4:33 AM, Nikolay Aleksandrov wrote:
On Aug 25, 2015, at 11:06 PM, David Miller wrote:
From: Nikolay Aleksandrov
Date: Tue, 25 Aug 2015 22:28:16 -0700
Certainly, that should be done and I will look into it, but the
essence of this patch is a bit different. The problem here is not
On Wed, Aug 26, 2015 at 8:57 PM, Pravin Shelar wrote:
> On Wed, Aug 26, 2015 at 8:08 PM, Jesse Gross wrote:
>> On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote:
>>> @@ -293,15 +615,13 @@ static netdev_tx_t geneve_xmit(struct sk_buff *skb,
>>> struct net_device *dev)
>> [...]
>>> +
On Wed, Aug 26, 2015 at 8:08 PM, Jesse Gross wrote:
> On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote:
>> diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c
>> index d05150c..a36a1de 100644
>> --- a/drivers/net/geneve.c
>> +++ b/drivers/net/geneve.c
>> @@ -138,16 +164,18 @@ static vo
Commit 62ee783bf1f8 ("smsc911x: Fix crash seen if neither ACPI nor OF is
configured or used") introduces an error check for the return value from
device_get_phy_mode() and bails out if there is an error. Unfortunately,
there are configurations where no phy is configured. Those configurations
now fa
Return -ENXIO if device property array access functions don't find
a suitable firmware interface.
This lets drivers decide if they should use available platform data
instead.
Cc: Rafael J. Wysocki
Signed-off-by: Guenter Roeck
---
v2: Added patch
v3: Document that device_property_read_string_arr
On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote:
> diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c
> index d05150c..a36a1de 100644
> --- a/drivers/net/geneve.c
> +++ b/drivers/net/geneve.c
> @@ -138,16 +164,18 @@ static void geneve_rx(struct geneve_sock *gs, struct
> sk_buff *skb)
On Wed, 2015-08-26 at 19:12 -0700, Eric Dumazet wrote:
> On Thu, 2015-08-27 at 08:56 +0800, chenweil...@huawei.com wrote:
> > From: Weilong Chen
> >
> > When try to merge several skbs to prior one, if the frag_list is
> > used and the the last one is a small packet, once the condition
> > "len <=
On Thu, 2015-08-27 at 08:56 +0800, chenweil...@huawei.com wrote:
> From: Weilong Chen
>
> When try to merge several skbs to prior one, if the frag_list is
> used and the the last one is a small packet, once the condition
> "len <= skb_tailroom(to)" is satisfied, we will get a wrong
> packet!
> Th
On Wed, Aug 26, 2015 at 2:54 PM, Pravin B Shelar wrote:
> Following patch create new tunnel flag which enable
> tunnel metadata collection on given device. These devices
> can be used by tunnel metadata based routing or by OVS.
> Geneve Consolidation patch get rid of collect_md_tun to
> simplify t
Very clear explanation，thank you! :-)
Zhangjie
-Original Message-
From: Eric Dumazet [mailto:eric.duma...@gmail.com]
Sent: Wednesday, August 26, 2015 9:10 PM
To: Zhangjie (HZ)
Cc: Jason Wang; netdev@vger.kernel.org; Qinchuanyu; Yewudi; liuyongan 00175866;
Wangbicheng; Yan Chen
Subject:
From: Weilong Chen
When try to merge several skbs to prior one, if the frag_list is
used and the the last one is a small packet, once the condition
"len <= skb_tailroom(to)" is satisfied, we will get a wrong
packet!
This patch just check frag_lists before the condtion to prevent
this from happeni
On Wednesday, August 26, 2015 04:25:59 PM Guenter Roeck wrote:
> On 08/26/2015 04:37 PM, Rafael J. Wysocki wrote:
> > On Wednesday, August 26, 2015 01:20:44 PM Guenter Roeck wrote:
> >> Return -ENXIO if device property array access functions don't find
> >> a suitable firmware interface.
> >>
> >>
From: Alexei Starovoitov
Date: Wed, 26 Aug 2015 15:57:38 -0700
> Make sure to indicate to tunnel driver that key.tun_id is set,
> otherwise gre won't recognize the metadata.
>
> Fixes: d3aa45ce6b94 ("bpf: add helpers to access tunnel metadata")
> Signed-off-by: Alexei Starovoitov
> ---
> With t
From: David Ahern
Date: Wed, 26 Aug 2015 12:36:15 -0700
> As such all we need is to push an eth header to the front of the skb
> for 1 loop through the stack and eth_header via dev_hard_header with
> NULL daddr is the simplest path to accomplish that. Any other path is
> just extra overhead.
And
From: Scott Feldman
Date: Wed, 26 Aug 2015 11:21:59 -0700
> Using another netdev doesn't seem right. Do we need a new device
> class for switches, and then create some common tool/interfaces for
> switch device class?
This is probably what we will need to do.
There has also been a discussion l
On Wed, Aug 26, 2015 at 5:08 PM, Stephen Hemminger
wrote:
> On Wed, 26 Aug 2015 15:41:26 -0700
> Cong Wang wrote:
>
>> Currently there is no check for if a qdisc is appropriate
>> to be used as the default qdisc. This causes we get no
>> error even we set the default qdisc to an inappropriate one
On Wed, 26 Aug 2015 15:41:26 -0700
Cong Wang wrote:
> Currently there is no check for if a qdisc is appropriate
> to be used as the default qdisc. This causes we get no
> error even we set the default qdisc to an inappropriate one
> but an error will be shown up later. This is not good.
>
> Also
On Wed, Aug 26, 2015 at 08:41:50PM +0200, Pablo Neira Ayuso wrote:
> On Fri, Aug 21, 2015 at 09:23:38AM -0700, Simon Horman wrote:
> > Hi Pablo,
> >
> > please consider these IPVS Updates for v4.3.
> >
> > I realise these are a little late in the cycle, so if you would prefer
> > me to repost the
On 08/26/2015 04:37 PM, Rafael J. Wysocki wrote:
On Wednesday, August 26, 2015 01:20:44 PM Guenter Roeck wrote:
Return -ENXIO if device property array access functions don't find
a suitable firmware interface.
This lets drivers decide if they should use available platform data
instead.
Cc: Raf
On Wednesday, August 26, 2015 01:20:44 PM Guenter Roeck wrote:
> Return -ENXIO if device property array access functions don't find
> a suitable firmware interface.
>
> This lets drivers decide if they should use available platform data
> instead.
>
> Cc: Rafael J. Wysocki
> Signed-off-by: Guent
Make sure to indicate to tunnel driver that key.tun_id is set,
otherwise gre won't recognize the metadata.
Fixes: d3aa45ce6b94 ("bpf: add helpers to access tunnel metadata")
Signed-off-by: Alexei Starovoitov
---
With this fix both vxlan and gretap are working with tc+bpf.
net/core/filter.c |
On Wed, 2015-08-26 at 15:49 -0700, Jeff Kirsher wrote:
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
[]
> @@ -2106,11 +2106,12 @@ int i40e_ndo_set_vf_port_vlan(struct net_device
> *netdev,
[]
> dev_err(&pf->
From: Shannon Nelson
The port.crc_errors is really an RX counter, so let's mark it as such.
Change-ID: I179afd3f8a95d45229bb4163a6aeb01f0d2d250b
Signed-off-by: Shannon Nelson
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
1 fil
From: Greg Rose
The existing comment is incorrect. Add new comment to point out that the
PF reset does not affect link but if the reset is changed to a different
type that does affect link then the link test would need to be moved to
before the reset.
Change-ID: I28d786f46e9465860babdee61c1dba5
From: Mitch Williams
Sparse cries when we compare an __le16 to a u16, almost like it cares
about architectures other than x86. Weird. Use the le16_to_cpu macro to
make it stop crying.
Change-ID: Id068f4d7868a2d3df234a791a76d15938f37db35
Signed-off-by: Mitch Williams
Tested-by: Andrew Bowers
Si
From: Mitch Williams
Turns out that 'inavlid' is an inavlid spelling for 'invalid'.
Change-ID: Ie1fe2d0f8d1ba75ab880594875ec2e4152a76f61
Signed-off-by: Mitch Williams
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
1 file change
This series contains updates to i40e and i40evf only.
Anjali provides a fix for i40e where the part is not receiving multicast
or VLAN tagged packets when in promiscuous mode. This can occur when a
software bridge is created on top of the device. Fixed the legacy and MSI
interrupt mode in the dr
From: Anjali Singhai Jain
Use CTLN1 instead of CTLN for the VF relative register space.
Change-ID: Iefba63faf0307af55fec8dbb64f26059f7d91318
Signed-off-by: Anjali Singhai Jain
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40evf/i40e_txrx.c | 28 ++
From: Greg Rose
Treat netqueues the same way we do virtual functions when someone wants to
run the ethtool offline diagnostic test.
Change-ID: Id48d2b933f1fd0db7be06305a93c6ebe3dc821f5
Signed-off-by: Greg Rose
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/inte
From: Greg Rose
The kernel notifies all VXLAN capable registered drivers, i.e. any
driver that implements ndo_add_vxlan_port(), of the addition of a
port so that the driver can track which ports are in use. There's
no need to log this - it just fills the system log with useless and
irksome noise
From: Anjali Singhai Jain
Add virtual channel offload capability to support RX polling mode in the
VF.
Change-ID: Ib643ae2a7506dfc75fc489fc207493fabefa4832
Signed-off-by: Jingjing Wu
Signed-off-by: Anjali Singhai Jain
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethe
From: Anjali Singhai Jain
This patch fixes the driver flow to take into account legacy interrupts.
Over time we added code that assumes MSIX is the only mode that the
driver runs in. It also enables a legacy workaround to trigger SWINT
when the TX ring has non-cache aligned descriptors pending an
From: Catherine Sullivan
We should be stopping the service task and flow director on
shutdown not on suspension.
Signed-off-by: Catherine Sullivan
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_main.c | 16 +---
1 file changed, 13 in
From: Catherine Sullivan
Bump version and update the copyright year for i40evf.
Change-ID: Iddb81b9dba09f0dc57ab54937b5821ecdd721ff6
Signed-off-by: Catherine Sullivan
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
drivers/net/
From: Neerav Parikh
Store the CEE TLV status returned by firmware to allow drivers to dump that
for debug purposes.
Change-ID: Ie3c4cf8cebabee4f15e1e3fdc4fc8a68bbca40ee
Signed-off-by: Neerav Parikh
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_
From: Neerav Parikh
This patch adds capability to update per VEB per TC statistics and dump
it via ethtool. It also adds a structure to hold VEB per TC statistics.
The fields can be filled by reading the GLVEBTC_* counters.
Change-ID: I28b4759b9ab6ad5a61f046a1bc9ef6b16fe31538
Signed-off-by: Neer
It is just another static flag which can be moved.
Cc: Jamal Hadi Salim
Signed-off-by: Cong Wang
---
include/net/sch_generic.h | 2 +-
net/sched/sch_api.c | 6 +++---
net/sched/sch_mq.c| 2 +-
net/sched/sch_mqprio.c| 2 +-
4 files changed, 6 insertions(+), 6 deletions(-)
diff
For those static flags, that is never changed dynamically,
we could just move them into qdisc->ops. This will be used
by the following patches.
Cc: Jamal Hadi Salim
Signed-off-by: Cong Wang
---
include/net/sch_generic.h | 3 ++-
net/sched/sch_api.c | 4 ++--
net/sched/sch_generic.c |
The main goal of this patchset is to improve the behavior of setting
the default qdisc. Current behavior has no error check, no check for
ingress and _can_ crash the kernel with some buggy implementation.
We only have flags for each instance of qdisc's, for flags like
if a qdisc is a fifo qdisc, t
Cc: Jamal Hadi Salim
Cc: Stephen Hemminger
Signed-off-by: Cong Wang
---
include/net/sch_generic.h | 2 +-
net/sched/sch_api.c | 11 +--
net/sched/sch_ingress.c | 1 +
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/include/net/sch_generic.h b/include/net/sch_gen
Currently there is no check for if a qdisc is appropriate
to be used as the default qdisc. This causes we get no
error even we set the default qdisc to an inappropriate one
but an error will be shown up later. This is not good.
Also, for qdisc's like HTB, kernel will just crash when
we use it as d
Relying on its name is a bad practice.
Cc: Jamal Hadi Salim
Signed-off-by: Cong Wang
---
include/net/sch_generic.h | 1 +
net/sched/sch_fifo.c | 6 --
net/sched/sch_generic.c | 1 +
3 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/include/net/sch_generic.h b/include/net
On 08/26/15 at 06:19pm, Jiri Benc wrote:
> might be a noise. However, there's definitely room for performance
> improvement here, the lwtunnel vxlan throughput is at about ~40% of the
> non-vxlan throughput. I did not spend too much time on analyzing this, yet,
> but it's clear the dst_entry layout
On 08/26/2015 11:54 PM, Alexei Starovoitov wrote:
On 8/26/15 2:00 PM, Daniel Borkmann wrote:
...
+reset:
+if (unlikely(limit++ >= MAX_REC_LOOP)) {
+net_notice_ratelimited("%s: reclassify loop, rule prio %u, "
+ "protocol %02x\n", tp->q->ops->id,
+
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
Acked-by: John W. Linville
---
drivers/net/geneve.c |4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/drivers/net/geneve.c b/drivers/net/geneve.c
index 897e1a3..95e9da0 100644
--- a/drivers/
Add netlink interface to configure Geneve UDP port number.
So that user can configure it for a Gevene device.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
Acked-by: John W. Linville
---
drivers/net/geneve.c | 25 +
include/uap
Following patch create new tunnel flag which enable
tunnel metadata collection on given device. These devices
can be used by tunnel metadata based routing or by OVS.
Geneve Consolidation patch get rid of collect_md_tun to
simplify tunnel lookup further.
Signed-off-by: Pravin B Shelar
---
v3-v4:
-
On packet transmit path geneve need to lookup route. Following
patch improves route lookup using more parameters.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
Acked-by: John W. Linville
---
drivers/net/geneve.c |3 +++
1 files changed, 3 insertions(+), 0 d
With help of tunnel metadata mode OVS can directly use
Geneve devices to implement Geneve tunnels.
This patch removes all of the OVS specific Geneve code
and make OVS use a Geneve net_device. Basic geneve vport
is still there to handle compatibility with current
userspace application.
Signed-off-b
geneve_core module handles send and receive functionality.
This way OVS could use the Geneve API. Now with use of
tunnel meatadata mode OVS can directly use Geneve netdevice.
So there is no need for separate module for Geneve. Following
patch consolidates Geneve protocol processing in single module
Following patches adds support for Geneve tunnel metadata
mode. OVS can make use of Geneve net-device with tunnel
metadata API from kernel.
This also allows us to consolidate Geneve implementation
from two kernel modules geneve_core and geneve to single
geneve module. geneve_core module was target
Introduce function udp_tun_rx_dst() to initialize tunnel dst on
receive path.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Acked-by: Thomas Graf
---
drivers/net/vxlan.c| 29 ++--
include/net/dst_metadata.h | 61
This change simplifies Geneve Tunnel hash table management.
Signed-off-by: Pravin B Shelar
Reviewed-by: Jesse Gross
Reviewed-by: John W. Linville
---
Rebased on updated patches.
---
drivers/net/geneve.c | 43 +--
1 files changed, 17 insertions(+), 26 d
On 8/26/15 2:00 PM, Daniel Borkmann wrote:
For classifiers getting invoked via tc_classify(), we always need an
extra function call into tc_classify_compat(), as both are being
exported as symbols and tc_classify() itself doesn't do much except
handling of reclassifications when tp->classify() re
* Guenter Roeck [150826 13:58]:
> Hi Tony,
>
> On 08/26/2015 01:16 PM, Tony Lindgren wrote:
> [ ... ]
>
> >>We may need two separate patches, one to fix up device_property_read_u32()
> >>to return -ENXIO, and one to fix smsc911x_probe_config() to ignore the error
> >>from device_get_phy_mode(),
* Guenter Roeck [150826 13:24]:
> Commit 62ee783bf1f8 ("smsc911x: Fix crash seen if neither ACPI nor OF is
> configured or used") introduces an error check for the return value from
> device_get_phy_mode() and bails out if there is an error. Unfortunately,
> there are configurations where no phy i
On 08/26/2015 05:09 PM, lucien xin wrote:
> On Thu, Aug 27, 2015 at 4:59 AM, Marcelo Ricardo Leitner
> wrote:
>> On Wed, Aug 26, 2015 at 04:42:21PM -0400, Vlad Yasevich wrote:
>>> On 08/26/2015 04:35 PM, Xin Long wrote:
in sctp_process_asconf(), we get address parameter from the beginning of
On 08/26/2015 05:03 PM, Xin Long wrote:
> in sctp_process_asconf(), we get address parameter from the beginning of
> the addip params. but we never check if it's really there. if the addr
> param is not there, it still can pass sctp_verify_asconf(), then to be
> handled by sctp_process_asconf(), it
On Mon, Aug 24, 2015 at 10:43:15AM -0700, Pravin B Shelar wrote:
> This change simplifies Geneve Tunnel hash table management.
>
> Signed-off-by: Pravin B Shelar
> Reviewed-by: Jesse Gross
Reviewed-by: John W. Linville
--
John W. LinvilleSomeday the world will need a hero, an
Corcodel Marian :
> Actually from r810x_pll_power_up function i removed function
> r810x_phy_power_up because is two situation.One run from rtl8169_phy_reset
> wich already power on interface after reset MII_BMCR and two i placed
> supplementary on __rtl8169_resume.
r810x_phy_power_up and r
On Thu, Aug 27, 2015 at 4:59 AM, Marcelo Ricardo Leitner
wrote:
> On Wed, Aug 26, 2015 at 04:42:21PM -0400, Vlad Yasevich wrote:
>> On 08/26/2015 04:35 PM, Xin Long wrote:
>> > in sctp_process_asconf(), we get address parameter from the beginning of
>> > the addip params. but we never check if it'
in sctp_process_asconf(), we get address parameter from the beginning of
the addip params. but we never check if it's really there. if the addr
param is not there, it still can pass sctp_verify_asconf(), then to be
handled by sctp_process_asconf(), it will not be safe.
so add detection in sctp_ver
On Mon, Aug 24, 2015 at 10:43:11AM -0700, Pravin B Shelar wrote:
> Add netlink interface to configure Geneve UDP port number.
> So that user can configure it for a Gevene device.
>
> Signed-off-by: Pravin B Shelar
> Reviewed-by: Jesse Gross
Acked-by: John W. Linville
--
John W. Linville
On Mon, Aug 24, 2015 at 10:43:09AM -0700, Pravin B Shelar wrote:
> On packet transmit path geneve need to lookup route. Following
> patch improves route lookup using more parameters.
>
> Signed-off-by: Pravin B Shelar
> Reviewed-by: Jesse Gross
> Acked-by: Thomas Graf
> ---
> drivers/net/genev
For classifiers getting invoked via tc_classify(), we always need an
extra function call into tc_classify_compat(), as both are being
exported as symbols and tc_classify() itself doesn't do much except
handling of reclassifications when tp->classify() returned with
TC_ACT_RECLASSIFY.
CBQ and ATM a
On Wed, Aug 26, 2015 at 04:42:21PM -0400, Vlad Yasevich wrote:
> On 08/26/2015 04:35 PM, Xin Long wrote:
> > in sctp_process_asconf(), we get address parameter from the beginning of
> > the addip params. but we never check if it's really there. if the addr
> > param is not there, it still can pass
Hi Tony,
On 08/26/2015 01:16 PM, Tony Lindgren wrote:
[ ... ]
We may need two separate patches, one to fix up device_property_read_u32()
to return -ENXIO, and one to fix smsc911x_probe_config() to ignore the error
from device_get_phy_mode(), and to bail out if device_property_read_u32()
returns
On Wed, Aug 26, 2015 at 4:49 AM, Chuck Ebbert wrote:
> On Wed, 26 Aug 2015 08:46:59 +
> Shaun Crampton wrote:
>
>> Testing our app at scale on Google¹s GCE, running ~1000 CoreOS hosts: over
>> approximately 1 hour, I see about 1 in 50 hosts hit one of the Oopses
>> below and then reboot (I¹m
Commit f8d960524328 ("sctp: Enforce retransmission limit during shutdown")
fixed a problem with excessive retransmissions in the SHUTDOWN_PENDING by not
resetting the association overall_error_count. This allowed the association
to better enforce assoc.max_retrans limit.
However, the same issue s
On Mon, Aug 24, 2015 at 10:43:08AM -0700, Pravin B Shelar wrote:
> Signed-off-by: Pravin B Shelar
> Reviewed-by: Jesse Gross
> Acked-by: Thomas Graf
> ---
> drivers/net/geneve.c |4 +---
> 1 files changed, 1 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/geneve.c b/drivers/net/
On 08/26/2015 04:35 PM, Xin Long wrote:
> in sctp_process_asconf(), we get address parameter from the beginning of
> the addip params. but we never check if it's really there. if the addr
> param is not there, it still can pass sctp_verify_asconf(), then to be
> handled by sctp_process_asconf(), it
On Wed, Aug 26, 2015 at 11:31 AM, Joe Stringer wrote:
> Expose the kernel connection tracker via OVS. Userspace components can
> make use of the CT action to populate the connection state (ct_state)
> field for a flow. This state can be subsequently matched.
>
> Exposed connection states are OVS_C
On 08/26/2015 10:23 PM, Arend van Spriel wrote:
On 08/26/2015 12:22 PM, Thierry Reding wrote:
From: Thierry Reding
The rate_control_cap_mask() function takes a parameter mcs_mask, which
GCC will take to be u8 * even though it was declared with a fixed size.
This causes the following warning:
in sctp_process_asconf(), we get address parameter from the beginning of
the addip params. but we never check if it's really there. if the addr
param is not there, it still can pass sctp_verify_asconf(), then to be
handled by sctp_process_asconf(), it will not be safe.
so add a code in sctp_verify
On 08/26/2015 12:22 PM, Thierry Reding wrote:
From: Thierry Reding
The rate_control_cap_mask() function takes a parameter mcs_mask, which
GCC will take to be u8 * even though it was declared with a fixed size.
This causes the following warning:
net/mac80211/rate.c: In function 'rate_co
1 - 100 of 204 matches
Mail list logo
