I don't think that this specific change should check firmware version, because it not dealing with hardware directly and it should work with older firmware too. Its restriction, that now it will support 2^30 members for QP instead of 2^32.
On Wed, Dec 1, 2010 at 11:12 PM, Or Gerlitz <[email protected]> wrote: > Aleksey Senin <[email protected]> wrote: >> Newest firmware does a IB/ETH protocol distiction. >> There are two bits in members_count field of multicast group table >> that store this info and its meaning : 0 - Infiniband, 1 - Ethernet. While >> changing >> members count for particular multicast group it important not to reset this >> information. > > Two quesions for this patch and et al friend patches to come somehow > soon: what happens if someone burns the new firmware on an IB system > which runs the mlx4 code before the patch? also, what happens if > someone runs the patched mlx4 on a system on which the firmware isn't > this newest, shouldn't a check for the firmware version to be >= X be > added to the driver? As for the 1st case (old code, new firmware), I > recall there was a similar situation couple of months or maybe little > more ago, Jack, I think you were involved in the solution, correct? > > Or. > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
