Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Oct 2011 22:09:43 +0300
From:      Kostik Belousov <kostikbel@gmail.com>
To:        Marcel Moolenaar <marcel@xcllnt.net>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, Marcel Moolenaar <marcel@freebsd.org>, src-committers@freebsd.org
Subject:   Re: svn commit: r226343 - head/sys/vm
Message-ID:  <20111013190943.GM1511@deviant.kiev.zoral.com.ua>
In-Reply-To: <7FB94714-3E47-4E1E-947A-AC62FF071135@xcllnt.net>
References:  <201110131620.p9DGKAM2022926@svn.freebsd.org> <20111013182025.GK1511@deviant.kiev.zoral.com.ua> <7FB94714-3E47-4E1E-947A-AC62FF071135@xcllnt.net>

next in thread | previous in thread | raw e-mail | index | archive | help

--HDfPXHCxbbzPc3vJ
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Thu, Oct 13, 2011 at 11:30:12AM -0700, Marcel Moolenaar wrote:
>
> On Oct 13, 2011, at 11:20 AM, Kostik Belousov wrote:
>
> > The two commits removed NX support for .data/.bss for 32bit binaries
> > Thon amd64. is is too unfortunate. Can we claim that only old
> > Thbinaries need this hack ?
>
> I don't know. When did our ABI for i386 change to have NX by default?
I think it changed de-facto when NX appears to be supported by hardware.
In other words, PF_R->PF_X was always considered a coincident, and not
a promise.

I think we can claim that the moment FreeBSD gained PAE support, it happen.

I would suggest to add a sysctl, say kern.elf32.readable_nx, with
the values:
0 - strictly follow segment permissions
1 - PF_R implies PF_X if p_osrel < 600000
2 - PF_R always implied PF_X
for 32bit binaries.

--HDfPXHCxbbzPc3vJ
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.18 (FreeBSD)

iEYEARECAAYFAk6XN3cACgkQC3+MBN1Mb4j0qACfRwkiBGTITzDPIyzStJFeWmFO
3tsAmgKLyQ5rEIgtMf3HPw2LLDYbOmNl
=iTtS
-----END PGP SIGNATURE-----

--HDfPXHCxbbzPc3vJ--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20111013190943.GM1511>