From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PULL] virtio/vhost: cross endian support Date: Thu, 2 Jul 2015 08:12:37 +0200 Message-ID: <20150702080916-mutt-send-email-mst@redhat.com> References: <20150701113150-mutt-send-email-mst@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Linus Torvalds Cc: KVM list , Network Development , Linux Kernel Mailing List , virtualization List-Id: virtualization@lists.linuxfoundation.org On Wed, Jul 01, 2015 at 12:03:59PM -0700, Linus Torvalds wrote: > On Wed, Jul 1, 2015 at 12:02 PM, Linus Torvalds > wrote: > > > > Doing a unconditional byte swap is faster and simpler than the crazy > > conditionals. > > Unconditional endianness not only makes for simpler and faster code, > it also ends up being easier to debug and add things like type > annotations for sparse. > > Linus At least this last one is well covered by these patches: this uses separate sparse types so all accesses are statically verified by sparse to use the correct accessor. -- MST