On 01/03/2025 0:51, Jakub Kicinski wrote: > On Wed, 26 Feb 2025 13:47:49 +0200 Tariq Toukan wrote: >> + if (table[i].speed == info->speed) { >> + if (!info->lanes || table[i].lanes == info->lanes) > > Hm, on a quick look it seems like lane count was added in all tables, > so not sure why the !info->lanes > it's for the case only speed was passed from ethtool (then ethtool passes 0 for lanes) >> + link_modes |= MLX5E_PROT_MASK(i);