On Thu, Jul 16, 2020 at 11:32 PM Arnd Bergmann <arnd@xxxxxxxx> wrote: > > On Fri, Jul 17, 2020 at 1:41 AM Atish Patra <atish.patra@xxxxxxx> wrote: > > > > From: Anup Patel <anup.patel@xxxxxxx> > > > > Currently, RISC-V reserves 1MB of fixmap memory for device tree. However, > > it maps only single PMD (2MB) space for fixmap which leaves only < 1MB space > > left for other kernel features such as early ioremap which requires fixmap > > as well. The fixmap size can be increased by another 2MB but it brings > > additional complexity and changes the virtual memory layout as well. > > If we require some additional feature requiring fixmap again, it has to be > > moved again. > > > > Technically, DT doesn't need a fixmap as the memory occupied by the DT is > > only used during boot. Thus, init memory section can be used for the same > > purpose as well. This simplifies fixmap implementation. > > > > Signed-off-by: Anup Patel <anup.patel@xxxxxxx> > > Signed-off-by: Atish Patra <atish.patra@xxxxxxx> > > The patch seems fine for the moment, but I have one concern for the > long run: > > > +#define DTB_EARLY_SIZE SZ_1M > > +static char early_dtb[DTB_EARLY_SIZE] __initdata; > > Hardcoding the size in .bss seems slightly problematic both for > small and for big systems. On machines with very little memory, > this can lead to running out of free pages before .initdata gets freed, > and it increases the size of the uncompressed vmlinux file by quite > a bit. > > On some systems, the 1MB limit may get too small. While most dtbs > would fall into the range between 10KB and 100KB, it can also be > much larger than that, e.g. when there are DT properties that include > blobs like device firmware that gets loaded into hardware by a kernel > driver. > I was not aware that we can do such things. Is there a current example of that ? > Is there anything stopping you from parsing the FDT in its original > location without the extra copy before it gets unflattened? > That's what the original code was doing. A fixmap entry was added to map the original fdt location to a virtual so that parse_dtb can be operated on a virtual address. But we can't map both FDT & early ioremap within a single PMD region( 2MB ). That's why we removed the DT mapping from the fixmap to .bss section. The other alternate option is to increase the fixmap space to 4MB which seems more fragile. > Arnd -- Regards, Atish
