From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/9] Replace the FAT filesystem code
Date: Mon, 10 Aug 2015 22:09:12 -0600 [thread overview]
Message-ID: <55C97568.1040900@wwwdotorg.org> (raw)
In-Reply-To: <1439003370-3671-1-git-send-email-swarren@wwwdotorg.org>
On 08/07/2015 09:09 PM, Stephen Warren wrote:
> The existing FAT filesystem implementation in U-Boot has some bugs that
> are tricky to fix cleanly without significant rework of the code. For
> example, see:
>
> http://lists.denx.de/pipermail/u-boot/2015-July/221054.html
> [PATCH] fat: handle paths that include ../
>
> This series replaces U-Boot's FAT filesystem implementation with the "ff"
> project; http://elm-chan.org/fsw/ff/00index_e.html. This project appears
> to be actively maintained, does not have at least the bug referenced in
> the patch link above, and is quite easy to integrate into U-Boot. An
> earlier version is also used in Barebox, so presumably it's had some
> level of testing/exposure there.
I've found a few issues with this series:
1) A warning when only building R/O FAT support, but not R/W FAT
support. I have fixed this locally.
2) Some functions only used by the SPL FAT support are missing, causing
build errors on some boards. I have fixed this locally. There are now no
new buildman failures on any architecture supported by buildman's
toolchains.
3) While the new FAT library (and the U-Boot integration code I wrote)
works on some test cases, others such as loading /zImage on an rpi, or
running ./test/fs/fs-test.sh fail. I'm in the process of investigating
these issues.
Still, I'd appreciate any thoughts on the general idea behind this series.
prev parent reply other threads:[~2015-08-11 4:09 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-08 3:09 [U-Boot] [PATCH 0/9] Replace the FAT filesystem code Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 1/9] disk: support host devices in dev_print() Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 2/9] fat: move to a different directory Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 3/9] fat: add ff.c implementation Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 4/9] fat: apply upstream bugfix to ff.c Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 5/9] fat: ffconf.h changes for U-Boot port Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 6/9] fat: port integer.h to U-Boot types Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 7/9] fat: add U-Boot to ff.c API conversion wrapper Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 8/9] fat: switch to new FAT implementation Stephen Warren
2015-08-08 3:09 ` [U-Boot] [PATCH 9/9] fat: remove old implementation Stephen Warren
2015-08-11 4:09 ` Stephen Warren [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55C97568.1040900@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox