Skip to content

Regime-C Cholesky reportedly host-synchronizes per front through its info check (blocks stream overlap of wide fronts); verify #110

Open

Description

@michel2323

Found while working on: PR #107 (performance prototypes; stream-pool experiment in bench/fact_fused.jl)

What: the prototype reports that _factor_panel_c! (src/numeric/factorize.jl, regime-C Cholesky) synchronizes the host once per front through its info check, so independent wide fronts cannot overlap on a stream pool (measured neutral for that reason) and the numeric phase is not host-synchronization-free as PLAN §3.9 states. The code calls _potrf_info_impl!, which was introduced in T09 to keep the status on the device, so either the vendor binding (vendor_potrf_info!, CUDACore.with_workspace plus the raw cusolverDn?potrf) synchronizes internally, or the report refers to another read. To verify with a CUDA profile of one refactorization on a matrix with several regime-C fronts (bench/profile_phases.jl): count the host synchronizations per factorization.

Why it is out of scope: #107 is bench-level; the vendor path is T09 code and overlap of wide fronts is T25 work. Blocks nothing; it caps experiment 5 if confirmed.

Suggested fix or plan change: if confirmed, an asynchronous vendor potrf whose info stays on the device without a workspace query per call (query the workspace size once at analysis), and the same for the LDLᵀ/LU regime-C paths. Owner note under T25.


Generated by Claude Code

Activity

  1. added
    found-by-agentOpened by an implementing agent for an out-of-scope finding
    performanceSpeed or memory of the numeric and solve phases; tracked in PERFORMANCE.md
    on Oct 9, 2026
  2. added 2 commits that reference this issue on Oct 9, 2026
  3. added
    triagedOwner has looked at this found-by-agent issue; it no longer holds the task chain
    on Oct 9, 2026
  4. michel2323 commented on Oct 9, 2026

    @michel2323
    MemberAuthor

    Triaged: first step of task T26 in TASKS.md — verify with a CUDA profile (host synchronizations per refactorization), then an asynchronous vendor potrf with info on the device if confirmed. Closed or documented by that PR.


    Generated by Claude Code

  5. added 2 commits that reference this issue on Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    found-by-agentOpened by an implementing agent for an out-of-scope findingperformanceSpeed or memory of the numeric and solve phases; tracked in PERFORMANCE.mdtriagedOwner has looked at this found-by-agent issue; it no longer holds the task chain

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions