On Tue, Nov 30, 2010 at 02:34:59PM +0200, Or Gerlitz wrote: > Jason Gunthorpe wrote: > > struct ib_nl_qp > > // String names for IB devices was a mistake, don't perpetuate it. > > __u32 ib_dev_if;
> Do you have a concrete suggestion and/or sketch for a patch > someone can work on to make this enumeration to take place? Oh, it is really easy, just add an ifindex member to struct ib_device. alloc_name can be extended to compute an appropriate index, see how this works in dev_new_index. Add a 'ifindex' to the ib_device's sysfs directory that exports this number. In future we'd want to see a netlink query to get the RDMA device list to replace trundling through sysfs. I think this is really important from a netlink perspective, there are going to be so many places you wan't to refer to an RDMA device. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
