В ядре Linux устранена следующая уязвимость:
can: bcm: исправить статистику приема/передачи кадров CAN
KCSAN обнаружил гонку данных в bcm_rx_handler(), когда два кадра CAN
были одновременно получены и обработаны за одну операцию приема двумя
разные процессоры. Используйте атомарные операции с (подписанными) длинными типами данных для доступа к
статистика в горячем пути для устранения жалобы KCSAN. Дополнительно упростите обновление и проверку переполнения статистики,
использование атомарных операций в отдельном bcm_update_[rx|tx]_stats()
функции.
Вариант rx работает под bcm_rx_update_lock, чтобы предотвратить
скачки при сбросе двух счетчиков приема; вариант tx работает под
bcm_tx_lock, и ему нужно только защититься от переполнения своего счетчика. Поскольку путь rx сбрасывает свои значения уже на уровне LONG_MAX/100, возникает
нет конфликта между двумя блокирующими доменами (bcm_rx_update_lock vs.
bcm_tx_lock) даже для операций, использующих оба пути. Обновление статистики приема и обновление Frames_filtered в
bcm_rx_changed() ранее выполнялись в двух отдельных
разделы bcm_rx_update_lock.
Для операции по приему подписка на всех интерфейсах
(ifindex == 0), bcm_rx_handler() может выполняться одновременно на разных
ЦП, поэтому сброс счетчика одним ЦП между этими двумя секциями может
оставитьframes_filtered больше, чемframes_abs, на другом процессоре, что приведет к
фиктивный (даже отрицательный) процент снижения в procfs. Обновите
статистика в том же критическом разделе, что и bcm_rx_changed(), чтобы закрыть
этот пробел, который также удаляет теперь ненужную дополнительную пару блокировки/разблокировки
вокруг расчета Traffic_flags.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: can: bcm: fix CAN frame rx/tx statistics KCSAN detected a data race within the bcm_rx_handler() when two CAN frames have been simultaneously received and processed in a single rx op by two different CPUs. Use atomic operations with (signed) long data types to access the statistics in the hot path to fix the KCSAN complaint. Additionally simplify the update and check of statistics overflow by using the atomic operations in separate bcm_update_[rx|tx]_stats() functions. The rx variant runs under bcm_rx_update_lock to prevent races when resetting the two rx counters; the tx variant runs under bcm_tx_lock and only needs to guard its own counter's overflow. As the rx path resets its values already at LONG_MAX / 100, there is no conflict between the two locking domains (bcm_rx_update_lock vs. bcm_tx_lock) even for ops that use both paths. The rx statistics update and the frames_filtered update in bcm_rx_changed() were previously performed in two separate bcm_rx_update_lock sections. For an rx op subscribed on all interfaces (ifindex == 0), bcm_rx_handler() can run concurrently on different CPUs, so a counter reset by one CPU between these two sections could leave frames_filtered larger than frames_abs on another CPU, producing a bogus (even negative) reduction percentage in procfs. Update the statistics in the same critical section as bcm_rx_changed() to close this gap, which also removes the now unneeded extra lock/unlock pair around the traffic_flags calculation.