Hi Joel, On Thu Feb 19, 2026 at 6:19 AM JST, Joel Fernandes wrote: > This series adds support for nova-core memory management, including VRAM > allocation, PRAMIN, VMM, page table walking, and BAR 1 read/writes. > These are critical for channel management, vGPU, and all memory > management uses. > > These patches depend on the following preparatory patches: > https://lore.kernel.org/all/[email protected]/T/#t > > All patches (including the preparatory patches from the other series) can > be found at: > git://git.kernel.org/pub/scm/linux/kernel/git/jfern/linux.git (branch nova/mm > or tag: nova-mm-v7-20260218) > > Earlier series of these patches: > https://lore.kernel.org/linux-fbdev/[email protected]/T/#t > https://lore.kernel.org/linux-fbdev/[email protected]/ > > Joel Fernandes (23): > nova-core: mm: Add support to use PRAMIN windows to write to VRAM > docs: gpu: nova-core: Document the PRAMIN aperture mechanism > nova-core: Add BAR1 aperture type and size constant > nova-core: gsp: Add BAR1 PDE base accessors > nova-core: mm: Add common memory management types > nova-core: mm: Add common types for all page table formats > nova-core: mm: Add MMU v2 page table types > nova-core: mm: Add MMU v3 page table types > nova-core: mm: Add unified page table entry wrapper enums > nova-core: mm: Add TLB flush support > nova-core: mm: Add GpuMm centralized memory manager > nova-core: mm: Add page table walker for MMU v2/v3 > nova-core: mm: Add Virtual Memory Manager > nova-core: mm: Add virtual address range tracking to VMM > nova-core: mm: Add multi-page mapping API to VMM > nova-core: mm: Add BAR1 user interface > nova-core: gsp: Return GspStaticInfo and FbLayout from boot() > nova-core: mm: Add BAR1 memory management self-tests > nova-core: mm: Add PRAMIN aperture self-tests > nova-core: gsp: Extract usable FB region from GSP > nova-core: fb: Add usable_vram field to FbLayout > nova-core: mm: Use usable VRAM region for buddy allocator > nova-core: mm: Add BarUser to struct Gpu and create at boot
Nice series, a few overall remarks before I dive deeper into each patch: - Use `gpu: nova-core:` (not just `nova-core:`) as the patch prefix. - There were a few clippy warnings/rustfmt diffs when I built it. - There are build failures introduced in patches 11 and 18 - basically the series doesn't build from 11 until the last patch. - This patchset is using the `module/mod.rs` pattern instead of `module.rs` for new modules. The latter is the preferred norm IIUC. The end result of the series looks coherent, but the ordering of patches makes it more tedious to review than it needs to be. We start with PRAMIN, then add some BAR1 types that are not used until 10+ patches later, switch to getting the framebuffer size from the GSP, add tests, go back to GSP, etc. It is difficult to follow all these intertwined topics (even though they eventually converge) and to know which of the previous patches are relevant to the current one. But there is a clear path, and after moving patches around (with only very trivial conflicts) I got the following ordering which is more logical IMHO: Phase 1: GSP plumbing - nova-core: gsp: Return GspStaticInfo and FbLayout from boot() - nova-core: gsp: Extract usable FB region from GSP - nova-core: fb: Add usable_vram field to FbLayout These constitute a logical change by themselves, by getting more information from the GSP to know how much VRAM we have. You could even display the result as a dev_info of dev_dbg to remove the only remaining `dead_code`. Phase 2: PRAMIN support - nova-core: mm: Add support to use PRAMIN windows to write to VRAM - docs: gpu: nova-core: Document the PRAMIN aperture mechanism PRAMIN is needed by everything that follows. Phase 3: GpuMm - nova-core: mm: Add common memory management types - nova-core: mm: Add TLB flush support - nova-core: mm: Add GpuMm centralized memory manager - nova-core: mm: Use usable VRAM region for buddy allocator The common memory management types patch and TLB give us all we need to introduce GpuMm, which makes it more accessible that after going through all the page table types which it doesn't depend on. This culminates with using the result of phase 1, which also allows you to get rid of the temporary 1MB window hack if you rearrange the code a bit. Phase 4: page tables / VMM - nova-core: mm: Add common types for all page table formats - nova-core: mm: Add MMU v2 page table types - nova-core: mm: Add MMU v3 page table types - nova-core: mm: Add unified page table entry wrapper enums - nova-core: mm: Add page table walker for MMU v2/v3 - nova-core: mm: Add Virtual Memory Manager - nova-core: mm: Add virtual address range tracking to VMM - nova-core: mm: Add multi-page mapping API to VMM The main course, required for BAR1 - these follow the original order. Phase 5: BAR1 - nova-core: Add BAR1 aperture type and size constant - nova-core: gsp: Add BAR1 PDE base accessors - nova-core: mm: Add BAR1 user interface - nova-core: mm: Add BarUser to struct Gpu and create at boot All the BAR1 stuff now happens in one place. There is certainly room for merging a few patches to avoid introducing dead code and eliminating just after. Phase 6: tests - nova-core: mm: Add PRAMIN aperture self-tests - nova-core: mm: Add BAR1 memory management self-tests I have done this reordering locally and it seems to build fine. I'll do a patch-by-patch review following this order, but I wanted to share it first for other reviewers of this revision as it makes the series more accessible IMHO.
