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

Merge pull request #103 from arutk/arutk-force-separate-mngmt-io-queue

Browse Source 
Do not allow top volume I/O on management queue
This commit is contained in:
Michał Mielewczyk 2019-04-05 13:22:40 +02:00 committed by GitHub
commit f9e5d019df
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/ocf_core.c
View File

@ -196,6 +196,12 @@ static inline int ocf_core_validate_io(struct ocf_io *io)
if (!io->end) if (!io->end)
return -EINVAL; return -EINVAL;
/* Core volume I/O must not be queued on management queue - this would
* break I/O accounting code, resulting in use-after-free type of errors
* after cache detach, core remove etc. */
if (io->io_queue == io->volume->cache->mngt_queue)
return -EINVAL;
return 0; return 0;
} }
@ -226,7 +232,7 @@ void ocf_core_submit_io_mode(struct ocf_io *io, ocf_cache_mode_t cache_mode)
ret = ocf_core_validate_io(io); ret = ocf_core_validate_io(io);
if (ret < 0) { if (ret < 0) {
io->end(io, -EINVAL); io->end(io, ret);
return; return;
} }
@ -400,7 +406,7 @@ static void ocf_core_volume_submit_flush(struct ocf_io *io)
ret = ocf_core_validate_io(io); ret = ocf_core_validate_io(io);
if (ret < 0) { if (ret < 0) {
ocf_io_end(io, -EINVAL); ocf_io_end(io, ret);
return; return;
} }
@ -442,7 +448,7 @@ static void ocf_core_volume_submit_discard(struct ocf_io *io)
ret = ocf_core_validate_io(io); ret = ocf_core_validate_io(io);
if (ret < 0) { if (ret < 0) {
ocf_io_end(io, -EINVAL); ocf_io_end(io, ret);
return; return;
} }