eepm icon indicating copy to clipboard operation
eepm copied to clipboard

Add videomemory allocation in switch-to-nvidia.sh

Open vlad196 opened this issue 10 months ago • 5 comments

  • add videomemory allocations in /run
  • delete duplicate libvulkan1

vlad196 avatar Apr 04 '24 14:04 vlad196

Source: https://download.nvidia.com/XFree86/Linux-x86_64/550.54.14/README/powermanagement.html

vlad196 avatar Apr 04 '24 14:04 vlad196

Я немного влезу с правками

Во-первых, такие вещи должны быть в пакете, так что правильнее открыть багу на nvidia_glx_common, тем более что там уже есть modprobe файл Во-вторых, NVreg_TemporaryFilePath должен быть в /var/tmp/, а не /run, /run лишь пример документации при чём не очень удачный В третьих, зачем мержить изменения основной ветки если они не требуют ребейса ? Лищний раз портить git log не за чем

Boria138 avatar Apr 07 '24 08:04 Boria138

  1. правильно, я лишь дополнил чужой коммит с nvidia-susprend nvidia-resume и nvidia-hibernate, который не работал бы без второй части с README
  2. Да, всё верно, я сейчас почитал, /run это тоже tmpfs каталог, его нужно поменять.
  3. Мержил из-за неправильной работы с форком. Обновлял форк перед тем, как заиметь обновлённый проект на компьютере.

vlad196 avatar Apr 07 '24 11:04 vlad196

Если у вас в репозитории есть ваши коммиты, то для обновления используйте git rebase -r <remote> Тогда никаких мержей не появится.

vitlav avatar Apr 08 '24 19:04 vitlav

Ждём!

vitlav avatar Apr 13 '24 20:04 vitlav

Я закрываю PR.

  1. Отдельным PR добавлю удаление дублирование libvulkan1
  2. Надо учесть 2 способа сохранения ресурсов видеопамяти, которые описаны в документации. Сделаю PR с новым предложением.

vlad196 avatar Jun 08 '24 11:06 vlad196