Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-04 Thread Ilya Maximets
Hi Eelco, Thanks for paying attention to this. I'll be appreciated if you'll test/review the solution when 'implementation level' discussion finished. Best regards, Ilya Maximets. On 04.07.2017 12:27, Eelco Chaudron wrote: > Hi Ilya/Bhanu, > > I see you guys started a discussion already, so

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-04 Thread Ilya Maximets
On 04.07.2017 14:37, Bodireddy, Bhanuprakash wrote: > Apologies for snipping the text. I did it to keep this thread readable. > >> >> Hi Darrell and Jan. >> Thanks for looking at this. I agree with Darrell that mixing implementations >> on >> two different levels is a bad idea, but as I already

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-04 Thread Bodireddy, Bhanuprakash
Apologies for snipping the text. I did it to keep this thread readable. > >Hi Darrell and Jan. >Thanks for looking at this. I agree with Darrell that mixing implementations on >two different levels is a bad idea, but as I already wrote in reply to >Bhanuprakash [2], there is no issues with

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-04 Thread Ilya Maximets
Message- > > From: ovs-dev-boun...@openvswitch.org > [mailto:ovs-dev-boun...@openvswitch.org] On Behalf Of Ilya Maximets > > Sent: Friday, 30 June, 2017 14:03 > > To: ovs-dev@openvswitch.org; Bhanuprakash Bodireddy > <bhanuprakash.bodire...@intel.com> >

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-04 Thread Eelco Chaudron
Hi Ilya/Bhanu, I see you guys started a discussion already, so just some input form my side... When I started looking into this, my hack looked very much like Ilya's, and I was flushing my tx queue every rx batch.I even kept track of which port had packets waiting to send, so I did not have

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-03 Thread Darrell Ball
penvswitch.org; Bhanuprakash Bodireddy <bhanuprakash.bodire...@intel.com> > Cc: Heetae Ahn <heetae82@samsung.com>; Ilya Maximets <i.maxim...@samsung.com> > Subject: [ovs-dev] [PATCH 0/3] Output packet batching. > > This patch-set inspired by [1] from Bh

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-03 Thread Jan Scheurich
heetae82@samsung.com>; Ilya Maximets > <i.maxim...@samsung.com> > Subject: [ovs-dev] [PATCH 0/3] Output packet batching. > > This patch-set inspired by [1] from Bhanuprakash Bodireddy. > Implementation of [1] looks very complex and introduces many pitfalls for > late

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-03 Thread Ilya Maximets
On 03.07.2017 13:15, Bodireddy, Bhanuprakash wrote: >> This patch-set inspired by [1] from Bhanuprakash Bodireddy. >> Implementation of [1] looks very complex and introduces many pitfalls for >> later code modifications like possible packet stucks. >> >> This version targeted to make simple and

Re: [ovs-dev] [PATCH 0/3] Output packet batching.

2017-07-03 Thread Bodireddy, Bhanuprakash
>This patch-set inspired by [1] from Bhanuprakash Bodireddy. >Implementation of [1] looks very complex and introduces many pitfalls for >later code modifications like possible packet stucks. > >This version targeted to make simple and flexible output packet batching on >higher level without