Thread (3 messages) flat view 3 messages, 3 authors, 2011-09-01

Re: linux-next: Tree for Aug 30 (jbd2 + bug.h)

From: Arnaud Lacombe <hidden>
Date: 2011-09-01 20:30:51
Also in: linux-next, lkml

Hi,

On Thu, Sep 1, 2011 at 7:43 AM, Arnd Bergmann [off-list ref] wrote:
On Wednesday 31 August 2011, Randy Dunlap wrote:
quoted
On Tue, 30 Aug 2011 17:16:05 +1000 Stephen Rothwell wrote:
quoted
Hi all,
[Sorry about the delay.  My build machine is being slow.
If this is alread fixed, sorry about the noise.]


When CONFIG_BUG is not enabled (I see this on x86_64):

fs/jbd2/transaction.c: In function 'jbd2_journal_dirty_metadata':
fs/jbd2/transaction.c:1176: error: implicit declaration of function '__WARN'


asm-generic/bug.h does not provide a version of __WARN() when
CONFIG_BUG is not enabled...
Hmm, my feeling is that we shouldn't do that either, and that jbd2 should
be changed. If we want a function that does what __WARN() does today, we
should probably make a conscious decision about what we want it to be called
and not have it start with "__".
Why is WARN_ON() not used here ?

 - Arnaud
       Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help