[OpenBIOS] [PATCH] OBP Memory deallocation
Olivier Danet
odanet at caramail.com
Tue Feb 11 01:44:27 CET 2014
On 10/02/2014 02:40, Mark Cave-Ayland wrote:
> On 09/02/14 22:59, Olivier Danet wrote:
>
> The first 2 parts of this patch look fine; however the last part seems
> absolutely unbelievable(!). I wonder if this could be side-effect of
> your change to the memory allocation function? It would definitely be
> worth checking this again once you've had a look at my comments from
> the previous memory allocation patch.
>
> ATB,
> Mark.
I knew you would not like that ...
Allocation :
- In the current code, 'arch/sparc32/lib.c:obp_memalloc(va,size,align)'
always calls
ofmem_claim_virt(va,size,align) with align=0. So allocating with va=0
does not work.
IIRC, there was also a problem ensuring that the physical and virtual
areas had the
same size and alignment.
Deallocation :
Here is a log of NextStep booting the ISO, with these crazy patches and
this option in OpenBIOS :
<option name="CONFIG_DEBUG_OFMEM" type="boolean" value="true"/>
and "#define CONFIG_DEBUG_MEM" in arch/sparc32/lib.c
... and a few manual linefeeds
Welcome to OpenBIOS v1.1 built on Feb 10 2014 23:08
Type 'help' for detailed information
Trying cdrom:d...
Trying cdrom...
Not a bootable ELF image
Loading a.out image...
Loaded 2772176 bytes
entry point is 0x4000
bootpath:
/iommu at 0,10000000/sbus at 0,10001000/espdma at 5,8400000/esp at 5,8800000/sd at 2,0
start_adr: 0
num_bytes: 8000000
Jumping to entry point 00004000 for type 00000005...
switching to new context:
obp_dumb_memalloc: virta 0x0, sz 7260
obp_memalloc: virta 0x0, sz 7260, align 7260
OFMEM: ofmem_claim phys=ffffffffffffffff size=00002000 align=00001c5c
OFMEM: warning: bad alignment 00001c5c rounded up to 00002000
OFMEM: ofmem_claim_virt virt=00000000 size=00002000 align=00001c5c
OFMEM: warning: bad alignment 00001c5c rounded up to 00002000
OFMEM: ofmem_claim_virt virt=00000000 size=00002000 align=00001000
OFMEM: ofmem_map_page_range fdffc000 -> 006fde000 00002000 mode 000000bc
obp_dumb_memfree: virta 0xfdffc000, sz 7260
OFMEM: ofmem_unmap fdffc000 00001c5c
OFMEM: unmap_page_range found fdffc000 -> 006fde000 00002000 mode 000000bc
OFMEM: ofmem_release_virt addr=fdffc000 size=00001c5c
OFMEM: ofmem_release_phys addr=006fde000 size=00001c5c
obp_dumb_memalloc: virta 0x0, sz 8192
obp_memalloc: virta 0x0, sz 8192, align 8192
OFMEM: ofmem_claim phys=ffffffffffffffff size=00002000 align=00002000
OFMEM: ofmem_claim_virt virt=00000000 size=00002000 align=00002000
OFMEM: ofmem_claim_virt virt=00000000 size=00002000 align=00001000
OFMEM: ofmem_map_page_range fdffa000 -> 006fde000 00002000 mode 000000bc
obp_dumb_memalloc: virta 0x0, sz 8192
obp_memalloc: virta 0x0, sz 8192, align 8192
OFMEM: ofmem_claim phys=ffffffffffffffff size=00002000 align=00002000
OFMEM: ofmem_claim_virt virt=00000000 size=00002000 align=00002000
OFMEM: ofmem_claim_virt virt=00000000 size=00002000 align=00001000
OFMEM: ofmem_map_page_range fdff6000 -> 006fdc000 00002000 mode 000000bc
obp_dumb_memalloc: virta 0x0, sz 636
obp_memalloc: virta 0x0, sz 636, align 636
OFMEM: ofmem_claim phys=ffffffffffffffff size=00001000 align=0000027c
OFMEM: warning: bad alignment 0000027c rounded up to 00001000
OFMEM: ofmem_claim_virt virt=00000000 size=00001000 align=0000027c
OFMEM: warning: bad alignment 0000027c rounded up to 00001000
OFMEM: ofmem_claim_virt virt=00000000 size=00001000 align=00001000
OFMEM: ofmem_map_page_range fdff4000 -> 006fdb000 00001000 mode 000000bc
obp_dumb_memalloc: virta 0x80000, sz 49152
obp_memalloc: virta 0x80000, sz 49152, align 49152
OFMEM: ofmem_claim phys=ffffffffffffffff size=0000c000 align=0000c000
OFMEM: warning: bad alignment 0000c000 rounded up to 00010000
OFMEM: ofmem_claim_virt virt=00080000 size=0000c000 align=0000c000
OFMEM: warning: bad alignment 0000c000 rounded up to 00010000
OFMEM: ofmem_claim_virt virt=00080000 size=0000c000 align=00000000
OFMEM: ofmem_map_page_range 00080000 -> 006fc0000 0000c000 mode 000000bc
OFMEM: mapping altered virt=00080000)
obp_dumb_memalloc: virta 0x8c000, sz 8192
obp_memalloc: virta 0x8c000, sz 8192, align 8192
OFMEM: ofmem_claim phys=ffffffffffffffff size=00002000 align=00002000
OFMEM: ofmem_claim_virt virt=0008c000 size=00002000 align=00002000
OFMEM: ofmem_claim_virt virt=0008c000 size=00002000 align=00000000
OFMEM: ofmem_map_page_range 0008c000 -> 006fd8000 00002000 mode 000000bc
OFMEM: mapping altered virt=0008c000)
obp_dumb_memalloc: virta 0x0, sz 32
obp_memalloc: virta 0x0, sz 32, align 32
OFMEM: ofmem_claim phys=ffffffffffffffff size=00001000 align=00000020
OFMEM: ofmem_claim_virt virt=00000000 size=00001000 align=00000020
OFMEM: ofmem_claim_virt virt=00000000 size=00001000 align=00001000
OFMEM: ofmem_map_page_range fdff0000 -> 006fda000 00001000 mode 000000bc
initsyms:
obp_dumb_memfree: virta 0xfdff4000, sz 636
OFMEM: ofmem_unmap fdff4000 0000027c
OFMEM: unmap_page_range found fdff4000 -> 006fdb000 00001000 mode 000000bc
OFMEM: ofmem_release_virt addr=fdff4000 size=0000027c
OFMEM: ofmem_release_phys addr=006fdb000 size=0000027c
obp_dumb_memfree: virta 0xfdff6000, sz 8192
OFMEM: ofmem_unmap fdff6000 00002000
OFMEM: unmap_page_range found fdff6000 -> 006fdc000 00002000 mode 000000bc
OFMEM: ofmem_release_virt addr=fdff6000 size=00002000
OFMEM: ofmem_release_phys addr=006fdc000 size=00002000
obp_dumb_memalloc: virta 0xc3000, sz 57344
obp_memalloc: virta 0xc3000, sz 57344, align 57344
OFMEM: ofmem_claim phys=ffffffffffffffff size=0000e000 align=0000e000
OFMEM: warning: bad alignment 0000e000 rounded up to 00010000
OFMEM: ofmem_claim_virt virt=000c3000 size=0000e000 align=0000e000
OFMEM: warning: bad alignment 0000e000 rounded up to 00010000
OFMEM: ofmem_claim_virt virt=000c3000 size=0000e000 align=00000000
OFMEM: ofmem_map_page_range 000c3000 -> 006fb0000 0000e000 mode 000000bc
OFMEM: mapping altered virt=000c3000)
NEXTSTEP boot v3.3.4.17
114436 memory
NEXTSTEP will start up in 10 seconds, or you can:
Type -v and press Return to start up NEXTSTEP with diagnostic messages
Type ? and press Return to learn about advanced startup options
Type any other character to stop NEXTSTEP from starting up automatically
boot:
Type 1 to use the English language and USA keyboard while installing
NEXTSTEP.
Tapez 2 pour installer NEXTSTEP avec un clavier et des messages francais.
Eingabe 3 fuer NEXTSTEP-Installation mit deutscher Sprache und Tastatur.
Premi 4 per installare NEXTSTEP usando lingua italiana e tastiera italiana.
Pulse 5 para usar el idioma y el teclado espanol en la instalacion de
NEXTSTEP.
Skriv 6 for att installera NEXTSTEP pa svenska med svenskt tangentbord.
---> 1
This program is for installing NEXTSTEP on a hard disk.
THIS IS NOT AN UPGRADE: ANY EXISTING NEXTSTEP FILES WILL BE DELETED.
If you have files on your hard disk that you want to keep,
quit this program and copy what you want to keep onto another disk.
Type 1 to prepare to install NEXTSTEP.
Type 2 to quit this installation program.
---> 1
Loading NEXTSTEP
obp_memalloc: virta 0xf0000000, sz 1490984, align 262144
OFMEM: ofmem_claim phys=ffffffffffffffff size=0016d000 align=00040000
OFMEM: ofmem_claim_virt virt=f0000000 size=0016d000 align=00040000
OFMEM: ofmem_claim_virt virt=f0000000 size=0016d000 align=00000000
OFMEM: ofmem_map_page_range f0000000 -> 006e40000 0016d000 mode 000000bc
obp_dumb_memalloc: virta 0x0, sz 122952
obp_memalloc: virta 0x0, sz 122952, align 122952
OFMEM: ofmem_claim phys=ffffffffffffffff size=0001f000 align=0001e048
OFMEM: warning: bad alignment 0001e048 rounded up to 00020000
OFMEM: ofmem_claim_virt virt=00000000 size=0001f000 align=0001e048
OFMEM: warning: bad alignment 0001e048 rounded up to 00020000
OFMEM: ofmem_claim_virt virt=00000000 size=0001f000 align=00001000
OFMEM: ofmem_map_page_range fdfb1000 -> 006e20000 0001f000 mode 000000bc
initsyms:
Reading NEXTSTEP configuration
obp_dumb_memalloc: virta 0x30000, sz 163840
obp_memalloc: virta 0x30000, sz 163840, align 163840
OFMEM: ofmem_claim phys=ffffffffffffffff size=00028000 align=00028000
OFMEM: warning: bad alignment 00028000 rounded up to 00040000
OFMEM: ofmem_claim_virt virt=00030000 size=00028000 align=00028000
OFMEM: warning: bad alignment 00028000 rounded up to 00040000
OFMEM: ofmem_claim_virt virt=00030000 size=00028000 align=00000000
OFMEM: ofmem_map_page_range 00030000 -> 006dc0000 00028000 mode 000000bc
OFMEM: mapping altered virt=00030000)
obp_memalloc: virta 0xf016e000, sz 4194304, align 4
OFMEM: ofmem_claim phys=ffffffffffffffff size=00400000 align=00000004
OFMEM: ofmem_claim_virt virt=f016e000 size=00400000 align=00000004
OFMEM: ofmem_claim_virt virt=f016e000 size=00400000 align=00000000
OFMEM: ofmem_map_page_range f016e000 -> 0069c0000 00400000 mode 000000bc
obp_memalloc: virta 0xf056e000, sz 1048576, align 4
OFMEM: ofmem_claim phys=ffffffffffffffff size=00100000 align=00000004
OFMEM: ofmem_claim_virt virt=f056e000 size=00100000 align=00000004
OFMEM: ofmem_claim_virt virt=f056e000 size=00100000 align=00000000
OFMEM: ofmem_map_page_range f056e000 -> 0068c0000 00100000 mode 000000bc
Starting NEXTSTEP
obp_dumb_memfree: virta 0xf0180000, sz 4120576
OFMEM: ofmem_unmap f0180000 003ee000
OFMEM: unmap_page_range found f0180000 -> 0069d2000 003ee000 mode 000000bc
OFMEM: ofmem_release_virt addr=f0180000 size=003ee000
OFMEM: ofmem_release_phys addr=0069d2000 size=003ee000
obp_dumb_memfree: virta 0xf056e000, sz 1048576
OFMEM: ofmem_unmap f056e000 00100000
OFMEM: unmap_page_range found f056e000 -> 0068c0000 00100000 mode 000000bc
OFMEM: ofmem_release_virt addr=f056e000 size=00100000
OFMEM: ofmem_release_phys addr=0068c0000 size=00100000
obp_dumb_memfree: virta 0x30000, sz 163840
OFMEM: ofmem_unmap 00030000 00028000
OFMEM: unmap_page_range found 00030000 -> 006dc0000 00028000 mode 000000bc
OFMEM: ofmem_release_virt addr=00030000 size=00028000
OFMEM: ofmem_release_phys addr=006dc0000 size=00028000
==================================================================================
The big problem is in :
obp_dumb_memfree: virta 0xf0180000, sz 4120576 (=0x3EE000)
No memory starting at 0xf0180000 was ever allocated!
There is instead something at 0xf016e000, sz 4194304 (0x400000)
There is certainly a relation, as the end match :
f016e000+400000 = F056E000
f0180000+3EE000 = F056E000
I tried a bit to trace Sun's OFW with gdb, but with no luck.
I don't know whether it is a bug in OpenBIOS, QEMU or NextSTEP.
It certainly does not look reasonable...
Olivier
More information about the OpenBIOS
mailing list