lkml.org 
[lkml]   [2015]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v6 6/8] virtio: add explicit big-endian support to memory accessors
On Fri, 24 Apr 2015 14:26:24 +0200
Greg Kurz <gkurz@linux.vnet.ibm.com> wrote:

> The current memory accessors logic is:
> - little endian if little_endian
> - native endian (i.e. no byteswap) if !little_endian
>
> If we want to fully support cross-endian vhost, we also need to be
> able to convert to big endian.
>
> Instead of changing the little_endian argument to some 3-value enum, this
> patch changes the logic to:
> - little endian if little_endian
> - big endian if !little_endian
>
> The native endian case is handled by all users with a trivial helper. This
> patch doesn't change any functionality, nor it does add overhead.
>
> Signed-off-by: Greg Kurz <gkurz@linux.vnet.ibm.com>
> ---
>
> Changes since v5:
> - changed endian checking helpers as suggested by Thomas (use || and line
> breaker)
>
> drivers/net/macvtap.c | 3 ++-
> drivers/net/tun.c | 3 ++-
> drivers/vhost/vhost.h | 3 ++-
> include/linux/virtio_byteorder.h | 24 ++++++++++++++----------
> include/linux/virtio_config.h | 3 ++-
> include/linux/vringh.h | 3 ++-
> 6 files changed, 24 insertions(+), 15 deletions(-)
>

Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com>



\
 
 \ /
  Last update: 2015-04-27 11:41    [W:0.211 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site