diff options
author | Ard Biesheuvel <ardb@kernel.org> | 2023-07-20 15:45:57 +0200 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-09-02 14:52:28 +0000 |
commit | dfb941d32a2f38c9177729e39c6a6515abbbad48 (patch) | |
tree | ac10316308155d458152888a8780c7fa8b14333e /IntelFsp2Pkg/Library/BaseDebugDeviceLibNull | |
parent | beafabdae49c873adecdb7511dbebe9d4ff5c8f0 (diff) | |
download | edk2-dfb941d32a2f38c9177729e39c6a6515abbbad48.tar.gz edk2-dfb941d32a2f38c9177729e39c6a6515abbbad48.tar.bz2 edk2-dfb941d32a2f38c9177729e39c6a6515abbbad48.zip |
OvmfPkg/IoMmuDxe: don't rely on TPLs to manage concurrency
Instead of relying on raising the TPL to protect the critical sections
that manipulate the global bitmask that keeps track of bounce buffer
allocations, use compare-and-exchange to manage the global variable, and
tweak the logic to line up with that.
Given that IoMmuDxe implements a singleton protocol that is shared
between multiple drivers, and considering the elaborate and confusing
requirements in the UEFP spec regarding TPL levels at which protocol
methods may be invoked, not relying on TPL levels at all is a more
robust approach in this case.
Link: https://bugzilla.redhat.com/show_bug.cgi?id=2211060
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Acked-by: Pedro Falcato <pedro.falcato@gmail.com>
Diffstat (limited to 'IntelFsp2Pkg/Library/BaseDebugDeviceLibNull')
0 files changed, 0 insertions, 0 deletions