Thanks for the review. Just realized that I should have squashed them into a single patch. Will do so before pushing them.
On Tue, May 6, 2014 at 10:10 PM, YAMAMOTO Takashi <yamam...@valinux.co.jp> wrote: >> Bug #1229225 >> >> Signed-off-by: Andy Zhou <az...@nicira.com> > > Reviewed-by: YAMAMOTO Takashi <yamam...@valinux.co.jp> > >> --- >> ofproto/bond.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/ofproto/bond.c b/ofproto/bond.c >> index c1a8448..c522ea3 100644 >> --- a/ofproto/bond.c >> +++ b/ofproto/bond.c >> @@ -888,7 +888,7 @@ bool >> bond_may_recirc(const struct bond *bond, uint32_t *recirc_id, >> uint32_t *hash_bias) >> { >> - if (bond->balance == BM_TCP && recirc_id) { >> + if (bond->balance == BM_TCP && bond->recirc_id) { >> if (recirc_id) { >> *recirc_id = bond->recirc_id; >> } >> -- >> 1.9.1 >> >> _______________________________________________ >> dev mailing list >> dev@openvswitch.org >> http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev