diff options
Diffstat (limited to 'src/plugins/acl')
| -rw-r--r-- | src/plugins/acl/acl_hash_lookup_doc.md (renamed from src/plugins/acl/hash_lookup.md) | 2 | ||||
| -rw-r--r-- | src/plugins/acl/acl_multicore_doc.md (renamed from src/plugins/acl/acl-plugin.md) | 6 |
2 files changed, 5 insertions, 3 deletions
diff --git a/src/plugins/acl/hash_lookup.md b/src/plugins/acl/acl_hash_lookup_doc.md index 95524643e25..cb93df04bff 100644 --- a/src/plugins/acl/hash_lookup.md +++ b/src/plugins/acl/acl_hash_lookup_doc.md @@ -1,4 +1,4 @@ -ACL plugin constant-time lookup design +ACL plugin constant-time lookup design {#acl_hash_lookup} ====================================== The initial implementation of ACL plugin performs a trivial for() cycle, diff --git a/src/plugins/acl/acl-plugin.md b/src/plugins/acl/acl_multicore_doc.md index 1b44bca959c..b2cf7b9c6d4 100644 --- a/src/plugins/acl/acl-plugin.md +++ b/src/plugins/acl/acl_multicore_doc.md @@ -1,4 +1,4 @@ -Multicore support for ACL plugin +Multicore support for ACL plugin {#acl_multicore} ================================ This captures some considerations and design decisions that I have made, @@ -20,16 +20,18 @@ at the time of replacing the old ACL being checked, with the new ACL. In case an acl_add_replace is being used to replace the rules -within the existing entry, a reallocation of am->acls[X].rules +within the existing entry, a reallocation of `am->acls[X].rules` vector will happen and potentially a change in count. acl_match_5tuple() has the following code: +```{.c} a = am->acls + acl_index; for (i = 0; i < a->count; i++) { r = a->rules + i; . . . +``` Ideally we should be immune from a->rules changing, but the problem arises if the count changes in flight, |