From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f178.google.com (mail-pl1-f178.google.com [209.85.214.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C447426B2CE for ; Fri, 3 Jul 2026 22:46:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783118761; cv=none; b=UTbbdIW/UUxLBUZNH0H2jNNxgGHaYWfIG7k46yz5ADVmPKX6+FzMdByQ+yGNyrykxFci0iRTLGv6NkMURtB6WaoQ/CEeXNKW2JkTEE2389xZCL4361V2DQAWdDUHBgxt05WAMnIi/6ZaWDwJ2OMM9T7KTU0kaT8xhuyhX7OPoRI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783118761; c=relaxed/simple; bh=PSjFwIIno+ebiVWym6GDA9Tme7ZjXlcJe7QBT7YkdM0=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=Q8i7bRh8/wQEYvZRtZij1HJGHVW04g2SGOLBvcou6/6Ktey2hwvBx93J2dbFTKHKZc1jti7gHodmSsixXzS5hSj0j2wn+ZdBs/9VcfVDIV474mLff1QZcww86HO8ooD8pql40lLX6wkawxmKX6Y4XKG9VwKHgx9QDaK1torGkGU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=VGF4vYKX; arc=none smtp.client-ip=209.85.214.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="VGF4vYKX" Received: by mail-pl1-f178.google.com with SMTP id d9443c01a7336-2caa57a41cdso9657255ad.1 for ; Fri, 03 Jul 2026 15:46:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783118760; x=1783723560; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=OF4WsiF3Ail0sbbjny0CNLup/vJTyjDY+iJarnPvWzk=; b=VGF4vYKXgqWBuxf7M9YtuIfZvZYB/SEXc1OMBDpW+ScjV2z6StX+KHl8H0l0IxS03S UkfHVTBImOnh1IvHAxqcrDuUFScByjYf0ymoyb/NUg5wpv2dUp6xru29hCNpzQnZX6Z1 v6DuiNs5nOBExcUYkX54+293dV+wRNmQyeETLTqIBb0T5ZknBdePTooundTmU/uiZOR3 yXQvFj5rBBxxyFuBIuKqwtgahC54ERX4nUfqqY7R2ehpOlqoOFGbHXgobMqdETQhn1nQ yQOaGExLflzVCd3SHgkr/i8v1vZS5++D4GM0h83EX3BkmcSsj3ZXdQ5T051ysXTCpJTz 3hdA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783118760; x=1783723560; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=OF4WsiF3Ail0sbbjny0CNLup/vJTyjDY+iJarnPvWzk=; b=maYEWXZdVTgqczh4FpHvs1iIswe4w9ITVkywqOtTAExuHsRhlJAGEiUrq0tIFilcDh FyTOWjn7nUb08SgWG+mNqni++eyS++8BmMp5myU1i8M4u+bIUlH/Akx7pI0jNh4cvkfH cVzkQgu7WrQkgGVfCSFvKR3pjWTU/ceTBbM84PO7iJ10AiXyICYBRMhPuKIWdDCp1Qco pWUDyv+klBM6uki6+9B4u6JrV8CGUwZ6ySIhLu3Tn5I2/GNSUGH6suTsf3iBAgba92U7 wML1gLGm99/t202YFidzqwN8m+0BWUsqQbX4e0aoidQsTPTAZJsy8tenuvh8ZfpnTqXX 61Ww== X-Forwarded-Encrypted: i=1; AHgh+RrY1SzHiVlF/9e+fmLr1eaNyBttEBmxjmuG8CrWpdLk5TJqZb2l1tSlzXDbFPyrQLpLX/72w9CjiL7/9mw=@vger.kernel.org X-Gm-Message-State: AOJu0Ywox41e0yA+S2GceDGZQ0Iwy0BGiAzDPWORPG/W2s9cUm8Blfdb FakZg8c/l2pAR0MNRQyb0SQ7P1K4R/Db5RKj3uBedNIz27ckeO/Z4qmQoKkPWg== X-Gm-Gg: AfdE7cnyB5ZPMacrEE8HaDFQqJkoQJZoBTP8aA5Gr+cBOSi8Q59TKoqFdHVj0OPhX+f egtnqKjsPQU8Ea9OqHn/aXfAHdZNwIqiZnmaBzJP4gTKZ9Odq13oqvDQCdUyfSHktZMGu+SQK9x xMXRb/yKG4/2mDSJTyYP4MdGtFLr/WxixXDV5aovZpRLz44VsAH6j4XOSty4ouVaNR46ZPsc2UO 8xzDSUgVFB3xF82/aQZSTe+iOO6LgXi4ugqRbl8dSeEN0Z+7b38kSQFZRo0V11SRg/VeHqp18nZ Iw7ZFU+q11H5O+BEpmjSLOqy1PL2J4Wj7cZl0ROxk7eQLsBJU5RTx6YKDeM2IbIQjQiAiKTRbxl CuKvT2aZakZU93lMM1tbDO3o+TpkbVG/CquJATqV7CwAP8zDiZp2tVBP+y04bGM4wY3/MNfMUdP rEDkKLf4DS5uhXvNfI2FydExCAsUlWfPgCiA4jcTvKF+vebZnGBYw6SLXYtZRQlM0gpBRd6Yshb RTzs9fMZQ== X-Received: by 2002:a17:902:da2d:b0:2c9:97a7:3278 with SMTP id d9443c01a7336-2cbb9ef3df4mr8758615ad.41.1783118759764; Fri, 03 Jul 2026 15:45:59 -0700 (PDT) Received: from ryzen.lan ([2601:644:8000:7a86::e34]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2cad712b12fsm14967025ad.25.2026.07.03.15.45.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Jul 2026 15:45:59 -0700 (PDT) From: Rosen Penev To: linux-btrfs@vger.kernel.org Cc: Chris Mason , David Sterba , linux-kernel@vger.kernel.org (open list) Subject: [PATCHv2] btrfs: compression: allocate buckets with workspace Date: Fri, 3 Jul 2026 15:45:57 -0700 Message-ID: <20260703224557.488297-1-rosenp@gmail.com> X-Mailer: git-send-email 2.55.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Convert 3 allocations into one. Simplifies code slightly by removing extra kfree calls. SLUB merges same/similar sized structures for the named caches, so not a huge size concern: grep 2112 /proc/slabinfo sighand_cache 593 1335 2112 15 8 Signed-off-by: Rosen Penev --- v2: use normal arrays fs/btrfs/compression.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c index ffb6b52863a7..c62b5148d5ac 100644 --- a/fs/btrfs/compression.c +++ b/fs/btrfs/compression.c @@ -651,9 +651,9 @@ struct heuristic_ws { u8 *sample; u32 sample_size; /* Buckets store counters for each byte value */ - struct bucket_item *bucket; + struct bucket_item bucket[BUCKET_SIZE]; /* Sorting buffer */ - struct bucket_item *bucket_b; + struct bucket_item bucket_b[BUCKET_SIZE]; struct list_head list; }; @@ -664,8 +664,6 @@ static void free_heuristic_ws(struct list_head *ws) workspace = list_entry(ws, struct heuristic_ws, list); kvfree(workspace->sample); - kfree(workspace->bucket); - kfree(workspace->bucket_b); kfree(workspace); } @@ -681,14 +679,6 @@ static struct list_head *alloc_heuristic_ws(struct btrfs_fs_info *fs_info) if (!ws->sample) goto fail; - ws->bucket = kzalloc_objs(*ws->bucket, BUCKET_SIZE); - if (!ws->bucket) - goto fail; - - ws->bucket_b = kzalloc_objs(*ws->bucket_b, BUCKET_SIZE); - if (!ws->bucket_b) - goto fail; - INIT_LIST_HEAD(&ws->list); return &ws->list; fail: -- 2.55.0