Remove deprecated variables in sysctl(2)
Variables HW_PHYSMEM and HW_USERMEM were deprecated 13 years ago, maybe we can remove them from sysctl(2)? It was originally in sysctl(3) if someone want to do some history research https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib/libc/gen/Attic/sysctl.3#rev1.178 Index: sysctl.2 === RCS file: /home/reposync/src/lib/libc/sys/sysctl.2,v retrieving revision 1.44 diff -u -p -r1.44 sysctl.2 --- sysctl.218 May 2021 05:26:26 - 1.44 +++ sysctl.25 Oct 2021 16:03:28 - @@ -289,13 +289,11 @@ privileges may change the value. .It Dv HW_NCPUONLINE Ta "integer" Ta "no" .It Dv HW_PAGESIZE Ta "integer" Ta "no" .It Dv HW_PERFPOLICY Ta "string" Ta "yes" -.It Dv HW_PHYSMEM Ta "integer" Ta "no" .It Dv HW_PHYSMEM64 Ta "int64_t" Ta "no" .It Dv HW_PRODUCT Ta "string" Ta "no" .It Dv HW_SENSORS Ta "node" Ta "not applicable" .It Dv HW_SETPERF Ta "integer" Ta "yes" .It Dv HW_SMT Ta "integer" Ta "yes" -.It Dv HW_USERMEM Ta "integer" Ta "no" .It Dv HW_USERMEM64 Ta "int64_t" Ta "no" .It Dv HW_UUID Ta "string" Ta "no" .It Dv HW_VENDOR Ta "string" Ta "no" @@ -343,11 +341,6 @@ Can be one of .Dq auto , or .Dq high . -.It Dv HW_PHYSMEM -The total physical memory, in bytes. -This variable is deprecated; use -.Dv HW_PHYSMEM64 -instead. .It Dv HW_PHYSMEM64 Pq Va hw.physmem The total physical memory, in bytes. .It Dv HW_PRODUCT Pq Va hw.product @@ -393,11 +386,6 @@ is set to If set to 1, enable simultaneous multithreading (SMT) on CPUs that support it. Disabled by default. -.It Dv HW_USERMEM -The amount of available non-kernel memory in bytes. -This variable is deprecated; use -.Dv HW_USERMEM64 -instead. .It Dv HW_USERMEM64 Pq Va hw.usermem The amount of available non-kernel memory in bytes. .It Dv HW_UUID Pq Va hw.uuid
Re: Remove deprecated variables in sysctl(2)
Solene Rapenne wrote: > Variables HW_PHYSMEM and HW_USERMEM were deprecated 13 years ago, > maybe we can remove them from sysctl(2)? > > It was originally in sysctl(3) if someone want to do some history > research > https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib/libc/gen/Attic/sysctl.3#rev1.178 In a better world, HW_PHYSMEM and HW_USERMEM should become 64-bit by default, aliases for the 64-bit hackery that was introduced. Then everyone is happy.
Re: Remove deprecated variables in sysctl(2)
Stuart Henderson wrote: > On 2021/10/05 18:11, Solene Rapenne wrote: > > Variables HW_PHYSMEM and HW_USERMEM were deprecated 13 years ago, > > maybe we can remove them from sysctl(2)? > > "deprecated" doesn't mean removed or disabled, it just means that one > shouldn't use them. These are still in system headers and supported by > the kernel, and could possibly be used by ports. Yep. Please do not remove.
Re: Remove deprecated variables in sysctl(2)
On 2021/10/05 18:11, Solene Rapenne wrote: > Variables HW_PHYSMEM and HW_USERMEM were deprecated 13 years ago, > maybe we can remove them from sysctl(2)? "deprecated" doesn't mean removed or disabled, it just means that one shouldn't use them. These are still in system headers and supported by the kernel, and could possibly be used by ports. > It was originally in sysctl(3) if someone want to do some history > research > https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib/libc/gen/Attic/sysctl.3#rev1.178 > > Index: sysctl.2 > === > RCS file: /home/reposync/src/lib/libc/sys/sysctl.2,v > retrieving revision 1.44 > diff -u -p -r1.44 sysctl.2 > --- sysctl.2 18 May 2021 05:26:26 - 1.44 > +++ sysctl.2 5 Oct 2021 16:03:28 - > @@ -289,13 +289,11 @@ privileges may change the value. > .It Dv HW_NCPUONLINE Ta "integer" Ta "no" > .It Dv HW_PAGESIZE Ta "integer" Ta "no" > .It Dv HW_PERFPOLICY Ta "string" Ta "yes" > -.It Dv HW_PHYSMEM Ta "integer" Ta "no" > .It Dv HW_PHYSMEM64 Ta "int64_t" Ta "no" > .It Dv HW_PRODUCT Ta "string" Ta "no" > .It Dv HW_SENSORS Ta "node" Ta "not applicable" > .It Dv HW_SETPERF Ta "integer" Ta "yes" > .It Dv HW_SMT Ta "integer" Ta "yes" > -.It Dv HW_USERMEM Ta "integer" Ta "no" > .It Dv HW_USERMEM64 Ta "int64_t" Ta "no" > .It Dv HW_UUID Ta "string" Ta "no" > .It Dv HW_VENDOR Ta "string" Ta "no" > @@ -343,11 +341,6 @@ Can be one of > .Dq auto , > or > .Dq high . > -.It Dv HW_PHYSMEM > -The total physical memory, in bytes. > -This variable is deprecated; use > -.Dv HW_PHYSMEM64 > -instead. > .It Dv HW_PHYSMEM64 Pq Va hw.physmem > The total physical memory, in bytes. > .It Dv HW_PRODUCT Pq Va hw.product > @@ -393,11 +386,6 @@ is set to > If set to 1, enable simultaneous multithreading (SMT) on CPUs that > support it. > Disabled by default. > -.It Dv HW_USERMEM > -The amount of available non-kernel memory in bytes. > -This variable is deprecated; use > -.Dv HW_USERMEM64 > -instead. > .It Dv HW_USERMEM64 Pq Va hw.usermem > The amount of available non-kernel memory in bytes. > .It Dv HW_UUID Pq Va hw.uuid >