[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Libunwind-devel] WIP patch: configurable cache size

From: Milian Wolff
Subject: Re: [Libunwind-devel] WIP patch: configurable cache size
Date: Fri, 25 Sep 2015 08:34:09 +0200
User-agent: KMail/5.0.44 pre (Linux/4.1.6-1-ARCH; KDE/5.15.0; x86_64; ; )

On Montag, 22. Dezember 2014 22:39:29 CEST Arun Sharma wrote:
> On Thu, Dec 11, 2014 at 7:12 PM, Milian Wolff <address@hidden> wrote:
> > On Thursday 11 December 2014 13:58:14 Milian Wolff wrote:
> >> Hey Arun,
> >>
> >> I'm still waiting for your mail you promised below ;-) Locally, I've
> >> rebased my patch on current master.
> >
> > FTR, attached you find the current state of my patch. Anything
> > missing/wrong, besides documentation?
> >
> >
> > -#define DWARF_LOG_UNW_CACHE_SIZE        7
> Why not retain DWARF_LOG_UNW_CACHE_SIZE and use it in a few places
> that use the constant 7?
> The rest of the code looks alright to me. Could you look into the make
> check failures introduced by the patch? I got 5 new failures.

Hey Arun,

there was a serious bug in the last version which was found by the unit tests.
Should be fixed now \o/

The problem was that empty caches where handled differently from a NONE cache
policy, leading to access of invalid pointers. This should be fixed now.
Additionally, the code is now using the default cache size at a central place,
and assumes that 0 indicates the default size. If no caching is wanted, then
one can simply set the policy to none.

Hope that's good now! There are still some test failures, but I also get them
in the unpatched master HEAD as of today.

Milian Wolff

Attachment: 0001-Make-DWARF-cache-size-configurable-via-unw_set_cache.patch
Description: Text Data

Attachment: signature.asc
Description: This is a digitally signed message part.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]