Bug 42730

Summary: mt7921: delayed reboot after unbind/bind - drivers/net/wireless/mediatek/mt76/mt7921
Product: Branch p10 Reporter: Iakunin Andrei <iakuninaa>
Component: kernel-image-un-defAssignee: Vitaly Chikunov <vt>
Status: ASSIGNED --- QA Contact: qa-p10 <qa-p10>
Severity: minor    
Priority: P5 CC: rider
Version: не указана   
Hardware: x86_64   
OS: Linux   
See Also: https://bugzilla.altlinux.org/show_bug.cgi?id=42627
Attachments:
Description Flags
inxi, basic system information
none
journalctl log none

Description Iakunin Andrei 2022-05-11 14:46:42 MSK 
Created attachment 10721 [details]
inxi, basic system information

Пытаясь найти решение для Ошибка 42627 наткнулся странное поведение.

В https://bugzilla.kernel.org/show_bug.cgi?id=215787 проблема с востановлением после подключения-отключения устройства. 

Но у меня если делаешь:
$echo 'dev_addr' > /sys/bus/pci/drivers/mt7921e/unbind
$echo 'dev_addr' > /sys/bus/pci/drivers/mt7921e/bind
связь востанавливается нормально, WiFi работает. Однако через какое-то время ноутбук зависает и вскоре перезагружается. Зависает при использовании сети (apt-get, firefox), но не не сразу, а через некоторое время при, например открытии очередной вкладки.   При этом в journalctl всё чисто.
Comment 1 Iakunin Andrei 2022-05-11 14:47:18 MSK 
Created attachment 10722 [details]
journalctl log
Comment 2 Vitaly Chikunov 2022-05-11 15:53:10 MSK 
Такое надо репотрить в апстрим.
Comment 3 Iakunin Andrei 2022-05-12 10:03:38 MSK 
В баге апстрима есть патч с предположительным решением. Попробую его применить и отпишусь о результах.
Comment 4 Iakunin Andrei 2022-05-16 10:17:46 MSK 
Похоже проблема с внезапной перезагрузкой решилась с исправлением Ошибка 42627.
Патч приложен там.
Comment 5 Vitaly Chikunov 2022-05-17 06:39:04 MSK 
Андрей, вы предлагаете применить этот патч к некоторым нашим ядрам не дожидаясь того как он придет из апсрима (предположительно в 5.19)?

#42627 обознчена как Importance 'major' этот баг как 'minor'.
Comment 6 Anton Farygin 2022-05-17 10:21:24 MSK 
На мой взгляд конечно лучше применить и занести в p10 - оборудование у нас есть и ошибка постоянно воспроизводится.
Comment 7 Vitaly Chikunov 2022-05-18 00:34:11 MSK 
Хорошо. Я добавил этот патч в
  un-def/sisyphus   вариант из апстрима
  std-def/sisyphus  ваш вариант с моими модификациями
  un-def/p10        -"-

Например, в вашем патче переставлены вызовы функций в отличии от апстрима.

  pci_free_irq_vectors(pdev);
+ mt76_free_device(&dev->mt76);

Переставил это местами, чтоб стало как в апстриме.

При следующем обновлении версии ядра в sisyphus они попадут в него.

Дла un-def/p10 навернео стоит не дожидаться новой версии и пересобрать пока предыдущая версия ещё на тестировании?
Comment 8 Anton Farygin 2022-05-18 08:32:57 MSK 
Спасибо! Спешки нет, пусть попадает в следующую версию.