Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

perf(scan): AVX-512 + vpclmulqdq scanner backend

Aperta
#9 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
30/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
github-actions, rust
Ambito
ci-cd, performance

Direzione di ricerca

Read src/scan/avx2.rs and src/scan/mod.rs, then run the proposed CPU-feature diagnostic in the CI workflow to establish whether vpclmulqdq is consistently available. If the prerequisite is met, done means adding the feature-gated AVX-512 path, dispatcher fallback, cross-check coverage, supported-runner validation, and benchmark evidence.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement

Context

Current scanner uses AVX2 + PCLMUL (128-bit). On CPUs supporting avx512bw + vpclmulqdq (Ice Lake / Sapphire Rapids / Zen 4+), a 128-byte chunk path could halve the loop iteration count.

Prerequisite: CPU support audit

This issue is gated on confirming that the project's actual build/CI hosts support vpclmulqdq. If not, ROI is 0 and the issue should be deferred indefinitely.

  • Local dev host: confirmed missing vpclmulqdq (Skylake-X / Skylake-SP — has avx512bw but not vpclmulqdq). Cannot test locally.

  • CI runners: ubuntu-latest runner CPUs vary by allocation. Add a one-line diagnostic to the workflow:

    - name: CPU features
      run: grep -oE '\b(avx2|avx512bw|vpclmulqdq|pclmulqdq)\b' /proc/cpuinfo | sort -u
    

    Collect output over several CI runs; only proceed if vpclmulqdq is reliably present.

If CI runners do not reliably provide vpclmulqdq, the only path to validating this is paid larger-runners or self-hosted runners.

Proposal (pending CPU confirmation)

  • New src/scan/avx512.rs mirroring avx2.rs with 128-byte chunks
  • Dispatcher (src/scan/mod.rs): AVX-512 → AVX2 → scalar fallback chain
  • New avx512 feature flag (default off) so release builds stay portable
  • Use _mm512_clmulepi64_epi128 for the inside-string prefix-XOR

Estimated impact

est. speedup
CPUs with avx512bw + vpclmulqdq ~1.5–2× scan throughput
Other CPUs 0 (dispatcher falls back)

Validation plan

  • scanner_crosscheck proptest extended to compare AVX-512 vs scalar
  • CI matrix on a runner confirmed to have vpclmulqdq
  • make bench 3-run median on supported hardware

Recommendation

Last in the perf followup queue. The CPU support situation is uncertain; if it turns out CI runners don't have vpclmulqdq, this is dead code we maintain forever. Do the cheap wins (#5 memchr, #6 pooling, #7 PGO, #8 micro-opts) first.

Lingua principale
Rust
Stelle
2
Fork
0
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di api7/lua-qjson

Tutte le issue di api7/lua-qjson

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.