[PATCH] OBP Memory deallocation
by Olivier Danet
During boot, NextSTEP do memory allocation & deallocation:
- Implement dumb_memfree()
- Unmap freed memory (during boot, all the virtual memory is probed,
so freed memory must be properly unmapped)
- The pointer given to dumb_memfree() does not always match the pointer
returned by dumb_memalloc().
In that case, try to find a memory area which includes the pointer
address.
Ugly, indeed...
Signed-off-by : Olivier Danet <odanet(a)caramail.com>
Index: arch/sparc32/lib.c
===================================================================
--- arch/sparc32/lib.c (révision 1257)
+++ arch/sparc32/lib.c (copie de travail)
@@ -239,10 +245,18 @@
return obp_memalloc(va, size, align);
}
-void obp_dumb_memfree(__attribute__((unused))char *va,
- __attribute__((unused))unsigned sz)
+void obp_dumb_memfree(char *va, unsigned size)
{
- DPRINTF("obp_dumb_memfree 0x%p (size %d)\n", va, sz);
+ phys_addr_t phys;
+ ucell cellmode;
+
+ DPRINTF("obp_dumb_memfree: virta 0x%x, sz %d\n", (unsigned int)va,
size);
+
+ phys = ofmem_translate(pointer2cell(va), &cellmode);
+
+ ofmem_unmap(pointer2cell(va), size);
+ ofmem_release_virt(pointer2cell(va), size);
+ ofmem_release_phys(phys, size);
}
/* Data fault handling routines */
Index: arch/sparc32/ofmem_sparc32.c
===================================================================
--- arch/sparc32/ofmem_sparc32.c (révision 1257)
+++ arch/sparc32/ofmem_sparc32.c (copie de travail)
@@ -133,17 +133,24 @@
/* Generate memory available property entry for Sparc32 */
void ofmem_arch_create_available_entry(phandle_t ph, ucell
*availentry, phys_addr_t start, ucell size)
{
- int i = 0;
+ int i;
- i += ofmem_arch_encode_physaddr(availentry, start);
+ i = ofmem_arch_encode_physaddr(availentry, start);
availentry[i] = size;
}
/* Unmap a set of pages */
void ofmem_arch_unmap_pages(ucell virt, ucell size)
{
- /* OFMEM re-maps the pages for us, so just ensure the TLB is in sync */
- srmmu_flush_whole_tlb();
+ unsigned i;
+ unsigned long pa;
+
+ for (i = 0; i < size; i += PAGE_SIZE) {
+ pa = find_pte(virt, 0);
+ *(uint32_t *)pa = 0;
+ virt += PAGE_SIZE;
+ }
+ srmmu_flush_whole_tlb();
}
/* Map a set of pages */
Index: arch/sparc32/romvec.h
===================================================================
--- arch/sparc32/romvec.h (révision 1257)
+++ arch/sparc32/romvec.h (copie de travail)
@@ -30,8 +30,8 @@
int obp_inst2pkg_handler(int dev_desc);
char *obp_dumb_memalloc(char *va, unsigned int size);
char *obp_dumb_memalloc_handler(char *va, unsigned int size);
-void obp_dumb_memfree(__attribute__((unused))char *va,
__attribute__((unused))unsigned sz);
-void obp_dumb_memfree_handler(__attribute__((unused))char *va,
__attribute__((unused))unsigned sz);
+void obp_dumb_memfree(char *va, unsigned size);
+void obp_dumb_memfree_handler(char *va, unsigned size);
char *obp_dumb_mmap(char *va, int which_io, unsigned int pa, unsigned
int size);
char *obp_dumb_mmap_handler(char *va, int which_io, unsigned int pa,
unsigned int size);
void obp_dumb_munmap(__attribute__((unused)) char *va,
__attribute__((unused)) unsigned int size);
Index: libopenbios/ofmem_common.c
===================================================================
--- libopenbios/ofmem_common.c (révision 1257)
+++ libopenbios/ofmem_common.c (copie de travail)
@@ -887,16 +887,25 @@
cr = *r;
*r = (**r).next;
free(cr);
-
return;
}
+ cr = *r;
+ for ( ; *r && ((**r).next)->start < ea; r = &(**r).next) {
+ cr = *r;
+ }
- for( ; *r && ((**r).next)->start < ea; r=&(**r).next ) {
+ if (((**r).next)->start == ea) {
+ /* Found matching entry */
+ cr = (**r).next;
+ (**r).next = cr->next;
+ free(cr);
+
+ } else {
+ /* Not exactly matching, delete the previous entry, with start
< ea */
+ free(cr->next);
+ cr->next = (*r)->next;
+ return;
}
-
- cr = (**r).next;
- (**r).next = cr->next;
- free(cr);
}
static int remove_range( phys_addr_t ea, ucell size, range_t **r )
===================================================================
5 years, 3 months
- 2
- 6
"next-property" reloaded !
by Olivier Danet
Hello !
I have tried to recreate the 3 known variants of next-property:
Standard : a) true "string" b) false c) false
Sun Ultra10 : a) -1 "string" b) nothing c) nothing
PowerPC iBook : a) -1 "string" b) -1 0 0 c) 0
Here is a tentative patch :
Index: forth/admin/devices.fs
===================================================================
--- forth/admin/devices.fs (révision 1269)
+++ forth/admin/devices.fs (copie de travail)
@@ -451,7 +451,7 @@
?active-package dup >r if
0 0
begin
- r@ next-property
+ r@ next-property-std
while
cr 2dup dup -rot type
begin ." " 1+ dup d# 26 >= until drop
Index: forth/device/property.fs
===================================================================
--- forth/device/property.fs (révision 1269)
+++ forth/device/property.fs (copie de travail)
@@ -63,14 +63,14 @@
;
\ From package (5.3.4.1)
-: next-property
+: next-property-std
( previous-str previous-len phandle -- false | name-str name-len true )
>r
2dup 0= swap 0= or if
2drop r> >dn.properties @
else
r> find-property dup if @ then
- ?dup if >prop.next @ then
+ dup if >prop.next @ then
then
?dup if
@@ -81,7 +81,57 @@
then
;
+: next-property-ppc
+( previous-str previous-len phandle -- false | name-str name-len true |
0 0 true )
+ >r
+ 2dup 0= swap 0= or if
+ 2drop r> >dn.properties @ true
+ else
+ r> find-property dup if @ then
+ dup if >prop.next @ true then
+ then
+ if
+ dup if
+ >prop.name @ dup cstrlen
+ ( phandle name-str name-len )
+ else
+ 0
+ ( 0 0 )
+ then
+ true
+ else
+ false
+ then
+;
+
+: next-property-std-bis
+ next-property-ppc
+ dup
+ if over 0=
+ if
+ 2drop
+ then
+ then
+;
+
+: next-property-s64
+ next-property-std
+ ?dup drop
+;
+
+: next-property
+ [IFDEF] CONFIG_PPC
+ next-property-ppc
+ [ELSE]
+ [IFDEF] CONFIG_SPARC64
+ next-property-s64
+ [ELSE]
+ next-property-std
+ [THEN]
+ [THEN]
+;
+
\
\ 5.3.5.4 Property value access
\
Index: forth/system/ciface.fs
===================================================================
--- forth/system/ciface.fs (révision 1269)
+++ forth/system/ciface.fs (copie de travail)
@@ -121,7 +121,7 @@
( buf prev prev_len )
- r> next-property if
+ r> next-property-std if
( buf name name_len )
dup 1+ -rot ci-strcpy drop 1
else
===================================================================
there are :
next-property-std = standard, used by Sparc32 "romvec" C code and by
forth code, like ".properties"
next-property-s64 = Sparc64 version, built from next-property-std
next-property-ppc = PowerPC version
next-property-std-bis = Alternate standard version, built from the PPC
one (e.g. when compiling for PPC target)
Tests :
showstack
cd screen
\ Sparc32
" " ?active-package next-property-std cr . type cr
" name" ?active-package next-property-std cr . type cr
" interrupts" ?active-package next-property-std cr . cr
" ttttt" ?active-package next-property-std cr . cr
\ PowerPC
" " ?active-package next-property-ppc cr . type cr
" name" ?active-package next-property-ppc cr . type cr
" interrupts" ?active-package next-property-ppc cr . . . cr
" ttttt" ?active-package next-property-ppc cr . cr
\ Sparc64
" " ?active-package next-property-s64 cr . type cr
" name" ?active-package next-property-s64 cr . type cr
" interrupts" ?active-package next-property-s64 cr
" ttttt" ?active-package next-property-s64 cr
" " ?active-package next-property-std-bis cr . type cr
" name" ?active-package next-property-std-bis cr . type cr
" interrupts" ?active-package next-property-std-bis cr . cr
" ttttt" ?active-package next-property-std-bis cr . cr
Currently all variants are available simultaneously.
Additional [IFDEF] can be used to hide some variants according
to compilation options, generate the -std from the -ppc,..
One more oddity:
NetBSD is not the only one to do strange things with properties.
In OpenBSD, one can find :
arch/sparc/dev/sbus.c:int sbus_testdma(sc, ca)
...
getpropint(0, "slave-only", 0)
...
arch/sparc/dev/sbus.c:int sbus_print(args, sbus)
...
getpropint(0, sl, 0)
...
Here, 0 should select the 'root' node.
It really looks like a bug in OpenBSD.
It does not prevent the OS from running though.
Olivier
5 years, 4 months
- 2
- 4
[PATCH] Add the prog interface field to class-code property
by BALATON Zoltan
The class-code property of PCI devices should also include the prog
interface field. This patch adds this.
Signed-off-by: BALATON Zoltan <balaton(a)eik.bme.hu>
===================================================================
--- drivers/pci.c (revision 1269)
+++ drivers/pci.c (working copy)
@@ -893,11 +893,13 @@
int status,id;
uint16_t vendor_id, device_id;
uint8_t rev;
+ uint8_t class_prog;
uint32_t class_code;
vendor_id = pci_config_read16(addr, PCI_VENDOR_ID);
device_id = pci_config_read16(addr, PCI_DEVICE_ID);
rev = pci_config_read8(addr, PCI_REVISION_ID);
+ class_prog = pci_config_read8(addr, PCI_CLASS_PROG);
class_code = pci_config_read16(addr, PCI_CLASS_DEVICE);
if (pci_dev) {
@@ -925,7 +927,7 @@
set_int_property(dev, "vendor-id", vendor_id);
set_int_property(dev, "device-id", device_id);
set_int_property(dev, "revision-id", rev);
- set_int_property(dev, "class-code", class_code << 8);
+ set_int_property(dev, "class-code", class_code << 8 | class_prog);
if (config->irq_pin) {
OLDWORLD(set_int_property(dev, "AAPL,interrupts",
===============================================================
5 years, 4 months
- 2
- 3
[commit] r1269 - trunk/openbios-devel/arch/sparc32
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:46 2014
New Revision: 1269
URL: http://tracker.coreboot.org/trac/openbios/changeset/1269
Log:
SPARC32: alter obp_dumb_memalloc() to handle allocation with va == 0
It seems that the obp_dumb_memalloc() function can be called with va == 0
which means that OpenBIOS should pick the next free virtual address itself.
Otherwise we end up accidentally mapping into zero page causing all sorts of
problems.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/arch/sparc32/lib.c
Modified: trunk/openbios-devel/arch/sparc32/lib.c
==============================================================================
--- trunk/openbios-devel/arch/sparc32/lib.c Sun Feb 16 17:26:43 2014 (r1268)
+++ trunk/openbios-devel/arch/sparc32/lib.c Sun Feb 16 17:26:46 2014 (r1269)
@@ -229,14 +229,28 @@
char *obp_dumb_memalloc(char *va, unsigned int size)
{
unsigned long align = size;
+ phys_addr_t phys;
+ ucell virt;
DPRINTF("obp_dumb_memalloc: virta 0x%x, sz %d\n", (unsigned int)va, size);
/* Solaris seems to assume that the returned value is physically aligned to size.
- e.g. it is used for setting up page tables. Fortunately this is now handled by
- ofmem_claim_phys() above. */
+ e.g. it is used for setting up page tables. */
- return obp_memalloc(va, size, align);
+ /* Claim physical memory */
+ phys = ofmem_claim_phys(-1, size, align);
+
+ /* Claim virtual memory - if va == NULL then we choose va address */
+ if (va == NULL) {
+ virt = ofmem_claim_virt((ucell)-1, size, align);
+ } else {
+ virt = ofmem_claim_virt(pointer2cell(va), size, 0);
+ }
+
+ /* Map the memory */
+ ofmem_map(phys, virt, size, ofmem_arch_default_translation_mode(phys));
+
+ return cell2pointer(virt);
}
void obp_dumb_memfree(char *va, unsigned size)
5 years, 5 months
- 1
- 0
[commit] r1268 - trunk/openbios-devel/libopenbios
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:43 2014
New Revision: 1268
URL: http://tracker.coreboot.org/trac/openbios/changeset/1268
Log:
OFMEM: ensure minimum alignment of PAGE_SIZE
NextStep appears to request an alignment of 4 for some of its large memory
ranges, and since this is an exact power of 2 then the code to ensure a
minimum alignment of PAGE_SIZE is accidentally bypassed. Move this to a
separate check so we can guarantee the minimum alignment requirement is always
observed.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/libopenbios/ofmem_common.c
Modified: trunk/openbios-devel/libopenbios/ofmem_common.c
==============================================================================
--- trunk/openbios-devel/libopenbios/ofmem_common.c Sun Feb 16 17:26:40 2014 (r1267)
+++ trunk/openbios-devel/libopenbios/ofmem_common.c Sun Feb 16 17:26:43 2014 (r1268)
@@ -428,13 +428,20 @@
{
phys_addr_t base = min;
range_t *r2;
- ucell old_align;
+ ucell old_align = align;
int i;
+ if( (align < PAGE_SIZE) ) {
+
+ /* Minimum alignment is page size */
+ align = PAGE_SIZE;
+
+ OFMEM_TRACE("warning: bad alignment " FMT_ucellx " rounded up to " FMT_ucellx "\n", old_align, align);
+ }
+
if( (align & (align-1)) ) {
/* As per IEEE1275 specification, round up to the nearest power of 2 */
- old_align = align;
if (old_align <= PAGE_SIZE) {
align = PAGE_SIZE;
} else {
@@ -447,8 +454,6 @@
OFMEM_TRACE("warning: bad alignment " FMT_ucellx " rounded up to " FMT_ucellx "\n", old_align, align);
}
- if( !align )
- align = PAGE_SIZE;
base = reverse ? max - size : min;
r2 = reverse ? NULL : r;
5 years, 5 months
- 1
- 0
[commit] r1267 - trunk/openbios-devel/arch/sparc32
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:40 2014
New Revision: 1267
URL: http://tracker.coreboot.org/trac/openbios/changeset/1267
Log:
SPARC32: unmap freed memory in ofmem_arch_unmap_pages()
During NextStep boot, all the virtual memory is probed so freed memory must be
properly unmapped.
Signed-off-by: Olivier Danet <odanet(a)caramail.com>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/arch/sparc32/ofmem_sparc32.c
Modified: trunk/openbios-devel/arch/sparc32/ofmem_sparc32.c
==============================================================================
--- trunk/openbios-devel/arch/sparc32/ofmem_sparc32.c Sun Feb 16 17:26:37 2014 (r1266)
+++ trunk/openbios-devel/arch/sparc32/ofmem_sparc32.c Sun Feb 16 17:26:40 2014 (r1267)
@@ -142,8 +142,16 @@
/* Unmap a set of pages */
void ofmem_arch_unmap_pages(ucell virt, ucell size)
{
- /* OFMEM re-maps the pages for us, so just ensure the TLB is in sync */
- srmmu_flush_whole_tlb();
+ unsigned long pa;
+ ucell i;
+
+ for (i = 0; i < size; i += PAGE_SIZE) {
+ pa = find_pte(virt, 0);
+ *(uint32_t *)pa = 0;
+ virt += PAGE_SIZE;
+ }
+
+ srmmu_flush_whole_tlb();
}
/* Map a set of pages */
5 years, 5 months
- 1
- 0
[commit] r1266 - trunk/openbios-devel/arch/sparc32
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:37 2014
New Revision: 1266
URL: http://tracker.coreboot.org/trac/openbios/changeset/1266
Log:
SPARC32: implement obp_dumb_memfree()
Signed-off-by: Olivier Danet <odanet(a)caramail.com>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/arch/sparc32/lib.c
trunk/openbios-devel/arch/sparc32/romvec.h
Modified: trunk/openbios-devel/arch/sparc32/lib.c
==============================================================================
--- trunk/openbios-devel/arch/sparc32/lib.c Sun Feb 16 17:26:34 2014 (r1265)
+++ trunk/openbios-devel/arch/sparc32/lib.c Sun Feb 16 17:26:37 2014 (r1266)
@@ -239,10 +239,18 @@
return obp_memalloc(va, size, align);
}
-void obp_dumb_memfree(__attribute__((unused))char *va,
- __attribute__((unused))unsigned sz)
+void obp_dumb_memfree(char *va, unsigned size)
{
- DPRINTF("obp_dumb_memfree 0x%p (size %d)\n", va, sz);
+ phys_addr_t phys;
+ ucell cellmode;
+
+ DPRINTF("obp_dumb_memfree: virta 0x%x, sz %d\n", (unsigned int)va, size);
+
+ phys = ofmem_translate(pointer2cell(va), &cellmode);
+
+ ofmem_unmap(pointer2cell(va), size);
+ ofmem_release_virt(pointer2cell(va), size);
+ ofmem_release_phys(phys, size);
}
/* Data fault handling routines */
Modified: trunk/openbios-devel/arch/sparc32/romvec.h
==============================================================================
--- trunk/openbios-devel/arch/sparc32/romvec.h Sun Feb 16 17:26:34 2014 (r1265)
+++ trunk/openbios-devel/arch/sparc32/romvec.h Sun Feb 16 17:26:37 2014 (r1266)
@@ -31,8 +31,8 @@
int obp_inst2pkg_handler(int dev_desc);
char *obp_dumb_memalloc(char *va, unsigned int size);
char *obp_dumb_memalloc_handler(char *va, unsigned int size);
-void obp_dumb_memfree(__attribute__((unused))char *va, __attribute__((unused))unsigned sz);
-void obp_dumb_memfree_handler(__attribute__((unused))char *va, __attribute__((unused))unsigned sz);
+void obp_dumb_memfree(char *va, unsigned size);
+void obp_dumb_memfree_handler(char *va, unsigned size);
char *obp_dumb_mmap(char *va, int which_io, unsigned int pa, unsigned int size);
char *obp_dumb_mmap_handler(char *va, int which_io, unsigned int pa, unsigned int size);
void obp_dumb_munmap(__attribute__((unused)) char *va, __attribute__((unused)) unsigned int size);
5 years, 5 months
- 1
- 0
[commit] r1265 - trunk/openbios-devel/libopenbios
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:34 2014
New Revision: 1265
URL: http://tracker.coreboot.org/trac/openbios/changeset/1265
Log:
SPARC32: copy the a.out header beneath load-base during load
NextSTEP's bootloader for Sparc32 expects that the 32 bytes a.out header
is copied just before the start address.
Since this is a compatibility hack for an implementation detail of OBP then
implement the functionality within a #define AOUT_HEADER_COPY section so
it can easily be removed if people really need a strictly compliant firmware.
Signed-off-by: Olivier Danet <odanet(a)caramail.com>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/libopenbios/aout_load.c
Modified: trunk/openbios-devel/libopenbios/aout_load.c
==============================================================================
--- trunk/openbios-devel/libopenbios/aout_load.c Sun Feb 16 17:26:31 2014 (r1264)
+++ trunk/openbios-devel/libopenbios/aout_load.c Sun Feb 16 17:26:34 2014 (r1265)
@@ -10,6 +10,12 @@
#define CONFIG_SPARC64_PAGE_SIZE_8KB
#endif
+/* NextStep bootloader on SPARC32 expects the a.out header directly
+ below load-base (0x4000) */
+#ifdef CONFIG_SPARC32
+#define AOUT_HEADER_COPY
+#endif
+
#include "libopenbios/sys_info.h"
#include "libopenbios/bindings.h"
#include "libopenbios/aout_load.h"
@@ -143,6 +149,11 @@
debug("Loaded %lu bytes\n", size);
debug("entry point is %#lx\n", start);
+#ifdef AOUT_HEADER_COPY
+ // Copy the a.out header just before start
+ memcpy((char *)(start - 0x20), &ehdr, 0x20);
+#endif
+
// Initialise saved-program-state
PUSH(addr_fixup(start));
feval("saved-program-state >sps.entry !");
5 years, 5 months
- 1
- 0
[commit] r1264 - in trunk/openbios-devel: arch/sparc32 forth/device
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:31 2014
New Revision: 1264
URL: http://tracker.coreboot.org/trac/openbios/changeset/1264
Log:
SPARC32: implement Forth get-msecs word
Switch obp_ticks over to be a pointer to the counter rather than its value, then
point this to the contents of a new Forth obp-ticks variable. This allows a
simple get-msecs word to be implemented for SPARC32 which simply reads the value
of the obp-ticks variable and places it on the stack.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/arch/sparc32/romvec.c
trunk/openbios-devel/arch/sparc32/romvec.h
trunk/openbios-devel/arch/sparc32/vectors.S
trunk/openbios-devel/forth/device/other.fs
Modified: trunk/openbios-devel/arch/sparc32/romvec.c
==============================================================================
--- trunk/openbios-devel/arch/sparc32/romvec.c Sun Feb 16 17:26:28 2014 (r1263)
+++ trunk/openbios-devel/arch/sparc32/romvec.c Sun Feb 16 17:26:31 2014 (r1264)
@@ -449,7 +449,7 @@
dstackcnt = dstacktmp;
}
-volatile uint32_t obp_ticks;
+volatile uint32_t *obp_ticks;
void *
init_openprom(void)
@@ -482,9 +482,11 @@
romvec0.pv_printf = obp_printf_handler;
romvec0.pv_abort = obp_abort_handler;
- /* Reset the tick counter */
- obp_ticks = 0;
- romvec0.pv_ticks = &obp_ticks;
+ /* Point to the Forth obp-ticks variable and reset */
+ fword("obp-ticks");
+ obp_ticks = cell2pointer(POP());
+ *obp_ticks = 0;
+ romvec0.pv_ticks = obp_ticks;
romvec0.pv_halt = obp_halt_handler;
romvec0.pv_synchook = &sync_hook;
Modified: trunk/openbios-devel/arch/sparc32/romvec.h
==============================================================================
--- trunk/openbios-devel/arch/sparc32/romvec.h Sun Feb 16 17:26:28 2014 (r1263)
+++ trunk/openbios-devel/arch/sparc32/romvec.h Sun Feb 16 17:26:31 2014 (r1264)
@@ -2,7 +2,7 @@
* romvec main C function and handler declarations
*/
-extern volatile uint32_t obp_ticks;
+extern volatile uint32_t *obp_ticks;
void *init_openprom(void);
int obp_devopen(char *str);
Modified: trunk/openbios-devel/arch/sparc32/vectors.S
==============================================================================
--- trunk/openbios-devel/arch/sparc32/vectors.S Sun Feb 16 17:26:28 2014 (r1263)
+++ trunk/openbios-devel/arch/sparc32/vectors.S Sun Feb 16 17:26:31 2014 (r1264)
@@ -208,9 +208,10 @@
ld [%l7 + %lo(counter_regs)], %l7
ld [%l7], %g0
sethi %hi(obp_ticks), %l7
- ld [%l7 + %lo(obp_ticks)], %l6
+ ld [%l7 + %lo(obp_ticks)], %l7
+ ld [%l7], %l6
add %l6, 10, %l6
- st %l6, [%l7 + %lo(obp_ticks)]
+ st %l6, [%l7]
jmp %l1
rett %l2
Modified: trunk/openbios-devel/forth/device/other.fs
==============================================================================
--- trunk/openbios-devel/forth/device/other.fs Sun Feb 16 17:26:28 2014 (r1263)
+++ trunk/openbios-devel/forth/device/other.fs Sun Feb 16 17:26:31 2014 (r1264)
@@ -93,12 +93,25 @@
\ 5.3.7.3 Time
+[IFDEF] CONFIG_SPARC32
+
+\ OBP tick value updated by timer interrupt
+variable obp-ticks
+
+: get-msecs ( -- n )
+ obp-ticks @
+ ;
+
+[ELSE]
+
0 value dummy-msecs
: get-msecs ( -- n )
dummy-msecs dup 1+ to dummy-msecs
;
-
+
+[THEN]
+
: ms ( n -- )
get-msecs +
begin dup get-msecs < until
5 years, 5 months
- 1
- 0
[commit] r1263 - in trunk/openbios-devel: arch/sparc32 drivers
by repository service
Author: mcayland
Date: Sun Feb 16 17:26:28 2014
New Revision: 1263
URL: http://tracker.coreboot.org/trac/openbios/changeset/1263
Log:
SPARC32: implement romvec.pv_ticks counter
romvec.pv_ticks is a pointer to an 'uptime' counter, calibrated in
milliseconds.
We use the [1st] CPU timer, running at 100Hz, to increment (+10) the counter.
This counter is used by NextStep to measure the 10 seconds delay during boot,
when asking about boot options.
Signed-off-by: Olivier Danet <odanet(a)caramail.com>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
Modified:
trunk/openbios-devel/arch/sparc32/openbios.c
trunk/openbios-devel/arch/sparc32/openprom.h
trunk/openbios-devel/arch/sparc32/romvec.c
trunk/openbios-devel/arch/sparc32/romvec.h
trunk/openbios-devel/arch/sparc32/vectors.S
trunk/openbios-devel/drivers/obio.c
trunk/openbios-devel/drivers/obio.h
Modified: trunk/openbios-devel/arch/sparc32/openbios.c
==============================================================================
--- trunk/openbios-devel/arch/sparc32/openbios.c Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/arch/sparc32/openbios.c Sun Feb 16 17:26:28 2014 (r1263)
@@ -23,6 +23,7 @@
#include "boot.h"
#include "romvec.h"
#include "openprom.h"
+#include "psr.h"
#include "libopenbios/video.h"
#define NO_QEMU_PROTOS
#include "arch/common/fw_cfg.h"
@@ -916,6 +917,11 @@
fword("find-device");
setup_uuid();
+
+ /* Enable interrupts */
+ temp = get_psr();
+ temp = (temp & ~PSR_PIL) | (13 << 8); /* Enable CPU timer interrupt (level 14) */
+ put_psr(temp);
}
extern struct _console_ops arch_console_ops;
Modified: trunk/openbios-devel/arch/sparc32/openprom.h
==============================================================================
--- trunk/openbios-devel/arch/sparc32/openprom.h Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/arch/sparc32/openprom.h Sun Feb 16 17:26:28 2014 (r1263)
@@ -117,7 +117,7 @@
void (*pv_reboot)(char *bootstr);
void (*pv_printf)(__const__ char *fmt, ...);
void (*pv_abort)(void);
- __volatile__ int *pv_ticks;
+ __volatile__ unsigned int *pv_ticks;
void (*pv_halt)(void);
void (**pv_synchook)(void);
Modified: trunk/openbios-devel/arch/sparc32/romvec.c
==============================================================================
--- trunk/openbios-devel/arch/sparc32/romvec.c Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/arch/sparc32/romvec.c Sun Feb 16 17:26:28 2014 (r1263)
@@ -449,6 +449,8 @@
dstackcnt = dstacktmp;
}
+volatile uint32_t obp_ticks;
+
void *
init_openprom(void)
{
@@ -479,6 +481,11 @@
romvec0.pv_reboot = obp_reboot_handler;
romvec0.pv_printf = obp_printf_handler;
romvec0.pv_abort = obp_abort_handler;
+
+ /* Reset the tick counter */
+ obp_ticks = 0;
+ romvec0.pv_ticks = &obp_ticks;
+
romvec0.pv_halt = obp_halt_handler;
romvec0.pv_synchook = &sync_hook;
romvec0.pv_v0bootargs = &obp_argp;
Modified: trunk/openbios-devel/arch/sparc32/romvec.h
==============================================================================
--- trunk/openbios-devel/arch/sparc32/romvec.h Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/arch/sparc32/romvec.h Sun Feb 16 17:26:28 2014 (r1263)
@@ -2,6 +2,7 @@
* romvec main C function and handler declarations
*/
+extern volatile uint32_t obp_ticks;
void *init_openprom(void);
int obp_devopen(char *str);
Modified: trunk/openbios-devel/arch/sparc32/vectors.S
==============================================================================
--- trunk/openbios-devel/arch/sparc32/vectors.S Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/arch/sparc32/vectors.S Sun Feb 16 17:26:28 2014 (r1263)
@@ -207,6 +207,10 @@
sethi %hi(counter_regs), %l7
ld [%l7 + %lo(counter_regs)], %l7
ld [%l7], %g0
+ sethi %hi(obp_ticks), %l7
+ ld [%l7 + %lo(obp_ticks)], %l6
+ add %l6, 10, %l6
+ st %l6, [%l7 + %lo(obp_ticks)]
jmp %l1
rett %l2
Modified: trunk/openbios-devel/drivers/obio.c
==============================================================================
--- trunk/openbios-devel/drivers/obio.c Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/drivers/obio.c Sun Feb 16 17:26:28 2014 (r1263)
@@ -287,7 +287,7 @@
counter_regs = (struct sun4m_timer_regs *)ofmem_map_io(base + (uint64_t)offset, sizeof(*counter_regs));
- counter_regs->cfg = 0xffffffff;
+ counter_regs->cfg = 0xfffffffe;
counter_regs->l10_timer_limit = 0;
counter_regs->cpu_timers[0].l14_timer_limit = 0x9c4000; /* see comment in obio.h */
counter_regs->cpu_timers[0].cntrl = 1;
Modified: trunk/openbios-devel/drivers/obio.h
==============================================================================
--- trunk/openbios-devel/drivers/obio.h Sun Feb 16 17:26:25 2014 (r1262)
+++ trunk/openbios-devel/drivers/obio.h Sun Feb 16 17:26:28 2014 (r1263)
@@ -88,7 +88,7 @@
* Registers of hardware timer in sun4m.
*/
struct sun4m_timer_percpu {
- volatile unsigned int l14_timer_limit; /* Initial value is 0x009c4000 */
+ volatile unsigned int l14_timer_limit; /* Initial value is 0x009c4000 = 10ms period*/
volatile unsigned int l14_cur_count;
};
5 years, 5 months
- 1
- 0