From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752603AbYIANCa (ORCPT ); Mon, 1 Sep 2008 09:02:30 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750876AbYIANCW (ORCPT ); Mon, 1 Sep 2008 09:02:22 -0400 Received: from gw-colo-pa.panasas.com ([66.238.117.130]:17898 "EHLO natasha.panasas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750831AbYIANCV (ORCPT ); Mon, 1 Sep 2008 09:02:21 -0400 Message-ID: <48BBE77D.7070007@panasas.com> Date: Mon, 01 Sep 2008 16:00:45 +0300 From: Boaz Harrosh User-Agent: Thunderbird 2.0.0.14 (X11/20080501) MIME-Version: 1.0 To: Ingo Molnar , Rusty Russell , "David S. Miller" , Ivo van Doorn , "John W. Linville" CC: Alexey Dobriyan , Andrew Morton , Theodore Tso , Linus Torvalds , Jan Beulich , linux-kernel Subject: [PATCHSET 0/5] BUILD_BUG_ON: error on none-const expressions Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 01 Sep 2008 12:59:57.0151 (UTC) FILETIME=[A2884EF0:01C90C32] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Submitted a patchset to fix BUILD_BUG_ON to no longer let through none compile-time constant expressions. This was debated a few times on LKML. The final solution is as proposed by Rusty Russell, which produces all expected results in my tests. [see: http://www.spinics.net/lists/kernel/msg772904.html] [PATCH 1/5] debug, x86: move BUILD_BUG_ON() ARRAY_SIZE and __FUNCTION__ Ingo this is your patch. Please verify? [PATCH 2/5] net/niu: Fix none-const BUILD_BUG_ON usage [PATCH 3/5] virtio: Fix none-const BUILD_BUG_ON usage [PATCH 4/5] rt2x00: Compiler warning unmasked by fix of BUILD_BUG_ON These three need maintainers approval. I hope I did not forget anyone [PATCH 5/5] debug: BUILD_BUG_ON: error on none-const expressions Finally after all call sites are fixed this can go in. (Rusty this one is From: you) I have only compiled ARCH=x86 (64/32) allmodconfig. So this might break on other ARCHs. It should spend a night in Linux-Next to expose these places. Thanks Boaz