Yes it will work.  I'll do check-all on transaction failure.

I'll also combine this commit with the previous one for clarity.

Thanks,
Alex Wang,


On Mon, Jun 9, 2014 at 2:39 PM, Ben Pfaff <b...@nicira.com> wrote:

> On Sun, Jun 01, 2014 at 10:04:30PM -0700, Alex Wang wrote:
> > This commit adds logic that marks the 'struct iface's when their
> > status need to be updated but the current status update transaction
> > fails.  So, the main thread can re-try the update in the next run.
> >
> > Bug #1256577
> >
> > Signed-off-by: Alex Wang <al...@nicira.com>
>
> Now that I see it here, I am not a fan of putting all of the ifaces on
> a list.  It seems error-prone.  I do not think that database
> transactions should fail often, and so I would prefer to just run the
> database refreshes on every port if the transaction fails.  Does that
> work OK?
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to