[PATCH 1/3] UBIFS: assert no fixup when writing a node
Artem Bityutskiy
dedekind1 at gmail.com
Wed Jun 1 06:03:29 EDT 2011
On Mon, 2011-05-30 at 14:56 -0400, Ben Gardiner wrote:
> The current free space fixup can result in some writing to the UBI volume
> when the space_fixup flag is set.
>
> To catch instances where UBIFS is writing to the NAND while the space_fixup
> flag is set, add an assert to ubifs_write_node().
>
> Signed-off-by: Ben Gardiner <bengardiner at nanometrics.ca>
>
> ---
> fs/ubifs/io.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
> index 166951e..db298de 100644
> --- a/fs/ubifs/io.c
> +++ b/fs/ubifs/io.c
> @@ -763,6 +763,8 @@ int ubifs_write_node(struct ubifs_info *c, void *buf, int len, int lnum,
> if (c->ro_error)
> return -EROFS;
>
> + ubifs_assert(!c->space_fixup);
> +
I've moved this assert upper to where all other assertions are placed.
I've also added this assertion to 'ubifs_wbuf_write_nolock()' because it
is the other often used write path.
And pushed to the UBIFS tree, thanks.
--
Best Regards,
Artem Bityutskiy (Артём Битюцкий)
More information about the linux-mtd
mailing list