On Thu, Nov 30, 2017 at 05:37:57AM +0000, Vishal Deep Ajmera wrote: > Today in OVS, when errors are encountered during the execution > of an action the entire batch of packets may be deleted (for e.g. > in processing push_tnl_action, if the port is not found in the > port_cache of PMD). The remaining actions continue to be executed > even though there are no packets to be processed. > > It is assumed that the code dealing with each action checks that > the batch is not empty before executing. Crashes may occur if the > assumption is not met. > > The patch makes OVS skip processing of further actions from the > action-set once a batch is emptied. Doing so centralizes the check > in one place and avoids the possibility of crashes. > > This change DOES NOT fix any existing bug in the code, only a > precautionary measure to avoid crashes if new actions does not > take care of empty batches. > > Signed-off-by: Vishal Deep Ajmera <[email protected]>
Thanks. I applied this to master. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
