[Freeipa-devel] [PATCH 92] Flush whole zone from cache during zone renaming/removal.

[Petr Spacek]

On 12/04/2012 02:36 PM, Adam Tkac wrote:
> On Thu, Nov 15, 2012 at 07:06:37PM +0100, Petr Spacek wrote:
>> >Hello,
>> >
>> >attached patch is preliminary implementation of selective zone flush.
>> >
>> >
>> >Implementation is not so straight-forward as I want to see. Before
>> >discussing the patch itself - can we consider per-zone caches? In
>> >that case, we can simply deallocate whole per-zone RBT and we are
>> >done.
>> >
>> >Pros:
>> >* Potentially better concurrency, simpler code, much less corner cases.
>> >
>> >Cons:
>> >* We have to look into Zone register before searching the cache.
>> >* It can limit concurrency ... with many extra small zones? I'm not sure.
> Hi Peter,
>
> In my opinion per-zone caches are better. Look into zone register isn't
> costly operation.
Second version of the patch with per-zone caches is attached. I cut all 
debugging code so this version could be considered as final.

-- 
Petr^2 Spacek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0092-2-Flush-whole-zone-from-cache-during-zone-renaming.patch
Type: text/x-patch
Size: 19136 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20121205/0954648b/attachment.bin>