[PATCH] UBIFS: Fix empty_log_bytes() computing error
Artem Bityutskiy
dedekind1 at gmail.com
Wed Jul 16 05:38:47 PDT 2014
On Wed, 2014-07-16 at 19:44 +0800, hujianyang wrote:
> First, I think the re-worked patch and my origin patch fix same
> problem. Once I think the error 'not CS node' is caused by power
> cut, so I research the code and find this race.
>
> But, after several days test, this error came out again and I
> think we don't catch the true cause. Then I find the error below.
>
> I think previous fixes(commit 642f63ed10) are useful but not solve
> my problem. Maybe you can change the commit log or just keep them
> because they can really fix a problem like that.
OK, I'll keep the patches, but will amend the commit message a bit and
tell there that they are fixing a theoretical issue. And I'll remove Cc:
stable so far.
> Yes, I'm not sure how to fix it is better and I was busy these
> days. You can fix this error in your way and cc the patch to me.
>
> But I don't know if this error is the true cause of my problem. I
> would like to report to you if I get something new.
I've created 2 patches based on your e-mail. I did not test them at all.
The assertion - I move it to 'ubifs_next_log_lnum()'. May be this is
incorrect, but looks like a good place for me on the first glance.
I guess you will continue testing, right? Please, use these patches and
let me know if they are wrong, if they help or not, etc. I'll send the 2
patches shortly.
Thanks!
--
Best Regards,
Artem Bityutskiy
More information about the linux-mtd
mailing list