[LEDE-DEV] [PATCH 2/4] v2: block.c: Add support for checking vfat filesystems
John Crispin
john at phrozen.org
Wed May 18 23:07:45 PDT 2016
On 18/05/2016 13:28, lede at daniel.thecshore.com wrote:
> From: Daniel Dickinson <lede at daniel.thecshore.com>
>
> v2: Fix mixup of dosfsck checking ext* and e2fsck checking vfat.
>
> vfat is a common filesystem which users may want to mount on an
> OpenWrt/LEDE device, so support peforming filesystem checks
> before mount for vfat.
has this series been test on device and did you verify that the fs check
now works properly for both those filesystems ?
John
>
> Signed-off-by: Daniel Dickinson <lede at daniel.thecshore.com>
> ---
> block.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/block.c b/block.c
> index 71ffd0b..5a584cb 100644
> --- a/block.c
> +++ b/block.c
> @@ -628,24 +628,30 @@ static void check_filesystem(struct blkid_struct_probe *pr)
> pid_t pid;
> struct stat statbuf;
> const char *e2fsck = "/usr/sbin/e2fsck";
> + const char *dosfsck = "/sbin/dosfsck";
> + const char *ckfs;
>
> /* UBIFS does not need stuff like fsck */
> if (!strncmp(pr->id->name, "ubifs", 5))
> return;
>
> - if (strncmp(pr->id->name, "ext", 3)) {
> + if (!strncmp(pr->id->name, "vfat", 4)) {
> + ckfs = dosfsck;
> + } else if (!strncmp(pr->id->name, "ext", 3)) {
> + ckfs = e2fsck;
> + } else {
> ULOG_ERR("check_filesystem: %s is not supported\n", pr->id->name);
> return;
> }
>
> - if (stat(e2fsck, &statbuf) < 0) {
> + if (stat(ckfs, &statbuf) < 0) {
> ULOG_ERR("check_filesystem: %s not found\n", e2fsck);
> return;
> }
>
> pid = fork();
> if (!pid) {
> - execl(e2fsck, e2fsck, "-p", pr->dev, NULL);
> + execl(ckfs, ckfs, "-p", pr->dev, NULL);
> exit(-1);
> } else if (pid > 0) {
> int status;
>
More information about the Lede-dev
mailing list