git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/16] packfile: carve out a new packfile store
@ 2025-08-19  8:19 Patrick Steinhardt
  2025-08-19  8:19 ` [PATCH 01/16] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
                   ` (22 more replies)
  0 siblings, 23 replies; 181+ messages in thread
From: Patrick Steinhardt @ 2025-08-19  8:19 UTC (permalink / raw)
  To: git

Hi,

information about a object database's packfiles is currently distributed
across two different structures:

  - `struct packed_git` contains the `next` pointer as well as the
    `mru_head`, both of which serve to store the list of packfiles.

  - `struct object_database` contains several fields that relate to the
    packfiles.

So we don't really have a central data structure that tracks our
packfiles, and consequently responsibilities aren't always clear cut.
A consequence for the upcoming pluggable object databases is that this
makes it very hard to move management of packfiles from the object
database level down into the object database source.

This patch series introduces a new `struct packfile_store`, which is
about to become the single source of truth for managing packfiles, and
carves out the packfile store subsystem.

This is the first step to make packfiles work with pluggable object
databases. Next steps will be to:

  - Move the `struct packed_git::next` and `struct packed::mru_head`
    pointers into the packfile store so that `struct packed_git` only
    tracks a single packfile.

  - Push the `struct packfile_store` down one level so that it's not
    hosted by the object database anymore, but instead by the object
    database source.

Thanks!

Patrick

---
Patrick Steinhardt (16):
      packfile: introduce a new `struct packfile_store`
      odb: move list of packfiles into `struct packfile_store`
      odb: move initialization bit into `struct packfile_store`
      odb: move packfile map into `struct packfile_store`
      odb: move MRU list of packfiles into `struct packfile_store`
      odb: move kept cache into `struct packfile_store`
      packfile: reorder functions to avoid function declaration
      packfile: refactor `prepare_packed_git()` to work on packfile store
      packfile: split up responsibilities of `reprepare_packed_git()`
      packfile: refactor `install_packed_git()` to work on packfile store
      packfile: always add packfiles to MRU when adding a pack
      packfile: introduce function to load and add packfiles
      packfile: move `get_multi_pack_index()` into "midx.c"
      packfile: remove `get_packed_git()`
      packfile: refactor `get_all_packs()` to work on packfile store
      packfile: refactor `get_packed_git_mru()` to work on packfile store

 builtin/backfill.c          |   2 +-
 builtin/cat-file.c          |   2 +-
 builtin/count-objects.c     |   2 +-
 builtin/fast-import.c       |   8 +-
 builtin/fsck.c              |   8 +-
 builtin/gc.c                |  12 +-
 builtin/grep.c              |   2 +-
 builtin/index-pack.c        |  10 +-
 builtin/pack-objects.c      |  22 ++--
 builtin/pack-redundant.c    |   4 +-
 builtin/receive-pack.c      |   2 +-
 builtin/repack.c            |   8 +-
 bulk-checkin.c              |   2 +-
 connected.c                 |   4 +-
 fetch-pack.c                |   4 +-
 http-backend.c              |   4 +-
 http.c                      |   4 +-
 http.h                      |   2 +-
 midx.c                      |  26 ++--
 midx.h                      |   2 +
 object-name.c               |   6 +-
 odb.c                       |  37 ++++--
 odb.h                       |  34 ++---
 pack-bitmap.c               |   4 +-
 pack-objects.c              |   2 +-
 packfile.c                  | 293 ++++++++++++++++++++++++--------------------
 packfile.h                  | 111 ++++++++++++++---
 server-info.c               |   2 +-
 t/helper/test-find-pack.c   |   2 +-
 t/helper/test-pack-mtimes.c |   2 +-
 transport-helper.c          |   2 +-
 31 files changed, 354 insertions(+), 271 deletions(-)


---
base-commit: c44beea485f0f2feaf460e2ac87fdd5608d63cf0
change-id: 20250806-b4-pks-packfiles-store-a44a608ca396


^ permalink raw reply	[flat|nested] 181+ messages in thread

end of thread, other threads:[~2025-09-29 21:39 UTC | newest]

Thread overview: 181+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-19  8:19 [PATCH 00/16] packfile: carve out a new packfile store Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 01/16] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
2025-08-19  9:47   ` Karthik Nayak
2025-08-20  4:58     ` Patrick Steinhardt
2025-08-19 17:32   ` Junio C Hamano
2025-08-20  4:58     ` Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 02/16] odb: move list of packfiles into " Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 03/16] odb: move initialization bit " Patrick Steinhardt
2025-08-19  9:57   ` Karthik Nayak
2025-08-19 16:24     ` Junio C Hamano
2025-08-20  8:04       ` Karthik Nayak
2025-08-22 23:50         ` Junio C Hamano
2025-08-26 12:19           ` [PATCH] Documentation: note styling for bit fields Karthik Nayak
2025-08-20  4:58     ` [PATCH 03/16] odb: move initialization bit into `struct packfile_store` Patrick Steinhardt
2025-08-20  6:24       ` Junio C Hamano
2025-08-19  8:19 ` [PATCH 04/16] odb: move packfile map " Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 05/16] odb: move MRU list of packfiles " Patrick Steinhardt
2025-08-20 12:44   ` Karthik Nayak
2025-08-20 19:20     ` Jeff King
2025-08-21  6:40       ` Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 06/16] odb: move kept cache " Patrick Steinhardt
2025-08-19 18:56   ` Junio C Hamano
2025-08-20  4:58     ` Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 07/16] packfile: reorder functions to avoid function declaration Patrick Steinhardt
2025-08-19 19:18   ` Junio C Hamano
2025-08-19  8:19 ` [PATCH 08/16] packfile: refactor `prepare_packed_git()` to work on packfile store Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 09/16] packfile: split up responsibilities of `reprepare_packed_git()` Patrick Steinhardt
2025-08-20 13:17   ` Karthik Nayak
2025-08-19  8:19 ` [PATCH 10/16] packfile: refactor `install_packed_git()` to work on packfile store Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 11/16] packfile: always add packfiles to MRU when adding a pack Patrick Steinhardt
2025-08-20 13:35   ` Karthik Nayak
2025-08-19  8:19 ` [PATCH 12/16] packfile: introduce function to load and add packfiles Patrick Steinhardt
2025-08-20 13:41   ` Karthik Nayak
2025-08-21  6:40     ` Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 13/16] packfile: move `get_multi_pack_index()` into "midx.c" Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 14/16] packfile: remove `get_packed_git()` Patrick Steinhardt
2025-08-20 13:50   ` Karthik Nayak
2025-08-21  6:40     ` Patrick Steinhardt
2025-08-20 13:51   ` Karthik Nayak
2025-08-19  8:19 ` [PATCH 15/16] packfile: refactor `get_all_packs()` to work on packfile store Patrick Steinhardt
2025-08-20 13:53   ` Karthik Nayak
2025-08-21  6:40     ` Patrick Steinhardt
2025-08-19  8:19 ` [PATCH 16/16] packfile: refactor `get_packed_git_mru()` " Patrick Steinhardt
2025-08-19 17:13 ` [PATCH 00/16] packfile: carve out a new " Junio C Hamano
2025-08-20 13:55 ` Karthik Nayak
2025-08-21  7:38 ` [PATCH v2 " Patrick Steinhardt
2025-08-21  7:38   ` [PATCH v2 01/16] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 02/16] odb: move list of packfiles into " Patrick Steinhardt
2025-08-25 23:42     ` Taylor Blau
2025-09-02  8:50       ` Patrick Steinhardt
2025-09-02 17:21         ` Taylor Blau
2025-09-02 17:42           ` Junio C Hamano
2025-09-03  5:58             ` Patrick Steinhardt
2025-09-11 23:16         ` Taylor Blau
2025-09-15  7:44           ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 03/16] odb: move initialization bit " Patrick Steinhardt
2025-08-26  1:40     ` Taylor Blau
2025-08-21  7:39   ` [PATCH v2 04/16] odb: move packfile map " Patrick Steinhardt
2025-08-26  1:41     ` Taylor Blau
2025-08-21  7:39   ` [PATCH v2 05/16] odb: move MRU list of packfiles " Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 06/16] odb: move kept cache " Patrick Steinhardt
2025-08-26  1:46     ` Taylor Blau
2025-09-02  8:50       ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 07/16] packfile: reorder functions to avoid function declaration Patrick Steinhardt
2025-08-26  1:47     ` Taylor Blau
2025-08-21  7:39   ` [PATCH v2 08/16] packfile: refactor `prepare_packed_git()` to work on packfile store Patrick Steinhardt
2025-08-26  1:58     ` Taylor Blau
2025-08-21  7:39   ` [PATCH v2 09/16] packfile: split up responsibilities of `reprepare_packed_git()` Patrick Steinhardt
2025-08-26  2:10     ` Taylor Blau
2025-09-02  8:50       ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 10/16] packfile: refactor `install_packed_git()` to work on packfile store Patrick Steinhardt
2025-08-26  2:11     ` Taylor Blau
2025-09-02  8:50       ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 11/16] packfile: always add packfiles to MRU when adding a pack Patrick Steinhardt
2025-08-27  1:04     ` Taylor Blau
2025-09-02  8:50       ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 12/16] packfile: introduce function to load and add packfiles Patrick Steinhardt
2025-08-27  1:12     ` Taylor Blau
2025-08-21  7:39   ` [PATCH v2 13/16] packfile: move `get_multi_pack_index()` into "midx.c" Patrick Steinhardt
2025-08-27  1:20     ` Taylor Blau
2025-08-21  7:39   ` [PATCH v2 14/16] packfile: remove `get_packed_git()` Patrick Steinhardt
2025-08-27  1:38     ` Taylor Blau
2025-09-02  8:50       ` Patrick Steinhardt
2025-09-11 23:25         ` Taylor Blau
2025-09-15  7:30           ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 15/16] packfile: refactor `get_all_packs()` to work on packfile store Patrick Steinhardt
2025-08-27  1:45     ` Taylor Blau
2025-09-02  8:51       ` Patrick Steinhardt
2025-09-11 23:33         ` Taylor Blau
2025-09-15  7:44           ` Patrick Steinhardt
2025-08-21  7:39   ` [PATCH v2 16/16] packfile: refactor `get_packed_git_mru()` " Patrick Steinhardt
2025-09-02 10:48 ` [PATCH v3 00/15] packfile: carve out a new " Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 01/15] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
2025-09-09  7:49     ` Karthik Nayak
2025-09-02 10:48   ` [PATCH v3 02/15] odb: move list of packfiles into " Patrick Steinhardt
2025-09-09  8:00     ` Karthik Nayak
2025-09-09 11:09       ` Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 03/15] odb: move initialization bit " Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 04/15] odb: move packfile map " Patrick Steinhardt
2025-09-09  8:22     ` Karthik Nayak
2025-09-09 11:01       ` Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 05/15] odb: move MRU list of packfiles " Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 06/15] odb: move kept cache " Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 07/15] packfile: reorder functions to avoid function declaration Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 08/15] packfile: refactor `prepare_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 09/15] packfile: split up responsibilities of `reprepare_packed_git()` Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 10/15] packfile: refactor `install_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 11/15] packfile: introduce function to load and add packfiles Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 12/15] packfile: move `get_multi_pack_index()` into "midx.c" Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 13/15] packfile: remove `get_packed_git()` Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 14/15] packfile: refactor `get_all_packs()` to work on packfile store Patrick Steinhardt
2025-09-02 10:48   ` [PATCH v3 15/15] packfile: refactor `get_packed_git_mru()` " Patrick Steinhardt
2025-09-02 16:40   ` [PATCH v3 00/15] packfile: carve out a new " Junio C Hamano
2025-09-11 23:34     ` Taylor Blau
2025-09-09  9:33   ` Karthik Nayak
2025-09-09 11:02 ` [PATCH v4 " Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 01/15] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 02/15] odb: move list of packfiles into " Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 03/15] odb: move initialization bit " Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 04/15] odb: move packfile map " Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 05/15] odb: move MRU list of packfiles " Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 06/15] odb: move kept cache " Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 07/15] packfile: reorder functions to avoid function declaration Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 08/15] packfile: refactor `prepare_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 09/15] packfile: split up responsibilities of `reprepare_packed_git()` Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 10/15] packfile: refactor `install_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 11/15] packfile: introduce function to load and add packfiles Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 12/15] packfile: move `get_multi_pack_index()` into "midx.c" Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 13/15] packfile: remove `get_packed_git()` Patrick Steinhardt
2025-09-11 23:37     ` Taylor Blau
2025-09-09 11:03   ` [PATCH v4 14/15] packfile: refactor `get_all_packs()` to work on packfile store Patrick Steinhardt
2025-09-09 11:03   ` [PATCH v4 15/15] packfile: refactor `get_packed_git_mru()` " Patrick Steinhardt
2025-09-10  7:35   ` [PATCH v4 00/15] packfile: carve out a new " Karthik Nayak
2025-09-11 23:40   ` Taylor Blau
2025-09-11 23:42     ` Taylor Blau
2025-09-15  7:25       ` Patrick Steinhardt
2025-09-15  8:54 ` [PATCH v5 " Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 01/15] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
2025-09-17 21:26     ` Justin Tobler
2025-09-23  9:34       ` Patrick Steinhardt
2025-09-24 21:56         ` Justin Tobler
2025-09-15  8:54   ` [PATCH v5 02/15] odb: move list of packfiles into " Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 03/15] odb: move initialization bit " Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 04/15] odb: move packfile map " Patrick Steinhardt
2025-09-17 22:15     ` Justin Tobler
2025-09-23  9:35       ` Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 05/15] odb: move MRU list of packfiles " Patrick Steinhardt
2025-09-17 21:59     ` Justin Tobler
2025-09-15  8:54   ` [PATCH v5 06/15] odb: move kept cache " Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 07/15] packfile: reorder functions to avoid function declaration Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 08/15] packfile: refactor `prepare_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 09/15] packfile: split up responsibilities of `reprepare_packed_git()` Patrick Steinhardt
2025-09-17 22:32     ` Justin Tobler
2025-09-23  9:34       ` Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 10/15] packfile: refactor `install_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 11/15] packfile: introduce function to load and add packfiles Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 12/15] packfile: move `get_multi_pack_index()` into "midx.c" Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 13/15] packfile: refactor `get_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 14/15] packfile: refactor `get_all_packs()` " Patrick Steinhardt
2025-09-15  8:54   ` [PATCH v5 15/15] packfile: refactor `get_packed_git_mru()` " Patrick Steinhardt
2025-09-23 10:16 ` [PATCH v6 00/15] packfile: carve out a new " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 01/15] packfile: introduce a new `struct packfile_store` Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 02/15] odb: move list of packfiles into " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 03/15] odb: move initialization bit " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 04/15] odb: move packfile map " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 05/15] odb: move MRU list of packfiles " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 06/15] odb: move kept cache " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 07/15] packfile: reorder functions to avoid function declaration Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 08/15] packfile: refactor `prepare_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 09/15] packfile: split up responsibilities of `reprepare_packed_git()` Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 10/15] packfile: refactor `install_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 11/15] packfile: introduce function to load and add packfiles Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 12/15] packfile: move `get_multi_pack_index()` into "midx.c" Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 13/15] packfile: refactor `get_packed_git()` to work on packfile store Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 14/15] packfile: refactor `get_all_packs()` " Patrick Steinhardt
2025-09-23 10:17   ` [PATCH v6 15/15] packfile: refactor `get_packed_git_mru()` " Patrick Steinhardt
2025-09-24 21:58   ` [PATCH v6 00/15] packfile: carve out a new " Justin Tobler
2025-09-25 16:08   ` Junio C Hamano
2025-09-26  5:26     ` Patrick Steinhardt
2025-09-28 22:05       ` Taylor Blau
2025-09-29 21:39         ` Patrick Steinhardt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).