lkml.org 
[lkml]   [2017]   [Oct]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRE: [PATCH v5 0/5] cramfs refresh for embedded usage
On Fri, 6 Oct 2017, Chris Brandt wrote:

> On Friday, October 06, 2017, Christoph Hellwig wrote:
> > This is still missing a proper API for accessing the file system,
> > as said before specifying a physical address in the mount command
> > line is a an absolute non-no.
> >
> > Either work with the mtd folks to get the mtd core down to an absolute
> > minimum suitable for you, or figure out a way to specify fs nodes
> > through DT or similar.
>
> On my system, the QSPI Flash is memory mapped and set up by the boot
> loader. In order to test the upstream kernel, I use a squashfs image and
> mtd-rom.
>
> So, 0x18000000 is the physical address of flash as it is seen by the
> CPU.
>
> Is there any benefit to doing something similar to this?
>
> /* File System */
> /* Requires CONFIG_MTD_ROM=y */
> qspi@18000000 {
> compatible = "mtd-rom";
> probe-type = "map_rom";
> reg = <0x18000000 0x4000000>; /* 64 MB*/
> bank-width = <4>;
> device-width = <1>;
>
> #address-cells = <1>;
> #size-cells = <1>;
>
> partition@800000 {
> label ="user";
> reg = <0x0800000 0x800000>; /* 8MB @ 0x18800000 */
> read-only;
> };
> };
>
>
> Of course this basically ioremaps the entire space on probe, but I think
> what you really want to do is just ioremap pages at a time (maybe..I
> might not be following your code correctly)

No need for ioremaping pages individually. This creates unneeded
overhead, both in terms of code execution and TLB trashing. With a
single map, the ARM code at least is smart enough to fit large MMU
descriptors when possible with a single TLB for a large region. And if
you're interested in XIP cramfs then you do have huge vmalloc space to
spare anyway.

As to the requirement for a different interface than a raw physical
address: I'm investigating factoring out the MTD partition parsing code
so it could be used with or without the rest of MTD. Incidentally, the
person who wrote the very first incarnation of MTD partitioning 17 years
ago was actually me, so with luck I might be able to figure out
something sensible.


Nicolas

\
 
 \ /
  Last update: 2017-10-06 18:31    [W:0.042 / U:0.032 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site