On Mon, Jul 23, 2012 at 03:16:24PM -0700, Andrew Morton wrote: > On Mon, 23 Jul 2012 09:28:23 +0100 > Catalin Marinas <catalin.marinas@xxxxxxx> wrote: > > > Hi Fengguang, > > > > On Mon, Jul 23, 2012 at 07:55:42AM +0100, Fengguang Wu wrote: > > > There are new compile warnings show up in > > > > > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git akpm > > > head: 37e2ad4953983527f7bdb6831bf478eedcc84082 > > > commit: ffacc905e2135e8408d43adbe0b8e21484b2c8e7 [244/309] atomic64_test: simplify the #ifdef for atomic64_dec_if_positive() test > > > config: sparc-allmodconfig (attached as .config) > > > > > > All warnings: > > > > > > lib/atomic64_test.c: In function 'test_atomic64': > > > lib/atomic64_test.c:131:2: warning: #warning Please implement atomic64_dec_if_positive for your architecture and select the above Kconfig symbol [-Wcpp] > > > > Is this a new warning in the sense that the string is different now? Or > > you never got it before on sparc64. I don't have a sparc toolchain at > > hand now but it looks to me like atomic64_test.c should have given the > > warning before this patch as atomic64_dec_if_positive() does not seem to > > be defined on sparc64. > > > > Before: > lib/atomic64_test.c:132:2: warning: #warning Please implement atomic64_dec_if_positive for your architecture, and add it to the IF above > > After: > lib/atomic64_test.c:131:2: warning: #warning Please implement atomic64_dec_if_positive for your architecture and select the above Kconfig symbol Exactly. That changed message text will be recognized by my dumb script as "new" warning and I cannot do much about this.. Fengguang -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
