[ovs-dev] [dev master 3/3] bridge: Re-try iface status update if the previous transaction fails.

Ben Pfaff blp at nicira.com
Mon Jun 9 21:39:55 UTC 2014


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 <alexw at 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?



More information about the dev mailing list