Skip to content

nimble : Add LRU store overflow callback - #2273

Open
abhi152 wants to merge 1 commit into
apache:masterfrom
abhi152:feat/lru_bond_overflow_strategy
Open

nimble : Add LRU store overflow callback#2273
abhi152 wants to merge 1 commit into
apache:masterfrom
abhi152:feat/lru_bond_overflow_strategy

Conversation

@abhi152

@abhi152 abhi152 commented Jul 23, 2026

Copy link
Copy Markdown
Prefer least-recently-used bond eviction over round-robin when
storage is full; apps can register ble_store_util_status_lru.

@abhi152

abhi152 commented Jul 25, 2026

Copy link
Copy Markdown
Author

@sjanc could you please review the PR and let me know if there are any comments

@sjanc

sjanc commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

looks ok to me, but please fix compliance check (and also make commit message follow nimble convention)

@abhi152

abhi152 commented Sep 8, 2026

Copy link
Copy Markdown
Author

Sure @sjanc. Thanks for reviewing the PR I will make necessary changes quickly and we can proceed to merge it.

@abhi152
abhi152 force-pushed the feat/lru_bond_overflow_strategy branch from 454c250 to 829fd86 Compare September 10, 2026 11:34

@abhi152 abhi152 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sjanc I have resolved the code compliance issues, so you can approve so that other workflows will be tested

Round-robin overflow eviction can drop recently used bonds when
storage is full. Add ble_store_util_status_lru so apps can prefer
least-recently-used eviction based on bond_count when the store
maintains that field.
@abhi152
abhi152 force-pushed the feat/lru_bond_overflow_strategy branch from 829fd86 to 5d9d9ef Compare September 10, 2026 12:11
@abhi152 abhi152 changed the title feat(nimble): add LRU store overflow callback nimble : Add LRU store overflow callback Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants