@@ -956,12 +956,10 @@ static void free_workspace(int type, struct list_head *ws)
static void btrfs_init_workspace_manager(int type)
{
- const struct btrfs_compress_op *ops = btrfs_compress_op[type];
- struct workspace_manager *wsm = ops->workspace_manager;
+ struct workspace_manager *wsm;
struct list_head *workspace;
- wsm->ops = ops;
-
+ wsm = btrfs_compress_op[type]->workspace_manager;
INIT_LIST_HEAD(&wsm->idle_ws);
spin_lock_init(&wsm->ws_lock);
atomic_set(&wsm->total_ws, 0);
@@ -109,7 +109,6 @@ enum btrfs_compression_type {
};
struct workspace_manager {
- const struct btrfs_compress_op *ops;
struct list_head idle_ws;
spinlock_t ws_lock;
/* Number of free workspaces */
We can infer the ops from the type that is now passed to all functions that would need it, this makes workspace_manager::ops redundant and can be removed. Signed-off-by: David Sterba <dsterba@suse.com> --- fs/btrfs/compression.c | 6 ++---- fs/btrfs/compression.h | 1 - 2 files changed, 2 insertions(+), 5 deletions(-)