github/ocf
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #483 from arutk/repart_fix

Browse Source 
Fix repartitioning in request refresh path
This commit is contained in:
Michał Mielewczyk 2021-03-31 11:32:28 +02:00 committed by GitHub
commit 642794dcd7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/engine/engine_common.c
View File

@ -150,9 +150,13 @@ static void ocf_engine_update_req_info(struct ocf_cache *cache,
start_sector = ocf_map_line_start_sector(req, idx); start_sector = ocf_map_line_start_sector(req, idx);
end_sector = ocf_map_line_end_sector(req, idx); end_sector = ocf_map_line_end_sector(req, idx);
ENV_BUG_ON(entry->status != LOOKUP_HIT &&
entry->status != LOOKUP_MISS &&
entry->status != LOOKUP_REMAPPED &&
entry->status != LOOKUP_INSERTED);
/* Handle return value */ /* Handle return value */
switch (entry->status) { if (entry->status == LOOKUP_HIT) {
case LOOKUP_HIT:
if (metadata_test_valid_sec(cache, entry->coll_idx, if (metadata_test_valid_sec(cache, entry->coll_idx,
start_sector, end_sector)) { start_sector, end_sector)) {
req->info.hit_no++; req->info.hit_no++;
@ -169,7 +173,9 @@ static void ocf_engine_update_req_info(struct ocf_cache *cache,
start_sector, end_sector)) start_sector, end_sector))
req->info.dirty_all++; req->info.dirty_all++;
} }
}
if (entry->status == LOOKUP_HIT || entry->status == LOOKUP_REMAPPED) {
if (req->part_id != ocf_metadata_get_partition_id(cache, if (req->part_id != ocf_metadata_get_partition_id(cache,
entry->coll_idx)) { entry->coll_idx)) {
/* /*
@ -178,17 +184,12 @@ static void ocf_engine_update_req_info(struct ocf_cache *cache,
entry->re_part = true; entry->re_part = true;
req->info.re_part_no++; req->info.re_part_no++;
} }
}
break;
case LOOKUP_INSERTED: if (entry->status == LOOKUP_INSERTED ||
case LOOKUP_REMAPPED: entry->status == LOOKUP_REMAPPED) {
req->info.insert_no++; req->info.insert_no++;
break;
case LOOKUP_MISS:
break;
default:
ENV_BUG();
break;
} }
/* Check if cache hit is sequential */ /* Check if cache hit is sequential */