Page MenuHomeFreeBSD

linuxkpi: Handle NULL cache pointer in kmem_cache_destroy()
ClosedPublic

Authored by markj on Oct 6 2021, 3:36 PM.

Details

Summary

This scenario arose when testing iwlwifi, when iwl_trans_pcie_alloc()
fails at the "wrong" point. Linux handles a NULL pointer here, so let's
do the same.

Diff Detail

Repository
R10 FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

markj requested review of this revision.Oct 6 2021, 3:36 PM
emaste added inline comments.
sys/compat/linuxkpi/common/src/linux_slab.c
195

We're already using unlikely()s here, maybe add it on this test?

This revision is now accepted and ready to land.Oct 6 2021, 4:22 PM

If you have a log from iwlwifi along with hardware information please email that over as well.

sys/compat/linuxkpi/common/src/linux_slab.c
195

It really doesn't make sense here, this function is very expensive and rarely called.