DRA718: CMA size too large

Hello Buddy

Our IPUx and DSPx memory map configuration are according to your comments. We understand that these are fine and we don't reduce it.

Error message rise at U-Boot (MLO) time.

These are the original U-Boot error message

U-Boot SPL 2016.05-00009-gb142ff9 (Apr 09 2018 - 14:14:49)
DRA722-GP ES2.1
Trying to boot from MMC2_2
failed to allocate carveout rsc: da 94000000, pa 0, len 4400000, flags 0
handle_resources failed: -12
load_elf_image_phdr returned error for core IPU2
Error loading remotecore IPU2!,Continuing with boot ...
## No elf image at address 0xa27ff000
Error loading remotecore DSP2!,Continuing with boot ...
## No elf image at address 0xa0fff000
Error loading remotecore IPU1!,Continuing with boot ...

We added some custom trace

alloc_mem: 1-len 983040(0x000f0000) align 8(0x00000008)
alloc_mem: 2-order 8(0x00000008), ilog2 19(0x00000013)
alloc_mem: 3-count 256(0x100) mask 0xff pageno 0x0, mem_count 0x9000

bitmap_find_next_zero_area: 0-mem_count 36864(0x9000), start 0(0x0), count 256(0x100)
bitmap_find_next_zero_area: 1-index 8(0x8) align_mask 255(0xff)
bitmap_find_next_zero_area: 2-index 256(0x100) align_mask 255(0xff)
bitmap_find_next_zero_area: 3-end 512(0x200), count 256(0x100)
bitmap_find_next_zero_area: 0-mem_count 36864(0x9000), start 257(0x101), count 256(0x100)
bitmap_find_next_zero_area: 1-index 512(0x200) align_mask 255(0xff)
bitmap_find_next_zero_area: 2-index 512(0x200) align_mask 255(0xff)
bitmap_find_next_zero_area: 3-end 768(0x300), count 256(0x100)

alloc_mem: 4-count 256(0x100) mask 0xff pageno 0x200, mem_count 0x9000

alloc_mem: 1-len 71303168(0x04400000) align 8(0x00000008)
alloc_mem: 2-order 15(0x0000000f), ilog2 26(0x0000001a)
alloc_mem: 3-count 32768(0x8000) mask 0x7fff pageno 0x0, mem_count 0x9000

bitmap_find_next_zero_area: 0-mem_count 36864(0x9000), start 0(0x0), count 32768(0x8000)
bitmap_find_next_zero_area: 1-index 8(0x8) align_mask 32767(0x7fff)
bitmap_find_next_zero_area: 2-index 32768(0x8000) align_mask 32767(0x7fff)
bitmap_find_next_zero_area: 3-end 65536(0x10000), count 32768(0x8000)

alloc_mem: 4-count 32768(0x8000) mask 0x7fff pageno 0x10000, mem_count 0x9000
failed to allocate carveout rsc: da 94000000, pa 0, len 4400000, flags 0
handle_resources failed: - 12

To avoid this error we have to increase,

1-We dont understand why CMA size must be increased to 512MB

2-Is there any way to debug SDK_VISION+RVC, by U-boot terminal messages?. It would be usefully print out some logs.

Hi Christian, 

Noticing three things, 

1. We noticed your logs are showing "Trying to boot from MMC2_2", which isn't what we expected. By default RVC release is configured to run from QSPI boot, which includes some startup code being added to the QSPI boot in u-boot.
2. Your previous comment mentioned the "#ifdef CONFIG_LATE_ATTACH_DMA_POOL" 
3. Is this being run on a TI-EVM or a custom board? If custom, do you have an TI-EVM you could run this on? 

Can you also provide a patch for your custom debug prints so we can attempt to replicate? 

Thanks,

David

Hello David

It is running on custom board without QSPI (MMC only). We notice that UBoot should be modified to launch RVC.

First at all we test dra71x-ipu2-fw.xem4 privided by TI on EVM (it runs booting from QSPI). Latter we run the same file on custom board and got the error message. We try with custom dra71x-ipu2-fw.xem4 file with the same error.

Please understand that the following source is a dirty and quick "looking for error" code.  We modify lateattach.c for custom board, so add it manually

void bitmap_set(unsigned long *map, int start, int nr)
{
	unsigned long *p = map + BIT_WORD(start);
	const int size = start + nr;
	int bits_to_set = BITS_PER_LONG - (start % BITS_PER_LONG);
	unsigned long mask_to_set = BITMAP_FIRST_WORD_MASK(start);

	printf("CHRISTIAN bitmap_set 0: start %d(%#x) count %d(%#x), bits_to_set %d(%#x), mask_to_set %ld(%#lx)\n", start, start, nr, nr, bits_to_set, bits_to_set, mask_to_set, mask_to_set);
	while (nr - bits_to_set >= 0) {
		*p |= mask_to_set;
		nr -= bits_to_set;
		bits_to_set = BITS_PER_LONG;
		mask_to_set = ~0UL;
		p++;
	}
	if (nr) {
		mask_to_set &= BITMAP_LAST_WORD_MASK(size);
		*p |= mask_to_set;
	}
}

unsigned long bitmap_find_next_zero_area(unsigned long *map,
					 unsigned long size,
					 unsigned long start,
					 unsigned int nr,
					 unsigned long align_mask)
{
	unsigned long index, end, i;
again:
   printf("CHRISTIAN bitmap_find_next_zero_area: 0-mem_count %ld(%#lx), start %ld(%#lx), count %d(%#x)\n", size, size, start, start, nr, nr);
	index = evm_find_next_zero_bit(map, size, start);
	printf("CHRISTIAN bitmap_find_next_zero_area: 1-index %ld(%#lx) align_mask %ld(%#lx)\n", index, index, align_mask, align_mask);
	/* Align allocation */
	index = (index + align_mask) & ~align_mask;
	printf("CHRISTIAN bitmap_find_next_zero_area: 2-index %ld(%#lx) align_mask %ld(%#lx)\n", index, index, align_mask, align_mask);
	end = index + nr;
   printf("CHRISTIAN bitmap_find_next_zero_area: 3-end %ld(%#lx), count %d(%#x)\n", end, end, nr, nr);
	if (end > size)
		return end;
	i = find_next_bit(map, end, index);
	if (i < end) {
		start = i + 1;
		goto again;
	}
	return index;
}

void *alloc_mem(unsigned long len, unsigned long align)
{
   unsigned long mask;
   unsigned long pageno=0;
   int count, i, b;
   unsigned int order;
   printf("\nCHRISTIAN alloc_mem: 1-len %ld(0x%08lx) align %ld(0x%08lx)\n", len, len, align, align);
   if (len < PAGE_SIZE)
      order = 0;
   else{
      order = __ilog2((len) - 1) - PAGE_SHIFT + 1;
      printf("CHRISTIAN alloc_mem: 2-order %d(0x%08x), ilog2 %d(0x%08x)\n", order, order, __ilog2((len) - 1), __ilog2((len) - 1));
   };
   count = (1 << order);
   mask = (1 << order) - 1;
   printf("CHRISTIAN alloc_mem: 3-count %d(%#x) mask %#lx pageno %#lx, mem_count %#lx\n", count, count, mask, pageno, mem_count);
   /*printf("CHRISTIAN mem_bitmap 0:\n");
   b=1;
   i=0;
   printf("\n%04d:", i);
   for(i=0; i<BITS_TO_LONGS(DRA7_RPROC_MAX_CMA_SIZE >> PAGE_SHIFT); i++){
      printf("%08lx,", mem_bitmap[i]);
      if(b>=18){
         printf("\n%04d:", i);
         b=0;
      };
      b++;
   };
   printf("\n");*/
   pageno = bitmap_find_next_zero_area(mem_bitmap, mem_count, 0, count, mask);
   /*printf("CHRISTIAN mem_bitmap 1:\n");
   b=1;
   i=0;
   printf("\n%04d:", i);
   for(i=0; i<BITS_TO_LONGS(DRA7_RPROC_MAX_CMA_SIZE >> PAGE_SHIFT); i++){
      printf("%08lx,", mem_bitmap[i]);
      if(b>=18){
         printf("\n%04d:", i);
         b=0;
      }
      b++;
   };
   printf("\n");*/
   debug("CHRISTIAN alloc_mem: count %d mask %#lx pageno %#lx\n", count, mask, pageno);
   printf("CHRISTIAN alloc_mem: 4-count %d(%#x) mask %#lx pageno %#lx, mem_count %#lx\n", count, count, mask, pageno, mem_count);
   if (pageno >= mem_count)
      return NULL;

   bitmap_set(mem_bitmap, pageno, count);
   /*printf("CHRISTIAN mem_bitmap 2:\n");
   b=1;
   i=0;
   printf("\n%04d:", i);
   for(i=0; i<BITS_TO_LONGS(DRA7_RPROC_MAX_CMA_SIZE >> PAGE_SHIFT); i++){
      printf("%08lx,", mem_bitmap[i]);
      if(b>=18){
         printf("\n%04d:", i);
         b=0;
      }
      b++;
   };
   printf("\n");*/
   return (void *)(mem_base + (pageno << PAGE_SHIFT));
}

Hi Christian, 

Can you provide the CMA base addresses as defined in lateattach.c (ex. DRA7_RPROC_CMA_BASE_IPU1, DRA7_RPROC_CMA_BASE_IPU2...)? The CMA sizes should be set to fit without conflicting with other partitions. 

Thanks,

David

Hi Christian,

I don't currently have a Robust RVC 4.0 source setup, so I can't cross verify. The tables your talking about look like they're in vision_sdk/vision_sdk/apps/build/tda2xx/*.xs. I believe it should either be here or in u-boot where these are being specified. The 0x94000000 at least I thought came from u-boot/board/ti/dra7xx/lateattach.c.

Did disabling CONFIG_LATE_ATTACH_DMA_POOL help?

Thanks,

David

Hello David

Disabling CONFIG_LATE_ATTACH_DMA_POOL solve it.

From U-Boot side is clear that address and range is in lateattach.c.

Could you confirm, where memory address configuration is located at vision_sdk?

Thanks in advance