Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 27 Jul 2013 13:47:52 -0700
From:      Andrey Zonov <zont@FreeBSD.org>
To:        Bruce Evans <brde@optusnet.com.au>
Cc:        Juli Mallett <jmallett@FreeBSD.org>, svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, Jilles Tjoelker <jilles@stack.nl>
Subject:   Re: svn commit: r253662 - in head: lib/libc/gen sys/vm tools/tools/sysdoc
Message-ID:  <51F431F8.1040803@FreeBSD.org>
In-Reply-To: <20130727170845.E872@besplex.bde.org>
References:  <201307260200.r6Q207cB015223@svn.freebsd.org> <20130726150949.GC14175@stack.nl> <51F2E970.3070303@FreeBSD.org> <CACVs6=_gLGtW01-rmxTmaVgMBJWjpFHMEQ2R36Nvf2er7B4v%2Bw@mail.gmail.com> <20130727170845.E872@besplex.bde.org>

next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
------enig2BWDOWUBELCWUQNFBHHBD
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On 7/27/13 12:47 AM, Bruce Evans wrote:
> On Fri, 26 Jul 2013, Juli Mallett wrote:
>=20
>> On Fri, Jul 26, 2013 at 2:26 PM, Andrey Zonov <zont@freebsd.org> wrote=
:
>>
>>> On 7/26/13 8:09 AM, Jilles Tjoelker wrote:
>>>> As noted in mail from Bruce Evans, please preserve the ABI of
>>>> VM_SWAPPING_ENABLED here.
>>>
>>> I don't think it's a big problem.  Internally we don't use it.  Good
>>> code uses sysctlbyname() instead of sysctl().  It doesn't seem to me
>>> this is very popular sysctl.  It's also CURRENT without MFC.
>>
>> That doesn't seem like a great argument.
>>
>> Why should we break the ABI for bad code that used that sysctl and all=

>> following?  It seems pretty trivial to preserve, and no-cost.  Also, t=
he
>> fact that it's in current misses the point completely: it breaks ABI w=
ith
>> older binaries, namely older binaries running on -CURRENT.  And not ju=
st
>> for that field.
>=20
> Especially since the main reason numbered sysctls exist is to support o=
ld
> binaries (and sources) that use them.
>=20

Does it look OK?

diff --git a/sys/vm/vm_param.h b/sys/vm/vm_param.h
index 44306be..0323473 100644
--- a/sys/vm/vm_param.h
+++ b/sys/vm/vm_param.h
@@ -82,8 +82,9 @@
 #define        VM_V_CACHE_MIN          7       /* cnt.v_cache_min */
 #define        VM_V_CACHE_MAX          8       /* cnt.v_cache_max */
 #define VM_V_PAGEOUT_FREE_MIN  9       /* cnt.v_pageout_free_min */
-#define VM_SWAPPING_ENABLED    10      /* swapping enabled */
-#define        VM_MAXID                11      /* number of valid vm ids=
 */
+/*#define      VM_PAGEOUT_ALGORITHM    10      pageout algorithm */
+#define VM_SWAPPING_ENABLED    11      /* swapping enabled */
+#define        VM_MAXID                12      /* number of valid vm ids=
 */

 #define CTL_VM_NAMES { \
        { 0, 0 }, \
@@ -96,6 +97,7 @@
        { "v_cache_min", CTLTYPE_UINT }, \
        { "v_cache_max", CTLTYPE_UINT }, \
        { "v_pageout_free_min", CTLTYPE_UINT}, \
+       { "obsolete_pageout_algorithm", CTLTYPE_INT}, \
        { "swap_enabled", CTLTYPE_INT},\
 }



--=20
Andrey Zonov


------enig2BWDOWUBELCWUQNFBHHBD
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.18 (Darwin)
Comment: GPGTools - http://gpgtools.org

iQEcBAEBAgAGBQJR9DH7AAoJEBWLemxX/CvTTl4H/Rbjk4vlXRxR2t/+z1Kcx4vS
5DpoM6DRfC3jZi3qfo2TxT600+5Ekyabhgpe6Gp3VM4TZVo0trC7lphPZ+I/0cv4
FWy9ElytnJIJYIlZcTOYXTLbD92itdkFKF2AqZL8Rttrt048CFy0XKYuHA9EIKSz
Tj1B+5EfG+6ymdmC4JcPqfTAZxuO20jOIUcTx/WCgne7l21bbloC+/AJ+kyAPfQF
HJAYktT134bpnP6lxigk67vTBALTskv08ucPXpKdVnxUpFy79riauweKKmNxibA2
TqczSDSULkSqC1ytTKCAWwhl5ZP9UsCO++RqSDag/rw8WpZPiOEm+h5Ykt9QNYE=
=Cdzu
-----END PGP SIGNATURE-----

------enig2BWDOWUBELCWUQNFBHHBD--



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