All of lore.kernel.org
 help / color / mirror / Atom feed
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.

      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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.