[coreboot] malloc.c problems of option rom

Jason Wang wangqingpei at gmail.com
Sun Aug 2 15:37:38 CEST 2009


updated the new, which changed the display value
static void setup(void)
{
    int size = (unsigned int)(&_eheap - &_heap) - HDRSIZE;
    *((hdrtype_t *) hstart) = FREE_BLOCK(size);
//    printf("%s memory size:0x%x,begin:0x%x end:0x%x,
hstart:0x%x\n",__func__,size,(unsigned int)&_heap,(unsigned
int)&_eheap,(unsigned int )*((hdrtype_t *) hstart));


}

static void *alloc(int len)
{
    hdrtype_t header;
    void *ptr = hstart;
    printf("%s %d  length:0x%x,ptr=0x%x \n",__func__,__LINE__,len,(unsigned
int )ptr);
    printf("%s %d memory begin:0x%x end:0x%x,
hstart:0x%x\n",__func__,__LINE__,(unsigned int)&_heap,(unsigned
int)&_eheap,(unsigned int ) hstart);
    /* Align the size. */
    len = (len + 3) & ~3;

    if (!len || len > 0xffffff)
        return (void *)NULL;
//    while(1);

    /* Make sure the region is setup correctly. */
    if (!HAS_MAGIC(*((hdrtype_t *) ptr))){
        setup();
    }
    if (!HAS_MAGIC(*((hdrtype_t *) ptr))){
    printf("set up error,*ptr=0x%x\n",(unsigned int)*((hdrtype_t *) ptr));
    }

    /* Find some free space. */
    do {
        header = *((hdrtype_t *) ptr);
        int size = SIZE(header);

        if (!HAS_MAGIC(header) || size == 0) {
            printf("memory allocator panic!!! the size=0x%x.
header=0x%x\n",size,(unsigned int)header);
            halt();
        }

        if (header & FLAG_FREE) {
            if (len <= size) {
                void *nptr = ptr + (HDRSIZE + len);
                int nsize = size - (HDRSIZE + len);

                /* If there is still room in this block,
                 * then mark it as such otherwise account
                 * the whole space for that block.
                 */

                if (nsize > 0) {
                    /* Mark the block as used. */
                    *((hdrtype_t *) ptr) = USED_BLOCK(len);

                    /* Create a new free block. */
                    *((hdrtype_t *) nptr) =
                        FREE_BLOCK(nsize);
                } else {
                    /* Mark the block as used. */
                    *((hdrtype_t *) ptr) = USED_BLOCK(size);
                }

                return (void *)(ptr + HDRSIZE);
            }
        }

        ptr += HDRSIZE + size;

    } while (ptr < hend);

    /* Nothing available. */
    return (void *)NULL;
}
result log:
Running option rom at ce00:0003
hello, initialize_usb
00:13.0 4387:1002.0 OHCI controller
Not supported.
00:13.1 4388:1002.1 OHCI controller
Not supported.
00:13.2 4389:1002.2 OHCI controller
Not supported.
00:13.3 438a:1002.3 OHCI controller
Not supported.
00:13.4 438b:1002.4 OHCI controller
Not supported.
00:13.5 4386:1002.5 EHCI controller
Not supported.
00:05.0 3038:1106.0 UHCI controller
alloc 82  length:0x238,ptr=0x3077e0
alloc 83 memory begin:0x77e0 end:0x577e0, hstart:0x300000
set up error,*ptr=0xa2c0e2cc
memory allocator panic!!! the size=0xc0e2cc. header=0xa2c0e2cc

On Sun, Aug 2, 2009 at 4:27 AM, Jason Wang <wangqingpei at gmail.com> wrote:

>
> Hi all,
>    I added some printf message into malloc.c, and find that the function
> setup() which used to init the memory seems not executed very well.
> I put my own malloc.c and the log attached to this mail. Hope some one can
> help me to find out the problems.
>
> static void setup(void)
> {
>     int size = (unsigned int)(&_eheap - &_heap) - HDRSIZE;
>
>     *((hdrtype_t *) hstart) = FREE_BLOCK(size);
>     printf("%s the memory size:0x%x,begin:0x%x end:0x%x,
> hstart:0x%x\n",__func__,size,(unsigned int)&_heap,(unsigned
> int)&_eheap,(unsigned int )*((hdrtype_t *) hstart));
>
>
> }
>
> static void *alloc(int len)
> {
>     hdrtype_t header;
>     void *ptr = hstart;
>     printf("%s length=0x%x, the ptr=0x%x\n",__func__,len,(unsigned int
> )*((hdrtype_t*)ptr));
>     printf("%s the memory begin:0x%x end:0x%x,
> hstart:0x%x\n",__func__,(unsigned int)&_heap,(unsigned int)&_eheap,(unsigned
> int )*((hdrtype_t *) hstart));
>     /* Align the size. */
>     len = (len + 3) & ~3;
>
>     if (!len || len > 0xffffff)
>         return (void *)NULL;
>
>     /* Make sure the region is setup correctly. */
>     if (!HAS_MAGIC(*((hdrtype_t *) ptr)))
>         setup();
>     if (!HAS_MAGIC(*((hdrtype_t *) ptr))){
>         printf("set up failed,ptr=0x%x\n",(unsigned int)*((hdrtype_t *)
> ptr));
>     }
>     /* Find some free space. */
>     do {
>         header = *((hdrtype_t *) ptr);
>         int size = SIZE(header);
>
>         if (!HAS_MAGIC(header) || size == 0) {
>             printf("memory allocator panic!!! the size=0x%x.
> header=0x%x\n",size,(unsigned int)header);
>             halt();
>         }
>
>         if (header & FLAG_FREE) {
>             if (len <= size) {
>                 void *nptr = ptr + (HDRSIZE + len);
>                 int nsize = size - (HDRSIZE + len);
>
>                 /* If there is still room in this block,
>                  * then mark it as such otherwise account
>                  * the whole space for that block.
>                  */
>
>                 if (nsize > 0) {
>                     /* Mark the block as used. */
>                     *((hdrtype_t *) ptr) = USED_BLOCK(len);
>
>                     /* Create a new free block. */
>                     *((hdrtype_t *) nptr) =
>                         FREE_BLOCK(nsize);
>                 } else {
>                     /* Mark the block as used. */
>                     *((hdrtype_t *) ptr) = USED_BLOCK(size);
>                 }
>
>                 return (void *)(ptr + HDRSIZE);
>             }
>         }
>
>         ptr += HDRSIZE + size;
>
>     } while (ptr < hend);
>
>     /* Nothing available. */
>     return (void *)NULL;
> }
> log:
> Attempting to init PCI bdf 06:05.0 (dev/ven 30381106)
> Copying option rom (size 130560) from 0xfff00000 to ce000
> Checking rom 0x000ce000 (sig aa55 size 255)
> Running option rom at ce00:0003
> hello, initialize_usb
> 00:13.0 4387:1002.0 OHCI controller
> Not supported.
> 00:13.1 4388:1002.1 OHCI controller
> Not supported.
> 00:13.2 4389:1002.2 OHCI controller
> Not supported.
> 00:13.3 438a:1002.3 OHCI controller
> Not supported.
> 00:13.4 438b:1002.4 OHCI controller
> Not supported.
> 00:13.5 4386:1002.5 EHCI controller
> Not supported.
> 00:05.0 3038:1106.0 UHCI controller
> alloc length=0x238, the ptr=0x0
> alloc the memory begin:0x7860 end:0x57860, hstart:0x0
> setup the memory size:0x4fffc,begin:0x7860 end:0x57860, hstart:0xaa04fffc
> set up failed,ptr=0x0
> memory allocator panic!!! the size=0x0. header=0x0
>
> --
> Jason Wang
> Peking University
>



-- 
Jason Wang
Peking University
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090802/a27c2de9/attachment.html>


More information about the coreboot mailing list