JWM wrote: > if(copy_from_user(&packet->mad,buf, > sizeof(struct ib_user_mad) + IB_MGMT_RMPP_HDR)) > > BTW - why is IB_MGMT_RMPP_HDR an enum? Why not a sizeof ? No structure was ever defined for just this (standard MAD header + RMPP header only). - Sean