This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [patch] Fix mallinfo() to accumulate results for all arenas.
- From: Paul Pluzhnikov <ppluzhnikov at google dot com>
- To: Ulrich Drepper <drepper at gmail dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Thu, 1 Mar 2012 12:32:34 -0800
- Subject: Re: [patch] Fix mallinfo() to accumulate results for all arenas.
- Authentication-results: mr.google.com; spf=pass (google.com: domain of ppluzhnikov@google.com designates 10.216.139.143 as permitted sender) smtp.mail=ppluzhnikov@google.com; dkim=pass header.i=ppluzhnikov@google.com
- References: <20110823223856.E97B0190B0B@elbrus2.mtv.corp.google.com><CALoOobO-6-cvFH64Cv3OHYkUvoNSa2jLe0ZSD4bMox9Mv36Ocw@mail.gmail.com><CAOPLpQc=Sk2v1XEdocuL+c5ByE4TDLVFaQmwo3JCmmqYKLCTFg@mail.gmail.com><CALoOobM4oFQ2AHUTmmx-psP-3+yDSqEisBVPsrczwwbZrtP63g@mail.gmail.com><CAOPLpQfqHGPKKhJ12F2gUy1-r98rO-BFFf0g4N11FF+GeT6Dwg@mail.gmail.com> <CALoOobOuUXD_AiOURN-50K61_90uBfB5RNdyOYb_pcn8mWmZeg@mail.gmail.com>
On Mon, Sep 5, 2011 at 6:09 PM, Paul Pluzhnikov <ppluzhnikov@google.com> wrote:
>
> On Mon, Sep 5, 2011 at 5:55 PM, Ulrich Drepper <drepper@gmail.com> wrote:
> > On Mon, Sep 5, 2011 at 17:43, Paul Pluzhnikov <ppluzhnikov@google.com>
> > wrote:
> >> That would appear to be a rather weak argument to keep it broken. If
> >> it can be fixed, let's fix it.
> >
> > No, it's not. ?What is now counted is what people git so far and code
> > expects.
>
> IMHO that's bordering on insane.
>
> libc can provide correct answer, or the wrong answer. You insist that
> it must continue to forever give the wrong answer, because some
> applications may expect that wrong answer.
>
> But what possible use can an application have for the wrong answer?
> Isn't that application broken (by definition), if providing correct
> answer breaks it?
Greetings,
I would like to ask glibc maintainers to re-consider rejection of the patch.
Original problem description:
http://cygwin.com/ml/libc-alpha/2011-08/msg00116.html
Confirmed still broken in current git trunk.
Thanks,
--
Paul Pluzhnikov