[edk2-devel] [PATCH 1/1] OvmfPkg/IoMmuDxe: add locking to IoMmuAllocateBounceBuffer

[Gerd Hoffmann]

  Hi,

> > On a quick review of the code, there appear to be other points that also
> > modify mReservedMemBitmap (IoMmuAllocateCommonBuffer() and
> > IoMmuFreeCommonBuffer()).  I'd guess that these also need to raise to
> > TPL_NOTIFY, but I'm not familiar with the code so I don't know if
> > there's anything that makes this unnecessary.
> >
> 
> Thanks for mentioning that - perhaps Gerd could have another look? (I
> merged the v2 already)

Patch sent.  Same pattern.  Didn't notice we have that twice (once for
explicitly allocating buffers and once for internally used bounce
buffers).

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107090): https://edk2.groups.io/g/devel/message/107090
Mute This Topic: https://groups.io/mt/100233359/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-