Roland Dreier <[EMAIL PROTECTED]> wrote on 01/30/2008 11:47:55 AM:
>
> > - /* First check that we have enough alignment */
> > - if ((*iova_start & ~PAGE_MASK) != (buffer_list[0].addr &
~PAGE_MASK))
> > - return ERR_PTR(-EINVAL);
>
> I don't think we want to remove this check. It prevents someone from
> trying to register a region with a virtual address that the hardware
> can't handle.
This initial check is redundant, given:
mask = buffer_list[0].addr ^ *iova_start;
and subsequent:
if (mask & ~PAGE_MASK)
return ERR_PTR(-EINVAL);
- Bryan
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
