[ovs-dev] [PATCH 5/5] vswitchd: LACP switch lacp_status to bit mask.

Ben Pfaff blp at nicira.com
Mon Feb 7 20:01:33 UTC 2011


On Mon, Feb 07, 2011 at 11:45:33AM -0800, Ethan Jackson wrote:
> Much of the LACP status information attached to an interface is
> moved from an enum to a bit mask in this commit.  The main reason
> to do this is to allow a link to be concurrently expired and
> defaulted.  With this commit, if a link enters an expired state,
> but has never had its partner information update, it will properly
> set the defaulted flag in its LACP messages.

You could leave the lacp_status member declared as "enum lacp_status"
now that you've defined that.

Thanks, with the other correction from your followup email, this looks
good.




More information about the dev mailing list