Date: Sat, 11 Apr 2009 22:48:35 -0600 From: Scott Long <scottl@samsco.org> To: Alan Cox <alc@cs.rice.edu> Cc: current@freebsd.org Subject: Re: [Fwd: svn commit: r190949 - head/sys/vm] Message-ID: <49E172A3.2000108@samsco.org> In-Reply-To: <49E11CE7.5050903@cs.rice.edu> References: <49E11CE7.5050903@cs.rice.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
Off the top of my head, I would almost expect this to conflict with Java. Haven't tried it, though, so I can't do anything more than speculate. However, I would encourage testing here. Scott Alan Cox wrote: > It's possible that this could cause problems for "exotic" languages that > do their own memory management, i.e., they don't use libc's malloc(). > If anyone experiences or hears of such things, please let me know so > that we can get the problems sorted out well before the 8.0 release. > > Alan > > > ------------------------------------------------------------------------ > > Subject: > svn commit: r190949 - head/sys/vm > From: > Alan Cox <alc@FreeBSD.org> > Date: > Sat, 11 Apr 2009 22:34:09 +0000 (UTC) > To: > src-committers@freebsd.org, svn-src-all@freebsd.org, > svn-src-head@freebsd.org > > To: > src-committers@freebsd.org, svn-src-all@freebsd.org, > svn-src-head@freebsd.org > > > Author: alc > Date: Sat Apr 11 22:34:08 2009 > New Revision: 190949 > URL: http://svn.freebsd.org/changeset/base/190949 > > Log: > Remove execute permission from the memory allocated by sbrk(). > > Pre-announced on: -arch (3/31/09) > Discussed with: rwatson > Tested by: marius (sparc64) > > Modified: > head/sys/vm/vm_unix.c > > Modified: head/sys/vm/vm_unix.c > ============================================================================== > --- head/sys/vm/vm_unix.c Sat Apr 11 22:07:19 2009 (r190948) > +++ head/sys/vm/vm_unix.c Sat Apr 11 22:34:08 2009 (r190949) > @@ -117,7 +117,7 @@ obreak(td, uap) > goto done; > } > rv = vm_map_insert(&vm->vm_map, NULL, 0, old, new, > - VM_PROT_ALL, VM_PROT_ALL, 0); > + VM_PROT_RW, VM_PROT_ALL, 0); > if (rv != KERN_SUCCESS) { > error = ENOMEM; > goto done; > > > ------------------------------------------------------------------------ > > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?49E172A3.2000108>