On Thu, Jun 13, 2019 at 04:25:13PM +0100, David Woodhouse wrote:
The alignment constraint is defined in the CSM specifications as "Bit mapped. First non-zero bit from the right is the alignment."
Use __fls() to sanitise the alignment given that definition, since passing a non-power-of-two alignment to _malloc() isn't going to work well. And cope with being passed zero, which was happening for the E820 table allocation from EDK2.
Thanks. I committed this change.
-Kevin