[openib-general] Re: [PATCH 01/13] [RFC] ipath basic headers

Robert Walsh rjwalsh at pathscale.com
Sat Dec 17 13:55:47 PST 2005


> Do we really need this ugly userspace emulation code in the kernel?

Probably not.  I'll look into removing it.

> What is this used for? Why can't yo use memcpy?

Our chip can only handle double-word copies.

> > +#define round_up(v,sz) (((v) + (sz)-1) & ~((sz)-1))
> 
> Please use ALIGN().

Fair enough.

> You seem to be introducing loads of new ioctls. Any reason you can't
> use sysfs and/or configfs?

I'll see what people here think of that idea.

> > +#define ips_get_ipath_ver(ipath_header) (((ipath_header) >> INFINIPATH_I_VERS_SHIFT) \
> > +        & INFINIPATH_I_VERS_MASK)
> 
> Please use static inlines instead for readability.

OK.

Regards,
 Robert.

-- 
Robert Walsh                                 Email: rjwalsh at pathscale.com
PathScale, Inc.                              Phone: +1 650 934 8117
2071 Stierlin Court, Suite 200                 Fax: +1 650 428 1969
Mountain View, CA 94043.





More information about the general mailing list