From: "Theodore Ts'o" <tytso@mit.edu>
To: "Wang, Yalin" <Yalin.Wang@sonymobile.com>
Cc: "'linux-arch-owner@vger.kernel.org'"
<linux-arch-owner@vger.kernel.org>,
"'linux-kernel@vger.kernel.org'" <linux-kernel@vger.kernel.org>,
"'linux-arm-msm-owner@vger.kernel.org'"
<linux-arm-msm-owner@vger.kernel.org>
Subject: Re: bug report about assert at [ kernel/fs/jbd2/transaction.c:2138 ]
Date: Thu, 21 Nov 2013 20:32:54 -0500 [thread overview]
Message-ID: <20131122013254.GA5923@thunk.org> (raw)
In-Reply-To: <35FD53F367049845BC99AC72306C23D10194484B7BB2@CNBJMBX05.corpusers.net>
On Thu, Nov 21, 2013 at 03:54:29PM +0800, Wang, Yalin wrote:
>
> I have a crash on Android phone during daily use.
>
> It crash at function __jbd2_journal_refile_buffer( ) :
>
> if (jh->b_transaction)
> assert_spin_locked(&jh->b_transaction->t_journal->j_list_lock);
>
>
> I have a look at the code,
> Seems this part are all handled by kernel ,
> So should be kernel BUG ,
>
> https://www.codeaurora.org/cgit/quic/la/kernel/msm/tree/fs/jbd2/commit.c?id=ccde7494b7174d810f316019b6e935fb222c6533
This commit ID references a truly antique kernel --- 2.6.29, dating
from January 9th, 2009. There is saying that in Internet times, two
years is infinity. So four years is two times infinity. :-)
If the reason why Sony is still shipping kernels that are this anique
is because you are stuck on !@#!?! proprietary binaries brought to you
by the letters 'B' or the letters 'Q', may I suggest that you add to
your procurement contracts a requirement that your downstream hardware
providers provide open source drivers?
(I'll note that some of enterprise server providers have succesfully
leaned on their peripheral providers for host bus adapters and
networking cards that the drivers not only be open source, but have to
be accepted in the upstream kernel. I'll gently suggest that handset
vendors first require in their procurement contracts that drivers at
the very least be open source, and then later, that said open source
drivers are accepted into the mainline kernel, since that way you
won't be responsible for forward porting out-of-tree drivers to
non-antique kernels.)
In any case, ext4 back in 2.6.29 had many bugs. It's not something
that the open source community is going to be all that interested in
supporting. Enterprise distributions pay long-suffering developers
$$$ to support ancient, old kernels. My advice to not ship new
products with ancient kernels.
If in fact you aren't really using that ancient a kernel, you need to
let us know the precise version of the kernel you are using.
Best regards,
- Ted
next prev parent reply other threads:[~2013-11-22 1:33 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-21 7:54 bug report about assert at [ kernel/fs/jbd2/transaction.c:2138 ] Wang, Yalin
2013-11-22 1:32 ` Theodore Ts'o [this message]
2013-11-22 5:42 ` Wang, Yalin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131122013254.GA5923@thunk.org \
--to=tytso@mit.edu \
--cc=Yalin.Wang@sonymobile.com \
--cc=linux-arch-owner@vger.kernel.org \
--cc=linux-arm-msm-owner@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox