From da3add97f8d4a1bb4d14989f5bd0b6f5f7bc9891 Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Sun, 9 Feb 2020 01:05:00 +0100 Subject: Move block writer and fragment table management out of block processor Signed-off-by: David Oberhollenzer --- lib/sqfs/block_processor/common.c | 25 ++++--------------------- lib/sqfs/block_processor/internal.h | 7 ++----- lib/sqfs/block_processor/serial.c | 6 +++--- lib/sqfs/block_processor/winpthread.c | 12 ++++++------ 4 files changed, 15 insertions(+), 35 deletions(-) (limited to 'lib/sqfs/block_processor') diff --git a/lib/sqfs/block_processor/common.c b/lib/sqfs/block_processor/common.c index 806a595..23991ea 100644 --- a/lib/sqfs/block_processor/common.c +++ b/lib/sqfs/block_processor/common.c @@ -20,30 +20,20 @@ void free_blk_list(sqfs_block_t *list) int block_processor_init(sqfs_block_processor_t *proc, size_t max_block_size, sqfs_compressor_t *cmp, unsigned int num_workers, - size_t max_backlog, size_t devblksz, sqfs_file_t *file) + size_t max_backlog, sqfs_block_writer_t *wr, + sqfs_frag_table_t *tbl) { proc->max_block_size = max_block_size; proc->num_workers = num_workers; proc->max_backlog = max_backlog; proc->cmp = cmp; - proc->file = file; - - proc->frag_tbl = sqfs_frag_table_create(0); - if (proc->frag_tbl == NULL) - return -1; - - proc->wr = sqfs_block_writer_create(file, devblksz, 0); - if (proc->wr == NULL) - return -1; + proc->frag_tbl = tbl; + proc->wr = wr; return 0; } void block_processor_cleanup(sqfs_block_processor_t *proc) { - if (proc->frag_tbl != NULL) - sqfs_frag_table_destroy(proc->frag_tbl); - if (proc->wr != NULL) - sqfs_block_writer_destroy(proc->wr); free_blk_list(proc->queue); free_blk_list(proc->done); free(proc->blk_current); @@ -51,13 +41,6 @@ void block_processor_cleanup(sqfs_block_processor_t *proc) free(proc); } -int sqfs_block_processor_write_fragment_table(sqfs_block_processor_t *proc, - sqfs_super_t *super) -{ - return sqfs_frag_table_write(proc->frag_tbl, proc->file, - super, proc->cmp); -} - int sqfs_block_processor_set_hooks(sqfs_block_processor_t *proc, void *user_ptr, const sqfs_block_hooks_t *hooks) { diff --git a/lib/sqfs/block_processor/internal.h b/lib/sqfs/block_processor/internal.h index 16f0edb..09e879b 100644 --- a/lib/sqfs/block_processor/internal.h +++ b/lib/sqfs/block_processor/internal.h @@ -61,10 +61,7 @@ struct sqfs_block_processor_t { unsigned int num_workers; size_t max_backlog; - sqfs_file_t *file; - sqfs_frag_table_t *frag_tbl; - sqfs_compressor_t *cmp; sqfs_block_t *frag_block; @@ -102,8 +99,8 @@ SQFS_INTERNAL void free_blk_list(sqfs_block_t *list); SQFS_INTERNAL int block_processor_init(sqfs_block_processor_t *proc, size_t max_block_size, sqfs_compressor_t *cmp, unsigned int num_workers, - size_t max_backlog, size_t devblksz, - sqfs_file_t *file); + size_t max_backlog, sqfs_block_writer_t *wr, + sqfs_frag_table_t *tbl); SQFS_INTERNAL void block_processor_cleanup(sqfs_block_processor_t *proc); diff --git a/lib/sqfs/block_processor/serial.c b/lib/sqfs/block_processor/serial.c index eedb19c..e18b67e 100644 --- a/lib/sqfs/block_processor/serial.c +++ b/lib/sqfs/block_processor/serial.c @@ -11,8 +11,8 @@ sqfs_block_processor_t *sqfs_block_processor_create(size_t max_block_size, sqfs_compressor_t *cmp, unsigned int num_workers, size_t max_backlog, - size_t devblksz, - sqfs_file_t *file) + sqfs_block_writer_t *wr, + sqfs_frag_table_t *tbl) { sqfs_block_processor_t *proc; @@ -22,7 +22,7 @@ sqfs_block_processor_t *sqfs_block_processor_create(size_t max_block_size, return NULL; if (block_processor_init(proc, max_block_size, cmp, num_workers, - max_backlog, devblksz, file)) { + max_backlog, wr, tbl)) { block_processor_cleanup(proc); return NULL; } diff --git a/lib/sqfs/block_processor/winpthread.c b/lib/sqfs/block_processor/winpthread.c index b16a17c..85416ab 100644 --- a/lib/sqfs/block_processor/winpthread.c +++ b/lib/sqfs/block_processor/winpthread.c @@ -98,8 +98,8 @@ sqfs_block_processor_t *sqfs_block_processor_create(size_t max_block_size, sqfs_compressor_t *cmp, unsigned int num_workers, size_t max_backlog, - size_t devblksz, - sqfs_file_t *file) + sqfs_block_writer_t *wr, + sqfs_frag_table_t *tbl) { sqfs_block_processor_t *proc; unsigned int i; @@ -117,7 +117,7 @@ sqfs_block_processor_t *sqfs_block_processor_create(size_t max_block_size, InitializeConditionVariable(&proc->done_cond); if (block_processor_init(proc, max_block_size, cmp, num_workers, - max_backlog, devblksz, file)) { + max_backlog, wr, tbl)) { goto fail; } @@ -178,8 +178,8 @@ sqfs_block_processor_t *sqfs_block_processor_create(size_t max_block_size, sqfs_compressor_t *cmp, unsigned int num_workers, size_t max_backlog, - size_t devblksz, - sqfs_file_t *file) + sqfs_block_writer_t *wr, + sqfs_frag_table_t *tbl) { sqfs_block_processor_t *proc; sigset_t set, oldset; @@ -199,7 +199,7 @@ sqfs_block_processor_t *sqfs_block_processor_create(size_t max_block_size, proc->done_cond = (pthread_cond_t)PTHREAD_COND_INITIALIZER; if (block_processor_init(proc, max_block_size, cmp, num_workers, - max_backlog, devblksz, file)) { + max_backlog, wr, tbl)) { goto fail_init; } -- cgit v1.2.3