flash_eraseall reports incorrect percentage

Fix flash_eraseall percentage reporting:

# flash_eraseall /dev/mtd4
Erasing 128 Kibyte @ 2800000 -- 100 % complete.

flash_eraseall
 * fix percentage reporting
 * exit()ing from main() is overkill, just return

Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubinize: correct subpage_size print and initialise vol_info to zero

J. Scott Merritt wrote:
> Dear list,
>
> I believe that I have stumbled upon two problems with the latest version
> of ubinize.c in mtd-utils - one serious, and one not so serious.
>
> - First, the easy one: At line 457, I believe that we should be printing
> args.subpage_size rather than ui.min_io_size.
>
> - More seriously, I believe that the "flags" field in the ubigen_vol_info
> structures is not being properly initialized.  Line 494 allocates memory
> for these structures with malloc, so they are not cleared to zero.  The
> read_section function updates the flag field if the AUTO-RESIZE flag is
> specified, but does not otherwise initialize or clear it.
>
> I reckon the latter problem could be repaired either with calloc, or by
> modifying read_section to more directly set/clear the flag.
>
> Thanks, Scott.

Reported-by: J. Scott Merritt <merrij3@rpi.edu>
Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.ubifs: make program version 1.1.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Fix favor LZO implementation

Use c->favor_percent, not c->favor_lzo.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Implement favor LZO method

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Introduce favor_lzo compressor options

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Make "c" global

We need to have access to "c" from various files.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Define the worst case compression constant

It will be used in the next patches.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

UBIFS: rename crc32 function

Zlib headers also declare crc32 symbol and it confilicts
with our crc32 fuction. So rename it.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

[PATCH] [MTD] mtd-utils: fec.c: bzero->memset, bcopy->memcpy, bcmp->memcmp

This patch alters fec.c so it does not use the deprecated bzero, bcopy
and bcmp functions.  This can help on platforms that do not have these
functions.

Signed-off-by: Mitch Davis <mitch.davis@symstream.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>

ubinize: allow an absent 'image' in the ubinize configuration.

Some recent changes to ubinize prevent the 'image' parameter from being
absent in the configuration file. The following patch fixes that.

Signed-off-by: Richard Titmuss <richard.titmuss@logitech.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

nanddump: Add Support for Quiet Option

Added support for the '-q,--quiet' option to suppress diagnostic output.
 Made the new option mutually-exclusive with the pretty print option.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nanddump: Clean-up Usage Output

Realign help usage output to make it more explict when a description
needs to be wrapped.

Use sentence case for the help usage output option short descriptions.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nanddump: Use Boolean Mnemonics from stdbool.h

Added include directive for stdbool.h and leveraged where appropriate
to improve code readability by making variable intent and usage more
explicit.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nanddump: Pass Real Names as Arguments to perror

Pass the MTD device node and dump file name as arguments to perror
rather than more ambigous, static messages on open failures.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nanddump: Utilize Standard Exit Mnemonics

Replace main exit and return status codes with equivalent mnemonics.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nanddump: Qualifier Clean-up

Static-qualified all globals except 'main' because they have no use
beyond file scope.
Constant-qualified MTD device and input positional parameter globals.
Constant-qualified argv array.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nandwrite: Add Support for Reading from Standard Input

Added suppport for reading in band data from standard input based on a
patch originally generated by Richard Titmuss <titmuss@slimdevices.com>
at <http://lists.slimdevices.com/pipermail/jive-checkins/2008-May/001918.html>.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nandwrite: Clean-up Usage Output

Realign help usage output to make it more explict when a description
needs to be wrapped.

Use sentence case for the help usage output option short descriptions.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nandwrite: Use Boolean Mnemonics from stdbool.h

Added include directive for stdbool.h and leveraged where appropriate
to improve code readability by making variable intent and usage more
explicit.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nandwrite: Pass Real Names as Arguments to perror

Pass the MTD device node and input file name as arguments to perror
rather than more ambigous, static messages on open failures.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nandwrite: Utilize Standard Exit Mnemonics

Replace main exit and return status codes with equivalent mnemonics.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

nandwrite: Qualifier Clean-up

Static-qualified all globals except 'main' because they have no use
beyond file scope.
Constant-qualified MTD device and input positional parameter globals.
Constant-qualified argv array.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

fs-tests: also preserve mount options when mounting again

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.ubifs: amend help print

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.ubifs: update ubifs-media.h

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubiupdatevol: remove non-existing option from help message

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.ubifs: check max_leb_count

Complain and exit if max. count of LEBs was not specified.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubirmvol: learn to remove volume by name

Add -N option to ubirmvol to support deleting volumes
by name.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: minor rename

We use term "volume ID", not "volume number". So change few
vol_num variables to vol_id to be consistent.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubimkvol: check for free LEBs

Check for free LEBs before trying to create a volume. This fixes
the following issue:

$ ubimkvol /dev/ubi0 -n 0 -N test_fs -m
ubimkvol: error!: cannot UBI create volume
          error 22 (Invalid argument)
Error: cannot create UBI volume

"Invalid argument" ? Not understandable. But the thing is that
-m makes ubimkvol use available space, which is zero, then it
passes zero to the ioctl, which fails in request validation
code, because 0 invalid volume size.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubiattach: minor help text fix

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: added missing linefeed in 'ubinfo'

Trivial patch to add a missing linefeed in static volume info:

| Data bytes:  113112 bytes (110.5 KiB)State:       OK

Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: allow 'ubiformat' to read from stdin

This patch allows to read the image from stdin and adds an
'--image-size' option to specify its size.  'upiupdatevol' has this
feature already and it is useful for 'ubiformat' too.

It adds a read_all() function which reads a block of data. Plain
'read(2)' can fail on

  nc -l -p 1234 | ubiformat -f - -S 25427968

else.

Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fixed and enhanced '--flash-image' option

The '--flash-image' option of 'ubiformat' requires an argument and
'.has_arg' must not be 0 hence.  The patch adds this option to the
commandline synopsis too.

Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: use 'stat(2)' instead of 'lstat(2)'

The UBI tools refuse to work with symlinks like
'/dev/ubi/bootloader -> ../ubi0_2' because they use 'lstat(2)' and get
information about the symlink but not about the device.  This is
unwanted and fixed by this patch.

This patch renames 'struct stat stat' variables to 'st' to avoid
compilation errors.

Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mtd-utils: add support for 4k pages.

Add support for 4K pages in nanddump & nandwrite.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Sebastian Siewior <bigeasy@linutronix.de>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.ubifs: remove debugging leftovers from Makefile

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

use $(MAKE) rather than make

If you execute straight `make`, this prevents the active make env from
being passed on to sub children.  This prevents parallel building as well
as build flag passing.

I also dropped the recursive make in ubi-utils/Makefile for the all target
as this causes problems when building in parallel.  All of the targets
that we actually care about are handled by the $(NTARGETS) target anyways.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.ubifs: set CFLAGS/LDLIBS properly

CFLAGS should only be set as a default instead of overriding the user's
choices.  LDLIBS is for adding libraries, not LDFLAGS.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mtd-utils: enable parallel build process

-j 2 build was not possible without this patch.

Signed-off-by: Sebastian Siewior <bigeasy@linutronix.de>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>

mkfs.ubifs: set program version as 1.0

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Add mkfs.ubifs

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

fs-tests: allow for symlink name too long in integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

fs-tests: fix max file name length in integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

fs-tests: fix symlink bug in integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubinize: fix static volumes generation

When static volumes is generated, the 'used_ebs' field of VID
header has to be calculated based on image file size, not on
the volume size. Fix this.

This patch also improves error reporting a little and re-arrages
the code so that all the validation is done in 'read_section()'

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-tests: fix run script

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-tests: fix makefile

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

uninize: add more ini-file validation

Check that volume names and IDs are unique

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubinize: validate number of sections

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Makefile: Add Optional ZLIB and LZO CPPFLAGS and LDFLAGS

This adds support for allowing the specification of unique CPPFLAGS and
LDFLAGS for both LZO and ZLIB. In addition, it allows for independently
specifying both PREFIX and EXEC_PREFIX.

The former is useful in some cross-compilation environments where the
LZO and ZLIB versions being built against are not those installed in the
standard locations.

The latter is useful when installing mtd-utils in a multi-architecture
installation area.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

ubiupdatevol: minor tweak

Undo my weird change which slipped accidentally to the commit.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubiupdate: support reading from stdio

1. Adds -s/--size switch;
2. Allows the image file to be specified as '-', meaning stdin; if used,
the size must be specified.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubiformat: reverse question about vid offset change

Users can pass the -O option to ubiformat to override the VID header
offset on the command-line. If for some reason, the new offset does
not match what is on flash, ubiformat currently prompts the user if
they'd rather use the *old* offsets.  This means that calling
ubiformat with --yes will not do what was requested on the
command-line.

This patch reverses the question so --yes is actually useful for
running in batch-mode.

Signed-off-by: Bernard Blackham <bernard@largestprime.net>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.jffs2: Add Support for Symlinks to Device Table

Added support for symbolic links to the grammar of the device table file
such that links that already exist in the root working directory may
have their permissions and ownership changed. Creating symbolic links
anew is still unsupported.

Added a note to this effect in the manual reference page.

Print out the type character when encountering an unsupported file type
(e.g. symbolic links when created anew).

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Signed-off-by: Josh Boyer <jwboyer@vader.jdub.homelinux.org>

mkfs.jffs2: Remove Incorrect Find Optimization

Remove performance optimization in find_filesystem_entry that prevented
the successful simultaneous use of --root and --devtable where the
latter is only used to fix-up permissions and ownership and to create
device nodes.

As it stood, the performance optimization prevented a successful find
where directory recursion was required or where the file being searched
for had any mode permission bits set.

See http://lists.infradead.org/pipermail/linux-mtd/2008-June/021997.html
for additional information.

Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Signed-off-by: Josh Boyer <jwboyer@vader.jdub.homelinux.org>

respect CFLAGS/CPPFLAGS from build environment

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Josh Boyer <jwboyer@gmail.com>

fs-tests: fix rename bug in integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: add symlinks to integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: check link count in integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: add rename to integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: add linking/unlinking to integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: make integrity test shrink faster

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: add mmapping to integrity test for ubifs

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: preserve mount options when mounting again

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: stop integrity test looping forever

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: fix bug in integrity test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: add simple performance test

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: make integrity test record truncations

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: make integrity test shrink better

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

fs-tests: allow for ENOSPC in test fwrite00

Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>

ubi-tests: fix compilation

libubi_open interface was changed, but the tests were not
amended. Fix this.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix typo in ubiformat

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Fix recv_image build harder

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

ubi-utils: remove unnecessary ubinize warning

... and fix help message.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: add ubiformat to Makefile

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fixe ubiupdatevol

-t option does not require image file.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix long options

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: make sure only one volume is auto-resize

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: improve input parameters checks

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mtd-utils: fix input parameter checks

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix percent output in ubiformat

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix the output print

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: improve input parameters checks

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: add ubiformat

A new utility to erase MTD devices and flash UBI images, while
preserving erase counters

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: add libscan

A library to scan MTD devices. For now it only reads EC header.
Later it may be improved if needed.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: add libmtd

Just collection of functions to make it easier to work with MTD devices.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: various clean-ups and preparations

This patch introduces many clean-ups, nicifications and preparations
to the following ubiformat utility. There are also fixes.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: use common.h in libubi

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

mkfs.jffs2.c: fix issue with crashing when using lzo compression

Signed-off-by: Siarhei Siamashka <siarhei.siamashka@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

compr_lzo.c: allocate enough memory for lzo compressor.

This is the same bug as in kernel, pointed out the LZO author
(Markus Oberhumer):

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=f2a11b158a24301e9158e9c873fa88e5eb775486

Signed-off-by: Riku Voipio <riku.voipio@iki.fi>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Use LSB locations for manpages

LSB says manpages belong to /usr/share/man, adjust makefile
defaults accordingly.

Signed-off-by: Riku Voipio <riku.voipio@iki.fi>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Teach libubi_open not to prirnt error message

... when it is expected that UBI is not necessarily present

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubinize: correctly calculate VID offset when min I/O unit is 1

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix a typo
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix printing typos

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: fix wrong variable printout

Signed-off-by: Zoltan Sogor <weth@inf.u-szeged.hu>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

ubi-utils: don't use argp.h

Don't include argp.h as it's glibc-specific, and not used anyway.

Signed-Off-By: Hamish Moffatt <hamish@cloud.net.au>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

Fix recv_image compilation with new glibc

struct ip_mreq is hidden behind __USE_MISC now