On Tue, May 14, 2019 at 3:15 PM Han Zhou wrote:
> From: Han Zhou
>
> Increase the limit on the number of open file descriptors, because
> SB DB may connect to large number of chassises.
>
> Signed-off-by: Han Zhou
> ---
> ovn/utilities/ovn-ctl | 8
> 1 file changed, 8 insertions(+)
>
On 15.05.2019 8:51, Ben Pfaff wrote:
> On Tue, May 14, 2019 at 07:23:59PM +0300, Ilya Maximets wrote:
>> Some tests uses 'kill -SEGV' to simulate segfault of a child process.
>> This causes test failures on CirrusCI because process hangs in DL state
>> for more than 10 seconds:
>
> That seems weir
Hallo, Sie haben eine Spende von € 2.000.000,00 EUR, bitte kontaktieren Sie
mich umgehend für weitere Informationen. Antworten Sie mit dem auf diese
E-Mail: richardpoverty...@gmail.com
___
dev mailing list
d...@openvswitch.org
https://mail.openvswit
On 5/14/19 3:08 PM, Ilya Maximets wrote:
Needed for the future post-copy live migration support for
vhost-user ports.
Signed-off-by: Ilya Maximets
---
lib/util.c | 15 +++
lib/util.h | 3 +++
vswitchd/ovs-vswitchd.c | 2 ++
3 files changed, 20 ins
On 5/14/19 3:08 PM, Ilya Maximets wrote:
From: Liliia Butorina
Post-copy Live Migration for vHost supported since DPDK 18.11 and
QEMU 2.12. New global config option 'vhost-postcopy-support' added
to control this feature. Ex.:
ovs-vsctl set Open_vSwitch . other_config:vhost-postcopy-suppo
On 25.04.2019 18:22, David Marchand wrote:
> We currently poll all available queues based on the max queue count
> exchanged with the vhost peer and rely on the vhost library in DPDK to
> check the vring status beneath.
> This can lead to some overhead when we have a lot of unused queues.
>
> To e
On 06.05.2019 13:43, Roni Bar Yanai wrote:
> Background
> ==
> OVS HW offload solution is consisted of forwarding and control. HW implements
> embedded switch that connects SRIOV VF's and forwards packets according to the
> dynamically configured HW rules (packets can be altered by HW rules
On 15.05.2019 8:33, Roi Dayan wrote:
>
>
> On 14/05/2019 19:20, Ilya Maximets wrote:
>> On 14.05.2019 18:24, Roi Dayan wrote:
>>>
>>>
>>> On 14/05/2019 18:16, Roi Dayan wrote:
>>
>>
>>
Hi,
I did a quick ping test between two ports and after this commit the
command o
Hi Ilya,
Thanks for the comment.
I think the suggested arch is very good and has many advantages, and in fact I
had something very similar as my internally first approach.
However, I had one problem: it doesn't solves the kernel case. It make sense
doing forwarding using dpdk also when OVS is ke
pinctrl_handle_buffered_packets can insert new elements in
buffered_packets_map hasmap and it runs concurrently with pinctrl_run
starting from commit 3594ffab6b4b. Fix possible races grabbing
pinctrl_mutex before running pinctrl_handle_buffered_packets
Fixes: 3594ffab6b4b ("ovn-controller: Add a n
On 15.05.2019 16:01, Roni Bar Yanai wrote:
> Hi Ilya,
>
> Thanks for the comment.
> I think the suggested arch is very good and has many advantages, and in fact
> I had something very similar as my internally first approach.
> However, I had one problem: it doesn't solves the kernel case. It make
This is the combined patch-set for dynamic flow API along with
the further work about splitting up netdev offloading from the
generic netdev related code and making different netdev flow
offloading implementations more or less consistent.
Version 4:
* Added dpctl related patch.
* No other chan
Since introduction of dynamic flow API for netdevs, tricky
accesses to uninitialized flow API are no longer possible.
So, ovs-dpctl doesn't support dumping HW offloaded flows now.
Claim this in docs and man pages. Additionally forbidden
'type' argument for 'ovs-dpctl dump-flows'.
Signed-off-by: Il
Current issues with Flow API:
* OVS calls offloading functions regardless of successful
flow API initialization. (ex. on init_flow_api failure)
* Static initilaization of Flow API for a netdev_class forbids
having different offloading types for different instances
of netdev with the same net
Flow API providers renamed to be consistent with parent module
'netdev-offload' and look more like each other.
'_rte_' replaced with more convenient '_dpdk_'.
We'll have following structure:
Common code:
lib/netdev-offload-provider.h
lib/netdev-offload.c
lib/netdev-offload.h
Pro
New module 'netdev-offload' created to manage different flow API
implementations. All the generic and provider independent code moved
there from the 'netdev' module.
Flow API providers further encapsulated.
The only function that was changed is 'netdev_any_oor'.
Now it uses offloading related hma
On Wed, May 15, 2019 at 12:53 AM aginwala wrote:
>
>
> On Tue, May 14, 2019 at 3:15 PM Han Zhou wrote:
>
>> From: Han Zhou
>>
>> Increase the limit on the number of open file descriptors, because
>> SB DB may connect to large number of chassises.
>>
>> Signed-off-by: Han Zhou
>> ---
>> ovn/ut
Bleep bloop. Greetings Ilya Maximets, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
ERROR: Improper whitespace around control block
#1065 FILE: lib/netdev-offload-tc.c:173:
H
Hi Aaron.
Robot complains about lines not touched by this patch.
This is strange.
Best regards, Ilya Maximets.
On 15.05.2019 19:05, 0-day Robot wrote:
> Bleep bloop. Greetings Ilya Maximets, I am a robot and I have tried out your
> patch.
> Thanks for your contribution.
>
> I encountered some
On 15.05.2019 19:13, Ilya Maximets wrote:
> Hi Aaron.
>
> Robot complains about lines not touched by this patch.
> This is strange.
I suspect that it's because of file renaming.
What git version you're using? Or maybe some special options?
>
> Best regards, Ilya Maximets.
>
> On 15.05.2019 19:
Ilya Maximets writes:
> On 15.05.2019 19:13, Ilya Maximets wrote:
>> Hi Aaron.
>>
>> Robot complains about lines not touched by this patch.
>> This is strange.
>
> I suspect that it's because of file renaming.
Yes, I think you're correct.
> What git version you're using? Or maybe some special
On 5/8/2019 4:13 PM, Harry van Haaren wrote:
This allows plugging-in of different subtable hash-lookup-verify
routines, and allows special casing of those functions based on
known context (eg: # of bits set) of the specific subtable.
Signed-off-by: Harry van Haaren
---
v9:
- Use count_1bits i
Next meeting June 12th 1700 UTC
May 15th minutes
Attendees: Asaf, David, Flavio, Frikkie, Ian, Simon, Thomas, Aaron,
Johann, Ophir, Roni, Kevin.
===
GENERAL
===
- Timing of meeting (Kevin)
-- Should we move it to an earlier time?
-- Some people from US west coast are attending
- DPDK 18
From: Numan Siddique
This new type is added for the following reasons:
- When a load balancer is created in an OpenStack deployment with Octavia
service, it creates a logical port 'VIP' for the virtual ip.
- This logical port is not bound to any VIF.
- Octavia service creates a servi
On Wed, May 15, 2019 at 11:14:15AM +0300, Ilya Maximets wrote:
> On 15.05.2019 8:51, Ben Pfaff wrote:
> > On Tue, May 14, 2019 at 07:23:59PM +0300, Ilya Maximets wrote:
> >> Some tests uses 'kill -SEGV' to simulate segfault of a child process.
> >> This causes test failures on CirrusCI because proc
From: Numan Siddique
This new type is added for the following reasons:
- When a load balancer is created in an OpenStack deployment with Octavia
service, it creates a logical port 'VIP' for the virtual ip.
- This logical port is not bound to any VIF.
- Octavia service creates a servi
On Wed, May 15, 2019 at 11:36 AM wrote:
>
> From: Numan Siddique
>
> This new type is added for the following reasons:
>
> - When a load balancer is created in an OpenStack deployment with
Octavia
> service, it creates a logical port 'VIP' for the virtual ip.
>
> - This logical port is no
Gentle ping on this patch. This fix is a bit critical to fix the
ovn-controller CPU usage.
Thanks
Numan
On Sat, May 11, 2019 at 2:34 PM Numan Siddique wrote:
>
>
> On Sat, May 11, 2019 at 2:09 PM Numan Siddique
> wrote:
>
>>
>>
>> On Sat, May 11, 2019 at 5:38 AM Ankur Sharma
>> wrote:
>>
>>>
On 5/8/2019 4:13 PM, Harry van Haaren wrote:
This commit moves some data-structures to be available
in the dpif-netdev.h header. This allows specific implementations
of the subtable lookup function to include just that header
file, and not require that the code exists in dpif-netdev.c
Signed-off
Hi Numan,
Just confirming, anything else pending from my side?
or given that patch has been Acked, hence someone will apply it to master?
Thanks
Regards,
Ankur
From: Numan Siddique
Sent: Wednesday, May 15, 2019 11:43 AM
To: Ankur Sharma
Cc: Han Zhou ; ovs-dev@openvswitch.org
Subject: Re: [ovs
On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote:
>
>
> On Wed, May 15, 2019 at 11:36 AM wrote:
> >
> > From: Numan Siddique
> >
> > This new type is added for the following reasons:
> >
> > - When a load balancer is created in an OpenStack deployment with
> Octavia
> > service, it creates
On Thu, May 16, 2019 at 12:20 AM Ankur Sharma
wrote:
> Hi Numan,
>
> Just confirming, anything else pending from my side?
> or given that patch has been Acked, hence someone will apply it to master?
>
I don't think you have anything pending unless the committer has any
comments.
Thanks for the
Bleep bloop. Greetings Numan Siddique, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
WARNING: Line is 190 characters long (recommended limit is 79)
#491 FILE: ovn/northd/ovn-nort
Acked-by: aginwala
On Wed, May 15, 2019 at 8:58 AM Han Zhou wrote:
>
>
> On Wed, May 15, 2019 at 12:53 AM aginwala wrote:
>
>>
>>
>> On Tue, May 14, 2019 at 3:15 PM Han Zhou wrote:
>>
>>> From: Han Zhou
>>>
>>> Increase the limit on the number of open file descriptors, because
>>> SB DB may
On Wed, May 15, 2019 at 6:34 AM Lorenzo Bianconi <
lorenzo.bianc...@redhat.com> wrote:
>
> pinctrl_handle_buffered_packets can insert new elements in
> buffered_packets_map hasmap and it runs concurrently with pinctrl_run
> starting from commit 3594ffab6b4b. Fix possible races grabbing
> pinctrl_mu
Bleep bloop. Greetings Numan Siddique, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
WARNING: Line is 190 characters long (recommended limit is 79)
#496 FILE: ovn/northd/ovn-nort
On Wed, May 15, 2019 at 11:55 AM Numan Siddique wrote:
>
>
> On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote:
>
>>
>>
>> On Wed, May 15, 2019 at 11:36 AM wrote:
>> >
>> > From: Numan Siddique
>> >
>> > This new type is added for the following reasons:
>> >
>> > - When a load balancer is crea
Aaron Conole writes:
> Ilya Maximets writes:
>
>> On 15.05.2019 19:13, Ilya Maximets wrote:
>>> Hi Aaron.
>>>
>>> Robot complains about lines not touched by this patch.
>>> This is strange.
>>
>> I suspect that it's because of file renaming.
>
> Yes, I think you're correct.
>
>> What git versio
Thanks for fixing this! a comment below:
On Monday, May 13, 2019, 6:31:28 AM CDT, Dumitru Ceara
wrote:
Add tunnel-id creation and parsing functions in encaps.h.
Reported-at: https://bugzilla.redhat.com/1708131
Reported-by: Haidong Li
Fixes: b520ca7 ("Support for multiple VTEP in OVN"
> > + OVS AF_XDP netdev is using the userspace datapath, the same datapath
> > + as used by OVS-DPDK. So it requires --disable-system for ovs-vswitchd
> > + and datapath_type=netdev when adding a new bridge.
>
> I don't think that '--disable-system' is needed. It doesn't affect anything.
>
Hi Ilya,
Thanks for your feedback.
On Mon, May 13, 2019 at 10:48 AM Ilya Maximets wrote:
>
> On 10.05.2019 2:54, William Tu wrote:
> > The patch introduces experimental AF_XDP support for OVS netdev.
> > AF_XDP, Address Family of the eXpress Data Path, is a new Linux socket type
> > built upon t
On 15.05.2019 23:32, Aaron Conole wrote:
> Aaron Conole writes:
>
>> Ilya Maximets writes:
>>
>>> On 15.05.2019 19:13, Ilya Maximets wrote:
Hi Aaron.
Robot complains about lines not touched by this patch.
This is strange.
>>>
>>> I suspect that it's because of file renaming.
On 16.05.2019 2:20, William Tu wrote:
>>> + OVS AF_XDP netdev is using the userspace datapath, the same datapath
>>> + as used by OVS-DPDK. So it requires --disable-system for ovs-vswitchd
>>> + and datapath_type=netdev when adding a new bridge.
>>
>> I don't think that '--disable-system' is
43 matches
Mail list logo