Re: [ovs-dev] [PATCH v8 1/3] dpif-netlink: Detect Out-Of-Resource condition on a netdev

2018-10-26 Thread Eelco Chaudron
On 26 Oct 2018, at 11:53, Simon Horman wrote: > On Fri, 26 Oct 2018 at 08:58, Eelco Chaudron wrote: > I have a general comment, don't know where to put it, so I put it here. Some hardware might have multiple tables. If one type of table is full the ENOSPC might be

Re: [ovs-dev] [PATCH v8 1/3] dpif-netlink: Detect Out-Of-Resource condition on a netdev

2018-10-26 Thread Simon Horman
On Fri, 26 Oct 2018 at 08:58, Eelco Chaudron wrote: > > > On 25 Oct 2018, at 16:00, Sriharsha Basavapatna wrote: > > > Hi Eelco, > > > > Thanks for your comments, please see my response below. > > On Fri, Oct 19, 2018 at 7:52 PM Eelco Chaudron > > wrote: > >> > >> On 18 Oct 2018, at 18:13,

Re: [ovs-dev] [PATCH v8 1/3] dpif-netlink: Detect Out-Of-Resource condition on a netdev

2018-10-26 Thread Eelco Chaudron
On 25 Oct 2018, at 16:00, Sriharsha Basavapatna wrote: Hi Eelco, Thanks for your comments, please see my response below. On Fri, Oct 19, 2018 at 7:52 PM Eelco Chaudron wrote: On 18 Oct 2018, at 18:13, Sriharsha Basavapatna via dev wrote: This is the first patch in the patch-set to

Re: [ovs-dev] [PATCH v8 1/3] dpif-netlink: Detect Out-Of-Resource condition on a netdev

2018-10-25 Thread Sriharsha Basavapatna via dev
Hi Eelco, Thanks for your comments, please see my response below. On Fri, Oct 19, 2018 at 7:52 PM Eelco Chaudron wrote: > > On 18 Oct 2018, at 18:13, Sriharsha Basavapatna via dev wrote: > > > This is the first patch in the patch-set to support dynamic > > rebalancing > > of offloaded flows. > >

Re: [ovs-dev] [PATCH v8 1/3] dpif-netlink: Detect Out-Of-Resource condition on a netdev

2018-10-19 Thread Eelco Chaudron
On 18 Oct 2018, at 18:13, Sriharsha Basavapatna via dev wrote: This is the first patch in the patch-set to support dynamic rebalancing of offloaded flows. The patch detects OOR condition on a netdev port when ENOSPC error is returned by TC-Flower while adding a flow rule. A new structure is