Hi Andrew,
On 11/11/2021 17:57, Andrew Cooper wrote:
There are exactly 3 callers of sort() in the hypervisor.
Both arm callers pass in NULL for the swap function. While this might seem
like an attractive option at first, it causes generic_swap() to be used which
forced a byte-wise copy. Provide real swap functions which the compiler can
optimise sensibly.
I understand the theory, but none of the two calls are in hot paths or
deal with large set on Arm. So I am rather hesitant to introduce
specialised swap in each case as it doesn't seem we will gain much from
this change.
Cheers,
--
Julien Grall