Re: [Qemu-devel] [PATCH 1/1] Update ARM emulation to be version 4t by default

[Vincent Sanders]

On Fri, May 29, 2009 at 10:59:49PM +0100, Paul Brook wrote:
> On Friday 29 May 2009, Vincent Sanders wrote:
> > Update ARM emulation to be version 4t by default and add v5 as a
> >  feature. Implementation is very similar to the way the v6 features are
> >  presented.
> 
> You really need to distinguish between v5t and v5te.
> 

This was intended to be a minimum change alteration. The current
implementation does not make this distinction and I dont feel it is my
place to alter core semantics in this way. 

I am unaware of the specifics of these differences as the ARM ARM is
unrevealing on the subject. It tends to imply there is ARMv4t and
ARMv5te. If you feel the need to add this distinction please feel free
and I will rebase my patches upon that work.

> You've also missed the different load behavior, which is probably the most 
> important difference.
> 

Please can you provide a reference to what you mean by this? I thought
I had provided all the relevant sections. If I have missed something
please can you spell it out (preferably by reference to the ARM ARM)
as I do not have the ARM ARM commited to memory and lack your insigt
on this issue.

Please can you confirm that there is nothing erroneous about this
patch or approach, just that it is missing some corner cases with
respect to loading into the program counter.

I would also repeat my previous query asking if I am wasting my time?
You appear to go silent on this whenever I ask you directly. No doubt
I will bump into you again during the forthcoming months and can ask
you face to face but I would prefer if your comments were on record.

-- 
Regards Vincent
http://www.kyllikki.org/