[PATCH v3] Add support for "info auxv" on NetBSD
John Baldwin
jhb@FreeBSD.org
Fri Mar 27 16:31:33 GMT 2020
On 3/26/20 4:26 PM, Kamil Rytarowski wrote:
> Ping?
>
> On 20.03.2020 18:27, Kamil Rytarowski wrote:
>> Register nbsd_auxv_parse() that overloads the default (Linux-style)
>> AUXV parsing. On NetBSD the type parameter is defined as int32_t
>> for all architectures.
I would tone down some of the rhetoric. FreeBSD uses the default AUXV
parsing, and I think Solaris does as well, so describing it as Linux-only
isn't very accurate.
(Similarly, I think the earlier reviews I saw around ptrace() claimed that
NetBSD was the only OS to use LWPs with ptrace() in the log messages in
effect which isn't really true as both Solaris and FreeBSD use LWPs
happily with ptrace(), just using a different convention.)
>> }
>> +
>> +/* NetBSD-specific parser for AUXV data with. NetBSD follows the ELF
>> + specification, contrary to some other ELF Operating Systems. */
I would also tone this down a bit, and at least reference the correct
specification. The ELF spec doesn't define the layout of auxv_t. The
per-architecture psABI documents do. Also, just saying that you follow
the spec doesn't help. I would suggest something like:
/* NetBSD-specific parser for AUXV entries. NetBSD always uses an int
to store the type as defined in the SVR4 psABI specifications rather
than long as assumed by the default parser. */
--
John Baldwin
More information about the Gdb-patches
mailing list