vllm_backend.rs 23.5 KB
Newer Older
1
// SPDX-FileCopyrightText: Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2
3
// SPDX-License-Identifier: Apache-2.0

4
//! # KV Manager (vLLM Backend)
5
6
//! A synchronous implementation of a block manager that handles MoveBlock signals for caching KV blocks.
//!
7
8
//! Uses [`HashCache`] for O(1) block lookups with active/inactive pool management.
//!
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
//! ## Block Operations
//! The KV manager processes four types of MoveBlock signals:
//!
//! ### Use
//! - Checks if block exists in active pool → increment reference count
//! - If in inactive pool → move to active pool
//! - If neither → try evicting from inactive pool to make room
//! - If inactive pool is empty → pre-empt the oldest running request
//!
//! ### Destroy
//! - Removes the block from the active pool
//!
//! ### Deref
//! - Decrements reference count of a block in active pool
//! - If count reaches zero → move block to inactive pool
//!
//! ### Promote
//! - Converts a partial block (uuid) into a full block (global block hash)
//!
//! ## Preemption
//! If a Use operation fails (typically due to insufficient space), a false boolean signal
//! is returned to the scheduler for preemption. Initial KV block allocations for new requests
31
//! should not fail due to the capacity checking during scheduling.
32
33
34
35
36
//!
//! ## NOTE
//! For simplicity (or non-simplicity), reference counting is tracked manually instead of using
//! the more idiomatic built-in Arc reference counter. This can be considered a shadow / mirror
//! implementation of the main block manager.
37
38
39
use crate::cache::HashCache;
use crate::common::protocols::{KvCacheEventSink, MoveBlock, PrefillCost};
use crate::common::sequence::ActiveSequence;
40
use dynamo_kv_router::protocols::{
Yan Ru Pei's avatar
Yan Ru Pei committed
41
42
43
    ExternalSequenceBlockHash, KvCacheEvent, KvCacheEventData, KvCacheRemoveData, KvCacheStoreData,
    KvCacheStoredBlockData, LocalBlockHash,
};
44
use dynamo_runtime::config::environment_names::mocker;
45
46
use dynamo_tokens::blocks::UniqueBlock;
use dynamo_tokens::{BlockHash, SequenceHash};
47
use std::collections::HashMap;
48
49
50
51
52
53
54
55
56
57
use std::env;
use std::sync::{Arc, LazyLock};
use std::time::{SystemTime, UNIX_EPOCH};

/// Check the env var to enable KV cache allocation/eviction trace logs.
static KV_CACHE_TRACE_ENABLED: LazyLock<bool> = LazyLock::new(|| {
    env::var(mocker::DYN_MOCKER_KV_CACHE_TRACE)
        .map(|v| v == "1" || v.eq_ignore_ascii_case("true"))
        .unwrap_or(false)
});
58
59

pub struct KvManager {
60
    cache: HashCache,
61
    block_size: usize,
62
    kv_event_sink: Option<Arc<dyn KvCacheEventSink>>,
Yan Ru Pei's avatar
Yan Ru Pei committed
63
64
    dp_rank: u32,
    next_event_id: u64,
65
66
67
}

impl KvManager {
68
    pub fn new(max_capacity: usize, block_size: usize) -> Self {
69
        Self::new_with_event_sink(max_capacity, block_size, None, 0)
70
71
    }

72
    pub fn new_with_event_sink(
73
74
        max_capacity: usize,
        block_size: usize,
75
        kv_event_sink: Option<Arc<dyn KvCacheEventSink>>,
Yan Ru Pei's avatar
Yan Ru Pei committed
76
        dp_rank: u32,
77
    ) -> Self {
78
        debug_assert!(max_capacity > 0, "max_capacity must be > 0");
79
        if kv_event_sink.is_some() {
Yan Ru Pei's avatar
Yan Ru Pei committed
80
            tracing::info!(
81
                "KvManager initialized with event sink for DP rank {dp_rank} with block_size {block_size}"
Yan Ru Pei's avatar
Yan Ru Pei committed
82
            );
83
        }
Yan Ru Pei's avatar
Yan Ru Pei committed
84

85
        KvManager {
86
            cache: HashCache::new(max_capacity),
87
            block_size,
88
            kv_event_sink,
Yan Ru Pei's avatar
Yan Ru Pei committed
89
90
            dp_rank,
            next_event_id: 0,
91
92
93
        }
    }

94
    /// Converts stored/removed blocks into KvCacheEventData and publishes if sink is available.
Yan Ru Pei's avatar
Yan Ru Pei committed
95
96
97
98
    fn publish_kv_event(
        &mut self,
        full_blocks: Vec<SequenceHash>,
        local_hashes: &[BlockHash],
99
        parent_hash: Option<u64>,
Yan Ru Pei's avatar
Yan Ru Pei committed
100
        is_store: bool,
101
        token_ids: Option<Vec<Vec<u32>>>,
102
    ) {
Yan Ru Pei's avatar
Yan Ru Pei committed
103
104
105
106
        if full_blocks.is_empty() {
            return;
        }

107
        if *KV_CACHE_TRACE_ENABLED {
108
109
            let active_len = self.cache.num_active();
            let inactive_len = self.cache.num_inactive();
110
            let free_blocks = self
111
112
                .cache
                .max_capacity()
113
114
115
                .saturating_sub(active_len)
                .saturating_sub(inactive_len);
            let event = if is_store { "allocation" } else { "eviction" };
116
117
118
119
            let timestamp_ms = SystemTime::now()
                .duration_since(UNIX_EPOCH)
                .unwrap_or_default()
                .as_millis() as u64;
120
121
122
123
124
125
126
127
            tracing::info!(
                event,
                timestamp_ms,
                block_ids = ?&full_blocks,
                block_size = self.block_size,
                free_blocks_after = free_blocks,
                active_blocks = active_len,
                inactive_blocks = inactive_len,
128
                total_blocks = self.cache.max_capacity(),
129
130
131
132
133
                dp_rank = self.dp_rank,
                "KV cache trace"
            );
        }

134
135
136
137
        let Some(ref sink) = self.kv_event_sink else {
            return;
        };

Yan Ru Pei's avatar
Yan Ru Pei committed
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
        let event_data = if is_store {
            let num_blocks = full_blocks.len();
            let local_hashes_slice = &local_hashes[local_hashes
                .len()
                .checked_sub(num_blocks)
                .expect("local hashes fewer than stored blocks")..];

            KvCacheEventData::Stored(KvCacheStoreData {
                parent_hash: parent_hash.map(ExternalSequenceBlockHash),
                blocks: full_blocks
                    .into_iter()
                    .zip(local_hashes_slice.iter())
                    .map(|(global_hash, local_hash)| KvCacheStoredBlockData {
                        block_hash: ExternalSequenceBlockHash(global_hash),
                        tokens_hash: LocalBlockHash(*local_hash),
153
                        mm_extra_info: None,
Yan Ru Pei's avatar
Yan Ru Pei committed
154
155
156
157
158
159
160
161
162
163
164
                    })
                    .collect(),
            })
        } else {
            KvCacheEventData::Removed(KvCacheRemoveData {
                block_hashes: full_blocks
                    .into_iter()
                    .map(ExternalSequenceBlockHash)
                    .collect(),
            })
        };
165
        // Use incremental event ID starting from 0 and incrementing by 1 for each event.
Yan Ru Pei's avatar
Yan Ru Pei committed
166
167
168
169
170
171
172
173
174
        let event_id = self.next_event_id;
        self.next_event_id += 1;

        let event = KvCacheEvent {
            event_id,
            data: event_data,
            dp_rank: self.dp_rank,
        };

175
        if let Err(e) = sink.publish(event, token_ids.as_deref()) {
Yan Ru Pei's avatar
Yan Ru Pei committed
176
            tracing::warn!("Failed to publish KV event: {e}");
177
178
179
        }
    }

180
181
182
183
184
185
186
187
    /// Process a MoveBlock instruction synchronously.
    ///
    /// For `MoveBlock::Use`, returns the number of blocks successfully allocated.
    /// On partial failure, blocks 0..N are committed but block N+1 could not be
    /// allocated. Callers should use the return value to track partial progress.
    ///
    /// For other variants, returns the total block count (they always succeed or panic).
    pub fn process(&mut self, event: &MoveBlock) -> usize {
188
        match event {
189
            MoveBlock::Use(hashes, local_hashes, token_ids, parent) => {
190
                let mut blocks_stored = Vec::<u64>::new();
191
192
                let mut stored_token_ids: Option<Vec<Vec<u32>>> =
                    token_ids.as_ref().map(|_| Vec::new());
193

194
                let mut parent_block: Option<&UniqueBlock> = parent.as_ref();
195
                let mut allocated = 0;
196
                for (i, hash) in hashes.iter().enumerate() {
197
                    // First check if it already exists in active blocks
198
                    if self.cache.contains_active(hash) {
199
                        // Block already active, just increment reference count
200
                        self.cache.increment_ref(hash);
201
                        parent_block = Some(hash);
202
                        allocated += 1;
203
204
205
206
                        continue;
                    }

                    // Then check if it exists in inactive and move it to active if found
207
                    if self.cache.reactivate(hash) {
208
                        parent_block = Some(hash);
209
                        allocated += 1;
210
211
212
213
                        continue;
                    }

                    // If at max capacity, evict the oldest entry from inactive blocks
214
215
                    if self.cache.is_at_capacity() {
                        let Some(evicted) = self.cache.evict_inactive() else {
216
                            return allocated;
217
                        };
Yan Ru Pei's avatar
Yan Ru Pei committed
218
219
220
221
                        tracing::trace!(
                            "Evicting block from inactive pool: {evicted:?}, dp_rank={}",
                            self.dp_rank
                        );
222
                        if let UniqueBlock::FullBlock(evicted_full_block) = evicted {
223
                            self.publish_kv_event(vec![evicted_full_block], &[], None, false, None);
224
225
226
227
                        }
                    }

                    // Now insert the new block in active blocks with reference count 1
228
                    self.cache.insert_active(hash.clone(), 1);
229
                    allocated += 1;
230
231
                    // Track blocks for trace/event
                    if let UniqueBlock::FullBlock(stored_full_block) = hash {
232
                        blocks_stored.push(*stored_full_block);
233
234
235
                        if let Some(ref mut stids) = stored_token_ids {
                            stids.push(token_ids.as_ref().unwrap()[i].clone());
                        }
236
                    }
237
                }
238
239
240
241
242
243

                let parent_hash = match parent_block {
                    None => None,
                    Some(UniqueBlock::FullBlock(block)) => Some(*block),
                    Some(UniqueBlock::PartialBlock(_)) => panic!("parent block cannot be partial"),
                };
244
245
246
247
248
249
250
                self.publish_kv_event(
                    blocks_stored,
                    local_hashes,
                    parent_hash,
                    true,
                    stored_token_ids,
                );
251
                return allocated;
252
            }
253

254
            MoveBlock::Destroy(hashes) => {
255
                let mut blocks_destroyed = Vec::<u64>::new();
Yan Ru Pei's avatar
Yan Ru Pei committed
256
257
                // Process blocks in order (already reversed by caller if needed)
                for hash in hashes.iter() {
258
                    self.cache.remove_active(hash).unwrap();
259
                    // Track blocks for batch sending
260
                    if let UniqueBlock::FullBlock(destroyed_full_block) = hash {
261
                        blocks_destroyed.push(*destroyed_full_block);
262
                    }
263
                }
264

265
                self.publish_kv_event(blocks_destroyed, &[], None, false, None);
266
            }
267

268
            MoveBlock::Deref(hashes) => {
Yan Ru Pei's avatar
Yan Ru Pei committed
269
270
                // Process blocks in order (already reversed by caller if needed)
                for hash in hashes.iter() {
271
                    // Decrement reference count and check if we need to move to inactive
272
273
                    if let Some(ref_count) = self.cache.get_active_ref_count(hash) {
                        if ref_count == 0 {
274
275
276
277
                            panic!("Negative reference count would be encountered after Deref.");
                        }

                        // If reference count reaches zero, remove from active and move to inactive
278
279
280
281
                        if ref_count == 1 {
                            self.cache.deactivate(hash);
                        } else {
                            self.cache.decrement_ref(hash);
282
283
284
285
                        }
                    }
                }
            }
286

287
            MoveBlock::Promote(uuid, hash, parent_hash, local_hash, promote_token_ids) => {
288
289
290
                let uuid_block = UniqueBlock::PartialBlock(*uuid);
                let hash_block = UniqueBlock::FullBlock(*hash);

Yan Ru Pei's avatar
Yan Ru Pei committed
291
                assert_eq!(
292
                    self.cache.remove_active(&uuid_block),
Yan Ru Pei's avatar
Yan Ru Pei committed
293
294
295
296
                    Some(1),
                    "uuid_block {uuid_block:?} should exist and be unique with ref_count=1"
                );

297
298
299
300
301
302
303
                let hash_ref_count = self.cache.get_active_ref_count(&hash_block);
                // Block is new if it's not in active and not in inactive
                let is_new = if hash_ref_count.is_some() {
                    false
                } else {
                    !self.cache.remove_inactive(&hash_block)
                };
304

305
306
                self.cache
                    .insert_active(hash_block, hash_ref_count.unwrap_or(0) + 1);
307
308

                if is_new {
309
310
311
312
313
314
315
                    self.publish_kv_event(
                        vec![*hash],
                        &[*local_hash],
                        *parent_hash,
                        true,
                        promote_token_ids.as_ref().map(|t| vec![t.clone()]),
                    );
316
                }
317
318
319
            }
        }

320
        1
321
322
    }

323
    /// Get the count of blocks that aren't in active or inactive pools
324
325
326
    pub fn probe_new_blocks(&self, blocks: &[UniqueBlock]) -> usize {
        blocks
            .iter()
327
            .filter(|&block| !self.cache.contains(block))
328
329
330
331
332
            .count()
    }

    /// Get the current capacity (active blocks + inactive blocks)
    pub fn current_capacity(&self) -> usize {
333
        self.cache.current_capacity()
334
335
336
337
    }

    /// Get the current capacity as a percentage of the maximum capacity
    pub fn current_capacity_perc(&self) -> f64 {
338
        self.cache.current_capacity() as f64 / self.cache.max_capacity() as f64
339
340
341
342
    }

    /// Get the number of active blocks
    pub fn num_active_blocks(&self) -> usize {
343
        self.cache.num_active()
344
345
    }

346
347
    /// Get the percentage of active blocks relative to maximum capacity
    pub fn get_active_perc(&self) -> f64 {
348
        self.cache.num_active() as f64 / self.cache.max_capacity() as f64
349
350
    }

351
352
    /// Get the number of inactive blocks
    pub fn num_inactive_blocks(&self) -> usize {
353
        self.cache.num_inactive()
354
355
356
357
    }

    /// Get the keys of inactive blocks
    pub fn get_inactive_blocks(&self) -> Vec<&UniqueBlock> {
358
        self.cache.inactive_keys().collect()
359
360
361
362
    }

    /// Get the keys of active blocks
    pub fn get_active_blocks(&self) -> Vec<&UniqueBlock> {
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
        self.cache.active_keys().collect()
    }

    pub fn max_capacity(&self) -> usize {
        self.cache.max_capacity()
    }

    pub fn block_size(&self) -> usize {
        self.block_size
    }

    pub fn dp_rank(&self) -> u32 {
        self.dp_rank
    }

    /// Direct access to active blocks map (for tests).
    pub fn active_blocks(&self) -> &HashMap<UniqueBlock, usize> {
        self.cache.active_blocks()
381
382
383
    }

    /// Check if a sequence can be scheduled and calculate cost if possible
384
385
    pub fn get_prefill_cost(&self, sequence: &ActiveSequence) -> PrefillCost {
        let seq_blocks = sequence.unique_blocks();
386
387
388
389
390

        // Find the longest prefix that exists in cache
        // We must stop at the first cache miss since KV states are computed sequentially
        let mut overlap_blocks = 0;
        for block in seq_blocks {
391
            if !self.cache.contains(block) {
392
393
394
395
396
397
398
399
400
401
                // First cache miss - can't use anything after this point
                break;
            }
            overlap_blocks += 1;
        }

        let new_blocks = seq_blocks.len() - overlap_blocks;
        // Clamp cached_tokens to handle partial blocks (last block may have < block_size tokens)
        let cached_tokens = (overlap_blocks * self.block_size).min(sequence.num_input_tokens());
        let new_tokens = sequence.num_input_tokens() - cached_tokens;
402

403
404
        PrefillCost {
            new_blocks,
405
            new_tokens,
406
        }
407
408
409
410
411
412
413
414
415
416
417
418
    }
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn test_failure_on_max_capacity() {
        // Create a KvManager with 10 blocks capacity
        let mut manager = KvManager::new(10, 16);

419
420
        // Helper function to use multiple blocks that returns the count allocated
        fn use_blocks(manager: &mut KvManager, ids: Vec<u64>) -> usize {
Yan Ru Pei's avatar
Yan Ru Pei committed
421
422
            let blocks: Vec<_> = ids.iter().map(|&id| UniqueBlock::FullBlock(id)).collect();
            let hashes: Vec<_> = ids.into_iter().collect();
423
            manager.process(&MoveBlock::Use(blocks, hashes, None, None))
424
425
426
427
        }

        // First use 10 blocks (0 to 9) in a batch
        let response = use_blocks(&mut manager, (0..10).collect());
428
        assert_eq!(response, 10, "Expected all 10 blocks allocated");
429
430
431
432

        // Verify we are at capacity
        assert_eq!(manager.current_capacity(), 10);

433
        // The 11th block should return 0, not panic
434
        let response = use_blocks(&mut manager, vec![10]);
435
436
437
        assert_eq!(
            response, 0,
            "Expected 0 blocks allocated when exceeding max capacity"
438
439
440
441
442
        );
    }

    #[test]
    fn test_block_lifecycle_stringent() {
Yan Ru Pei's avatar
Yan Ru Pei committed
443
444
        // Create a KvManager with 10 blocks capacity (no KV event publisher for tests)
        let mut manager = KvManager::new(10, 16);
445
446
447

        // Helper function to use multiple blocks
        fn use_blocks(manager: &mut KvManager, ids: Vec<u64>) {
Yan Ru Pei's avatar
Yan Ru Pei committed
448
449
            let blocks: Vec<_> = ids.iter().map(|&id| UniqueBlock::FullBlock(id)).collect();
            let hashes: Vec<_> = ids.into_iter().collect();
450
            manager.process(&MoveBlock::Use(blocks, hashes, None, None));
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
        }

        // Helper function to destroy multiple blocks
        fn destroy_blocks(manager: &mut KvManager, ids: Vec<u64>) {
            let blocks = ids.into_iter().map(UniqueBlock::FullBlock).collect();
            manager.process(&MoveBlock::Destroy(blocks));
        }

        // Helper function to deref multiple blocks
        fn deref_blocks(manager: &mut KvManager, ids: Vec<u64>) {
            let blocks = ids.into_iter().map(UniqueBlock::FullBlock).collect();
            manager.process(&MoveBlock::Deref(blocks));
        }

        // Helper function to check if active blocks contain expected blocks with expected ref counts
        fn assert_active_blocks(manager: &KvManager, expected_blocks: &[(u64, usize)]) {
            assert_eq!(
                manager.active_blocks().len(),
                expected_blocks.len(),
                "Active blocks count doesn't match expected"
            );

            for &(id, ref_count) in expected_blocks {
                let block = UniqueBlock::FullBlock(id);
                assert!(
                    manager.active_blocks().contains_key(&block),
477
                    "Block {id} not found in active blocks",
478
479
480
481
                );
                assert_eq!(
                    manager.active_blocks().get(&block),
                    Some(&ref_count),
482
                    "Block {id} has wrong reference count",
483
484
485
486
487
488
489
490
491
492
493
                );
            }
        }

        // Helper function to check if inactive blocks contain expected blocks
        fn assert_inactive_blocks(
            manager: &KvManager,
            expected_size: usize,
            expected_blocks: &[u64],
        ) {
            let inactive_blocks = manager.get_inactive_blocks();
494
            let inactive_blocks_count = manager.num_inactive_blocks();
495
496
497
498
499
500
501
502
503
504

            assert_eq!(
                inactive_blocks_count, expected_size,
                "Inactive blocks count doesn't match expected"
            );

            for &id in expected_blocks {
                let block = UniqueBlock::FullBlock(id);
                assert!(
                    inactive_blocks.iter().any(|&b| *b == block),
505
                    "Block {id} not found in inactive blocks",
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
                );
            }
        }

        // First use blocks 0, 1, 2, 3, 4 in a batch
        use_blocks(&mut manager, (0..5).collect());

        // Then use blocks 0, 1, 5, 6 in a batch
        use_blocks(&mut manager, vec![0, 1, 5, 6]);

        // Check that the blocks 0 and 1 are in active blocks, both with reference counts of 2
        assert_active_blocks(
            &manager,
            &[(0, 2), (1, 2), (2, 1), (3, 1), (4, 1), (5, 1), (6, 1)],
        );

        // Now destroy block 4
        destroy_blocks(&mut manager, vec![4]);

        // And deref blocks 3, 2, 1, 0 in this order as a batch
        deref_blocks(&mut manager, vec![0, 1, 2, 3]);

        // Check that the inactive_blocks is size 2 (via num_objects) and contains 3 and 2
        assert_inactive_blocks(&manager, 2, &[3, 2]);
        assert_active_blocks(&manager, &[(0, 1), (1, 1), (5, 1), (6, 1)]);

        // Now destroy block 6
        destroy_blocks(&mut manager, vec![6]);

        // And deref blocks 5, 1, 0 as a batch
        deref_blocks(&mut manager, vec![0, 1, 5]);

        // Check that the inactive_blocks is size 5, and contains 0, 1, 2, 3, 5
        assert_inactive_blocks(&manager, 5, &[0, 1, 2, 3, 5]);
        assert_active_blocks(&manager, &[]);

        // Now use 0, 1, 2, 7, 8, 9 as a batch
        use_blocks(&mut manager, vec![0, 1, 2, 7, 8, 9]);

        // Check that the inactive_blocks is size 2, and contains 3 and 5
        assert_inactive_blocks(&manager, 2, &[3, 5]);
        assert_active_blocks(&manager, &[(0, 1), (1, 1), (2, 1), (7, 1), (8, 1), (9, 1)]);

        // Test the new_blocks method - only block 4 should be new out of [0,1,2,3,4]
        let blocks_to_check: Vec<UniqueBlock> = vec![0, 1, 2, 3, 4]
            .into_iter()
            .map(UniqueBlock::FullBlock)
            .collect();
        assert_eq!(manager.probe_new_blocks(&blocks_to_check), 1);

        // Now use blocks 10, 11, 12 as a batch
        use_blocks(&mut manager, vec![10, 11, 12]);

        // Check that the inactive_blocks is size 1 and contains only 5
        assert_inactive_blocks(&manager, 1, &[5]);
561
562

        use_blocks(&mut manager, vec![13]);
563
    }
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631

    #[test]
    fn test_chunked_prefill_parent_hash() {
        use std::sync::Mutex;

        use crate::common::sequence::ActiveSequence;

        #[derive(Default)]
        struct CapturingSink {
            events: Mutex<Vec<KvCacheEvent>>,
        }

        impl KvCacheEventSink for CapturingSink {
            fn publish(
                &self,
                event: KvCacheEvent,
                _block_token_ids: Option<&[Vec<u32>]>,
            ) -> anyhow::Result<()> {
                self.events.lock().unwrap().push(event);
                Ok(())
            }
        }

        let block_size = 64;
        let tokens: Vec<u32> = (0..512).collect(); // 8 blocks
        let mut seq = ActiveSequence::new(tokens, 100, Some(block_size), true, false);

        let sink = Arc::new(CapturingSink::default());
        let mut manager =
            KvManager::new_with_event_sink(256, block_size, Some(sink.clone() as _), 0);

        // Chunk 1: allocate blocks 0-3
        let signal = seq.prepare_allocation(256).unwrap();
        manager.process(&signal);
        seq.commit_allocation(256);

        // Chunk 2: allocate blocks 4-7
        let signal = seq.prepare_allocation(512).unwrap();
        manager.process(&signal);
        seq.commit_allocation(512);

        let events = sink.events.lock().unwrap();
        assert_eq!(events.len(), 2, "expected two store events");

        // First event: parent_hash should be None (starts from root)
        let KvCacheEventData::Stored(ref store1) = events[0].data else {
            panic!("expected store event");
        };
        assert!(
            store1.parent_hash.is_none(),
            "first chunk should have no parent"
        );

        // Second event: parent_hash should be the seq_hash of block 3
        // (the last block from the first chunk)
        let KvCacheEventData::Stored(ref store2) = events[1].data else {
            panic!("expected store event");
        };
        let expected_parent = seq.unique_blocks()[3].clone();
        let UniqueBlock::FullBlock(expected_hash) = expected_parent else {
            panic!("expected full block");
        };
        assert_eq!(
            store2.parent_hash,
            Some(ExternalSequenceBlockHash(expected_hash)),
            "second chunk's parent should be block 3's seq_hash"
        );
    }
632
}