On 4/2/2012 9:02 AM, Or Gerlitz wrote: > On 4/2/2012 3:51 PM, Or Gerlitz wrote: >> can you add these prints and send me the output after attempting to >> cat the rate file? > > okay, on a system which has IB on port 1 and Ethernet on port 2, using > this patch > I get these prints: >> ib_link_query_port active_speed 4 >> rate_show ret 0 for ib_query_port dev mlx4_0 port 1 link 1 >> eth_link_query_port active_speed 4 >> rate_show ret 0 for ib_query_port dev mlx4_0 port 2 link 2 > > but if forcing port 2 link layer to be IB as well, which means we will > land in ib_link_query_port for an Ethernet port, I get the below > >> echo ib > /sys/bus/pci/devices/0000:07:00.0/mlx4_port2 >> ib_link_query_port active_speed 4 >> rate_show ret 0 for ib_query_port dev mlx4_0 port 1 link 1 >> ib_link_query_port active_speed 7 >> rate_show ret 0 for ib_query_port dev mlx4_0 port 2 link 1 > > So when doing the MAD_IFC port info query command on Ethernet port, the > firmware returns the > value of seven which isn't among the IB speeds and we are remained with > rate=-1 in rate_show > of drivers/infiniband/core/sysfs.c
libibumad (and infiniband-diags) are not yet RoCE ready AFAIK. Fixing that at least for libibumad is minor. Ira can comment on infiniband-diags. > It should be pretty simple to come with patch to that situation, but I > want to better understand > what happens on your system, waiting for the output... I think there are 3 main issues here: 1. EINVAL can be returned from rate_show and hence "Invalid argument" rate string should be handled in libibumad. I think this was Bart's original point. 2. Why is rate_show returning EINVAL ? I think that's what you're trying to isolate with the additional printks you sent Bart for sysfs.c. 3. link_layer ethernet should also be handled which is the issue you raised. -- Hal > 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
