On 10 July 2013 18:10, Fabien Chouteau <chout...@adacore.com> wrote:
> --- a/include/qemu/bswap.h
> +++ b/include/qemu/bswap.h
> @@ -432,6 +432,11 @@ static inline uint32_t le32_to_cpupu(const uint32_t *p)
> return ldl_le_p(p);
> }
>
> +static inline uint16_t be16_to_cpupu(const uint16_t *p)
> +{
> + return lduw_be_p(p);
> +}
> +
So these functions are preceded by the comment:
/* Legacy unaligned versions. Note that we never had a complete set. */
which suggests that we shouldn't be adding new ones.
I think the correct answer is to use lduw_be_p() directly.
(If so it's tempting to get rid of the legacy functions
completely since the change would be pretty mechanical.)
thanks
-- PMM
