diff options
author | Ben Pfaff <blp@nicira.com> | 2012-08-21 13:51:01 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2012-08-21 14:02:55 -0700 |
commit | 4133cc197f7a498ed9d4644d48cdf97dfb71d54f (patch) | |
tree | 1f251bbf0be3b591e407ef4338e490b521ef611b /vswitchd | |
parent | 8be92d8c88c8ee87ed7d69e50681829a243d5350 (diff) |
bond: Tag flows according to their hash bucket, not just their slave.
The bonding code is supposed to tag flows two ways:
- According to the chosen bond slave, to make it easy to invalidate all
of the flows assigned to a given slave.
- According to the hash value for a flow, to make it easy to invalidate
all of the flows that hash into the same bucket.
However, the code wasn't actually applying the hash-based tags. This
meant that rebalancing didn't take effect immediately, and so after
rebalancing we could get log messages like this:
inconsistency in subfacet (actions were: 5) (correct actions: 4)
specifying some flow that was moved by the rebalance.
This commit fixes the problem by applying the hash-based tags.
Bug #12847.
Reported-by: Pratap Reddy <preddy@nicira.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
Diffstat (limited to 'vswitchd')
0 files changed, 0 insertions, 0 deletions