perf: is_executable calls fs_access for every file on every reload

Abierto Apto para principiantes
  • ¿Algo no está bien?
#3,383 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
72/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
lua, neovim
Área
devtools, performance

Línea de trabajo

Empieza por localizar utils.is_executable y los dos puntos de llamada descritos: la creación de un nuevo FileNode y Explorer:reload, donde lstat ya está disponible. Comprueba cómo el stat expone el tipo de archivo y los bits de ejecución, y luego compara el comportamiento con las comprobaciones existentes de fs_access. El trabajo estará terminado cuando los archivos regulares sin bits de ejecución omitan esa llamada al sistema, mientras que los directorios, los enlaces y los archivos ejecutables conserven su comportamiento actual.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Problem

utils.is_executable calls vim.uv.fs_access(path, "X") for:

  • every new FileNode, and
  • every existing node on every Explorer:reload.

The node's lstat result is already available at both call sites.

Impact

fs_access costs 7.7µs per call vs 1.9µs for lstat. is_executable is about 11% of reload_explorer on a fully expanded 16.8k-node tree.

Proposed fix

Pass the existing stat into is_executable. A regular file with none of the user, group or other execute bits set can never pass access(X), even as root, so return false without the syscall. Files with execute bits, directories and links still use fs_access, happy to follow up with a PR soon.

Environment: macOS, Nvim v0.13.0-dev, LuaJIT 2.1. Test repo: 16.8k files and folders, 3.1k git status entries (modified, untracked, ignored). Measured headless with vim.uv.hrtime() and the LuaJIT profiler (jit.p).

Lenguaje dominante
Lua
Estrellas
8.7k
Forks
642
Merge medio
1 d 4 min
PR fusionados (30 d)
11

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de nvim-tree/nvim-tree.lua

Todos los issues de nvim-tree/nvim-tree.lua

Issues similares

Más issues de Lua

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.