Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
S
slash
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
uplex-varnish
slash
Commits
1a721622
Unverified
Commit
1a721622
authored
Jan 18, 2024
by
Nils Goroll
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
fellow_cache: support freeing an fcs which contains itself
parent
0eaed11d
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
4 additions
and
2 deletions
+4
-2
fellow_cache.c
src/fellow_cache.c
+4
-2
No files found.
src/fellow_cache.c
View file @
1a721622
...
...
@@ -1427,6 +1427,7 @@ static void
fellow_cache_seg_free(struct buddy_returns *memret,
struct fellow_cache_seg *fcs, unsigned deref)
{
struct buddy_ptr_extent mem;
CHECK_OBJ_NOTNULL(fcs, FELLOW_CACHE_SEG_MAGIC);
...
...
@@ -1457,8 +1458,7 @@ fellow_cache_seg_free(struct buddy_returns *memret,
FCS_INCORE, FCS_DISK);
fellow_cache_lru_chgbatch_apply(lcb);
}
if
(
fcs
->
alloc
.
ptr
)
AN
(
buddy_return_ptr_extent
(
memret
,
&
fcs
->
alloc
));
TAKE(mem, fcs->alloc);
AZ(fcs->fcs_onlru);
assert(fcs->refcnt == deref);
fcs->refcnt = 0;
...
...
@@ -1467,6 +1467,8 @@ fellow_cache_seg_free(struct buddy_returns *memret,
* at this point, the fcs is not consistent in all cases, e.g. FCS_EVICT
* has no memory - but this is the point where it does no longer exist
*/
if (mem.ptr)
AN(buddy_return_ptr_extent(memret, &mem));
}
/*
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment