Page MenuHomeFreeBSD

Add missing M_NOWAIT flag
ClosedPublic

Authored by rstone on Oct 23 2019, 4:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 6 2024, 2:32 AM
Unknown Object (File)
Jan 29 2024, 9:36 PM
Unknown Object (File)
Jan 13 2024, 2:49 AM
Unknown Object (File)
Dec 24 2023, 8:09 PM
Unknown Object (File)
Dec 20 2023, 1:03 AM
Unknown Object (File)
Nov 12 2023, 4:03 PM
Unknown Object (File)
Sep 11 2023, 4:10 AM
Unknown Object (File)
Aug 22 2023, 3:15 PM

Details

Summary

The LinuxKPI linux_dma code calls PCTRIE_INSERT with a
mutex held, but does not set M_NOWAIT when allocating
nodes, leading to a potential panic. All of this code
can handle an allocation failure here, so prefer an
allocation failure to sleeping on memory.

Also fix a related case where NOWAIT/WAITOK was not
specified. In this case it's not clear whether sleeping
is allowed so be conservative and assume not. There are
a lot of other paths in this code that can fail due to
a lack of memory anyway.

Sponsored by: Dell EMC Isilon
MFC After: 1 week

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This looks correct.
I agree with the IRC comment about a kassert in uma_zalloc for the flags, but that's not needed here.

This revision is now accepted and ready to land.Oct 23 2019, 4:36 PM
This revision was automatically updated to reflect the committed changes.