Re: [coreboot] #146: memalign faild with 4k boundary

[coreboot]

#146: memalign faild with 4k boundary
-------------------------------------+--------------------------------------
   Reporter:  wangqingpei@…          |          Owner:  somebody
       Type:  defect                 |         Status:  new     
   Priority:  blocker                |      Milestone:          
  Component:  libpayload             |        Version:  v2      
   Keywords:  memalign crashed       |   Dependencies:          
Patchstatus:  there is no patch      |  
-------------------------------------+--------------------------------------

Comment(by wangqingpei@…):

in libpayload/drivers/usb/uhci.c line 127
 the function uhci_init is used for initializing the UHCI controller.

{{{
 hci_t *
 uhci_init (pcidev_t addr)
 {
         int i;
         hci_t *controller = new_controller ();
         printf("the  malloc 1\n");
         controller->instance = malloc (sizeof (uhci_t));
         controller->start = uhci_start;
         controller->stop = uhci_stop;
         controller->reset = uhci_reset;
         controller->shutdown = uhci_shutdown;
         controller->packet = uhci_packet;
         controller->bulk = uhci_bulk;
         controller->control = uhci_control;
         controller->create_intr_queue = uhci_create_intr_queue;
         controller->destroy_intr_queue = uhci_destroy_intr_queue;
         controller->poll_intr_queue = uhci_poll_intr_queue;
         for (i = 0; i < 128; i++) {
                 controller->devices[i] = 0;
         }
         init_device_entry (controller, 0);
         UHCI_INST (controller)->roothub = controller->devices[0];

controller->bus_address = addr;
         controller->reg_base = pci_read_config32 (controller->bus_address,
 0x20) & ~1;  /* ~1 clears the register type indicator that is set to 1 for
 IO space */

/* kill legacy support handler */
         uhci_stop (controller);
         mdelay (1);
         uhci_reg_write16 (controller, USBSTS, 0x3f);
         pci_write_config32 (controller->bus_address, 0xc0, 0x8f00);
         printf("the memalign 2\n");
         UHCI_INST (controller)->framelistptr = memalign (0x1000, 1024 *
 sizeof (flistp_t *));   /* 4kb aligned to 4kb */
 /* in this line which use mamalign to malloc 4Kb with 4k boundary, i
 tracked with memalign, find it failed at
 align_regions = allocate_region(align_regions, align,
 (size/align<99)?100:((size/align)+1));
 */

memset (UHCI_INST (controller)->framelistptr, 0,
                 1024 * sizeof (flistp_t));
         printf("the step 3 passed\n");

}}}
 if i narrow the boundary from 0x1000 to 0x10, then the crash disappeared.
 what's why i though the function memalign has an bug, in my view, even the
 memory allocate failed, it should not crashed. but instead , just return
 with error