On 13/12/2024 12:33, Carlo Nonato wrote: > > > Using paste.debian: > > https://paste.debian.net/1339647/ 1. Issue I mentioned with prefixing with double underscore 2. Generic helper should not be named ext_regions 3. s/skip_size/min_bank_size/ And still you need to convince others about 128MB limit because I'm not sure. Imagine, that our kernel+dtb+ramdisk is > 128MB and your first bank is 128MB. With your solution this would fail. Now, imagine that you sort your banks and start with the biggest one. You don't care about its size. It's the biggest one so if it does not fit, then that's not your problem. ~Michal
- Re: [PATCH v11 03/12] xen/arm: permit non dire... Michal Orzel
- Re: [PATCH v11 03/12] xen/arm: permit non ... Carlo Nonato
- Re: [PATCH v11 03/12] xen/arm: permit non ... Julien Grall
- Re: [PATCH v11 03/12] xen/arm: permit... Carlo Nonato
- Re: [PATCH v11 03/12] xen/arm: pe... Andrea Bastoni
- Re: [PATCH v11 03/12] xen/arm: pe... Michal Orzel
- Re: [PATCH v11 03/12] xen/arm: pe... Carlo Nonato
- Re: [PATCH v11 03/12] xen/arm: pe... Michal Orzel
- Re: [PATCH v11 03/12] xen/arm: pe... Carlo Nonato
- Re: [PATCH v11 03/12] xen/arm: pe... Carlo Nonato
- Re: [PATCH v11 03/12] xen/arm: pe... Michal Orzel
- Re: [PATCH v11 03/12] xen/arm: pe... Carlo Nonato
- [PATCH v11 11/12] xen/arm: make consider_modules() availabl... Carlo Nonato
- [PATCH v11 09/12] xen: add cache coloring allocator for dom... Carlo Nonato
- [PATCH v11 01/12] xen/common: add cache coloring common cod... Carlo Nonato
- [PATCH v11 06/12] tools: add support for cache coloring con... Carlo Nonato
- Re: [PATCH v11 06/12] tools: add support for cache col... Anthony PERARD