Roland Dreier wrote:
> + dev = kmalloc(sizeof *dev + device->phys_port_cnt * sizeof *port, > + GFP_KERNEL); > + if (!dev) > + return; > + > + for (i = 1; i <= device->phys_port_cnt; i++) {Seems like this is implicitly assuming that the IB device is a CA. Maybe we should give up the ghost and stop trying to support IB switches?
I can go either way here. Would someone want to use this module on a switch? If so, I can fix up the code so that it can work on one, or at least change the checks that the device is a CA.
I thought that someone was running at least ib_mad on a switch. I'm just not sure which other modules are likely to run on them.
- Sean _______________________________________________ openib-general mailing list [email protected] http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
