diff options
author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-05-26 09:57:14 -0400 |
---|---|---|
committer | Artem Bityutskiy <dedekind1@gmail.com> | 2011-07-04 03:54:26 -0400 |
commit | 549c999a768a7a144c60a0faa58f34c48f39112b (patch) | |
tree | c1bdefb19fafb74164331c21ebaee7b2a987441c /fs/ubifs/commit.c | |
parent | b0af8dfdd67699e25083478c63eedef2e72ebd85 (diff) |
UBIFS: return EROFS in case of broken commit
If commit failed and it is in broken state, UBIFS switches to R/O mode. Most
operations return -EROFS in this case, except of commit which returns -EINVAL.
Make it return -EROFS too for consistency. This is also important for our power
cut emulation testing.
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'fs/ubifs/commit.c')
-rw-r--r-- | fs/ubifs/commit.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ubifs/commit.c b/fs/ubifs/commit.c index 87cd0ead8633..8ab03d12d5c0 100644 --- a/fs/ubifs/commit.c +++ b/fs/ubifs/commit.c | |||
@@ -418,7 +418,7 @@ int ubifs_run_commit(struct ubifs_info *c) | |||
418 | 418 | ||
419 | spin_lock(&c->cs_lock); | 419 | spin_lock(&c->cs_lock); |
420 | if (c->cmt_state == COMMIT_BROKEN) { | 420 | if (c->cmt_state == COMMIT_BROKEN) { |
421 | err = -EINVAL; | 421 | err = -EROFS; |
422 | goto out; | 422 | goto out; |
423 | } | 423 | } |
424 | 424 | ||
@@ -444,7 +444,7 @@ int ubifs_run_commit(struct ubifs_info *c) | |||
444 | * re-check it. | 444 | * re-check it. |
445 | */ | 445 | */ |
446 | if (c->cmt_state == COMMIT_BROKEN) { | 446 | if (c->cmt_state == COMMIT_BROKEN) { |
447 | err = -EINVAL; | 447 | err = -EROFS; |
448 | goto out_cmt_unlock; | 448 | goto out_cmt_unlock; |
449 | } | 449 | } |
450 | 450 | ||