svn commit: r266574 - in stable/8: lib/libc/sys sys/vm

[Peter Holm]

Author: pho
Date: Fri May 23 05:35:43 2014
New Revision: 266574
URL: http://svnweb.freebsd.org/changeset/base/266574

Log:
  MFC r265534:
  
  msync(2) must return ENOMEM and not EINVAL when the address is outside the
  allowed range or when one or more pages are not mapped. This according to
  The Open Group Base Specifications Issue 7.
  
  Sponsored by:	EMC / Isilon storage division

Modified:
  stable/8/lib/libc/sys/msync.2
  stable/8/sys/vm/vm_mmap.c
Directory Properties:
  stable/8/   (props changed)
  stable/8/lib/   (props changed)
  stable/8/lib/libc/   (props changed)
  stable/8/lib/libc/sys/   (props changed)
  stable/8/sys/   (props changed)
  stable/8/sys/vm/   (props changed)

Modified: stable/8/lib/libc/sys/msync.2
==============================================================================
--- stable/8/lib/libc/sys/msync.2	Fri May 23 05:15:17 2014	(r266573)
+++ stable/8/lib/libc/sys/msync.2	Fri May 23 05:35:43 2014	(r266574)
@@ -87,11 +87,13 @@ The
 .Fa addr
 argument
 is not a multiple of the hardware page size.
-.It Bq Er EINVAL
-The
+.It Bq Er ENOMEM
+The addresses in the range starting at
+.Fa addr
+and continuing for
 .Fa len
-argument
-is too large or negative.
+bytes are outside the range allowed for the address space of a
+process or specify one or more pages that are not mapped.
 .It Bq Er EINVAL
 The
 .Fa flags
@@ -99,7 +101,7 @@ argument
 was both MS_ASYNC and MS_INVALIDATE.
 Only one of these flags is allowed.
 .It Bq Er EIO
- An error occurred while writing at least one of the pages in
+An error occurred while writing at least one of the pages in
 the specified region.
 .El
 .Sh SEE ALSO

Modified: stable/8/sys/vm/vm_mmap.c
==============================================================================
--- stable/8/sys/vm/vm_mmap.c	Fri May 23 05:15:17 2014	(r266573)
+++ stable/8/sys/vm/vm_mmap.c	Fri May 23 05:35:43 2014	(r266574)
@@ -497,7 +497,7 @@ msync(td, uap)
 	case KERN_SUCCESS:
 		return (0);
 	case KERN_INVALID_ADDRESS:
-		return (EINVAL);	/* Sun returns ENOMEM? */
+		return (ENOMEM);
 	case KERN_INVALID_ARGUMENT:
 		return (EBUSY);
 	case KERN_FAILURE: