<br><tt><font size=2>Roland Dreier <rdreier@cisco.com> wrote on 01/30/2008
11:47:55 AM:<br>
> <br>
>  > -       /* First check that we have enough
alignment */<br>
>  > -       if ((*iova_start & ~PAGE_MASK)
!= (buffer_list[0].addr &  ~PAGE_MASK))<br>
>  > -               return
ERR_PTR(-EINVAL);<br>
> <br>
> I don't think we want to remove this check.  It prevents someone
from<br>
> trying to register a region with a virtual address that the hardware<br>
> can't handle.<br>
</font></tt>
<br><tt><font size=2>This initial check is redundant, given:</font></tt>
<br>
<br><tt><font size=2>        mask = buffer_list[0].addr
^ *iova_start;</font></tt>
<br>
<br><tt><font size=2>and subsequent:</font></tt>
<br>
<br><tt><font size=2>        if (mask &
~PAGE_MASK)</font></tt>
<br><tt><font size=2>           
    return ERR_PTR(-EINVAL);</font></tt>
<br>
<br><tt><font size=2>- Bryan</font></tt>
<br>
<br>