From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:35186)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <blauwirbel@gmail.com>) id 1QMQE7-0006JE-0C
	for qemu-devel@nongnu.org; Tue, 17 May 2011 15:46:43 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <blauwirbel@gmail.com>) id 1QMQE6-0002rp-1p
	for qemu-devel@nongnu.org; Tue, 17 May 2011 15:46:42 -0400
Received: from mail-qy0-f180.google.com ([209.85.216.180]:54772)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <blauwirbel@gmail.com>) id 1QMQE5-0002ri-Vt
	for qemu-devel@nongnu.org; Tue, 17 May 2011 15:46:42 -0400
Received: by qyk10 with SMTP id 10so506922qyk.4
	for <qemu-devel@nongnu.org>; Tue, 17 May 2011 12:46:41 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <BANLkTikWkPu8NBEUAAjtyw33G8RyJuwKKw@mail.gmail.com>
References: <BANLkTiksFDsZRin=drOg2hk5uCtsBpNKwg@mail.gmail.com>
	<BANLkTikWkPu8NBEUAAjtyw33G8RyJuwKKw@mail.gmail.com>
From: Blue Swirl <blauwirbel@gmail.com>
Date: Tue, 17 May 2011 22:46:21 +0300
Message-ID: <BANLkTikE4giYMpLz+kVX4t8buUCVEekRwg@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [Qemu-devel] [PATCH 01/11] Move user emulator stuff from
 cpu-exec.c to user-exec.c
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-devel <qemu-devel@nongnu.org>

On Tue, May 17, 2011 at 5:52 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
> On 14 May 2011 20:36, Blue Swirl <blauwirbel@gmail.com> wrote:
>
>> +#if defined(__i386__)
>
>> +#elif defined(__x86_64__)
>
>> +#elif defined(_ARCH_PPC)
>
>> +#elif defined(__alpha__)
>
>> +#elif defined(__sparc__)
>
>> +#elif defined(__arm__)
>
>> +#elif defined(__mc68000)
>
>> +#elif defined(__ia64)
>
>> +#elif defined(__s390__)
>
>> +#elif defined(__mips__)
>
>> +#elif defined(__hppa__)
>
> ...if we're moving this code about anyway would it be possible
> to split the per-host-arch specific code out into separate files?
> I find these source files with enormous ifdef ladders pretty hard
> to read.

There are two problems which the #ifdeffery tackles: how to get PC
from signal state (OS dependent, also arch dependent) and how to
determine whether the access was a write (arch dependent).

Splitting by arch would not help much, but would it make sense to
split by OS and push the pieces to each user emulator? The arch
dependent but OS independent stuff for is_write could be moved to some
common .h file.