Re: [RESEND][PATCH] against mmap.c (do_mmap_pgoff) and a note

DervishD
2003-06-16T09:52:35Z

* Jörn Engel <joern@wohnheim.fh-wedel.de> dixit:
> > - if (!len)
> > + if (len == 0)
> No change.

Just cosmethic, ask Dave S. Miller, he is the author of the
change, I'm just doing the patch for him.

> > return addr;
> >
> > - if (len > TASK_SIZE)
> > - return -EINVAL;
> > -
> > len = PAGE_ALIGN(len);
> >
> > + if (len > TASK_SIZE || len == 0)
> > + return -EINVAL;
> > +
>
> PAGE_ALIGN(0) = 0
> PAGE_ALIGN(1) = PAGE_SIZE
>
> Again, no change.

There is a change in archs where TASK_SIZE is the entire
addressable space (like sparc64). Ask Dave S., again. The problem did
arise when TASK_SIZE is ~0. Then semantics change.