cvs commit: src/sys/vm vm_map.c

[Brian Fundakowski Feldman]

On Mon, Jun 28, 2004 at 03:22:33PM -0400, Andrew Gallatin wrote:
> Andrew Gallatin [gallatin at FreeBSD.org] wrote:
> > gallatin    2004-06-28 19:15:40 UTC
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/vm               vm_map.c 
> >   Log:
> >   Fix alpha - the use of min() on longs was loosing the high bits and
> >   returning wrong answers, leading to strange values vm2->vm_{s,t,d}size.
> 
> Why are min() and max() inlines which operate on ints?  This seems
> like a real landmine for 64-bit platforms..

Also, why is GCC not generating the correct warnings?  The values passed
in were definitely a 64-bit type.  Thanks for finding and fixing this.
The inlines seem to exist to work around the effect of using macros
unknowingly on statements with side effects.  These should really be
MIN(), and there seems to have been an extra tab that crept in.  Do
you think you could change those things?

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green at FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\