Sakura-IT / SonnetAmiga

Reimplementation of WarpOS supporting Sonnet Crescendo 7200 and other PowerPC PCI cards (mirror of CVS development repository).
MIT License
39 stars 3 forks source link

IRA does not like 1005 hunk #4

Closed rkujawa closed 7 years ago

rkujawa commented 9 years ago

While investigating optimisations issues, I discovered it is impossible to use IRA on binaries generated with the new version of vasm and vlink (with the memory attribute set to $1005).

Running IRA on such binary results in: Hunk...:00001005 NOT SUPPORTED.

I'll report this issue to author of IRA and ask him to add support for such hunks.

rkujawa commented 9 years ago

Frank agreed to implement it.

rkujawa commented 9 years ago

Newest source code release (2.09 beta) supports it. Available here: http://sonnet.uguu.waw.pl/ira/ Still needs some testing.

DvdBoon commented 9 years ago

Are all relevant hunks now supported? ($3E9, $3EA, $3EB)? I know we only talked about the code and data hunk, but not about the BSS hunk.

rkujawa commented 9 years ago

Looks like BSS is supported too.

I'm more concerned about PPC code, have to do some more testing, but I think it sometimes is trying to disassemble the PPC code instead of treating it as data. I added disassembly to the build process.

make disasm68k runs ira on all interesting executables.

The result is later archived into disasm68k.lha by the build script (perhaps it should be a make target too).

See the latest build results: http://sonnet.uguu.waw.pl/builds/20150316-2037/ esp. disassembly of template and getinfo_ppc. It does not look correct to me, but maybe I am doing something wrong ;).

DvdBoon commented 9 years ago

I always have to fine-tune ira (using a config file) to get exactly what I want, so I am not surprised :-)

DvdBoon commented 7 years ago

This still needed with the move to system which patches during loading?

DvdBoon commented 7 years ago

No longer needed as the special hunks will be phased out in favour of the LoadSeg() patch.