CVSROOT: /cvs Module name: src Changes by: clau...@cvs.openbsd.org 2023/10/08 01:44:52
Modified files: sys/net : ifq.c ifq.h Log message: Revert commitid: KtmyJEoS0WWxmlZ5 --- Protect interface queues with read once and mutex. Reading atomic values need at least read once and writing values should have a mutex. This is what mbuf queues already do. Add READ_ONCE() to ifq and ifiq macros for len and empty. Convert ifq_set_maxlen() to a function that grabs ifq_mtx. OK mvs@ --- ifq_set_maxlen() is called before the ifq_mtx is initalized and this at least crashes WITNESS kernels on boot. Reported-by: syzbot+7b218ef53432b5d56...@syzkaller.appspotmail.com