Skip to content

Add support for AXFS filesystem type

Thomas Petazzoni buildroot 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest posting 2/20 of 2 patches 0 repliesNew
  2. v2 posting 1/20 of 2 patches 0 repliesSuperseded

Patches in v2 posting 2/2 No applies recorded yet

None of the 2 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [Buildroot] [PATCH v2 0/2] Add support for AXFS filesystem type

Hello,

Quoted 4 lines · Ariel D'Alessandro
Quoted 2 lines

You received some feedback on both patches. Could you address this feedback, and submit a new version of the patches?

Yes, right away.

Excellent, thanks!

Quoted 4 lines

Moreover, "mkfs.axfs tool can receive an additional argument using the -i flag which points to a user-created file in the XML format; it indicates which pages the user desires to have marked for XIP within the file system image." [1]

So probably a Config.in option to specify the path to this file should be added.

Quoted 10 lines
Quoted 5 lines

I remember there was even a tool to
run on the target to "measure" the filesystem accesses, and give some hints on which pages of data should be stored uncompressed vs. which pages should be stored compressed. Is this actually implemented in AXFS ?

Yes. The CONFIG_AXFS_PROFILING option is added in order to enable the profiling tool. Then simply running on the target platform, the AXFS profiler will collect data and write the information it could gather to /proc/axfs/volume0. See [1] for more information.

Good. This sort of into can be added in the help text of the Config.in option.

Quoted 5 lines
Quoted 1 line

If so, do you plan to add support for such functionality ?

Yes, it would be nice to add support for this functionality in order to explode the entire AXFS capability. I'll do this as a set of follow-up patches if that's OK, and let the -a flag (XIP all) for now.

Sounds like a very good plan.

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com