[ovs-dev] [PATCH] ofproto: Return error codes for Rule insertions"

Aravind Prasad raja.avi at gmail.com
Wed Jul 11 18:39:09 UTC 2018


 >Hi Ben/All,
>If possible, Kindly hold reviewing this patch for now. Expecting an
approval from my Org. Sorry for the inconvenience >caused and thanks for
the support.
>Will get back and intimate for the review as soon as possible after the
approval (expecting it to take not more than a week).  >And thanks again
for understanding.

>>OK.

Hi Ben/All,
The patch changes are available for review. Thanks a lot for the continued
support.
Kindly review the patch and let me know your views.
Hi Ben,
In response to your query, as I had previously suggested, we could have
Hardware
implementations with ofproto and netdev providers. Only static checks could
be
possible in Rule-construct phase. The Rule-insert API call could
potentially fail
due to any of dynamic HW failures, runtime errors and so on. And incase of
Hybrid mode implementations(coexistence of normal and Openflow Switch
functioning), the possibility of rule-insertion failures could be higher.
Kindly let me know your views.

Thanks,
Aravind Prasad S

On Tue, Jul 10, 2018 at 10:02 PM Ben Pfaff <blp at ovn.org> wrote:

> OK.
>
> On Tue, Jul 10, 2018 at 02:58:47PM +0530, Aravind Prasad wrote:
> > Hi Ben/All,
> >
> > If possible, Kindly hold reviewing this patch for now. Expecting an
> > approval from my Org. Sorry for the inconvenience caused and thanks for
> the
> > support.
> >
> > Will get back and intimate for the review as soon as possible after the
> > approval (expecting it to take not more than a week).  And thanks again
> for
> > understanding.
> >
> > Thanks,
> > Aravind Prasad S
> >
> > On Tue, Jul 10, 2018 at 7:06 AM Aravind Prasad <raja.avi at gmail.com>
> wrote:
> >
> > >
> > > Currently, rule_insert() API doesnot have return value. There are some
> > > possible
> > > > scenarios where rule insertions can fail at run-time even though the
> > > static
> > > > checks during rule_construct() had passed previously.
> > > > Some possible scenarios for failure of rule insertions:
> > > > **) Rule insertions can fail dynamically in Hybrid mode (both
> Openflow
> > > and
> > > > Normal switch functioning coexist) where the CAM space could get
> suddenly
> > > > filled up by Normal switch functioning and Openflow gets devoid of
> > > > available space.
> > > > **) Some deployments could have separate independent layers for HW
> rule
> > > > insertions and application layer to interact with OVS. HW layer
> > > > could face any dynamic issue during rule handling which application
> could
> > > > not have predicted/captured in rule-construction phase.
> > > > Rule-insert errors for bundles are not handled in this pull-request.
> > > > Will be handled in upcoming pull request.
> > >
> > > >> I don't think that ofproto-dpif can ever see such a failure.  Are
> you
> > > >> planning to submit an ofproto provider that exercises this behavior?
> > >
> > > Hi Ben,
> > >
> > > These type of errors are possible in actual Hardware implementations.
> > > It is possible that ofproto and netdev providers could be implemented
> > > for a actual HW.
> > > Usually, in such cases, in the rule construct phase, all the static
> > > checks like verifying the qualifiers and actions could be done and the
> > > other related verifications.
> > > But during the rule insert phase, it is possible that the rule
> insertion
> > > may get failed in HW (runtime errors, HW errors and so on).
> > > Hence, we need a way to rollback for rule-insert phase also.
> > > Kindly let me know your views.
> > >
> > > Thanks,
> > > Aravind Prasad S
> > >
> > >
> > > On Tue, Jul 10, 2018 at 3:45 AM Ben Pfaff <blp at ovn.org> wrote:
> > >
> > >> On Mon, Jul 09, 2018 at 01:02:08PM +0530, Aravind Prasad S wrote:
> > >> > Currently, rule_insert() API doesnot have return value. There are
> some
> > >> possible
> > >> > scenarios where rule insertions can fail at run-time even though the
> > >> static
> > >> > checks during rule_construct() had passed previously.
> > >> > Some possible scenarios for failure of rule insertions:
> > >> > **) Rule insertions can fail dynamically in Hybrid mode (both
> Openflow
> > >> and
> > >> > Normal switch functioning coexist) where the CAM space could get
> > >> suddenly
> > >> > filled up by Normal switch functioning and Openflow gets devoid of
> > >> > available space.
> > >> > **) Some deployments could have separate independent layers for HW
> rule
> > >> > insertions and application layer to interact with OVS. HW layer
> > >> > could face any dynamic issue during rule handling which application
> > >> could
> > >> > not have predicted/captured in rule-construction phase.
> > >> > Rule-insert errors for bundles are not handled in this pull-request.
> > >> > Will be handled in upcoming pull request.
> > >> >
> > >> > Signed-off-by: Aravind Prasad S <raja.avi at gmail.com>
> > >>
> > >> I don't think that ofproto-dpif can ever see such a failure.  Are you
> > >> planning to submit an ofproto provider that exercises this behavior?
> > >>
> > >> Thanks,
> > >>
> > >> Ben.
> > >>
> > >
>


More information about the dev mailing list