[PATCH] malloc: Add missing arena lock in mallinfo [BZ #22408]
Siddhesh Poyarekar
siddhesh@gotplt.org
Wed Nov 15 04:44:00 GMT 2017
On Wednesday 15 November 2017 09:23 AM, DJ Delorie wrote:
> I don't feel too strongly about this because the code is the same in the
> end, but it seems like extra work with a weak justification. These are
> the types of requirements that may put off a first-time contributor.
> Obviously this doesn't apply to Florian ;-)
I probably feel stronger about it because of having to sift through
commits that mixed in multiple bug fixes in a past life. For example,
RHEL-5 may want the arena lock fix in malloc_info but may not want the
change in malloc_info output format, in which case downstream will end
up having to figure out which parts should get excluded. In this
particular case it is simple enough to do the separation (which is why I
was OK with the idea of calling the commit as "fixing multiple problems"
but not completely happy), but it can get very complicated very fast.
Siddhesh
More information about the Libc-alpha
mailing list