[ofw] [patch] [winverb] fix BS in WinVerbs when getting an event for non IB port

[Hefty, Sean]

> During our testing we got a BS in WinVerbs. It happens when we running a
> VPI scenario in which one port is IB and the second port is ETH.  The test
> bring down the ETH link, as a result an event is created and low level
> driver calls to winverbs to handle it.

We need to find the right way to handle this, and I'm not sure what that is.  Winverbs layers as a filter device for an IB device.  If an HCA acts as two different types of devices, then maybe it should report itself as two different devices to Windows..?  Then all ports from that device would be the same, with the HCA driver making whatever adjustments and mappings are necessary between a port number viewed by a user versus the physical port of the HCA.  Or winverbs may need to be fully extended to handle different link layers.

Whatever solution is chosen needs to apply to winmad as well.  How are other dual-port devices reported in Windows?

> Since the IB always should be the first port, the simple fix is to check
> that port number of the event is less or equal for PortCount this promise
> that port object is exist for the port.

I don't like this assumption.