[Libre-soc-bugs] [Bug 1173] provide an option to switch ISACaller to use a different Mem class that uses mmap.mmap instead of a dict

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=1173

Luke Kenneth Casson Leighton <lkcl at lkcl.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|change ISACaller's Mem      |provide an option to switch
                   |class to use mmap.mmap      |ISACaller to use a
                   |instead of a dict           |different Mem class that
                   |                            |uses mmap.mmap instead of a
                   |                            |dict
                 CC|                            |lkcl at lkcl.net

--- Comment #1 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Jacob Lifshay from comment #0)
> This is needed so there's a contiguous chunk of memory that can be passed to
> host syscalls.

no: do not replave the existing Mem class. too low level at this point.
a cascade of errors may result in blowing the budget fixing bugs caused
by massive disruption for many weeks.

write a *new* one and provide an option to ISACaller to use it
(another bool flag).

you'll need to propagate that up all the way to pypowersim
(and through all unit test infrastructure).

make ABSOLUTELY certain to add it as the LAST parameter to all
functions (and to ISACaller itself).

run_tst, class Program, the whole lot. bit of a pain but it is what
it is.

-- 
You are receiving this mail because:
You are on the CC list for the bug.