From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1KYNHw-0004zr-Gw
	for qemu-devel@nongnu.org; Wed, 27 Aug 2008 11:50:28 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1KYNHs-0004yH-9S
	for qemu-devel@nongnu.org; Wed, 27 Aug 2008 11:50:27 -0400
Received: from [199.232.76.173] (port=40756 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1KYNHs-0004yC-5C
	for qemu-devel@nongnu.org; Wed, 27 Aug 2008 11:50:24 -0400
Received: from ag-out-0708.google.com ([72.14.246.251]:5784)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1KYNHr-0005Xw-5V
	for qemu-devel@nongnu.org; Wed, 27 Aug 2008 11:50:23 -0400
Received: by ag-out-0708.google.com with SMTP id 31so6969438agc.5
	for <qemu-devel@nongnu.org>; Wed, 27 Aug 2008 08:50:21 -0700 (PDT)
Message-ID: <48B5778E.8000709@codemonkey.ws>
Date: Wed, 27 Aug 2008 10:49:34 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [patch] add byteordered types
References: <48B53E23.5040107@redhat.com>	<200808271456.57273.paul@codesourcery.com>	<48B56645.60206@redhat.com>	<20080827.084738.-674761106.imp@bsdimp.com>
	<48B56D8D.7050002@redhat.com>
In-Reply-To: <48B56D8D.7050002@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Reply-To: qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: qemu-devel@nongnu.org
Cc: Paul Brook <paul@codesourcery.com>, Gerd Hoffmann <kraxel@redhat.com>

Gerd Hoffmann wrote:
> M. Warner Losh wrote:
>   
>> In message: <48B56645.60206@redhat.com>
>>             Gerd Hoffmann <kraxel@redhat.com> writes:
>> : Paul Brook wrote:
>> : > On Wednesday 27 August 2008, Gerd Hoffmann wrote:
>> : >> +static inline le16 write_le16(uint16_t cpu) \
>> : > 
>> : > This is IMHO a bad name for this function. It doesn't write anything.
>> : 
>> : I'm not that happy the name too and certainly open for better
>> : suggestions I could use instead of read/write.
>> : 
>> : The alternatives I can think of are not very nice either:
>> : 
>> :  (1) get/set -- same problem as read/write.
>> :  (2) make_foo() instead of write_foo() -- "make" is too generic IMHO.
>> :  (3) create_foo() instead of write_foo() -- no nice symmetric
>> :      replacement for read_foo().
>> : 
>> : better ideas anyone?
>>
>> Using the existing names that linux/FreeBSD/NetBSD/OpenBSD are using?
>>     
>
> cpu_to_* is taken already by the not-typechecked macros.  And converting
> the whole qemu tree in one go so we could reuse the names is a bit
> unrealistic IMHO.
>   

Personally, I dislike the whole struct thing.  I even further dislike 
having multiple sets of conversion functions that are used in different 
places in the code.

Are we sure that this is something that we want to do?

Regards,

Anthony Liguori

> cheers,
>   Gerd
>
>
>