Hi, On 10/05/2022 11:15, Lin Liu wrote:
ext2 has nothing to do with this logic.
This code was a verbatim copy from Linux. Looking at the history, I am not sure there was even a connection with the ext2 filesystem (I guess this is what you mean?).
So I would drop this and simply say that we could use the new helpers.
Clean up the code with xen/byteswap.h which now has an unsigned long helper. No functional change. Signed-off-by: Lin Liu <[email protected]>
Other that what I wrote above: Acked-by: Julien Grall <[email protected]> Cheers, -- Julien Grall
