On Wed, 2004-09-22 at 18:37, Sean Hefty wrote: > I'll start submitting comments in the form of patches. > I'll try to keep each patch as small > and contained as possible.
OK. Thanks. > This first one is fairly easy. > I didn't see where struct ib_mad_buf was used. It's needed in the PCI mapping/unmapping. In i386, PPC, and some other architectures, the mapping is a nop so it might appear unused. I don't think the void *buf pointer is needed. > If it's needed, can it be merged directly into ib_mad_private_header? Sure. I will post a patch for this. -- Hal _______________________________________________ openib-general mailing list [EMAIL PROTECTED] http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
