[PATCH] UBIFS: Fix empty_log_bytes() computing error
hujianyang
hujianyang at huawei.com
Wed Jul 16 04:44:15 PDT 2014
On 2014/7/16 19:03, Artem Bityutskiy wrote:
> On Fri, 2014-07-11 at 10:13 +0800, hujianyang wrote:
>> Hi Artem,
>>
>> After several days testing, the error we fixed in commit 642f63ed10,
>> "UBIFS: fix fatal race condition" comes out again. This commit seems
>> not working for my problem and I have to research into it again.
>
> Do you say that your original patch fixed the problem, and the re-worked
> "fix fatal race condition" patch does not fix it, so it is incorrect?
>
> Or do you say that probably the original patch also did not fix the
> problem, so both do not really fix the issue?
>
> Or do you say that "fix fatal race condition" fixes only part of the
> problem, and there is another problem, which needs fixing too?
>
> Or you are not sure?
>
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.
>
> Do you want me to create 2 patches based on this information and send
> them to you, our you submit them? They should have 'Cc:
> stable at vger.kernel.org'.
>
> Thanks!
>
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.
Thanks~!
Hu
More information about the linux-mtd
mailing list