[Top][All Lists]

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

Re: [Qemu-ppc] [Qemu-devel] [PATCH] target-ppc: Init dcache and icache s

From: Andreas Färber
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH] target-ppc: Init dcache and icache size for e500 user mode
Date: Fri, 13 Apr 2012 13:40:59 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120312 Thunderbird/11.0

Am 12.04.2012 19:24, schrieb Scott Wood:
> On 04/12/2012 11:59 AM, Andreas Färber wrote:
>> Am 10.04.2012 22:04, schrieb Meador Inge:
>>> commit f7aa558396dd0f6b7a2b22c05cb503c655854102 pulled the dcache and icache
>>> line size initialization inside of a '#if !defined(CONFIG_USER_ONLY)' block.
>>> This is not correct because instructions like 'dcbz' need the dcache size
>>> initialized even for user mode.
>>> Signed-off-by: Meador Inge <address@hidden>
>> Looks okay and compiles,
>> Reviewed-by: Andreas Färber <address@hidden>
>> Scott, are you planning to review this e500 patch? Or should I go ahead
>> and apply?
> I'm OK with it, though it may make more sense for USER_ONLY to just pick
> an arbitrary cache line size (probably 32) than to try to imitate a
> specific core.

Meador, what was your specific problem: The value being zero or not
matching real hardware?

Scott's suggestion would avoid some #ifdef'ery so I'd prefer that if
possible. I'm planning for a PULL later today, so let me know.


SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

reply via email to

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