On Tue, 2017-05-23 at 06:19 -0700, Dupuis, Chad wrote: > If there are multiple descriptors for a particular type in a clear virtual > link we receive, we will not process it correctly but rather take the last > value. This can cause us not to not flap the virtual link as the value from > the descriptors that we compare against the our stored FCF or fc_lport values > may not match. > > Change is to do a comparison when processing the each descriptor instead of at > the end and then set a bool if we need to do the reset. Did you perhaps mean "Change this" instead of "Change is"? Anyway: Reviewed-by: Bart Van Assche <Bart.VanAssche@xxxxxxxxxxx>