qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 3/3] arm: semihosting: Wire up A64 HLT 0xf000


From: Leon Alrae
Subject: Re: [Qemu-devel] [PATCH 3/3] arm: semihosting: Wire up A64 HLT 0xf000
Date: Thu, 23 Apr 2015 12:00:18 +0100
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0

Hi Christopher,

On 09/04/2015 18:57, Peter Maydell wrote:
> On 9 April 2015 at 18:23, Christopher Covington
> <address@hidden> wrote:
>> On Fri, Mar 27, 2015 at 12:40 PM, Peter Maydell
>> <address@hidden> wrote:
>>> You need to have the semihosting_enabled check here rather
>>> than in the do_interrupt code, because otherwise we won't
>>> behave correctly in the disabled case.
>>
>> Do you have suggestions for getting semihosting_enabled defined in
>> translate-a64.c? I'm likely doing something dumb, but while #include
>> "sysemu/sysemu.h" at first seemed like the obvious approach, and
>> appears to work for -softmmu, I'm getting errors with that when
>> building -linux-user.
> 
> sysemu.h is an ugly grab-bag of things that are specific to the
> system emulator config. Semihosting is always enabled for linux-user,
> so it doesn't have an equivalent switch.
> 
> Let's clean this up a bit, by creating a new include/exec/semihost.h,

Do you happen to have this clean up available somewhere? I just want to
make sure I'm not duplicating the work as my patches will touch the same
area.

Thanks,
Leon



reply via email to

[Prev in Thread] Current Thread [Next in Thread]