[OpenBIOS] [commit] r739 - trunk/openbios-devel/libopenbios

[Mark Cave-Ayland]

Mark Cave-Ayland wrote:

> This points the finger at ofmem_update_mmu_translations()

In fact, it is possible to narrow down the culprit even further. If I 
comment out the set_property() call in ofmem_update_mmu_translations() 
then everything works fine. If I leave it in but change ncells to 
(ncells - 1) then that works too (but then of course the property array 
is one cell short).

It almost seems to suggest some kind of off-by-one or alignment error in 
the property setting code which is clobbering memory in Forth somewhere, 
but the specification seems to suggest that properties aren't aligned 
and a quick eye-ball of the property code hasn't yielded anything obvious.

Can anyone else see something I've missed?

Mark.

-- 
Mark Cave-Ayland - Senior Technical Architect
PostgreSQL - PostGIS
Sirius Corporation plc - control through freedom
http://www.siriusit.co.uk
t: +44 870 608 0063

Sirius Labs: http://www.siriusit.co.uk/labs